为什么在 Javascript 中不鼓励在 while 语句中赋值？

Question

我刚刚编写了以下运行良好的循环：

let position = 0;
// tslint:disable-next-line:no-conditional-assignment
while ((position = source.indexOf('something interesting', position)) > 0) {
    // do something interesting with something interesting
  position += 1;
}

但请注意，我必须添加 tslint 例外。显然有人 已决定不允许在 while 有条件的。然而，这是我所知道的最直接的代码模式 特殊情况。

如果这不是“允许的”（或者我应该说“不鼓励”），那该怎么办 推动这个的人认为正确的代码模式是？

Answer 1

正如亚当在 cmets 中的示例，无条件赋值的替代方案可能如下所示：

let position = 0;
while (true) {
  position = source.indexOf('something interesting', position);
  if (position < 0) {
    break;
  }
  // do something interesting with something interesting
  position += 1;
}

如果禁用 curly，语法可能会变得更加紧凑，这会在 if/for/do/while 语句上强制使用大括号，即使 K&R Ch 3 根本不强制使用这些大括号* .这可以很好地比较这些规则的总体动机。 TSLint 在its homepage 上定义了它的用途：

TSLint 是一种可扩展的静态分析工具，用于检查 TypeScript 代码的可读性、可维护性和功能错误。

是否完全可以正确使用无大括号的if 语句，尤其是在您对缩进和格式非常认真的情况下？当然。但是，也完全有可能误用该功能，like in the docs for curly:

if (foo === bar)
    foo++;
    bar++;

您的问题假设“显然有人认为它是不可取的”，但更准确的说法是 这种模式是跨代码库的重要错误来源，足以让 @987654334 的作者@为特定模式提供了内置检测和反馈。该模式不是“不受欢迎的”、“不允许的”或“不鼓励的”，而是足够稀有且被滥用，足以使其检测默认开启。

与tslint 中的所有规则一样，您可以使用tslint.json 或tslint.yaml 来configure the default rule set，并且您和您的团队可以选择禁用curly 和no-conditional-assignment 等规则。如果这种情况通常是您的编码风格中的误报（看起来是这样），或者如果您将 tslint 应用于该模式常见且正确使用的现有代码库，这可能特别有用。但是，我认为一个团队声称（正如其他人在这个问题上所做的那样）该模式容易出现=/== 拼写错误和其他错误并且正确的使用不足以证明允许代码库中的模式。

---

*注意：虽然 K&R 不强制使用大括号，但它在 p56 上确实指出“缩进明确显示了您想要的内容，但编译器没有收到消息，并将 else 与内部 @987654344 相关联@. 这种错误很难找到；当有嵌套的 ifs 时，最好使用大括号。”这表明即使是原始 K&R 文本也支持合理的样式规则或超出其语法支持的限制。

Answer 2

它使代码的意图更难确定。使用赋值作为表达式是不寻常的 - 如果你在一个条件中看到它，你有多确定代码的作者打算使用赋值，或者它可能是错字？如果代码编写者打算比较这个值怎么办？ EG

while ((position = source.indexOf('something interesting', position)) > 0) {

对

while ((position == source.indexOf('something interesting', position)) > 0) {

这两者非常不同。 （在这种特殊情况下，Typescript 会警告您与布尔值和数字的比较可能是一个错误，但在 Javascript 中不会发生这种情况，在其他一些情况下也不会发生这种情况。）

对于上面的代码，您可以可能使用正则表达式，而不是在条件内部分配，在条件外部分配，甚至更好（尽管在不知道内容的情况下无法确定） // do something interesting with something interesting)。

不幸的是，它看起来很重复，但另一种选择是

let position = 0;
const getPos = () => {
  position = source.indexOf('something interesting', position);
};
getPos();
while (position > 0) {
  // do something interesting with something interesting
  getPos();
}

Answer 3

然而，对于这种特殊情况，这是我所知道的最直接的代码模式。

其实不然。更直接和更容易阅读的模式如下：

let position = 0;
while (position >= 0) {
    // do something interesting with something interesting
  position = source.indexOf('something interesting', position) + 1;
}

上面的代码有效，因为如果找不到字符串，indexOf 会返回 -1。