【问题标题】:How to avoid global variables in JavaScript?如何避免 JavaScript 中的全局变量?
【发布时间】:2010-12-22 22:49:39
【问题描述】:

我们都知道global variables 绝不是最佳实践。但是有几个例子是没有它们就很难编码。您使用什么技术来避免使用全局变量?

例如,给定以下场景,你怎么会不使用全局变量?

JavaScript 代码:

var uploadCount = 0;

window.onload = function() {
    var frm = document.forms[0];

    frm.target = "postMe";
    frm.onsubmit = function() {
        startUpload();
        return false;
    }
}

function startUpload() {
    var fil = document.getElementById("FileUpload" + uploadCount);

    if (!fil || fil.value.length == 0) {
        alert("Finished!");
        document.forms[0].reset();
        return;
    }

    disableAllFileInputs();
    fil.disabled = false;
    alert("Uploading file " + uploadCount);
    document.forms[0].submit();
}

相关标记:

<iframe src="test.htm" name="postHere" id="postHere"
  onload="uploadCount++; if(uploadCount > 1) startUpload();"></iframe>

<!-- MUST use inline JavaScript here for onload event
     to fire after each form submission. -->

此代码来自具有多个&lt;input type="file"&gt; 的网络表单。它一次上传一个文件以防止大量请求。它通过POST向 iframe 执行此操作,等待触发 iframe onload 的响应,然后触发下一次提交。

您不必专门回答这个例子,我只是提供它以供参考,在这种情况下我无法想出避免全局变量的方法。

【问题讨论】:

标签: javascript global-variables


【解决方案1】:

最简单的方法是将您的代码包装在一个闭包中,然后手动将您需要全局的变量公开到全局范围:

(function() {
    // Your code here

    // Expose to global
    window['varName'] = varName;
})();

为了解决 Crescent Fresh 的评论:为了从场景中完全删除全局变量,开发人员需要更改问题中假设的一些内容。它看起来更像这样:

Javascript:

(function() {
    var addEvent = function(element, type, method) {
        if('addEventListener' in element) {
            element.addEventListener(type, method, false);
        } else if('attachEvent' in element) {
            element.attachEvent('on' + type, method);

        // If addEventListener and attachEvent are both unavailable,
        // use inline events. This should never happen.
        } else if('on' + type in element) {
            // If a previous inline event exists, preserve it. This isn't
            // tested, it may eat your baby
            var oldMethod = element['on' + type],
                newMethod = function(e) {
                    oldMethod(e);
                    newMethod(e);
                };
        } else {
            element['on' + type] = method;
        }
    },
        uploadCount = 0,
        startUpload = function() {
            var fil = document.getElementById("FileUpload" + uploadCount);

            if(!fil || fil.value.length == 0) {    
                alert("Finished!");
                document.forms[0].reset();
                return;
            }

            disableAllFileInputs();
            fil.disabled = false;
            alert("Uploading file " + uploadCount);
            document.forms[0].submit();
        };

    addEvent(window, 'load', function() {
        var frm = document.forms[0];

        frm.target = "postMe";
        addEvent(frm, 'submit', function() {
            startUpload();
            return false;
        });
    });

    var iframe = document.getElementById('postHere');
    addEvent(iframe, 'load', function() {
        uploadCount++;
        if(uploadCount > 1) {
            startUpload();
        }
    });

})();

HTML:

<iframe src="test.htm" name="postHere" id="postHere"></iframe>

您不需要在&lt;iframe&gt; 上需要内联事件处理程序,它仍然会在每次使用此代码加载时触发。

关于加载事件

这是一个测试用例,表明您不需要内联 onload 事件。这取决于引用同一服务器上的文件(/emptypage.php),否则您应该能够将其粘贴到页面中并运行它。

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"
    "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
<head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8"/>
    <title>untitled</title>
</head>
<body>
    <script type="text/javascript" charset="utf-8">
        (function() {
            var addEvent = function(element, type, method) {
                if('addEventListener' in element) {
                    element.addEventListener(type, method, false);
                } else if('attachEvent' in element) {
                    element.attachEvent('on' + type, method);

                    // If addEventListener and attachEvent are both unavailable,
                    // use inline events. This should never happen.
                } else if('on' + type in element) {
                    // If a previous inline event exists, preserve it. This isn't
                    // tested, it may eat your baby
                    var oldMethod = element['on' + type],
                    newMethod = function(e) {
                        oldMethod(e);
                        newMethod(e);
                    };
                } else {
                    element['on' + type] = method;
                }
            };

            // Work around IE 6/7 bug where form submission targets
            // a new window instead of the iframe. SO suggestion here:
            // http://stackoverflow.com/q/875650
            var iframe;
            try {
                iframe = document.createElement('<iframe name="postHere">');
            } catch (e) {
                iframe = document.createElement('iframe');
                iframe.name = 'postHere';
            }

            iframe.name = 'postHere';
            iframe.id = 'postHere';
            iframe.src = '/emptypage.php';
            addEvent(iframe, 'load', function() {
                alert('iframe load');
            });

            document.body.appendChild(iframe);

            var form = document.createElement('form');
            form.target = 'postHere';
            form.action = '/emptypage.php';
            var submit = document.createElement('input');
            submit.type = 'submit';
            submit.value = 'Submit';

            form.appendChild(submit);

            document.body.appendChild(form);
        })();
    </script>
