JSLint - 不要改变参数当使用“参数”时？

Question

所有，

我使用 JSLint 来验证我的 JS 文件。在我最近的项目中，我使用以下格式为许多 JavaScript 函数设置默认值（更详细的 此处)：

function(a, b, option) {
    option = arguments.length > 2 ? option : "some default value";
    // ...
}

然而，这会导致最新版本的 JSLint 产生以下错误：

"Do not mutate parameter 'option' when using 'arguments'."

我知道使用更常见的方法来分配默认值（即 option = option || {};)抑制错误；但是，如果我打算将 falsey 值传递给 option，这将产生不正确的行为。

解决这个问题的唯一方法是引入一个新变量吗？例如：

var option2 = arguments.length > 2 ? option : "some default value";

Answer 1

我猜 JSLint 会警告您，因为您尝试通过使用参数关键字检查来修改输入参数之一。但是，JSHint 在尝试您的代码时确实给了我任何警告。

解决您问题的方法是检查 option 是否已定义，这样您就可以通过发送虚假值来解决问题：

function(a, b, option) {
    if(typeof option === "undefined") {
        option = "some default value"; 
    }
    // ...
}

如果您发现编写此 typeof 很麻烦 每次检查，创建一个 isDef 函数：

function isDef(param) {
    return typeof param !== "undefined";
}
function(a, b, option) {
    option = isDef(option) ? option : "some default value"; 
    // ...
}

// Simon A.

Answer 2

在大多数情况下，你最好这样做：

function(a, b, option) {
    option = option || "some default value";
    // ...
}

或

function(a, b, option) {
    if ( !option || typeof option !== 'string' ) { 
        option = "some default value";
    }
    // ...
}