显式typeof ==“未定义”检查vs只是检查其存在？

Question

假设x是一个对象...有没有这样做的任何好处：

if (typeof x.foo != "undefined") 

与做

if (x.foo) 

？

这个问题上来，因为我正在读这篇博客文章： http://www.nczonline.net/blog/2010/03/09/custom-events-in-javascript/

在他的榜样，他的作用：

function EventTarget(){ 
    this._listeners = {}; 
} 

EventTarget.prototype = { 

    constructor: EventTarget, 

    addListener: function(type, listener){ 
    if (typeof this._listeners[type] == "undefined"){ 
     this._listeners[type] = []; 
    } 

    this._listeners[type].push(listener); 

在这种情况下this._listeners【类型】绝不会只是什么一个array--所以是不正确的，这将是清洁在这种情况下只是做

addListener: function(type, listener){ 
    if (!this._listeners[type]){ 
     this._listeners[type] = []; 
    } 

    this._listeners[type].push(listener); 

？

此外，作为一个侧面的问题，我不知道为什么他在做：

EventTarget.prototype = { 

    constructor: EventTarget 

不是默认的构造已设置为事件目标（“本”），当你调用新的EventTarget（） ？

Answer 1

小心truthy values。如果x.foo是

• 假

  if (x.foo)将无法​​运行

• 空
• 不确定
• “”
• NaN的

凡为if (typeof x.foo !== "undefined") {个对值是否undefined

替代检查检查是

if (x.foo !== undefined) {和if (x.foo !== void 0) {

做提防undefined可以覆盖一个局部变量

undefined = true是一个有效的声明，会破坏你的所有代码。当然，你永远不会看到这个代码在生产中，所以你不必屏蔽它，它只是需要谨慎的东西。

我个人倾向于使用

if (x.foo != null) { 
    ... 
} 

很多，检查两个null和undefined。

[编辑]

在您的具体实例它要么一个Array或undefined所以!foo是安全的。我个人更喜欢专门检查undefined，以便用户知道我只希望它在未定义时运行，而不是在null或false或""时运行。这使代码更加明确/自我记录。

至于

EventTarget.prototype = { 

    constructor: EventTarget 

如果覆盖EventTarget.prototype一个新的对象，然后将EventTarget.prototype.constructor财产发生丢失，需要重新设置。

如果您只是通过调用EventTarget.prototype.method = ...来扩展原型，则不需要再设置.constructor。

Answer 2

第一个明确检查x.foo是undefined，而if (x.foo)正在检查以查看x.foo是否真实。

http://11heavens.com/falsy-and-truthy-in-javascript