#在做過大量的程式碼審查後,我經常看到一些重複的錯誤,以下是糾正這些錯誤的方法。
一:在循環之前測試數組是否為空
$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 // ... } }
這不是你可能要修正這5個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://dev.to/klnjmm/5-bad-habits-to-lose-in-php-2j98
翻譯網址:https:/ /learnku.com/php/t/49583
以上是你可能要修正這5個PHP編碼小陋習!的詳細內容。更多資訊請關注PHP中文網其他相關文章!