根据具体的情况，一般的开发人员往往比优秀的开发人员的效率低 10%~20%。优秀的开发人员的效率更高，因为他们拥有丰富的经验和良好的编程习惯。不良的编程习惯将会影响到效率。本文通过展示一些良好的编程习惯，帮助您成为更优秀的程序员。
这些良好的编程习惯不仅能提高效率，还能让您编写出在应用程序的整个生命周期中易于维护的代码。编写出来的代码可能需要大量的维护；应用程序的维护是一笔很大的开支。养成良好的编程习惯能够提高设计质量（比如模块化），从而使代码更加容易理解，因此维护就更加容易，同时也降低维护成本。
不良的编程习惯会造成代码缺陷，使其难以维护和修改，并且很可能在修改时又引入其他缺陷。以下是 5 个良好的编程习惯，能够帮助 PHP 代码避免这些缺陷：
下一小节将详细介绍这些习惯。
使用良好的命名
使用良好的命名是最重要的编程习惯，因为描述性强的名称让代码更加容易阅读和理解。代码是否好理解取决于是否能在未来维护它。即便代码不带有注释，如果它很容易理解，将大大方便日后的更改。这个习惯的目标是让您编写的代码像书本一样容易阅读和理解。
不良习惯：含糊的或无意义的名称
清单 1 中的代码包含过短的变量名、难以辨认的缩写词，并且方法名不能反映该方法的功能。如果方法名给人的感觉是它应该做这件事情，而实际中它却做另外的事情，这将带来严重的问题，因为它会误导人。
清单 1. 不良习惯：含糊的或无意义的名称
<br><?php<br><br>function getNBDay($d)<BR>{<BR>switch($d) {<BR>case 5:<BR>case 6:<BR>case 7:<BR>return 1;<BR>default:<BR>return ($d + 1);<BR>}<BR>}<br><br>$day = 5;<br><br>$nextDay = getNBDay($day);<br><br>echo ("Next day is: " . $nextDay . "\n");<br><br>?><br>Copy after login |
良好习惯：说明性强并且简洁的名称
清单 2 中的代码体现了良好的编程习惯。新的方法名具有很强的说明性，反映了方法的用途。同样，更改后的变量名也更具说明性。惟一的保持最短的变量是<font face="新宋体">$i</font>，在本清单中，它是一个循环变量。尽管很多人不赞同使用过短的名称，但在循环变量中使用还是可以接受的（甚至有好处），因为它明确表明了代码的功能。
清单 2. 良好习惯：说明性强并且简洁的名称
<br><?php<br><br>define ('MONDAY', 1);<BR>define ('TUESDAY', 2);<BR>define ('WEDNESDAY', 3);<BR>define ('THURSDAY', 4);<BR>define ('FRIDAY', 5);<BR>define ('SATURDAY', 6);<BR>define ('SUNDAY', 7);<br><br>/*<BR>*<BR>* @param $dayOfWeek<BR>* @return int Day of week, with 1 being Monday and so on.<BR>*/<BR>function findNextBusinessDay($dayOfWeek)<BR>{<BR>$nextBusinessDay = $dayOfWeek;<br><br>switch($dayOfWeek) {<BR>case FRIDAY:<BR>case SATURDAY:<BR>case SUNDAY:<BR>$nextBusinessDay = MONDAY;<BR>break;<BR>default:<BR>$nextBusinessDay += 1;<BR>break;<BR>}<br><br>return $nextBusinessDay;<BR>}<br><br>$day = FRIDAY;<br><br>$nextBusDay = findNextBusinessDay($day);<br><br>echo ("Next day is:" . $nextBusDay . "\n");<br><br>?><br>Copy after login |
我们鼓励您将大的条件拆分为一个方法，然后用能够描述该条件的名字命名方法。这个技巧能够提高代码的可读性，并且能够将条件具体化，使之能够被提取甚至重用。如果条件发生变化，更新方法也很容易。因为方法拥有一个有意义的名字，所以它能反映代码的用途，让代码更容易阅读。
分成更小的部分
专心解决一个问题之后再继续编程，这样会让您更轻松。在解决一个紧急的问题时，如果继续编程，会使函数越来越长。从长远来说，这并不是一个问题，但您要记得回过头来将它重构为更小的部分。
重构是个不错的主意，但您应该养成编写更短、功能更集中的代码。短的方法能够在一个窗口中一次看完，并且容易理解。如果方法过长，不能在一个窗口中一次看完，那么它就变得不容易理解，因为您不能快速地从头到尾了解它的整个思路。
构建方法时，您应该养成这样的习惯，让每个方法只完成一件事情。这个习惯很好，因为：首先，如果方法只完成一件事情，那么它就更容易被重用；其次，这样的方法容易测试；第三，这样的方法便于理解和更改。
不良习惯：过长的方法（完成很多件事情）
清单 3 展示了一个很长的函数，其中存在很多问题。它完成很多件事情，因此不够紧凑。它也不便于阅读、调试和测试。它要做的事情包括遍历一个文件、构建一个列表、为每个对象赋值、执行计算等等。
清单 3. 不良习惯：过长的函数
<br><?php<br><br>function writeRssFeed($user)<BR>{<BR>// Get the DB connection information<br><br><BR>// look up the user's preferences...<BR>$link = mysql_connect('mysql_host', 'mysql_user', 'mysql_password')<BR>OR die(mysql_error());<br><br>// Query<BR>$perfsQuery = sprintf("SELECT max_stories FROM user_perfs WHERE user= '%s'",<BR>mysql_real_escape_string($user));<br><br>$result = mysql_query($query, $link);<br><br>$max_stories = 25; // default it to 25;<br><br>if ($row = mysql_fetch_assoc($result)) {<BR>$max_stories = $row['max_stories'];<BR>}<br><br>// go get my data<BR>$perfsQuery = sprintf("SELECT * FROM stories WHERE post_date = '%s'",<BR>mysql_real_escape_string());<br><br>$result = mysql_query($query, $link);<br><br><BR>$feed = "<rss version=\"2.0\">" .<br>"<channel>" .<br>"<title>My Great Feed</title>" .<br>"<link>http://www.example.com/feed.xml</link>" .<br>"<description>The best feed in the world</description>" .<br>"<language>en-us</language>" .<br>"<pubDate>Tue, 20 Oct 2008 10:00:00 GMT</pubDate>" .<br>"<lastBuildDate>Tue, 20 Oct 2008 10:00:00 GMT</lastBuildDate>" .<br>"<docs>http://www.example.com/rss</docs>" .<br>"<generator>MyFeed Generator</generator>" .<br>"<managingEditor>editor@example.com</managingEditor>" .<br>"<webMaster>webmaster@example.com</webMaster>" .<br>"<ttl>5</ttl>";<br><br>// build the feed...<br>while ($row = mysql_fetch_assoc($result)) {<br>$title = $row['title'];<br>$link = $row['link'];<br>$description = $row['description'];<br>$date = $row['date'];<br>$guid = $row['guid'];<br><br>$feed .= "<item>";<br>$feed .= "<title>" . $title . "</title>";<br>$feed .= "<link>" . $link . "</link>";<br>$feed .= "<description> " . $description . "</description>";<br>$feed .= "<pubDate>" . $date . "</pubDate>";<br>$feed .= "<guid>" . $guid . "</guid>";<br>$feed .= "</item>";<br>}<br><br>$feed .= "</rss";<br><br>// write the feed out to the <SPAN style="TEXT-DECORATION: underline"><STRONG><SPAN style="COLOR: #07519a">server</SPAN></STRONG></SPAN>...<BR>echo($feed);<br><br>}<br><br>?><br>Copy after login |
如果多编写几个这样的方法，维护就成了真正的难题了。
良好习惯：易管理、功能专一的方法
清单 4 将原来的方法改写为更加紧凑、易读的方法。在这个示例中，将一个很长的方法分解为几个短方法，并且让每个短方法负责一件事情。这样的代码对将来的重用和测试都是大有裨益的。
清单 4. 良好习惯：易管理、功能专一的方法
<br><?php<br><br>function createRssHeader()<BR>{<BR>return "<rss version=\"2.0\">" .<br>"<channel>" .<br>"<title>My Great Feed</title>" .<br>"<link>http://www.example.com/feed.xml</link>" .<br>"<description>The best feed in the world</description>" .<br>"<language>en-us</language>" .<br>"<pubDate>Tue, 20 Oct 2008 10:00:00 GMT</pubDate>" .<br>"<lastBuildDate>Tue, 20 Oct 2008 10:00:00 GMT</lastBuildDate>" .<br>"<docs>http://www.example.com/rss</docs>" .<br>"<generator>MyFeed Generator</generator>" .<br>"<managingEditor>editor@example.com</managingEditor>" .<br>"<webMaster>webmaster@example.com</webMaster>" .<br>"<ttl>5</ttl>";<br>}<br><br>function createRssFooter()<br>{<br>return "</channel></rss>";<br>}<br><br>function createRssItem($title, $link, $desc, $date, $guid)<br>{<br>$item .= "<item>";<br>$item .= "<title>" . $title . "</title>";<br>$item .= "<link>" . $link . "</link>";<br>$item .= "<description> " . $description . "</description>";<br>$item .= "<pubDate>" . $date . "</pubDate>";<br>$item .= "<guid>" . $guid . "</guid>";<br>$item .= "</item>";<br>return $item;<br>}<br><br>function getUserMaxStories($db_link, $default)<br>{<br>$perfsQuery = sprintf("SELECT max_stories FROM user_perfs WHERE user= '%s'",<br>mysql_real_escape_string($user));<br><br>$result = mysql_query($perfsQuery, $db_link);<br><br>$max_stories = $default;<br><br>if ($row = mysql_fetch_assoc($result)) {<br>$max_stories = $row['max_stories'];<br>}<br><br>return $max_stories;<br>}<br><br>function writeRssFeed($user)<br>{<br>// Get the DB connection information<br>$settings = parse_ini_file("rss_server.ini");<br><br>// look up the user's preferences...<br>$link = mysql_connect($settings['db_host'], $settings['user'],<br>$settings['password']) OR die(mysql_error());<br><br>$max_stories = getUserMaxStories($link, 25);<br><br>// go get my data<br>$newsQuery = sprintf("SELECT * FROM stories WHERE post_date = '%s'",<br>mysql_real_escape_string(time()));<br><br>$result = mysql_query($newsQuery, $link);<br><br>$feed = createRssHeader();<br><br>$i = 0;<br>// build the feed...<br>while ($row = mysql_fetch_assoc($result)) {<br>if ($i < $max_stories) {<BR>$title = $row['title'];<BR>$link = $row['link'];<BR>$description = $row['description'];<BR>$date = $row['date'];<BR>$guid = $row['guid'];<br><br>$feed .= createRssItem($title, $link, $description, $date, $guid);<br><br>$i++;<BR>} else {<BR>break;<BR>}<BR>}<br><br>mysql_close($link);<br><br>$feed .= createRssFooter();<br><br>// write the feed out to the server...<BR>echo($feed);<BR>}<BR>?><br>Copy after login |
将长方法拆分为短方法也是有限制的，过度拆分将适得其反。因此，不要滥用这个良好的习惯。将代码分成大量的片段就像没有拆分长代码一样，都会造成阅读困难。
为代码添加注释
要为代码添加良好的注释有时似乎和编写代码一样难。要了解应该为哪些内容添加注释并不容易，因为我们常常倾向于注释代码当前做的事情。注释代码的目的是不错的主意。在函数的不是很明显的头部代码块中，告诉读者方法的输入和输出，以及方法的最初目标。
注释代码当前做什么是很常见的，但这是不必要的。如果代码很复杂，不得不注释它当前在做什么，这将暗示您应该重写代码，让它更容易理解。学会使用良好的名称和更短的方法，在不提供注释说明其用途的情况下提高代码的可读性。
不良习惯：函数注释过多或不足
清单 5 中的注释仅告诉读者代码在做什么 — 它正在通过一个循环进行迭代或添加一个数字。但它忽略了它为什么做当前的工作。这使维护该代码的人员不知道是否可以安全地更改代码（不引入新缺陷）。
清单 5. 不良习惯：函数注释过多或不足
<br><?php<br><br>class ResultMessage<BR>{<BR>private $severity;<BR>private $message;<br><br>public function __construct($sev, $msg)<BR>{<BR>$this->severity = $sev;<br>$this->message = $msg;<br>}<br><br>public function getSeverity()<br>{<br>return $this->severity;<br>}<br><br>public function setSeverity($severity)<br>{<br>$this->severity = $severity;<br>}<br><br>public function getMessage()<br>{<br>return $this->message;<br>}<br><br>public function setMessage($msg)<br>{<br>$this->message = $msg;<br>}<br>}<br><br>function cntMsgs($messages)<br>{<br>$n = 0;<br>/* iterate through the messages... */<br>foreach($messages as $m) {<br>if ($m->getSeverity() == 'Error') {<br>$n++; // add one to the result;<br>}<br>}<br>return $n;<br>}<br><br>$messages = array(new ResultMessage("Error", "This is an error!"),<br>new ResultMessage("Warning", "This is a warning!"),<br>new ResultMessage("Error", "This is another error!"));<br><br>$errs = cntMsgs($messages);<br><br>echo("There are " . $errs . " errors in the result.\n");<br><br>?><br>Copy after login |
良好习惯：带注释的函数和类
清单 6 中的注释告诉读者类和方法的目的。该注释解释了为什么代码在做当前的工作，这对未来维护代码十分有用。可能需要根据条件变更而修改代码，如果能够轻松了解代码的目的，则修改起来很容易。
清单 6. 良好习惯：带注释的函数和类
<br><?php<BR>/**<BR>* The ResultMessage class holds a message that can be returned<BR>* as a result of a process. The message has a severity and<BR>* message.<BR>*<BR>* @author nagood<BR>*<BR>*/<BR>class ResultMessage<BR>{<BR>private $severity;<BR>private $message;<br><br>/**<BR>* Constructor for the ResultMessage that allows you to assign<BR>* severity and message.<BR>* @param $sev See {@link getSeverity()}<BR>* @param $msg<BR>* @return unknown_type<BR>*/<BR>public function __construct($sev, $msg)<BR>{<BR>$this->severity = $sev;<br>$this->message = $msg;<br>}<br><br>/**<br>* Returns the severity of the message. Should be one<br>* "Information", "Warning", or "Error".<br>* @return string Message severity<br>*/<br>public function getSeverity()<br>{<br>return $this->severity;<br>}<br><br>/**<br>* Sets the severity of the message<br>* @param $severity<br>* @return void<br>*/<br>public function setSeverity($severity)<br>{<br>$this->severity = $severity;<br>}<br><br>public function getMessage()<br>{<br>return $this->message;<br>}<br><br>public function setMessage($msg)<br>{<br>$this->message = $msg;<br>}<br>}<br><br><br>/*<br>* Counts the messages with the given severity in the array<br>* of messages.<br>*<br>* @param $messages An array of ResultMessage<br>* @return int Count of messages with a severity of "Error"<br>*/<br>function countErrors($messages)<br>{<br>$matchingCount = 0;<br>foreach($messages as $m) {<br>if ($m->getSeverity() == "Error") {<br>$matchingCount++;<br>}<br>}<br>return $matchingCount;<br>}<br><br>$messages = array(new ResultMessage("Error", "This is an error!"),<br>new ResultMessage("Warning", "This is a warning!"),<br>new ResultMessage("Error", "This is another error!"));<br><br>$errs = countErrors($messages);<br><br>echo("There are " . $errs . " errors in the result.\n");<br><br>?><br>Copy after login |
处理错误
根据大众的经验，如果要编写健壮的应用程序，错误处理要遵循 80/20 规则：80% 的代码用于处理异常和验证，20% 的代码用于完成实际工作。在编写程序的基本逻辑（happy-path）代码时经常这样做。这意味着编写适用于基本条件的代码，即所有的数据都是可用的，所有的条件符合预期。这样的代码在应用程序的生命周期中可能很脆弱。另一个极端是，甚至需要花大量时间为从未遇到过的条件编写代码。
这一习惯要求您编写足够的错误处理代码，而不是编写对付所有错误的代码，以致代码迟迟不能完成。
不良习惯：根本没有错误处理代码
清单 7 中的代码演示了两个不良习惯。第一，没有检查输入的参数，即使知道处于某些状态的参数会造成方法出现异常。第二，代码调用一个可能抛出异常的方法，但没有处理该异常。当发生问题时，代码的作者或维护该代码的人员只能猜测问题的根源。
清单 7. 不良习惯：不处理错误条件
<br><?php<br><br>// Get the actual name of the<BR>function convertDayOfWeekToName($day)<BR>{<BR>$dayNames = array(<BR>"Sunday",<BR>"Monday",<BR>"Tuesday",<BR>"Wednesday",<BR>"Thursday",<BR>"Friday",<BR>"Saturday");<BR>return $dayNames[$day];<BR>}<br><br>echo("The name of the 0 day is: " . convertDayOfWeekToName(0) . "\n");<BR>echo("The name of the 10 day is: " . convertDayOfWeekToName(10) . "\n");<BR>echo("The name of the 'orange' day is: " . convertDayOfWeekToName('orange') . "\n");<br><br>?><br>Copy after login |
良好习惯：处理异常
清单 8 展示了以有意义的方式抛出和处理异常。额外的错误处理不仅使代码更加健壮，它还提高代码的可读性，使代码更容易理解。处理异常的方式很好地说明了原作者在编写方法时的意图。
清单 8. 良好习惯：处理异常
<br><?php<br><br>/**<BR>* This is the exception thrown if the day of the week is invalid.<BR>* @author nagood<BR>*<BR>*/<BR>class InvalidDayOfWeekException extends Exception { }<br><br>class InvalidDayFormatException extends Exception { }<br><br>/**<BR>* Gets the name of the day given the day in the week. Will<BR>* return an error if the value supplied is out of range.<BR>*<BR>* @param $day<BR>* @return unknown_type<BR>*/<BR>function convertDayOfWeekToName($day)<BR>{<BR>if (! is_numeric($day)) {<BR>throw new InvalidDayFormatException('The value \'' . $day . '\' is an ' .<BR>'invalid format for a day of week.');<BR>}<br><br>if (($day > 6) || ($day < 0)) {<BR>throw new InvalidDayOfWeekException('The day number \'' . $day . '\' is an ' .<BR>'invalid day of the week. Expecting 0-6.');<BR>}<br><br>$dayNames = array(<BR>"Sunday",<BR>"Monday",<BR>"Tuesday",<BR>"Wednesday",<BR>"Thursday",<BR>"Friday",<BR>"Saturday");<BR>return $dayNames[$day];<BR>}<br><br>echo("The name of the 0 day is: " . convertDayOfWeekToName(0) . "\n");<br><br>try {<BR>echo("The name of the 10 day is: " . convertDayOfWeekToName(10) . "\n");<BR>} catch (InvalidDayOfWeekException $e) {<BR>echo ("Encountered error while trying to convert value: " . $e->getMessage() . "\n");<br>}<br><br>try {<br>echo("The name of the 'orange' day is: " . convertDayOfWeekToName('orange') . "\n");<br>} catch (InvalidDayFormatException $e) {<br>echo ("Encountered error while trying to convert value: " . $e->getMessage() . "\n");<br>}<br><br>?><br>Copy after login |
虽然检查参数是一种确认 — 如果您要求参数处于某种状态，这将对使用方法的人很有帮助 — 但是您应该检查它们并抛出有意义的异常：
切忌使用复制粘贴
您可以从其他地方将代码复制粘贴到自己的代码编辑器，但这样做有利也有弊。好的一面是，从一个示例或模板中复制代码能够避免很多错误。不好的一面是，这容易带来大量的类似编程方式。
一定要注意，不要将代码从应用程序的一部分复制粘贴到另一部分。如果您采用这种方式，请停止这个不良的习惯，然后考虑将这段代码重写为可重用的。一般而言，将代码放置到一个地方便于日后的维护，因为这样只需在一个地方更改代码。
不良习惯：类似的代码段
清单 9 给出了几个几乎一样的方法，只是其中的值不同而已。有一些工具可以帮助找到复制粘贴过来的代码（参见参考资料）。
清单 9. 不良习惯：类似的代码段
<br><?php<BR>/**<BR>* Counts the number of messages found in the array of<BR>* ResultMessage with the getSeverity() value of "Error"<BR>*<BR>* @param $messages An array of ResultMessage<BR>* @return unknown_type<BR>*/<BR>function countErrors($messages)<BR>{<BR>$matchingCount = 0;<BR>foreach($messages as $m) {<BR>if ($m->getSeverity() == "Error") {<br>$matchingCount++;<br>}<br>}<br>return $matchingCount;<br>}<br><br>/**<br>* Counts the number of messages found in the array of<br>* ResultMessage with the getSeverity() value of "Warning"<br>*<br>* @param $messages An array of ResultMessage<br>* @return unknown_type<br>*/<br>function countWarnings($messages)<br>{<br>$matchingCount = 0;<br>foreach($messages as $m) {<br>if ($m->getSeverity() == "Warning") {<br>$matchingCount++;<br>}<br>}<br>return $matchingCount;<br>}<br><br>/**<br>* Counts the number of messages found in the array of<br>* ResultMessage with the getSeverity() value of "Information"<br>*<br>* @param $messages An array of ResultMessage<br>* @return unknown_type<br>*/<br>function countInformation($messages)<br>{<br>$matchingCount = 0;<br>foreach($messages as $m) {<br>if ($m->getSeverity() == "Information") {<br>$matchingCount++;<br>}<br>}<br>return $matchingCount;<br>}<br><br>$messages = array(new ResultMessage("Error", "This is an error!"),<br>new ResultMessage("Warning", "This is a warning!"),<br>new ResultMessage("Error", "This is another error!"));<br><br>$errs = countErrors($messages);<br><br>echo("There are " . $errs . " errors in the result.\n");<br>?><br>Copy after login |
良好习惯：带参数的可重用函数
清单 10 展示了修改后的代码，它将复制的代码放到一个方法中。另一个方法也进行了更改，它现在将任务委托给新的方法。构建通用的方法需要花时间设计，并且这样做使您能停下来思考，而不是本能地使用复制粘贴。但有必要进行更改时，对通用的方法投入的时间将得到回报。
清单 10. 良好习惯：带参数的可重用函数
<br><?php<BR>/*<BR>* Counts the messages with the given severity in the array<BR>* of messages.<BR>*<BR>* @param $messages An array of ResultMessage<BR>* @return int Count of messages matching $withSeverity<BR>*/<BR>function countMessages($messages, $withSeverity)<BR>{<BR>$matchingCount = 0;<BR>foreach($messages as $m) {<BR>if ($m->getSeverity() == $withSeverity) {<br>$matchingCount++;<br>}<br>}<br>return $matchingCount;<br>}<br><br>/**<br>* Counts the number of messages found in the array of<br>* ResultMessage with the getSeverity() value of "Error"<br>*<br>* @param $messages An array of ResultMessage<br>* @return unknown_type<br>*/<br>function countErrors($messages)<br>{<br>return countMessages($messages, "Errors");<br>}<br><br>/**<br>* Counts the number of messages found in the array of<br>* ResultMessage with the getSeverity() value of "Warning"<br>*<br>* @param $messages An array of ResultMessage<br>* @return unknown_type<br>*/<br>function countWarnings($messages)<br>{<br>return countMessages($messages, "Warning");<br>}<br><br>/**<br>* Counts the number of messages found in the array of<br>* ResultMessage with the getSeverity() value of "Warning"<br>*<br>* @param $messages An array of ResultMessage<br>* @return unknown_type<br>*/<br>function countInformation($messages)<br>{<br>return countMessages($messages, "Information");<br>}<br><br>$messages = array(new ResultMessage("Error", "This is an error!"),<br>new ResultMessage("Warning", "This is a warning!"),<br>new ResultMessage("Error", "This is another error!"));<br><br>$errs = countErrors($messages);<br><br>echo("There are " . $errs . " errors in the result.\n");<br><br>?><br>Copy after login |
Conclusion
If you develop the good habits discussed in this article when writing PHP code, you will be able to build code that is easy to read, understand, and maintain. Maintainable code built this way will reduce the risk of debugging, fixing, and extending the code.
Using good names and shorter methods can improve the readability of your code. The purpose of commenting code is to facilitate code understanding and expansion. Handling errors appropriately makes your code more robust. Finally, stop using copy-paste to keep your code clean and improve reusability.
![[Web front-end] Node.js quick start](https://img.php.cn/upload/course/000/000/067/662b5d34ba7c0227.png)
