前两天 Code Review 的时候被指出了这个问题。
如题，是
bool func() {
if (xxx) return false;
if (xxx) return true;
xxx;
return true;
}
还是
bool func() {
if (xxx) {
if (xxx) return false;
else return true;
} else {
xxx;
return true;
}
}
好像前者一般比较清爽，缩进层次也少。我只看过王垠在他的博客里强烈支持过第二种。
 |
blueskitJul 3, 2018 via Android
1
王垠为什么支持呢？
|
 |
2
你给的两个例子，逻辑都不一样。
至于使用那种形式，还是看逻辑啊。 如果是并列条件，使用第一种，如果是前提条件，使用前套逻辑。这样的代码符合人类的思维，好懂。 王垠又不是啥权威，写文章很多，无论是工业产品，还是开源项目，并没有什么建树，并不觉得他说什么就有道理。 我觉得除了 80 列是个铁律，别的都不是完全的定死的。 |
 |
3
Guard 有时候会因为逻辑 上没思考全导致部分条件没有覆盖而进入之后的代码流程，强制 if else 2 分能确保覆盖所有情况，养成习惯之后能保证处理到所有 case，不管是有意还是无意。再加上王垠用的是 java，用 if else ide 能提示有些 case 没处理
用这种习惯之后几乎没有负担的被动技能来减少代码出错的可能性其实也是比较务实的做法 |
 |
1010011010Jul 3, 2018
4
我喜欢第一种，过程比较清晰
|
 |
sugarsaltJul 3, 2018
5
case 多的话，switch
|
 |
lighteningJul 3, 2018
6
代码首先要反映程序员的逻辑，怎么想就怎么写。
两种常见情景： 1 ）写到这里，哎呀这个变量如果是 b 就不对了，先 return 掉 2 ）写到这里，发现有以下两种可能的情况 显然 1 ）应该用 early return，2 ）应该用 if else |
 |
jatesunJul 3, 2018
9
第一种，第二种可读性不高
|
 |
jorneyrJul 3, 2018
10
bool func() {
if (xxx) { return false }; if (xxx) { return true }; xxx; return true; } |
 |
Nick66Jul 3, 2018
11
一会简写一会不简写最恶心
|
 |
12
if (xxx) return false;
else return true; 这个代码太骚了。 |
|
VVVVVEXJul 3, 2018
13
GCC LLVM
Linux *BSD QEMU Python Django node Vim Firefox Chrome React Rust 这些项目都是遵循 80 列，有比 90 后年龄大的，也有比 90 后年轻的。 别的项目我不太清楚，gayhub 上的总会有很多超过 80 列的个人项目，数量上一定是取胜的。 |
 |
crayygyJul 3, 2018 via iPhone
14
就你提供的代码风格来看，这第二种写出来会被打的...不管哪种语言，if else 还是尽量都应该括号保证有合理的 scope，也方便以后添加逻辑，除非是保护性的代码，if ==null return 这种我觉得不需要括号，直接一行话搞定。
|
 |
auinJul 3, 2018
15
第二种是真的神奇，一会有括号，一会没括号，而且逻辑和第一个也不一样，我更喜欢 #10 的写法
|
 |
sagaxuJul 3, 2018 via Android
16
10 个变量二分就是 1024 种组合，来个 10 层嵌套吗？
形式上的完整覆盖，和逻辑上的完整覆盖，目的是哪个？ |
 |
chenyu8674Jul 3, 2018
17
跟代码没关系，本质上还是看写代码的人怎么理解这段判断关系
|
 |
zarteJul 3, 2018
18
第二种，
只返回一个的时候第一种 |
 |
qY3209HZitEb5ZtyJul 3, 2018
20
```
bool func() { if (xxx) { return xxx ? false : true; } xxx; return true; } ``` |
 |
ChiehJul 3, 2018 via Android
21
原则上用第二种，如果较短的 if else 两行能放下的话就用第一种
|
 |
22
|
 |
akiraJul 3, 2018
23
前面的 嵌套越少越好
|
 |
ConradGJul 3, 2018
24
分支多，或分支条件琐碎的，多用第一种。
分支少，尤其二择多用第二种。 |
 |
cuzfinalJul 3, 2018
25
第一种避免箭头形代码。
|
 |
newtype0092Jul 3, 2018
26
第二种逻辑更完整，不过可读性差，需要配合 ide 检查，个人觉得这种更适合自动生成而不是手动去写。
手写代码还是用第一种更合理，逻辑更清楚，而且一定程度上能反应写代码的思路。 有些人觉得第二种写法这样的冗余工作有助于减少出错，但我认为如果手写这种代码的话眼花出错的概率更高，而且看的头大干扰了自己的思路更容易写错。 人的思考还是应该求巧，求全是靠各种重复冗余来保证，这是机器该做的，与其不辞辛劳去做冗余工作，不如造一个机器帮自己偷懒。 |
 |
AltairTJul 3, 2018 via iPhone
29
王垠的文章我看过，貌似不是你第二种那样。他明明说了类似意思的话 1 避免箭头代码的方法是将条件反转，提前返回（就是你说的防御代码）;2 为了防止遗漏情况，有 if 就必须有 else。
这两条也不矛盾啊。还是你没理解清楚 |
 |
pynixJul 3, 2018
30
前期可能会先走逻辑(约定参数都有效)，，，，后期会慢慢加参数验证。。。
|
 |
morethanseanJul 3, 2018
31
第一种，当年写 WE 养成的习惯 2333333
|
 |
luozicJul 4, 2018
32
直接 if，如果分支切分覆盖不对，还不是一样坑。
|
1
Select Language