0

有问题的代码：

(function(window){
    ps = {},
    ps = window.ps,
    ps.subscriptions = [],
    ps.subscribe = function(name, callback){
        ps.subscriptions.push({"name": name, "callback": callback});
        return [name,callback];
    },
    ps.unsubscribe = function(args){
        for(x=0;x<ps.subscriptions.length;x++){
            if(ps.subscriptions[x].name == args[0], ps.subscriptions[x].callback == args[1])
                ps.subscriptions.splice(x, 1);
        }
    },
    ps.publish = function(name, args){
        var temp = [];
        if(ps.subscriptions.length > 0){
            for(var x=0;x<ps.subscriptions.length;x++) {
                if(ps.subscriptions[x].name == name)
                    temp.push({"fn":ps.subscriptions[x].callback});
            }
            for(x=0;x<temp.length;x++){
                temp[x].fn.apply(this,[args]);
            }
        }
    };

    console.log(window.ps);
    console.log(ps);
})(window);

我的问题：

  1. 我假设所有方法/变量都在global范围内声明，对吗？我问是因为这个假设驱动了我的其他问题，我想确保我没有遗漏任何东西。
  2. ps = window.ps如果在上面的行中他已经ps在全局范围内声明了，那么这样做有什么意义呢？正在做的事情不等同于ps = ps吗？
  3. 如果问题2中提到的上述逻辑有一个名称或约定，它是什么？我想不出一个好的方法来谷歌我所看到的。

如果有人有更好的方法来描述该技术并想更新我的问题标题以使其更容易找到，请随时这样做。

4

1 回答 1

1

那是糟糕的代码。一个更好的方法是：

(function(window){
    var ps = window.ps || {},
    ps.subscriptions = [],
    // ...

    window.ps = ps;
})(this);

为了更好的衡量，我会添加

"use strict"; // turn on "strict" mode

一开始也是。

于 2013-09-29T13:16:16.107 回答