推薦:《php視頻教程》
在做過大量的代碼審查后,我經(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 語句匯總
function foo(user $user) { if (!$user->isdisabled()) { // ... // long process // ... }}這不是 php 特有的情況,不過我經(jīng)常碰到此類情況。你可以通過提前返回來減少縮進(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);這段代碼可能在微笑,但是我碰巧寫了一段時(shí)間。而且我仍然看到很多!不用結(jié)合echo和sprintf,我們可以簡(jiǎn)單地使用printf方法。
$name = "john doe";printf('bonjour %s', $name);五:通過組合兩種方法檢查數(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來檢查值是否不是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