轉(zhuǎn)載自 PHP 論壇:https://learnku.com/php/t/49583
在做過(guò)大量的代碼審查后,我經(jīng)常看到一些重復(fù)的錯(cuò)誤,以下是糾正這些錯(cuò)誤的方法括袒。
在循環(huán)之前測(cè)試數(shù)組是否為空
$items = [];
// ...
if (count($items) > 0) {
foreach ($items as $item) {
// process on $item ...
}
}
foreach
以及數(shù)組函數(shù) (array_*
) 可以處理空數(shù)組。
- 不需要先進(jìn)行測(cè)試
- 可減少一層縮進(jìn)
$items = [];
// ...
foreach ($items as $item) {
// process on $item ...
}
將代碼內(nèi)容封裝到一個(gè) if 語(yǔ)句匯總
function foo(User $user) {
if (!$user->isDisabled()) {
// ...
// long process
// ...
}
}
這不是 PHP 特有的情況稿茉,不過(guò)我經(jīng)常碰到此類情況锹锰。你可以通過(guò)提前返回來(lái)減少縮進(jìn)。
所有主要方法處于第一個(gè)縮進(jìn)級(jí)別
function foo(User $user) {
if ($user->isDisabled()) {
return;
}
// ...
// 其他代碼
// ...
}
多次調(diào)用 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']
}
我們經(jīng)常需要檢查變量是否已定義漓库,php 提供了 isset 函數(shù)可以用于檢測(cè)該變量城须,而且該函數(shù)可以一次接受多個(gè)參數(shù),所以一下代碼可能更好:
$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);
看到這段代碼你可能會(huì)想笑米苹,不過(guò)我的確這樣寫(xiě)了一段時(shí)間糕伐,而且我仍然會(huì)看到很多這樣寫(xiě)的!其實(shí)echo
和 sprintf
并不需同時(shí)使用蘸嘶,printf
就可以完全實(shí)現(xiàn)打印功能良瞧。
$name = "John Doe";
printf('Bonjour %s', $name);
通過(guò)組合兩種方法檢查數(shù)組中是否存在鍵
$items = [
'one_key' => 'John',
'search_key' => 'Jane',
];
if (in_array('search_key', array_keys($items))) {
// process
}
我經(jīng)常看到的最后一個(gè)錯(cuò)誤是in_array
和array_keys
的聯(lián)合使用训唱。所有這些都可以使用array_key_exists
替換褥蚯。
$items = [
'one_key' => 'John',
'search_key' => 'Jane',
];
if (array_key_exists('search_key', $items)) {
// process
}
我們還可以使用isset
來(lái)檢查值是否不是null
。
if (isset($items['search_key'])) {
// process
}
討論請(qǐng)前往專業(yè)的 PHP 論壇:https://learnku.com/php/t/49583