讨论一个业务代码判断的写法问题，这两种写法你们更喜欢那种？

nuanshen · 4 小时 49 分钟前 · 1608 次点击

//写法 1 ，不满足条件跳过业务处理
for(){
  if(!condition){
    continue;
  }
  //businessCode
}

//写法 2,满足条件执行业务处理
for(){
  if(condition){
    //businessCode
  }
}

37 条回复 • 2022-06-17 17:25:14 +08:00

RedBeanIce 4 小时 48 分钟前 via iPhone

分场景。绝大部分提前返回。1

May725 4 小时 46 分钟前

更倾向于提前返回，避免嵌套

TWorldIsNButThis 4 小时 34 分钟前 via iPhone

一般不用 for
使用语义化的迭代工具
map filter

yazinnnn 4 小时 31 分钟前 3

collection.filter { } //过滤满足条件
.map { } //执行业务,返回业务结果
.reduce { } //聚合业务结果

yolee599 4 小时 27 分钟前 via Android

提前 return ，可以提高运行效率，代码也美观

chendy 4 小时 18 分钟前

看内容多长……
如果不长的话就套在写法 1 里
如果比较长就写成写法 2 提前退出

但是还有一种情况是好几个 contine / break 的（ sonar 会警告
这种情况一般就抽个方法，在方法里 return

newaccount 4 小时 18 分钟前

看情况。business 部分行数不多用 2 ，多了用 1 。但用 1 的时候可能会声明变量以去掉 not 判断，因为快速浏览时感叹号不明显导致漏看。如果感叹号左右增加空格提高可读性又得配置 checkstyle ，麻烦

dcsuibian 4 小时 17 分钟前

喜欢第 1 种，少一层嵌套。

AlekoShen 3 小时 53 分钟前

一般 1 不过如果判断条件复杂会用 3,4 楼的写法

VeryZero 3 小时 51 分钟前

大多数情况肯定第一种好啊，可以显著降低阅读时的心智负担

DrakeXiang 3 小时 50 分钟前

我一般都是 2 ，除非业务逻辑比较长，然后只有一个 condition 的情况下可能会选择 1

Detector 3 小时 50 分钟前

第一种，不管是写起来是读起来，对我而言都舒服太多

ericls 3 小时 47 分钟前 via iPhone

@yazinnnn 有些语言这样写会产生实际的性能问题虽然看起来比较舒服

我个人喜欢第一种但是你要写成别的我也没意见

MakHoCheung 3 小时 13 分钟前

第一种是一个模式，忘了叫啥，Swift 的 guard 语句就是这样

xuelu520 3 小时 10 分钟前

写法一吧，
第二种如果业务代码很多，很容易造成多重嵌套。

buried 3 小时 5 分钟前

写法一，避免太多的缩进

hidemyself 3 小时 4 分钟前

第一种，卫语句

SteveWoo 3 小时 3 分钟前

看这个模块重不重要。如果出错了风险大不大。一般情况第一种。
重要场景，个人一定会 if 和 else 成对出现来实现，宁愿有一堆 if else 嵌套
for(){
if {
if {
} else{
}
} else {
if {
} else{
// 根据算法多次与产品确认，这个场景就是啥也不做
}
}
}