编程感想:
由于工作需要,今天review了一个同事的代码,发现一个函数中竟然出现了10几个if else分支,顺着逻辑看下去好像也无可厚非,每一处逻辑判断都是必须的。但总觉得有些不对劲,这样的代码显得特别繁琐,以批判的心态再次审视代码,终于发现了问题所在:特殊情况处理
在我们编程的过程中,经常需要对值为null的情况特殊处理,有时候甚至会为了一个特殊情况使代码增加一倍。每当这个时候你是否想过null真的是特殊情况吗,其实你也可以认为他是存在的只是值为空,只需要在以后的逻辑中需要的地方处理一下。
举个例子来说明这个问题:有两个集合A和B ,其中A是有序的,需要对B进行排序然后放在A的后面。
A和B都是从别的变量赋值得到,很有可能得到的是null。
极端的情况你可以这样来做
if(null==A)
{
if(null==B){}
else{}
.......
return ret;
}
else
{
if(null==B)
{}
else{}
....
return ret;
}
按照特殊逻辑一般化的方法,我们可以这样来做:首先判断A,B是否为null,如果为null我们就给他赋值一个空集合,比如数组:
if(null==A)A=array();
if(null==B)B=array();
sort(B);
merge(A,B);
这样做代码是不是简洁了许多呢!