【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  阅读(1349)  评论(0编辑  收藏  举报

导航