据说优秀的代码没有 else？

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

The specified image does not exist

This topic created in 3122 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

❮

❯

natscat

Apr 19, 2018

101

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

lightening

Apr 19, 2018

102

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

floyda

Apr 19, 2018

103

优秀的代码没有 if 的

afpro

Apr 19, 2018

104

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

dayoushen

Apr 19, 2018

105

支持 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 ;

hslx111

Apr 19, 2018

106

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

awing

Apr 19, 2018

107

是时候祭出这段代码了
```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:
```

MasterC

Apr 19, 2018

108

@awing #107 为什么不用 switch case

guog

Apr 19, 2018 via Android

109

@MasterC Python🌚

hansight2018

Apr 19, 2018 via Android

110

refer to "guard clause"

bobuick

Apr 19, 2018

111

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

MeteorCat

Apr 19, 2018

112

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

undefined8

Apr 19, 2018

113

优秀程序员 tab 是 8 个空格

yanyuechuixue

Apr 19, 2018

114

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

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

Chingim

Apr 20, 2018

115

求比较, 求更优雅的方案

akira

Apr 20, 2018

116

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

Chingim

Apr 20, 2018 via Android

117

@akira show your code

elvodn

Apr 20, 2018

118

@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")
}
}
```

v2gba

Apr 20, 2018

119

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

akira

Apr 20, 2018

120

@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 , 这种过于技巧性的代码不建议在日常工作使用。其实你的第二种写法就非常好了

akira

Apr 20, 2018

121

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

elvodn

Apr 20, 2018

122

@akira go 里 switch 是默认 break 的

goinghugh

Apr 20, 2018

123

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

zhouquan03

Apr 20, 2018

124

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

EanCuznaivy

Apr 20, 2018

125

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

loveour

Apr 20, 2018

126

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

pmispig

Apr 20, 2018

127

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

hatcloud

Apr 20, 2018

128

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