function definitions not hoisted

后端 未结 2 2034
没有蜡笔的小新
没有蜡笔的小新 2020-12-18 07:30

W.r.t Hoisting of fxn definitions.

相关标签:
2条回答
  • 2020-12-18 07:44

    You should avoid using conditionally created functions.

    For example, assume the following code:

    if (false){
     function foo(){
      console.log(1)
     }
    }
    foo()
    

    Firefox will not hoist the function and this will result in ReferenceError: foo is not defined. Chrome, however, hoists the function nonetheless and prints 1. So obviously you have deal with different browser behaviour. Therefore, do not do things like that at all (or use function expressions if you really want to).

    Also see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Statements/function

    Functions can be conditionally declared, that is, a function statement can be nested within an if statement. Most browsers other than Mozilla will treat such conditional declarations as an unconditional declaration and create the function whether the condition is true or not, see this article for an overview. Therefore they should not be used, for conditional creation use function expressions.

    Especially look at the linked article which somewhat explains the issue you are seeing. So Chrome seems to have changed something in that regard. But again, do not use conditionally created functions.

    And note that, as FREEZE commented, you should use 'use strict'; which would not allow such code but throws an exception instead.

    0 讨论(0)
  • 2020-12-18 07:50

    The code you have is invalid in strict mode. Functions don't get hoisted out of blocks (or at least they shouldn't), function declarations inside blocks were completely illegal until ES6. You should write

    "use strict";
    var foo;
    if (true) {
      foo = function() {
        alert(1)
      };
    } else {
      foo = function() {
        alert(2)
      };
    }
    foo()
    

    to get the desired behaviour with reproducible and expected results.

    Did I miss something or, did console stop hoisting on fxn's!

    Looks like V8 was updated to align with the ES6 spec. It does "hoist" them to the function/top scope, but only when the declaration is actually encountered (in your case, conditionally).

    0 讨论(0)
提交回复
热议问题