</body>
</html>

每次我在 Safari、Firefox、IE 6、7 和 8 中单击提交按钮时都会触发警报。

【讨论】:

  • 或者提供某种访问器。我同意。
  • 当人们投反对票时,这对他们解释他们投反对票的理由很有帮助。
  • 我没有投反对票。但是,说 window['varName'] = varName 与在闭包之外进行全局 var 声明相同。 var foo = "bar"; (function() { alert(window['foo']) })();
  • 您回答的是标题,而不是问题。我不喜欢那样。将闭包习语放在内联事件处理程序的引用上下文中(因为问题的核心)会更好。
  • Crescent Fresh,我回答了这个问题。为了避免全局函数,需要重新设计问题的假设。此答案以问题的假设(例如内联事件处理程序)为指导,并允许开发人员仅选择必要的全局访问点,而不是全局范围内的所有内容。
【解决方案2】:

有些东西会在全局命名空间中——也就是说,无论您从内联 JavaScript 代码中调用什么函数。

一般来说,解决方案是将所有内容都包装在一个闭包中:

(function() {
    var uploadCount = 0;
    function startupload() {  ...  }
    document.getElementById('postHere').onload = function() {
        uploadCount ++;
        if (uploadCount > 1) startUpload();
    };
})();

并避免使用内联处理程序。

【讨论】:

  • 在这种情况下需要内联处理程序。当表单提交到 iframe 时,以编程方式设置的 onload 处理程序不会触发。
  • @Josh:哇,真的吗? iframe.onload = ... 不等于 &lt;iframe onload="..."?
【解决方案3】:

有时在 JavaScript 中使用全局变量是有意义的。但不要让它们像那样直接挂在窗外。

相反,创建一个“命名空间”对象来包含您的全局变量。对于奖励积分,把所有东西都放在那里,包括你的方法。

【讨论】:

  • 我该怎么做?创建一个命名空间对象来包含我的全局变量?
【解决方案4】:

我建议module pattern。

YAHOO.myProject.myModule = function () {

    //"private" variables:
    var myPrivateVar = "I can be accessed only from within YAHOO.myProject.myModule.";

    //"private" method:
    var myPrivateMethod = function () {
        YAHOO.log("I can be accessed only from within YAHOO.myProject.myModule");
    }

    return  {
        myPublicProperty: "I'm accessible as YAHOO.myProject.myModule.myPublicProperty."
        myPublicMethod: function () {
            YAHOO.log("I'm accessible as YAHOO.myProject.myModule.myPublicMethod.");

            //Within myProject, I can access "private" vars and methods:
            YAHOO.log(myPrivateVar);
            YAHOO.log(myPrivateMethod());

            //The native scope of myPublicMethod is myProject; we can
            //access public members using "this":
            YAHOO.log(this.myPublicProperty);
        }
    };

}(); // the parens here cause the anonymous function to execute and return

【讨论】:

  • 我会 +1,因为我理解这一点并且它非常有帮助,但我仍然不清楚在我只使用一个全局变量的情况下这会有多有效。如果我错了,请纠正我,但是执行此函数并返回它会导致返回的对象存储在YAHOO.myProject.myModule 中,这是一个全局变量。对吗?
  • @Josh:全局变量并不邪恶。全局变量_S_ 是邪恶的。尽量减少全局变量的数量。
  • 每次您想访问“模块”公共属性/方法之一时,都会执行整个匿名函数,对吗?
  • @UpTheCreek:不,不会。它只执行一次,当程序在最后一行遇到关闭 () 时,返回的对象将被分配给 myModule 属性,其中包含 myPrivateVar 和 myPrivateMethod 的闭包。
  • 漂亮,正是我正在寻找的。这允许我将我的页面逻辑与我的 jquery 事件处理分开。一个问题,通过 XSS 攻击,攻击者仍然可以访问 YAHOO.myProject.myModule 对吗?让外部函数不命名并放一个 (); 不是更好吗?最后,围绕 $(document).ready 以及?也许修改 YAHOO.myProct.myModule 属性的元对象?我刚刚在 js 理论上投入了相当多的时间,现在正在尝试将其组合在一起。
