PHP劇的ビフォーアフター1 if文を真偽値で置き換える

リファクタリング前

$flag = false;

if (empty($x)) {
    $flag = true; 
}
このようなコードをみかけたことはありますか?
問題点
  • $flagへの代入が2か所ある
  • if文は可読性が悪い

リファクタリング後

if文を使う代わりに、真偽値をそのまま代入しましょう。

$flag = empty($x);

すっきりしましたね!
カテゴリ: