https://www./blog-zk2Soxs3RE.htm 在做過大量的代碼審查后,我經??吹揭恍┲貜偷腻e誤,以下是糾正這些錯誤的方法。 一:在循環(huán)之前測試數組是否為空 $items = [];// ...if (count($items) > 0) { foreach ($items as $item) { // process on $item ... }} foreach 以及數組函數 (array_* ) 可以處理空數組。
$items = [];// ...foreach ($items as $item) { // process on $item ...} 二:將代碼內容封裝到一個 if 語句匯總 function foo(User $user) { if (!$user->isDisabled()) { // ... // long process // ... }} 這不是 PHP 特有的情況,不過我經常碰到此類情況。你可以通過提前返回來減少縮進。 所有主要方法處于第一個縮進級別 function foo(User $user) { if ($user->isDisabled()) { return; }
// ... // 其他代碼 // ... }
三:多次調用 isset 方法 你可能遇到以下情況: $a = null; $b = null; $c = null; // ...
if (!isset($a) || !isset($b) || !isset($c)) { throw new Exception("undefined variable"); }
// 或者
if (isset($a) && isset($b) && isset($c) { // process with $a, $b et $c }
// 或者
$items = []; //... if (isset($items['user']) && isset($items['user']['id']) { // process with $items['user']['id'] }
我們經常需要檢查變量是否已定義,php 提供了 isset 函數可以用于檢測該變量,而且該函數可以一次接受多個參數,所以一下代碼可能更好: $a = null; $b = null; $c = null; // ...
if (!isset($a, $b, $c)) { throw new Exception("undefined variable"); }
// 或者
if (isset($a, $b, $c)) { // process with $a, $b et $c }
// 或者
$items = []; //... if (isset($items['user'], $items['user']['id'])) { // process with $items['user']['id'] }
四:echo和sprintf方法一起使用 $name = "John Doe";echo sprintf('Bonjour %s', $name); 這段代碼可能在微笑,但是我碰巧寫了一段時間。而且我仍然看到很多!不用結合echo 和sprintf ,我們可以簡單地使用printf 方法。 $name = "John Doe";printf('Bonjour %s', $name); 五:通過組合兩種方法檢查數組中是否存在鍵 $items = [ 'one_key' => 'John', 'search_key' => 'Jane', ];
if (in_array('search_key', array_keys($items))) { // process }
我經常看到的最后一個錯誤是in_array 和array_keys 的聯合使用。所有這些都可以使用array_key_exists 替換。 $items = [ 'one_key' => 'John', 'search_key' => 'Jane', ];
if (array_key_exists('search_key', $items)) { // process }
我們還可以使用isset 來檢查值是否不是null 。 if (isset($items['search_key'])) { // process}
|