【Code Style】多余判断

           代码中经常充斥着很多多余的判断，不仅仅是增加了代码量，而且还有可能引入bug，同时也增加了理解的时间。

       

 1     public static String get(String key){
 2         Object returnObj = prop.getProperty(key);
 3 
 4         if (returnObj != null) {
 5             return (String)returnObj;
 6         }
 7         else { 
 8             return null;
 9         }
10      }

      上面的代码一共10行，有一个if else的判断，修改这个代码可以分为两个阶段，第一个阶段是使用卫语句（guard condition） ，这样的话，直接就没有else，这样看起来更加直观。

 1     public static String get(String key){
 2         Object returnObj = prop.getProperty(key);
 3 
 4         if (returnObj != null) {
 5             return (String)returnObj;
 6         }
 7         
 8         return null;
 9         
10      }

    下一步改进来说，就是尽量少使用 非语句 ！，不用思维上的转折，理解起来也不容易出现误解。（由原来的!= 变化为 ==）

public static String get(String key){
        Object returnObj = prop.getProperty(key);

        if (returnObj == null) {
            return null;
        }
        return (String)returnObj;
        
     }

    当然终极的代码就是一语抵千行

1        public static String get(String key) {
2         return (String) prop.getProperty(key);
3     }

   什么话都不用说了，大家也都明白了，前面那些都是废话，写了也都是白写，不如这最后的一句话，清晰明了，简单易懂。

 

posted on 2013-04-03 10:39  南郭先生kaka  阅读(1353)  评论(0)  编辑  收藏  举报

导航