【解决方案5】:
window.onload = function() {
  var frm = document.forms[0];
  frm.target = "postMe";
  frm.onsubmit = function() {
    frm.onsubmit = null;
    var uploader = new LazyFileUploader();
    uploader.startUpload();
    return false;
  }
}

function LazyFileUploader() {
    var uploadCount = 0;
    var total = 10;
    var prefix = "FileUpload";  
    var upload = function() {
        var fil = document.getElementById(prefix + uploadCount);

        if(!fil || fil.value.length == 0) {    
            alert("Finished!");
            document.forms[0].reset();
            return;
         }

        disableAllFileInputs();
        fil.disabled = false;
        alert("Uploading file " + uploadCount);
        document.forms[0].submit();
        uploadCount++;

        if (uploadCount < total) {
            setTimeout(function() {
                upload();
            }, 100); 
        }
    }

    this.startUpload = function() {
        setTimeout(function() {
            upload();
        }, 100);  
    }       
}

【讨论】:

  • 如何在 iframe 上的 onload 处理程序中增加 uploadCount?这是至关重要的。
  • 好的,我明白你在这里做了什么。不幸的是,这不一样。这会分别触发所有上传,但同时(从技术上讲,它们之间有 100 毫秒)。当前的解决方案是按顺序上传它们,这意味着在第一次上传完成之前不会开始第二次上传。这就是为什么需要内联 onload 处理程序的原因。以编程方式分配处理程序不起作用,因为它只在第一次触发。内联处理程序每​​次都会触发(无论出于何种原因)。
  • 我仍然会 +1,因为我确实看到这是隐藏全局变量的有效方法
  • 您可以为每次上传创建一个 iframe 以维护各个回调。
  • 我认为这最终是一个很好的答案,只是被特定示例的要求稍微蒙上了一层阴影。基本上这个想法是创建一个模板(对象)并使用“新”来实例化它。这可能是最好的答案,因为它真正避免了全局变量,即不妥协
【解决方案6】:

首先,要避免全局 JavaScript 是不可能的,总会有一些东西在全局范围内徘徊。即使您创建了一个名称空间(这仍然是一个好主意),该名称空间也将是全局的。

但是,有许多方法可以避免滥用全局范围。最简单的两个是使用闭包,或者由于您只有一个需要跟踪的变量,只需将其设置为函数本身的属性(然后可以将其视为static 变量)。

关闭

var startUpload = (function() {
  var uploadCount = 1;  // <----
  return function() {
    var fil = document.getElementById("FileUpload" + uploadCount++);  // <----

    if(!fil || fil.value.length == 0) {    
      alert("Finished!");
      document.forms[0].reset();
      uploadCount = 1; // <----
      return;
    }

    disableAllFileInputs();
    fil.disabled = false;
    alert("Uploading file " + uploadCount);
    document.forms[0].submit();
  };
})();

* 请注意,uploadCount 的递增发生在内部

函数属性

var startUpload = function() {
  startUpload.uploadCount = startUpload.count || 1; // <----
  var fil = document.getElementById("FileUpload" + startUpload.count++);

  if(!fil || fil.value.length == 0) {    
    alert("Finished!");
    document.forms[0].reset();
    startUpload.count = 1; // <----
    return;
  }

  disableAllFileInputs();
  fil.disabled = false;
  alert("Uploading file " + startUpload.count);
  document.forms[0].submit();
};

我不知道为什么uploadCount++; if(uploadCount &gt; 1) ... 是必要的,因为看起来条件总是为真。但是,如果您确实需要对变量进行全局访问,那么我上面描述的 function property 方法将允许您在变量实际上不是全局的情况下这样做。

<iframe src="test.htm" name="postHere" id="postHere"
  onload="startUpload.count++; if (startUpload.count > 1) startUpload();"></iframe>

但是,如果是这种情况,那么您可能应该使用对象字面量或实例化对象并以正常的 OO 方式进行处理(如果您喜欢,您可以使用模块模式)。

