javascript:名称空间污染

Question

我正试图将我的插件提交到mozilla站点，但我收到了这个该死的警告：

​

​

代码(在mf_options.js中)非常简单(我认为问题只存在于“起始存储”和“结束存储”之间：

// start Storage
var url = "http://mafiaafire.com";
var ios = Components.classes["@mozilla.org/network/io-service;1"]
          .getService(Components.interfaces.nsIIOService);
var ssm = Components.classes["@mozilla.org/scriptsecuritymanager;1"]
          .getService(Components.interfaces.nsIScriptSecurityManager);
var dsm = Components.classes["@mozilla.org/dom/storagemanager;1"]
          .getService(Components.interfaces.nsIDOMStorageManager);
var uri = ios.newURI(url, "", null);
var principal = ssm.getCodebasePrincipal(uri);
var storage = dsm.getLocalStorageForPrincipal(principal, "");
// end Storage



function display_blocked_list1() {

  var list = storage.getItem('domain_list_original');
  if (list !== undefined) {

    var strSingleLineText = list.replace(new RegExp( " ", "g" ),    "<br>"  );

    var status = document.getElementById("div1");
 status.innerHTML = strSingleLineText;
   }

  var list2 = storage.getItem('domain_list_redirect');
  if (list2 !== undefined) {

    // Strip out all line breaks.
    var strSingleLineText2 = list2.replace(new RegExp( " ", "g" ),  "<br>"  );

    var status2 = document.getElementById("div2");
 status2.innerHTML = strSingleLineText2;
   }


  var list3 = storage.getItem('list_expiry_date');

  if (list3 !== undefined) {



var dateArray = list3.split(",");
var future_date = new Date(dateArray[0],dateArray[1],dateArray[2]);
future_date.setDate(future_date.getDate()+2);

    var status2 = document.getElementById("div3");
 status2.innerHTML = future_date;

  }
// ##################################################

}

Answer 1

将您的代码包装在一个函数信封中，以便您的var位于该函数体的本地，并显式地将您想要的任何全局对象附加到全局对象。

(function (global) {

  // your code here

  global.myGlobalVar = myVar

}(this));

Answer 2

你一定要看看这个链接。然而，我也收到了这条消息，我非常确定我的代码不包含任何(污染)全局变量。

但如果这正是您使用的代码，那么您声明的任何函数和变量都将是全局的。在最简单的情况下，将代码包装在匿名函数调用中：

(function() {
    // your code here
}());

如果您需要一个全局变量，因为您必须从XUL元素调用，请确保您只有一个函数。在上面的函数调用中使用

window.YourPluginNamespace = {
    // all functions or "subspaces" here
};

Answer 3

问题是您使用了太多的全局变量，这些变量是在函数外部定义的。

想象一下这样的场景:我的插件Foo使用一个名为sheep的变量。

var sheep = 10;

您的插件Bar使用了一个也称为sheep的变量

var sheep = 20;

当我访问sheep时，如何才能确保你的插件没有修改它？这也是插件使用匿名函数foo = function() {的原因，因为它们是本地函数。

要使您的全局变量更具局部性，请将整个脚本包装在一个匿名函数中：

(function() {
  var sheep = 10;
}());

现在，你可以对sheep做任何你想做的事情，它将是本地的。但是，请记住，如果您计划使应用程序更加复杂，则需要一些更好的搭建。这个方法不是完全防弹的，也不是完全可伸缩的。