類的大括號在後面 不是另起一行php
變量名首字母小寫 駝峯模式 [a-z][a-zA-Z0-9]*spa
註釋要另起一行,而不是跟在代碼後面,code
移除註釋的代碼段要it
swtich 至少包含3個case 不然就用if吧io
if等不能嵌套超過3次function
類中的方法不能超過20個,超過的話 就拆分把class
移除沒有用的參數變量
移除沒用的變量方法
if必需要跟elseim
if老是跟着大括號
代碼中不要有太多的return
switch 要加default
以下代碼
if (condition) { return true; } else { return false; } //或者 if(a==b){ return true; }else{ return false; } 應該寫成 return condition; return a==b;
//直接返回 function compute_duration_in_milliseconds() { $duration = ((($hours * 60) + $minutes) * 60 + $seconds ) * 1000 ; return $duration; } Compliant Solution function compute_duration_in_milliseconds() { return ((($hours * 60) + $minutes) * 60 + $seconds ) * 1000; }
//出現重複參數 function run() { prepare('action1'); // Non-Compliant - 'action1' is duplicated 3 times execute('action1'); release('action1'); } //正確的作法 ACTION_1 = 'action1'; function run() { prepare(ACTION_1); execute(ACTION_1); release(ACTION_1); }
//布爾值直接判斷 if ($booleanVariable == true) { /* ... */ } if ($booleanVariable != true) { /* ... */ } if ($booleanVariable || false) { /* ... */ } doSomething(!false); Compliant Solution if ($booleanVariable) { /* ... */ } if (!$booleanVariable) { /* ... */ } if ($booleanVariable) { /* ... */ } doSomething(true);