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

你一定要看看这个链接。但是我也收到了这条消息，我很确定我的代码不包含任何（污染的）全局变量。

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

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

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

在上面的函数调用中创建它

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

Answer 2

将代码包装在函数信封中，以使var对于该函数体是本地的，并将您想要的全局内容显式附加到全局对象。

(function (global) {

  // your code here

  global.myGlobalVar = myVar

}(this));

Answer 3

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

想象一下这种情况：我的插件 Foo 使用了一个名为sheep的变量。

var sheep = 10;

你的插件栏，使用了一个名为sheep的变量：

var sheep = 20;

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

要使全局变量更加本地化，​​请将整个脚本包装在匿名函数中：

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

现在，您可以使用sheep执行任何操作，它将是本地的。但请记住，如果您计划使应用程序更复杂，则需要更好的脚手架。这种方法不是完全防弹或可扩展的......