javascript - 这段代码泄漏了什么/为什么？

Question

我被告知此代码在 previous post 中泄漏了 a1、a2、a3。

function t1()
  {
  var a=document.forms['f1'].elements;
  a1="Please fill out all fields";
  a2="Please enter your name using only letters and dashes";
  a3="Please enter a valid email address";
  a4="Password must be atleast 6 characters";
  if(c0(a,a1,'fb2')&&c1(a,a2,'fb2')&&c2(a[2],a3,'fb2')&&c3(a[3],a4,'fb2'))
    {
    return 1;
    }
  else
    {
    return 0;
    }
  }

Answer 1

在上一篇文章中，“泄漏”似乎指的是变量 a1 到 a4 没有在函数范围内正确声明 t1()。如果没有 var 关键字，它们将成为全局变量。

var a1="Please fill out all fields";
var a2="Please enter your name using only letters and dashes";
var a3="Please enter a valid email address";
var a4="Password must be atleast 6 characters";

您可能打算以逗号 , 而不是分号 ; 结束每一行，这会将它们全部链接到一个语句中，共享第一个 var 关键字。在那种情况下，它们就不会成为全局性的。只需确保使用 ; 终止最后一个。但是，我认为像我上面所做的那样，将每个语句都保留为自己的语句更具可读性和更好的做法。

有很多关于 JS 变量泄漏的博客文章。 Here's one I just found to get you started...