据说优秀的代码没有 else？

• 请不要在回答技术问题时复制粘贴 AI 生成的内容

This topic created in 2980 days ago, the information mentioned may be changed or developed.

多个 if else 嵌套的代码估计大家都见过。。。头疼！

但是其实很多 if else 可以简化成一个 if，不需要 else，举个例子：

$books = getByAFunc()
if($books) {
   do something ...
   return something
}else {
   do something ....
   return something
}

看上去没什么毛病，但是根据业务需求，大部分是可以这么写：

if($books) {
     do something...
     return something
}
do something...
return something

这只是举个例子，很多时候满足条件的只有一种情况，那我们就可以针对这一种情况处理，其他情况直接 return ！

还要一种写法是先处理可能出现的异常错误，然后最后执行下来的就是正确情况！

147 replies • 2018-04-20 21:07:11 +08:00

1 2

❮

❯

101

natscat

Apr 19, 2018

我觉得程序里面应该把每一种可能的执行路径都写清楚
所以我是支持 if...else...这种的

102

lightening

Apr 19, 2018

看情况。如果 if 是作为过滤掉一些非正常情况，那么早早 return 掉是可以接受。如果本来就是两种需要正常处理的情况，当然是 if ... else 更清晰。

103

floyda

Apr 19, 2018

优秀的代码没有 if 的

104

afpro

Apr 19, 2018

优秀的代码不拘泥于有没有 if 这种细节整体的可读性更重要

105

dayoushen

Apr 19, 2018

支持 if-else 匹配，但不支持 if-else if-else if ..-else
如下：
char *p = new char[10];
if(condition == true)
{
do_something()
delete []p;
p = NULL;
return some;
}
do_else();
delete []p;
p = NULL;
return some;
你要释放两次指针，只有 if 容易内存泄漏。完美 if-else
bool bRet;
char *p = new char[10];
if(condition == true)
{
do_something()
bRet = true;
}
else
{
do_else()
bRet = false;
}
do_else();
delete []p;
p = NULL;
return bRet ;

106

hslx111

Apr 19, 2018

else 可以有，但是多层嵌套不应该

107

awing

Apr 19, 2018

是时候祭出这段代码了
```python
elif (rootKey == "mode"):
# Capitalize it just for good measure
value = value.upper()
if (value == "LOITER"):
self.vehicleWrapper.vehicle.mode = dronekit.VehicleMode("LOITER")
elif (value == "STABILIZE"):
self.vehicleWrapper.vehicle.mode = dronekit.VehicleMode("STABILIZE")
elif (value == "ALT_HOLD"):
self.vehicleWrapper.vehicle.mode = dronekit.VehicleMode("ALT_HOLD")
elif (value == "GUIDED"):
self.vehicleWrapper.vehicle.mode = dronekit.VehicleMode("GUIDED")
elif (value == "AUTO"):
self.vehicleWrapper.vehicle.mode = dronekit.VehicleMode("AUTO")
elif (value == "RTL"):
self.vehicleWrapper.vehicle.mode = dronekit.VehicleMode("RTL")
elif (value == "BRAKE"):
self.vehicleWrapper.vehicle.mode = dronekit.VehicleMode("BRAKE")
else:
```

108

MasterC

Apr 19, 2018

@awing #107 为什么不用 switch case

109

guog

Apr 19, 2018 via Android

@MasterC Python🌚

110

hansight2018

Apr 19, 2018 via Android

refer to "guard clause"

111

bobuick

Apr 19, 2018

什么鬼。
反了吧，优秀的代码应该要有 else，如果没 else 说明这个 if 可以再抽象到一个函数里去。如果是简单的 if，应该是存在三元表达式的方式表达。

112

MeteorCat

Apr 19, 2018

涉及业务多的时候,你就知道了

113

undefined8

Apr 19, 2018

优秀程序员 tab 是 8 个空格

114

yanyuechuixue

Apr 19, 2018

认同一部分吧, 如果是发生了某条件需要做一件事, 而不发生的时候不需要做, 那么最好不用 else 只用 if, 提升效率. 而且最好 if 的条件为真的概率要更大.

如果需要分开处理, 那还是用 else 吧. 避免不了的.(当然你要说用两个 if 的话我没话说, 但效率会下降吧?我猜.)

115

Chingim

Apr 20, 2018

求比较, 求更优雅的方案

116

akira

Apr 20, 2018

@Chingim 根据与或关系应该是可以获得 0-3，4 个数字

117

Chingim

Apr 20, 2018 via Android

@akira show your code

118

elvodn

Apr 20, 2018

@Chingim 错误排除用 early return，双分支 if...else... 多分支应该用 switch 吧
``` go
func getPlanDone(isSunny, isHappy bool) {
switch {
case isSunny && isHappy:
println("plan-A-step-1")
println("plan-A-step-2")
case isSunny:
println("plan-B-step-1")
println("plan-B-step-2")
case isHappy:
println("plan-C-step-1")
println("plan-C-step-2")
default:
println("plan-D-step-1")
println("plan-D-step-2")
}
}
```

119

v2gba

Apr 20, 2018

优秀的代码没有 if
优秀的代码只有优秀的客户才配用只有一种可能没有任何 if 没有任何异常如果发生了那么就是客户不够优秀

120

akira

Apr 20, 2018

@Chingim 满足你的愿望。

function getPlanDone(isSunny, isHappy)
{
var steps = ["D","B","C","A"];
var c = isSunny + isHappy * 2;

console.log( 'plan-' + steps[c] + '-step-1' );
console.log( 'plan-' + steps[c] + '-step-2' );
}

getPlanDone( true, true); //A
getPlanDone( true, false); //B
getPlanDone( false, true); //C
getPlanDone( false, false); //D

PS , 这种过于技巧性的代码不建议在日常工作使用。其实你的第二种写法就非常好了

121

akira

Apr 20, 2018

@elvodn 没看过 go，好奇的问下，若 isSunny, isHappy 都为真，那么这个 switch 的前 3 个表达式的值都是真。这种情况下 switch 是怎么区分的呢

122

elvodn

Apr 20, 2018

@akira go 里 switch 是默认 break 的

123

goinghugh

Apr 20, 2018

@awing python 可以用字典的方式实现 switch case

124

zhouquan03

Apr 20, 2018

强烈反对写 if 不写 else
LZ 多写代码就会发现漏了 else 逻辑多么可怕

125

EanCuznaivy

Apr 20, 2018

有些 IDE 会提示你有不必要的 else ……

126

loveour

Apr 20, 2018

@zhouquan03
赞同。不明白非得不要 else 意义何在。。

127

pmispig

Apr 20, 2018

优秀的代码必须有 else,证明你有考虑到这种情况。

128

hatcloud

Apr 20, 2018

@loveour
其实不是反对 else 而是 else 带来的嵌套问题，就一层的话我觉得没什么问题。
我的理解是多层 if-else 很容易使代码呈现树状结构，很不易于他人阅读，而事实上大部分情况下，多层嵌套的 if-else 都可以通过 if-return 来替换成比较线性的代码。
@Chingim #115 的例子就很好