PHP 是一门自由度很高的编程语言。它是动态语言,对程序员有很大的宽容度。作为 PHP 程序员,要想让你的代码更有效,需要了解不少的规范。很多年来,我读过很多编程方面的书籍,与很多资深程序员也讨论过代码风格的问题。具体哪条规则来自哪本书或者哪个人,我肯定不会都记得,但是本文(以及接下来的另一篇文章) 表达了我对于如何写出更好的代码的观点:能经得起考验的代码,通常是非常易读和易懂的。这样的代码,别人可以更轻松的查找问题,也可以更简单的复用代码。
降低函数体的复杂度
在方法或者函数体里,尽可能的降低复杂性。相对低一些的复杂性,可以便于别人阅读代码。另外,这样做也可以减少代码出问题的可能性,更易修改,有问题也更易修复。
在函数里减少括号数量
尽可能少的使用 if, elseif, else 和 switch 这些语句。它们会增加更多的括号。这会让代码更难懂、更难测试一些(因为每个括号都需要有测试用例覆盖到)。总是有办法来避免这个问题的。
代理决策 ("命令,不用去查询(Tell, don't ask)")
有的时候 if 语句可以移到另一个对象里,这样会更清晰些。例如:
if($a->somethingIsTrue()) { $a->doSomething(); }
可以改成:
$a->doSomething();
这里,具体的判断由 $a 对象的 doSomething() 方法去做了。我们不需要再为此做更多的考虑,只需要安全的调用 doSomething() 即可。这种方式优雅的遵循了命令,不要去查询原则。我建议你深入了解一下这个原则,当你向一个对象查询信息并且根据这些信息做判断的时候都可以适用这条原则。
使用map
有时可以用 map 语句减少 if, elseif 或 else 的使用,例如:
if($type==='json') { return $jsonDecoder->decode($body);}elseif($type==='xml') { return $xmlDecoder->decode($body);}else{ throw new /LogicException( 'Type "'.$type.'" is not supported' );}
可以精简为:
$decoders= ...;// a map of type (string) to corresponding Decoder objects if(!isset($decoders[$type])) { thrownew/LogicException( 'Type "'.$type.'" is not supported' );}
这样使用 map 的方式也让你的代码遵循扩展开放,关闭修改的原则。
强制类型
很多 if 语句可以通过更严格的使用类型来避免,例如:
if($a instanceof A) { // happy path return $a->someInformation();}elseif($a=== null) { // alternative path return 'default information';}
可以通过强制 $a 使用 A 类型来简化:
return $a->someInformation();
新闻热点
疑难解答