嵌套If条件写法最佳实践咨询：两种代码实现方案对比

结论 #

方案2是该场景下行业公认的更优实践，你的判断是对的。

两种方案的对比分析 #

方案1（嵌套if写法） #

case colorElementEnum.ICON:
        color = colors.buttonText;
        if (isHighContrast) {
          if (!titleColor) {
            if (disabled) {
              color = colors.lightGray;
            }
          }
        }
        return color;

这种写法的问题在于无意义的多层嵌套：当所有嵌套条件都没有对应独立的分支处理逻辑，只是递进判断是否满足最终执行条件时，嵌套只会增加代码阅读的心智成本，后续新增条件还会进一步提升嵌套层数，不符合「扁平优先」的代码规范，大部分团队的编码规范都会要求条件嵌套层数不超过2层。

方案2（合并条件写法） #

case colorElementEnum.ICON:
        color = colors.buttonText;
        if (isHighContrast && !titleColor && disabled) {
              color = colors.lightGray;
        }
        return color;

这种写法的优势非常明确：

• 可读性更强：读者可以一眼识别出三个判断条件是与逻辑关系，需同时满足才会执行赋值操作，不需要逐层向内看嵌套结构
• 可维护性更高：后续调整条件时直接修改if后的判断语句即可，不需要调整大括号的嵌套结构，降低出错概率
• 符合编码规范：绝大多数主流语言的官方编码规范、业内通用开发指南都推荐将无中间分支逻辑的并列与条件合并写

特殊场景说明 #

如果后续你的逻辑调整，不同的条件层级需要增加独立的处理逻辑，比如isHighContrast为真时还需要额外修改其他变量，那此时拆分嵌套是合理的，不需要强制合并条件。

内容的提问来源于stack exchange，提问作者madu