【讨论】:

  • 你绝对可以避免全局作用域。在您的“闭包”示例中,只需从开头删除“var startUpload =”,该函数将被完全封闭,在全局级别无法访问。在实践中,许多人更喜欢公开单个变量,其中包含对其他所有内容的引用
  • @derrylwc 在这种情况下,startUpload 是您所指的单个变量。从闭包示例中删除 var startUpload = 意味着内部函数将永远无法执行,因为没有对它的引用。避免全局范围污染的问题与startUpload 使用的内部计数器变量uploadCount 有关。此外,我认为 OP 正试图避免使用内部使用的 uploadCount 变量污染此方法之外的任何范围。
  • 如果匿名闭包中的代码将上传器添加为事件监听器,那么只要发生适当的事件,它当然“将能够被执行”。
【解决方案7】:

使用闭包。像这样的东西给你一个全局以外的范围。

(function() {
    // Your code here
    var var1;
    function f1() {
        if(var1){...}
    }

    window.var_name = something; //<- if you have to have global var
    window.glob_func = function(){...} //<- ...or global function
})();

【讨论】:

  • 我过去的做法是定义一个特定的全局变量对象,并将所有全局变量附加到该对象上。我将如何将其包装在闭包中?可悲的是,评论框限制不允许我嵌入典型代码。
【解决方案8】:

对于中小型项目,使用闭包可能没问题。但是,对于大型项目,您可能希望将代码拆分为模块并将它们保存在不同的文件中。

因此我写了jQuery Secret plugin来解决问题。

在您使用此插件的情况下,代码将如下所示。

JavaScript:

// Initialize uploadCount.
$.secret( 'in', 'uploadCount', 0 ).

// Store function disableAllFileInputs.
secret( 'in', 'disableAllFileInputs', function(){
  // Code for 'disable all file inputs' goes here.

// Store function startUpload
}).secret( 'in', 'startUpload', function(){
    // 'this' points to the private object in $.secret
    // where stores all the variables and functions
    // ex. uploadCount, disableAllFileInputs, startUpload.

    var fil = document.getElementById( 'FileUpload' + uploadCount);

    if(!fil || fil.value.length == 0) {
        alert( 'Finished!' );
        document.forms[0].reset();
        return;
    }

    // Use the stored disableAllFileInputs function
    // or you can use $.secret( 'call', 'disableAllFileInputs' );
    // it's the same thing.
    this.disableAllFileInputs();
    fil.disabled = false;

    // this.uploadCount is equal to $.secret( 'out', 'uploadCount' );
    alert( 'Uploading file ' + this.uploadCount );
    document.forms[0].submit();

// Store function iframeOnload
}).secret( 'in', 'iframeOnload', function(){
    this.uploadCount++;
    if( this.uploadCount > 1 ) this.startUpload();
});

window.onload = function() {
    var frm = document.forms[0];

    frm.target = "postMe";
    frm.onsubmit = function() {
        // Call out startUpload function onsubmit
        $.secret( 'call', 'startUpload' );
        return false;
    }
}

相关标记:

<iframe src="test.htm" name="postHere" id="postHere" onload="$.secret( 'call', 'iframeOnload' );"></iframe>

打开你的Firebug,你会发现根本没有全局变量,甚至没有函数:)

有关完整文档,请参阅here。

有关演示页面,请参阅this。

Source code on GitHub.

【讨论】:

    【解决方案9】:

    对于“保护”单个全局变量:

    function gInitUploadCount() {
        var uploadCount = 0;
    
        gGetUploadCount = function () {
            return uploadCount; 
        }
        gAddUploadCount= function () {
            uploadCount +=1;
        } 
    }
    
    gInitUploadCount();
    gAddUploadCount();
    
    console.log("Upload counter = "+gGetUploadCount());

    我是 JS 的新手,目前在一个项目中使用它。 (我感谢任何评论和批评)

    【讨论】:

      【解决方案10】:

      另一种方法是创建一个对象,然后向它添加方法。

      var object = {
        a = 21,
        b = 51
      };
      
      object.displayA = function() {
       console.log(object.a);
      };
      
      object.displayB = function() {
       console.log(object.b);
      };
      

      通过这种方式,只有对象'obj'被暴露并且附加了方法。相当于将其添加到命名空间中。

      【讨论】:

        【解决方案11】:

        我是这样用的:

        {
            var globalA = 100;
            var globalB = 200;
            var globalFunc = function() { ... }
        
            let localA = 10;
            let localB = 20;
            let localFunc = function() { ... }
        
            localFunc();
        }
        

        对于所有全局范围使用“var”,对于本地范围使用“let”。

        【讨论】:

          猜你喜欢
          • 1970-01-01
          • 2012-12-19
          • 2011-03-24
          • 1970-01-01
          • 2016-03-19
          • 2012-02-02
          • 2015-12-26
          • 1970-01-01
          • 1970-01-01
          相关资源
          最近更新 更多