Home/ Questions/Q 6248313
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-24T13:02:02+00:00

http://dl.dropbox.com/u/331982/Sandbox/comsat.html <– code http://dl.dropbox.com/u/331982/Sandbox/comsat.js <– location of javascript file So, This worked when I

  • 0

http://dl.dropbox.com/u/331982/Sandbox/comsat.html <– code

http://dl.dropbox.com/u/331982/Sandbox/comsat.js <– location of javascript file

So, This worked when I had everything all in one file, but I’m not a huge fan of having giant script tags in my HTML.

I get the error that one of the Raphael methods is undefined…. =\

How do I load my script correctly?

Everything is loaded in the <head>

<head>
  <meta http-equiv="content-type" content="text/html; charset=UTF-8">
  <title>Comsat</title>

    <script type="text/javascript" src="http://dl.dropbox.com/u/331982/Sandbox/raphael-min.js"></script> 
    <script type="text/javascript" src="http://code.jquery.com/jquery-1.6.2.min.js"></script>
    <script type="text/javascript" src="http://dl.dropbox.com/u/331982/Sandbox/comsat.js"></script> 


  <style type="text/css">
    body{
    background: url(http://i.imgur.com/Ks6Pd.jpg);
}
  </style>

</head>
Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    It looks to me like the error is that initPlayerSelect() is not defined and I think that’s because that function is not in the global scope, but rather it’s defined inside another function scope and thus is not reachable from initComSat().

    You will need to make it so that initPlayerSelect() can be reached from inside the initComSat() function.

    One way to do that is to move the initPlayerSelect() code out of the block it’s in and put it right before initComSat().

    FYI, this particular error has nothing to do with using JS files or any order of JS files – it’s purely a function scoping issue.

    The next error I see is caused because you are running jQuery DOM access code before the page has been loaded here in comsat.js:

    /*
    DOM Containers
*/
var cs_playerList = [];
var cs_players = $("ul.players");
// End DOM Containers

    This code fails at script load time, which causes cs_players to be null or undefined which later causes initPlayerSelect() to fail.

    Code like that can’t be run until the page has been loaded. jQuery has $(document).ready() to run code when the page is loaded.;

    • 0