在做過大量的代碼審查后，我經常看到一些重復的錯誤，以下是糾正這些錯誤的方法，
在回圈之前測驗陣列是否為空
$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
}
轉載請註明出處，本文鏈接：https://www.uj5u.com/houduan/204716.html
標籤：PHP