請糾正這 5 個 PHP 編碼小(xiǎo)陋習

2022-03-03 查看(1680)


在循環之前測試數組是否爲空

$items = [];
// ...
if (count($items) > 0) {
    foreach ($items as $item) {
        // process on $item ...
    }
}複制代碼

foreach 以及數組函數 (array_*) 可以處理空數組。

  • 不需要先進行測試

  • 可減少一(yī)層縮進

$items = [];
// ...
foreach ($items as $item) {
    // process on $item ...
}複制代碼

将代碼内容封裝到一(yī)個 if  語句彙總

function foo(User $user) {
    if (!$user->isDisabled()) {
        // ...
        // long process
        // ...
    }
}複制代碼

這不是 PHP 特有的情況,不過我(wǒ)經常碰到此類情況。你可以通過提前返回來減少縮進。

所有主要方法處于第一(yī)個縮進級别

function foo(User $user) {
    if ($user->isDisabled()) {
        return;
    }

    // ...
    // 其他代碼
    // ...
}複制代碼

多次調用 isset 方法

你可能遇到以下(xià)情況:

$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']
}複制代碼

我(wǒ)們經常需要檢查變量是否已定義,php 提供了 isset 函數可以用于檢測該變量,而且該函數可以一(yī)次接受多個參數,所以一(yī)下(xià)代碼可能更好:

$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方法一(yī)起使用

$name = "John Doe";
echo sprintf('Bonjour %s', $name);複制代碼

看到這段代碼你可能會想笑,不過我(wǒ)的确這樣寫了一(yī)段時間,而且我(wǒ)仍然會看到很多這樣寫的!其實echosprintf 并不需同時使用,printf 就可以完全實現打印功能。

$name = "John Doe";
printf('Bonjour %s', $name);複制代碼

通過組合兩種方法檢查數組中(zhōng)是否存在鍵

$items = [
    'one_key' => 'John',
    'search_key' => 'Jane',
];

if (in_array('search_key', array_keys($items))) {
    // process
}複制代碼

我(wǒ)經常看到的最後一(yī)個錯誤是in_arrayarray_keys的聯合使用。所有這些都可以使用array_key_exists替換。

$items = [
    'one_key' => 'John',
    'search_key' => 'Jane',
];

if (array_key_exists('search_key', $items)) {
    // process
}複制代碼

我(wǒ)們還可以使用isset來檢查值是否不是null

if (isset($items['search_key'])) {
    // process
}複制代碼

讨論請前往專業的 PHP 論壇:learnku.com/php/t/49583


作者:Summer__
鏈接:https://juejin.cn/post/6873259938903244807
來源:稀土掘金
著作權歸作者所有。商(shāng)業轉載請聯系作者獲得授權,非商(shāng)業轉載請注明出處。


掃二維碼與項目經理溝通

我(wǒ)們在微信上24小(xiǎo)時期待你的聲音

解答本文疑問/技術咨詢/運營咨詢/技術建議/互聯網交流

鄭重聲明:郑州禾木网络技术有限公司網絡科技有限公司以外(wài)的任何單位或個人,不得使用該案例作爲工(gōng)作成功展示!