奇妙なJavaScript関数バインディング。この制御フローは正しいですか？

Question

私は「makeExpression」の呼び出しに達したときに

function PParser() { 
.... 
makeExpression = function (lexemes, index) { 

    return makeNumber(lexemes, index); 
} 

makeDeclaration = function(lexemes, index) 
{ 
    if (lexemes[index].TokenType != LALLPLexer.VAR) { 
     throw "Expected VAR at index " + index; 
    } 

    if (lexemes[index + 1].TokenType != LALLPLexer.ID) { 
     throw "Expected ID at index " + index + 1; 
    } 

    if (lexemes[index + 2].TokenType != LALLPLexer.ASSIGN) { 
     throw "Expected ASSIGN at index " + index + 2; 
    } 

    var expressionNodeResult = makeExpression(lexemes, index + 3); 

... 

、私はちょうど上で定義された関数に移動する制御フローを期待していた（A.jsそれを呼び出す）、同じ.jsファイルに次の関数定義を持っています。しかし、代わりに、 "makeExpression"という別の関数が完全に異なる.jsファイル（B.js）で呼び出されます。

function Controller() 
{ 
... 

this.parseToStatement = function(statementText) 
{ 
    makeExpression = function(expressionNode) 
    { 
     return new IntLiteral(expressionNode.Content); 
    } 

    try { 
     statement = parser.parseStatement(new LALLPLexer().lex(statementText)); 

     if (statement.NodeType == LALLPParser.DECLARATION) { 

      return new Declaration(statement.Id, makeExpression(statement.Expression)); 
     } 
    } 
    catch (exception) { 
     statement = new UnknownStatement(statementText); 
    } 

    return statement; 
} 
} 

私はなぜそれがわかりません。興味深いことに、上記の "parseStatement"という行は、 "makeExpression"呼び出しからの呼び出しチェーンの上にあります。これは正しいjavascriptの動作ですか、もしそうなら、なぜこの動作が期待されるのですか？意図した動作をどうやって得ることができますか？

Answer 1

私はこれについて完全にはわかりませんが、jsは標準で変数をローカルに作成しないので、これらの関数はすべてグローバルに（つまりwindow変数に）アタッチされる可能性があり、したがって互いに上書きする可能性があります。

これらの定義にすべてvarを追加してください（または適切な場合はthis.）。多分それが助けになるでしょう。