提高代码质量的技巧



不要将参数作为变量使用

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