如何优雅合并多段if语句?保持逻辑不变的代码优化方案咨询
合并分散if语句的方案(保持原逻辑不变)
嘿,这个问题很好解决!首先我们先拆解原代码的核心逻辑,再针对性地合并:
- 变量
c的赋值只和a == Humans::Men有关,和b的取值完全独立 - 变量
d、e的赋值由b的枚举值决定,且b的枚举值是互斥的(一个变量不可能同时等于Animals::Cat、Dog或Elephant)
基于这个逻辑,我们可以用两种清晰的方式合并代码:
方案1:用else if合并互斥的b判断
因为b的枚举值互斥,所以用else if替代独立的if,既减少冗余判断,又完全保留原逻辑:
// 处理a的独立条件,和原逻辑一致 if (a == Humans::Men) { c = 1; } // 合并b的互斥判断 if (b == Animals::Cat) { d = 1; e = 1; } else if (b == Animals::Dog) { d = 1; } else if (b == Animals::Elephant) { e = 1; }
方案2:用switch语句处理枚举(更推荐)
对于枚举类型的判断,switch语句的可读性和性能都更优,同样完全匹配原逻辑:
// 处理a的独立条件 if (a == Humans::Men) { c = 1; } // 用switch处理b的枚举判断 switch (b) { case Animals::Cat: d = 1; e = 1; break; case Animals::Dog: d = 1; break; case Animals::Elephant: e = 1; break; // 原逻辑未处理其他枚举值,所以default分支可以留空或省略 default: break; }
额外优化(可选)
如果想让代码更紧凑,c的赋值可以用三元运算符简化(但可读性略降):
c = (a == Humans::Men) ? 1 : 0; // 原逻辑中c初始为0,非Men时保持0,等价
内容的提问来源于stack exchange,提问作者Math_Enthusiast
相关产品推荐
相关产品推荐

