提高代码质量的技巧

不要将参数作为变量使用

int Sample( int inputVal ) {  
inputVal = inputVal * CurrentMultiplier( inputVal );  
inputVal = inputVal + CurrentAdder( inputVal );  
...  
return inputVal; <-- 1  
} 

这个例子中，inputVal是一个参数，并对其进行计算，最后返回一个修改后的值。单看这是没问题的，但这样做的弊端是，再要用inputVal去干别的，如果再使用inputVal就出问题了；所以最好的处理是用变量来代替参数：

int Sample( int inputVal ) {  
int workingVal = inputVal;  
workingVal = workingVal * CurrentMultiplier( workingVal );  
workingVal = workingVal + CurrentAdder( workingVal );  
...  
return workingVal;<-- 1  
}  

只要一个return语句

public function foo() {  
    if (true) {  
        return true;  
    } else {  
        return false;  
    }  
}  

改为

public function foo() {  
    $flag = true;  
    if (true) {  
        $flag = true;  
    } else {  
        $flag = false;  
    }  
    return $flag;  
}  

尽可能的把变量定义在靠近使用它的地方

int foo = 3;  
int bar = 5;  
// 一大段使用"bar"的代码，  
// 但没用到"foo"  
// ...     
baz(foo); 

可以改为

int bar = 5;  
// 一大段使用"bar"的代码，  
// 但没用到"foo"  
// ...         
int foo = 3;  
baz(foo); 

让代码更简洁

如果你发现写了大量的代码来解决一个简单的问题，你很可能做错了。下面的boolean用法是一个很好的例子：

if (numMines > 0) { 
enabled=true; 
} 
else { 
enabled=false; 
}

简化为

enabled = numMines > 0;

避免不必要的开销

for($i=0;$i<count($array);$i++){...} 

改为

$count=count($array);
for($i=0;$i<$count;$i++){...} 

---

本文参考http://blog.csdn.net/qinglianluan/article/details/14450653