关于条件表达式的样式或错误的 JSLINT 警告

Question

我正在尝试修复 JSLINT 警告的两件事，但我不知道如何修复。哪一行应该替换为什么代码？我已在下面粘贴了有问题的代码和有关它们的警告。预先非常感谢您的建议/答复。

JSLINT 说：

第 137 行出现问题：需要条件表达式，但看到的是赋值。

while(elem = document.getElementById("optionsDiv"+g))

第 140 行出现问题：预期为“{”，但看到的是“return”。

return g;

---

function isAnyOptionVisible()
{
    var g=0;
    while(elem = document.getElementById("optionsDiv"+g))
    {
        if(elem.className==="optionsDivVisible")
            return g;
        g++;
    }
    return -1;
}

Answer 1

条件没有任何问题，除了 elem 如果未在外部作用域中声明，则可能是全局的。如果未声明，请事先声明：

var elem, g=0;    

如果您想让 JSLint 满意，您可以添加与 null 的显式比较：

while((elem = document.getElementById("optionsDiv"+g)) !== null)

我没有看到任何有关“Expected {”的内容。然而，将 g++ 与大括号放在同一行是奇怪的风格。

请记住，JSlint 部分取决于 Crockford 的个人偏好，您不必总是同意。

编辑：好的，第二个错误是因为 JSLint 希望您将返回值放在大括号中，例如：

    if(elem.className==="optionsDivVisible")
    {
        return g;
    }
    g++;

这也是我更喜欢编码的样式。我确实发现它避免了某些错误。然而，这又是主观的。明显的权衡是它增加了两条线。