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

[nuanshen]

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

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

[jiangzhizhou]

这两种都可以，但是 for 一般必须被 Stream 替换。

[stillsilly]

我写代码 18 年了，从没用过 continue

[unco020511]

我喜欢 1

[kinghly]

减少嵌套

[clorischan]

个人习惯是循环内用真判断, 满足条件时执行.
函数内使用假判断, 参数不符合要求提前返回.
e.g:
DoSomethings (things)
{
if (things is null) throw
if (things is empty) return

for (thing in things)
{
if (thing is not nothing)
{
//do thing
}
}
}

[lujiaosama]

循环外第一种提前返回, 循环内第二种我不喜欢用 continue

[icylogic]

第一种视觉上嵌套少，但 npath 复杂度仍然是升高的，函数最终还是要控制 npath （或者类似的标准）以适配人脑 cache 大小

[a68UkLHpycW7ImyV]

大部分情况下都是第 1 种

[wu67]

建议看看《重构》, 有对这个场景的说明.
按我个人理解, 是分支少就直接 if else.
if else 过多, 需要打断嵌套地狱, 就使用‘错误先行’, 即提前返回, 后续的代码就能少一层{}

[cheng6563]

哪种{}小用哪种

[evan1024]

一般保持团队一致即可，推荐引入代码风格检查工具

[irisdev]

2 吧，不喜欢用 continue

[changz]

卫语句

[alen0206]

1

[siweipancc]

第二种容易写成无限套娃，看得人头大。
我写的业务代码缩进超过两次就当场重构了。

我的猜想是喜欢第二种的大部分原因来源产品模糊的需求，那怎么避免呢？ if 判断精确判断到产品的原话不就得了，好处是出问题直接甩锅，至于代码维护的相信后人智慧