javascript - 函数参数属性的赋值 'a' Eslint

Question

我正在尝试使用自定义函数对对象数组进行排序。
所以，我定义了我的函数

   sortingAlgo = (a, b) => {
        // we need to use custom sorting algorithm to meet our requirements.
        a.pos = a.pos.toUpperCase();
        b.pos = b.pos.toUpperCase();
        if(a.pos.slice(0, 1) === b.pos.slice(0, 1)) {
          if (parseInt(a.pos.slice(1)) < parseInt(b.pos.slice(1))){
            return -1;
          }
          if (parseInt(a.pos.slice(1)) > parseInt(b.pos.slice(1))){
            return 1;
          }
          return 0;
        } else {
          if (parseInt(a.pos.slice(0, 1)) < parseInt(b.pos.slice(0, 1))){
            return -1;
          }
          if (parseInt(a.pos.slice(0, 1)) > parseInt(b.pos.slice(0, 1))){
            return 1;
          }
          return 0;

        }
      }

我这样调用它。

const sortedProducts = data.data.sort(this.sortingAlgo);

现在我得到了 Eslint 错误提示。

**

• 169:5 错误 为函数参数“a”的属性赋值无参数重新分配
• 170:5 错误 为函数参数“b”的属性赋值无参数重新分配

**

我搜索了一下，发现这条规则是airBnB制定的。所以我不认为禁用这条规则是正确的。
那么有人可以告诉我如何解决这个问题或者禁用此规则是否可以。

Answer 1

无需重新分配函数参数 (a, b) 的 pos 属性。您应该只分配一个新变量(aPos, bPos)，这是最佳实践，这也是 ESLint 提示的原因。

您应该避免不必要的 mutations/side-effects尽可能在代码中防止错误并创建整体更高效的程序架构。

更具体地说，在这种情况下..

ESLint: Disallow Reassignment of Function Parameters (no-param-reassign)

const sortingAlgo = (a, b) => {
    // we need to use custom sorting algorithm to meet our requirements.
    let aPos = a.pos.toUpperCase(),
        bPos = b.pos.toUpperCase();

    if (aPos.slice(0, 1) === bPos.slice(0, 1)) {
        if (parseInt(aPos.slice(1)) < parseInt(bPos.slice(1))) {
            return -1;
        }
        if (parseInt(aPos.slice(1)) > parseInt(bPos.slice(1))) {
            return 1;
        }
        return 0;
    } else {
        if (parseInt(aPos.slice(0, 1)) < parseInt(bPos.slice(0, 1))) {
            return -1;
        }
        if (parseInt(aPos.slice(0, 1)) > parseInt(bPos.slice(0, 1))) {
            return 1;
        }
        return 0;

    }
}