Why is my javascript function sometimes "not defined"?

Posted by harpo on Stack Overflow See other posts from Stack Overflow or by harpo
Published on 2008-09-30T16:38:57Z Indexed on 2010/03/21 5:01 UTC
Read the original article Hit count: 420

Filed under:

Problem: I call my javascript function, and sometimes I get the error 'myFunction is not defined'.

But it is defined.

For example. I'll occasionally get 'copyArray is not defined' even in this example:

function copyArray( pa ) {
    var la = [];
    for (var i=0; i < pa.length; i++)
    	la.push( pa[i] );
    return la;
}

Function.prototype.bind = function( po ) {
    var __method = this;
    var __args = [];

    // sometimes errors -- in practice I inline the function as a workaround
    __args = copyArray( arguments );

    return function() {
    	/* bind logic omitted for brevity */
    }
}

As you can see, copyArray is defined right there, so this can't be about the order in which script files load.

I've been getting this in situations that are harder to work around, where the calling function is located in another file that should be loaded after the called function. But this was the simplest case I could present, and appears to be the same problem.

It doesn't happen 100% of the time, so I do suspect some kind of load-timing-related problem. But I have no idea what.

@Hojou: That's part of the problem. The function in which I'm now getting this error is itself my addLoadEvent, which is basically a standard version of the common library function.

@James: I understand that, and there is no syntax error in the function. When that is the case, the syntax error is reported as well. In this case, I am getting only the 'not defined' error.

@David: The script in this case resides in an external file that is referenced using the normal <script src="file.js"></script> method in the page's head section.

@Douglas: Interesting idea, but if this were the case, how could we ever call a user-defined function with confidence? In any event, I tried this and it didn't work.

@sk: This technique has been tested across browsers and is basically copied from the prototype library.

© Stack Overflow or respective owner

Related posts about JavaScript