为什么 jshint 不能将赋值识别为表达式？

Question

我需要如何修改这些行以使 jshint 满意？

赋值是一个表达式。为什么 jshint 不明白这一点？显然解释器会这样做。

Line 572: while(bookmark_element=bookmark_list[iterator++])

Expected a conditional expression and instead saw an assignment.


Line 582: while(bookmark_element=bookmark_list[iterator++])

Expected a conditional expression and instead saw an assignment.


Line 623: while(element_iterator=element_iterator.nextSibling)

Expected a conditional expression and instead saw an assignment.

Answer 1

如果你真的想听 JSHint，将表达式转换为布尔值：

while (!!(bookmark_element=bookmark_list[iterator++]))

! means: Something that evaluates to true is converted to false,
         something that evaluates to false is converted to true.

所以，!! 的意思是：将某些东西转换为条件表示。

Answer 2

/*jshint boss:true */

使用options 进行实验。

Answer 3

我确信 jshint 可以很好地理解这个表达式，只是大多数写 if (a = b) 的人实际上是指 if (a == b)，所以这会产生一个警告。

由于您的代码是您想要的，您可以添加一个显式测试：

while ((element_iterator = element_iterator.nextSibling) !== null) { ... }

Answer 4

报错至少有两种解决方法，根据JSHint docs.

1. 在条件语句前添加/*jshint boss:true */
2. 将语句括在额外的括号中，即while ((element_iterator = element_iterator.nextSibling)) {...}

就个人而言，我认为用额外的括号括起来是最佳做法，因为它可以保持错误检查，但在代码方面仍然很有意义。在之前添加!! 实际上什么也没做，只是将表达式来回转换为真/假两次。

Answer 5

这是一个表达式，你可以像这样修改它以使用 JSHint（虽然它不是很好）：

while(element_iterator.nextSibling) {
    element_iterator = element_iterator.nextSibling;

对于你的最后一个例子。但是，您不需要这样做。 JSHint 只是一种帮助您改善编码习惯和纠正错误的工具，但鉴于您所拥有的内容清晰、简洁，并且（在我看来）是最好的方式 - 请忽略这些消息。

Answer 6

jshint 无法判断您是否真的打算在条件块中进行分配，或者这是否真的应该是一个比较。担心的是其他人可能有同样的疑问。

在迭代器的情况下，我认为你没问题。

Answer 7

我遇到了这个错误，因为我在函数前面的声明中有一个尾随逗号：

this.foo = "bar",   // <- Error was here

this.myfunc = function() {
   ...   // <- Error reported on this line
};

_{（很难找到，但强化了我的观点，即 linter 通常是正确的，这是我的代码错误。如果我在全局范围内禁用了警告 - 甚至在那个地方 - 错误仍然会存在.</lecture>)}

Answer 8

在您的 Javascript 文件中添加 /* jshint expr: true */ 注释，它将不再显示警告。