请问这种判断方法叫什么名字？从哪里出来的？

大家好，我相信不少人都常用这样一种判断方法：

if (status == 1) {
          doSomething();
          return;
      }
     if (status == 2) {
          doSomething();
          return;
      }
      doSomething();

还有另外一种写法是：

if (status == 1) {
          doSomething();
           }
     else if (status == 2) {
          doSomething();
          return;
      }
     else
      doSomething();

     
我常用第一种写法，因为我觉得可读性更好也更简洁，但是现在我们team里有人不知道第一种写法，认为很难懂，我不知道怎么说服他。所以请问大家第一种写法出自哪里？叫什么名字？你们经常用第一种吗？（不讨论if/else可以重构成其他模式)

评论

23 楼 lijie250 2007-05-08  

论情况而定，每个人有自己的风格，没必要去强求别人
但是我经常用第一种，是在简单的判断情况下

22 楼 coolwangyu 2007-05-08  

第二种写法虽然很符合逻辑，但是如果在每个if中都有return的特殊情形下，应该重构为第一种，这样结构更清晰。  当然，如果也可以选择case来使用。
如果，你们doSomething();  不同的话，比如doSomething11();doSomething22();doSomething33();
可以考虑用Strategy继续重构。    

21 楼 Godlikeme 2007-05-08  

1,2,3 magic number啊，
专门看了refactoring  conditional expressions一章，实在是太简单的情况，没有必要区分。

20 楼 yangke250 2007-05-08  

我被强制要求第二种！

19 楼 抛出异常的爱 2007-05-08  

你想想如果加3哪个好加就用哪种。。。。

18 楼 dovecat 2007-05-08  

...反正少用点这类的东东比较好.

17 楼 过儿oO 2007-05-08  

kdekid 写道

如果 status 是 primitive，为什么不用

switch (status) {
  case 1:
    doCase1(); break;
  case 2:
    doCase2(); break;
  default:
    doDefault();
}
return;

如果status不是primitive，可以考虑用对象+策略模式重构。

我一遇到判断条件多了就写这种，省不少代码，哈哈

16 楼 kdekid 2007-05-08  

如果 status 是 primitive，为什么不用

switch (status) {
  case 1:
    doCase1(); break;
  case 2:
    doCase2(); break;
  default:
    doDefault();
}
return;

如果status不是primitive，可以考虑用对象+策略模式重构。

15 楼 calmness 2007-05-08  

个人更倾向于第二种，不过和LZ写的有点出入

if (status == 1) {   
       doSomething();   
} else if (status == 2) {   
        doSomething();    
} else  doSomething();   
return;

第二种的表达更清晰些，像这样的问题最好是有个代码的规范最好。

14 楼 一衣带水 2007-05-08  

我大约记得在 测试驱动开发(实用指南) 一书里面的重构那一章有说到这一种说法的;习惯了用第一种

13 楼 过儿oO 2007-05-08  

一样的
但是这个编码格式简直。。
能不能好好调调，看着也清晰啊

12 楼 hgq0011 2007-05-08  

看个人的思维方式吧，有的人比较容易接受第一种，有的容易接收第二种，我似乎两种都写过。这种逻辑判断最好加上简单明了的注释，不然以后会难维护。

11 楼 zelsa 2007-05-08  

个人用第二种，可读性比较强，至少一看就知道这是一组的。

10 楼 温柔一刀 2007-05-08  

dovecat 写道

刚想起来,第二种其实就是短路法.

我喜欢第一种,第二种太啰唆了

但是不管用那种，遇到用这样代码的情况往往代码是需要改进的

9 楼 Qieqie 2007-05-08  

仅是输入参数验证、条件判断时，我使用-用法1：
if (event == null) {
    return;
    //or throw exception here, just like NullPointerException or IllegalArgumentException!
}
if (event.isExpired()) {
    return;
}
events.add(event);

策略/类别判断时，假如不使用switch，我使用-用法2：
if (event.getSource() == 1) {
    //put some statements, or call another method
    //not return statement here
}
else if (event.getSource == 2) {
    //put some statements, or call another method
    //not return statement here
}

用法3：
使用设计模式(策略配置+模式)：
   略


------------------
怎么使用都没问题，你要书写人习惯，并保持良好的缩进对齐。

8 楼 抛出异常的爱 2007-05-08  

第一种扩展时比
第二种扩展时
限制多一点。。。
个人比较喜欢第二种。

7 楼 BIGN 2007-05-08  

还是让自己去接受别人吧

6 楼 pioneer21th 2007-05-08  

Qieqie 写道

有资深工程师吗，看看他对你们编码有什么规范没有？

能够统一的话不错，不统一也不妨碍
事情还多着了，不必死磕这一点，也不要说服别人

有几位都是有几年工作经验的工程师，我也有几年工作经验，只是他们几个在这个组呆很长时间了，我刚来几个月

5 楼 dovecat 2007-05-08  

刚想起来,第二种其实就是短路法.

4 楼 dovecat 2007-05-08  

...我不太赞成用第一种写法,
不记得是在哪里看到的了.
用
if
if
if
这样一条条的判断对性能有影响,当然你可以忽略不计,除非很大的循环.
用
if
else if
else if
则只命中一个条件,其余的if 条件jvm都会跳过.