“return”后不需要“else”。（没有其他回报）

Question

我正在使用es lint清除代码中的错误。我遇到了这个错误：

“return”后不需要“else”。（没有其他回报）

} else {

我总是在返回后使用其他语句。有什么我可能忽略了的吗？

if (cctot <= 3 && cctot > 0) {
    alert('Credit under $3.00 not allowed');
    return Number.MIN_SAFE_INTEGER; // important to return 0 so we can check for these conditions for validation
} else {
    cctot *= -1;
}
return precise(cctot);

Answer 1

虽然规则正确地指出，else块是不必要的，这是一种样式偏好，但我会为可读性和最重要的可扫描性添加额外的考虑。

对于编写这段代码的开发人员和解释它的机器来说，这可能是一个风格点，仅此而已。但是对于需要修复错误、增强功能、进行代码审查等的开发人员来说，快速扫描代码并查看其他块的能力有助于识别逻辑分支。

在几行独立的代码中很容易看出意图，但是在数百行具有if的代码中，其他块可以作为有用的标识符，很像其他常见的视觉实践，如缩进、换行和命名约定。

Answer 2

这是一种代码风格偏好。您不需要else，而是可以将else代码直接放在if下面。这是因为如果if成功，那么函数就结束了，因此无论如何都不会到达else代码。

所以这个：

if (condition) {
  return foo;
} else {
  // do bar
}

return baz

相当于：

if (condition) {
  return foo;
}

// do bar

return baz

这种风格在不同的编程社区中似乎有所不同。Go开发人员几乎总是会省略else，而我已经看到更多的JS开发人员包含它。

虽然我更喜欢省略其他，但这又是纯粹的偏好问题。不要让它让你太担心。人们可能会对这种事情变得教条，但这真的没那么重要。

Answer 3

这基本上是说，如果在if部分中有return，则if语句的else部分是不必要的。这就是它所期望的：

if (cctot <= 3 && cctot > 0) {
      alert('Credit under $3.00 not allowed');
      return Number.MIN_SAFE_INTEGER; // important to return 0 so we can check for these conditions for validation
}
cctot *= -1;

一般来说，这：

if (condition) {
  return something;
} else {
  // do another thing
}

return anotherThing;

类似于：

if (condition) {
  return something;
}

// do another thing
return anotherThing;

在使用return语句执行if之后，不需要使用else部分，因为if下面的代码只有在所述条件未满足时才会运行。