声明变量，如果和使用外

Question

考虑下面的代码：

var factory = function(someCondition) { 
    var para1 = 'some value'; 
    if(someCondition) 
     var para2 = 'some other value'; 

    return new MyClass(para1, para2); 
} 

我知道这个代码是perfectly legal，即使para2声明的if内部，而是外部使用。

我的问题是：这是 - 虽然合法 - 由主要的JavaScript风格指南认为是不好的风格？如果是这样，通过哪些方法以及建议的替代方案？

只是要清楚：我知道关于提升和变量不是作用域而是函数作用域的事实。

Answer 1

这不应该是一个坏的风格，这可能有点混乱。

任何方式的JavaScript会像下面这样做，因为hoisting，所以更好的写也像下面的。

var factory = function(someCondition) { 
    var para1 = 'some value'; 
    var para2; //declare it here 
    if(someCondition) 
     para2 = 'some other value'; //assign the value here 

    return new MyClass(para1, para2); 
} 

的ES6版本引入一个新的关键字让赋予变量块作用域。

Answer 2

技术上变量是“悬挂”，代码将执行这样的：代码执行之前在此范围内

var para1 = 'some value'; 
var para2; 
if(someCondition) 
    para2 = 'some other value'; 

一个范围内的所有声明var声明变量。因此，它在实践中没有太大的区别。不过，我认为这是一种很好的风格，可以像上面那样明确地编写代码，以便明确该范围中存在哪些变量。在条件内“隐藏”变量声明可能会导致混淆。

Answer 3

JavaScript的 '吊装' 的一个例子：

var para2 = "my value"; 

var factory = function(someCondition) { 
    var para1 = 'some value'; 
    console.log(para2); 
    if(someCondition) 
     var para2 = 'some other value'; 

    return new MyClass(para1, para2); 
} 

factory(true); // undefined 

上面的代码将日志 '未定义'。

Answer 4

JavaScript使用功能范围，因此，我会写这样

var factory = function(someCondition) { 
    var para1 = 'some value'; 
    var para2 = 'some other value'; 
    if(someCondition) 
    { 
     // some code.. 
    } 

    return new MyClass(para1, para2); 
} 

Answer 5

我会说这是所有权利。

var factory = function(someCondition) { 
    var para1 = 'some value'; 
    if(someCondition){ 
     var para2 = 'some other value'; 
     alert(para2); 
    } 

    alert(para2); 
    return new MyClass(para1, para2); 
} 

我已添加警报以便您可以关注。这里。在if语句中，您声明了一个名为para2的变量。当这段代码运行时，直接在它下面的alert函数显示'其他值'！这就说得通了。可能更不明智的是if语句之外的第二个提示还显示“其他值”！

这是怎么回事。你的para2变量在被称为块的内部被声明。块是出现在打开和关闭括号内的任何内容 - {和}。在许多编程语言中，块内声明的变量是该块本身范围的一部分。这意味着这些变量是本地的，不能在块之外访问。

JavaScript不像那些许多“其他”编程语言。 JavaScript不支持块范围。

查看更多here