Question

我打开了一段我写过的js文件，虽然它正在运行，但我认为我发现了一个错误。（JS不是我的主要语言）

我有这个：

if( myvar = fieldval.match(mypattern))
  {
     //Do Stuff
  }

所以我想我明白了。这是正确的陈述吗？： javascript赋值操作的计算结果为分配的值。

我在w3schools上测试过

<script type="text/javascript">
var str="The rain in SPAIN stays mainly in the plain"; 
var patt1=/ain/gi;
var test
document.write(test=str.match(patt1));
</script>

它写了“ain，AIN，ain，ain”，我可能期望它写“真”或不写，因为布尔值true不是字符串。我的思路和最终结论是否正确。（我在这方面问我的想法，因为我没有很多正式的CS培训。）

Answer 1

这是一个正确的陈述。 myvar的新值已经过测试：

if ( myvar = fieldval.match(mypattern) )

当String.match方法无法找到匹配项时，它会返回null。 !!null === false，因此不评估if-block。找到任何非空匹配时，条件为真，并对该块进行求值。

在这种情况下，if语句很可能是正确的，并且以下是：

if ( (myvar = fieldval.match(mypattern)) !== null )

Answer 2

Rob W是正确的，但是在这样的if语句中放置赋值是非常糟糕的做法。将来，任何一个人（包括你自己）都会在那个声明中低头，以确定这是不是你的意思。

我强烈推荐道格拉斯·克罗克福德的演讲：http://www.youtube.com/watch?v=taaEzHI9xyY它会让任何人（Js dev或不是）更好的开发者观看它，因为你会考虑你的编码风格的含义以及未来的维护者可能会假设的内容。

Answer 3

函数string.match（）返回匹配数组，如果未找到匹配项，则返回null。代码有效，因为null在if语句中求值为false，而数组求值为true。

确实，str.match的结果存储在myvar中，然后myvar被计算为boolean。它按预期工作。

Answer 4

match返回匹配值的数组，而不是布尔值。请改用test。

javascript评估赋值if（x = myfunc（））

4 个答案: