1.if层次过多。

举一个很有趣的例子,假如世界上只有两种烤鸭,在北京的叫做北京烤鸭,不在北京的叫做非北京烤鸭,写烂代码的程序员思路如下:

如果烤鸭不在地球,它一定不是北京烤鸭;
如果烤鸭不在陆地,它一定不是北京烤鸭;
如果烤鸭不在亚洲,它一定不是北京烤鸭;
如果烤鸭不在中国,它一定不是北京烤鸭;
如果烤鸭不在北京,它一定不是北京烤鸭;
否则,这是北京烤鸭。

伪代码如下:

if(!烤鸭在地球){
输出非北京烤鸭。
}else{
if(!烤鸭在陆地){
输出非北京烤鸭。
}else{
if(!烤鸭在亚洲){
输出非北京烤鸭。
}else{
if(!烤鸭在亚洲){
输出非北京烤鸭。
}else{
if(!烤鸭在中国){
输出非北京烤鸭。
}else{
if(!烤鸭在北京){
输出非北京烤鸭。
}else{
输出北京烤鸭。
}

好把,如果真有哪个程序员是这样写代码的,那么我真的很想一巴掌把它拍到火星去。不要这样嵌套好吗?就算你真的是这么想,也可以这样写:

if(!烤鸭在地球){
输出非北京烤鸭。
}else if(!烤鸭在陆地){
输出非北京烤鸭。
}else if(!烤鸭在亚洲){
输出非北京烤鸭。
}else if(!烤鸭在中国){
输出非北京烤鸭。
}else if(!烤鸭在北京){
输出非北京烤鸭。
}else{
输出北京烤鸭。
}

这个时候你可以发现原来前四个条件的内容都是相同的,那么可以这样写:

if(!烤鸭在地球
||!烤鸭在陆地
||!烤鸭在亚洲
||!烤鸭在中国
||!烤鸭在北京){
输出非北京烤鸭。
}else{
输出北京烤鸭。
}

当你发现前面if的条件过分臃肿的时候,可以考虑反过来会不会更容易
最终变成:

if(烤鸭在北京){
输出北京烤鸭。
}else{
输出非北京烤鸭。
}

可见,逻辑简化可弥补思维的不足。
这里有两个公式:
公式1:

if(条件1){
//做事情A
}else if(条件2){
//做事情A

可以转换为:

if(条件1 || 条件2){
//做事情A
if(条件1){
if(条件2){
//做事情A

可以转换为:

if(条件1 && 条件2){
//做事情A

如果条件太多,且实在不知道怎么合并,可以把条件单独放到一个函数:

public void doSomething(){
if(isRoastDuckNotInBeijing()){
输出非北京烤鸭。
}else{
输出北京烤鸭。
}
}

public boolean isRoastDuckNotInBeijing(){
return

注意,如果条件可读性强:比如sex == “man”就不要把条件放到单独函数。

2.学会使用return降低层次

public void doSomething(){
if(firstNum!=0){
if(secondNum!=0){
//doSomething

如果改为:

public void doSomething(){
if(firstNum == 0)
return;
if(secondNum == 0)
return;
//doSomething

看起来好多了。

3.学会使用函数分割代码

我是在上大学的时候学会写代码的。有一点解不开的疑惑:为什么要写函数?我把代码从头写到尾逻辑不是更加通顺吗?每个教代码的老师,总是在开始的时候给我们讲什么是变量,什么是函数,怎么用?我好想说一句:喂,我还没决定要用函数呢!

讲这部分之前,我先讲讲这个为什么要写函数,讲为什么要写函数前,我不得不提提二分查找法

举个例子:
有十六张扑克牌按照编号从小的顺序排列,我让你找出编号排名第11的,你怎么找呢?

按照顺序一本本找,你要找11次,如果我找的是最后一张,需要找16次(当然这里我们不讨论倒着找)

使用二分查找:分成一半,前面是1-8,后面是9-16,然后再分成一半9-12和13-16,然后再分成一半9-10和11-12,然后我找到了。我找了4次。发现了吗?速度比顺序查找要快,而且我最多只需要找4次。当查找的范围越大,二分查找相对顺序查找的优势就越明显,平均速度也更快。

好了,懂了二分查找,我们再来弄清楚为什么要写函数
举个例子:假设我要写100行代码,伪代码如下:

public void 写100行代码(){
写1-50行代码();
写51-100行代码();
}

public void 写1-50行代码(){
写1-25行代码();
写26-50行代码();
}

public void 写51-100行代码(){
写51-75行代码();
写75-100行代码();
}

这样如果我找写第70行代码写什么,只要使用二分法的原理:先找写100行代码(),然后找写51-100行代码();以此类推。

所以我们写函数的目的就在此:把代码分割成小的部分,更容易查找。

这样做有个前提:就是你的函数名必须有意义并且表达准确,否则就更浪费时间,如上文的写1-50行代码()写51-100行代码()函数名调换,内容不变,识别起来难度就会变大,你先找写1-50行代码(),发现它其实是写后面50行代码,这会让你在逻辑上纠结一段时间,所以再次强调:函数名一定要有意义

另外还有一点,分割项目不要一次分的太细比如上面的例子,我也可以这样写(为了显示效果,我没写省略号,就一行行写了):

public void 写100行代码(){
写1-5行代码();
写6-10行代码();
写11-15行代码();
写16-20行代码();
写21-25行代码();
写26-30行代码();
写31-35行代码();
写35-40行代码();
写41-45行代码();
写46-50行代码();
写51-55行代码();
写56-60行代码();
写61-65行代码();
写66-70行代码();
写71-75行代码();
写76-80行代码();
写81-85行代码();
写86-90行代码();
写91-95行代码();
写96-100行代码();
}

发现了吗?这段代码的可读性明显变差了。所以一般情况下这种分割函数(里面每一句都调用其他函数),分割函数代码不要超过10句,最好不要超过5句

和分割函数相对应的,还有一种我称作逻辑函数,这种函数都是做具体的工作,一般不要超过20行,如下:

public void

特殊情况1:有一堆的参数赋值/传递:
比如录入个人信息:有身高,体重,爱好,优点,缺点,性别,父名,母名…这种可能会有很多数据,可能远远超过20行,那么,就把这部分代码单独分割:

public void outputUserInfo(){
System.out.println(getUserInfo());
}

public UserInfo getUserInfo(){
UserInfo userInfo = new UserInfo();
userInfo.setHigh(10);
userInfo.setHeavy(1000);
userInfo.setHobby("泡妞");
userInfo.setAdvantage("很帅");
userInfo.setDisadvantage("我是世界上最帅的!");
userInfo.setFatherName("爸爸");
userInfo.setMotherName("妈妈");
//...此处省略万字
return

大概是这样,其实真的无法分割吗?并没有这样说法。上面的内容可分为:身体信息,偏好信息和亲友信息,于是上面的一个UserInfo就可以分割为BodyInfo,LoveInfo和RelativesInfo,于是又可以保持在20行之内了
ヾ(◍°∇°◍)ノ゙

特殊情况2:逻辑分支,回调和异常处理
逻辑分支例子:

if(!烤鸭在地球){
输出非北京烤鸭。
输出当前地点。
}else if(!烤鸭在陆地){
输出非北京烤鸭。
输出当前地点。
}else if(!烤鸭在亚洲){
输出非北京烤鸭。
输出当前地点。
}else if(!烤鸭在中国){
输出非北京烤鸭。
输出当前地点。
}else if(!烤鸭在北京){
输出非北京烤鸭。
输出当前地点。
}else{
输出北京烤鸭。
}

回调例子:

public void clickButton(){
button.setOnClickListener(
new OnClickListener(){
@Overriade
public void onClick(View v){
输出当前地点。
输出北京烤鸭。
}
}
);
}

异常处理例子:

public void methodName(){
try{
输出北京烤鸭。
输出当前地点。
}catch(Exception e){
e.printStackTrace();
}
}

上面三种情况:选择分支,回调和异常处理都会导致代码臃肿,可读性变差,我通常都会把里面的部分单独列到新的:
第一种:
合成为:输出结果(boolean isRoastDuckInBeijing,String address)
因为条件分支往往好起名字,所以我通常会这样写

第二种:
clickButtonEvent(){}
因为我不能保证除了输出北京烤鸭信息外是否会做些别的什么,通常这部分的修改频率较高

第三种:
public void methodNameThrowException() throw Exception{}
通常我会在原名称后加上ThrowException提醒自己抛出异常,这样既显示了区别也降低层次,看起来更舒服了:

public void methodName(){
try{
methodNameThrowException();
}catch(Exception e){
e.printStackTrace();
}
}

public void methodNameThrowException() throw

总结:

1.学会用if…else if代替多层次if…else
2.学会用逻辑运算符&&和||代替多层if和多分支if … else if
3.学会用return降低层次
4.函数参数要尽量少,太多的把它合成一个类
5.函数名要易读
6.函数要尽量短小
7.有多层次或多分支的,使用新的函数代替代码块。