ESLint Stylistic 项目中 indent-binary-ops 规则的行为变更分析

在 ESLint Stylistic 项目中，indent-binary-ops 规则用于控制二进制操作符（如 &&、||）的缩进风格。最近该规则的一次重构引发了一个值得注意的行为变更，特别是在处理操作符与花括号位于同一行的特殊情况时。

问题背景

在重构前的版本中，当二进制操作符与左花括号位于同一行时，规则会强制要求后续的操作数保持与第一个操作符相同的缩进级别。例如：

{ aaaaa &&
    bbbbb }

然而，重构后的版本改变了这一行为，现在要求后续操作数与花括号保持对齐：

{ aaaaa &&
  bbbbb }

技术分析

这种行为变化源于重构过程中移除了一个特定的条件判断逻辑。原逻辑会检查以下情况：

1. 操作符是否为逻辑运算符（&& 或 ||）
2. 节点起始行是否与左侧token相同
3. 节点起始列是否不等于该行缩进长度

这个条件原本确保了当逻辑运算符出现在行内时，后续操作数需要保持一致的缩进级别。移除后，规则现在更倾向于让操作数与包含它们的块结构（如花括号）对齐。

最佳实践建议

虽然两种缩进方式在技术上都可行，但从代码可读性和一致性角度考虑，更推荐的做法是将逻辑运算符单独放在新行上：

{ 
  aaaaa &&
  bbbbb 
}

这种写法具有以下优势：

1. 清晰地区分了块结构和逻辑表达式
2. 保持了操作符和操作数的一致对齐
3. 更易于阅读和维护
4. 避免了边缘情况的歧义

规则演进思考

这类规则的行为变更反映了代码风格指导原则的演进过程。在制定或调整代码风格规则时，我们需要权衡：

• 规则的严格性与灵活性
• 常见用例与边缘情况的处理
• 历史代码兼容性与新的最佳实践

对于团队项目，建议明确约定这类边界情况的处理方式，并在ESLint配置中做相应设置，以避免因工具版本更新导致的不一致问题。