更改爲比較運營商

有人告訴我，使用一個支持它的庫，即使前面的代碼沒有使用參數。而且我正在使用賦值運算符而不是比較。此外，真正的分支是reductant。更改爲比較運營商

以前的代碼：

if ($profilepic = $row['profilepic']){ 
    $profilepic = $row['profilepic']; 
}else{ 
    $profilepic = "DamjuNoImage"; 
}

目前代碼：

$profilepic = $row['profilepic']; 
$noimage = "DamjuNoImage"; 

if ($profilepic != $row['profilepic']){ 
     echo $noimage; 
    }

我想仔細檢查，看看我是否固定，即使我已經測試了它的代碼。

來源

2012-06-01 taeja87

你需要回去研究基本面。應該讓人大跌眼鏡的是，在當前版本中，「if」塊永遠不會做任何事情。也幾乎可以肯定的是，代碼沒有達到你想要的目標（因爲它實際做了什麼，沒有意義）。 – Jon

當我測試了它，目前的代碼做了什麼，我打算做的事。 – taeja87

@ taeja87你有什麼打算* *什麼關係呢？ – Sampson

下列要求：

$profilepic = $row['profilepic']; 
if ($profilepic != $row['profilepic']){ 
    echo $noimage; 
}

你的if語句將永遠不會做任何事情。第一行使得$profilepic等於$row['profilepic']（不管是一個空字符串，null或什麼具備的，你），和條件檢查，如果他們不相等 - 這絕不是因爲你剛拍完他們相等的情況下。

如果你想提供一個默認的形象，做這樣的事情：

if (empty($row["profilepic"])) { 
    $profilepic = $row["profilepic"]; 
} else { 
    $profilepic = "DamJuNoImage"; 
}

或寫入更簡潔：

// Assign value of column if not empty, otherwise "DamJuNoImage" 
$profilepic = empty($row["profilepic"]) 
    ? "DamJuNoImage" 
    : $row["profilepic"];

如果配置PIC欄是空的，我們分配了一個默認值爲$profilepic。從那裏我們可以迴應我們的形象。

echo "<img src='{$profilepic}' />";

演示：http://codepad.org/KXuQyVma

來源

2012-06-01 15:22:27 Sampson

謝謝！我決定使用更簡潔的一個。 – taeja87

@ taeja87我對這部分做了重要改變;請使用更新後的版本。 – Sampson

我明白了。非常感謝。學到了新東西。 – taeja87

更改爲比較運營商

回答

相關問題