PHP if文条件式内で変数宣言は一般的？

Question

前提

filter_input関数について調べていると馴染みのないif文を見かけました。
ネットで調べてもあまりこのような書き方について言及している記事を見つけられませんでした。

該当のソースコード

if ( $username = filter_input( INPUT_POST, 'username' ) )  {
    echo 'ユーザー名が入力されました。';
    echo $username;
}
else {
    echo 'ユーザー名が入力されていません;';
}

疑問1

条件式で比較演算子「==」「===」を使うべきところ、
「=」にしちゃってたというミスはよくあると思いますが、
意図的に「=」を使うのは一般的な記法でしょうか？
（実務でこのようなコードを書くと、上記のようなミスを見分けられず不便そうに思いました。）

疑問2

このコードは以下と同等でしょうか？

$username = filter_input( INPUT_POST, 'username' );
if ( $username )  {
    echo 'ユーザー名が入力されました。';
    echo $username;
}
else {
    echo 'ユーザー名が入力されていません。';
}

初歩的な質問かもしれませんがよろしくお願いいたします。

Answer 1

実務でこのようなコードを書くと、上記のようなミスを見分けられず不便そうに思いました。

そうですね、実務ではこのような書き方はNGとコーディング規約で制限することも多いかと思いますが、決して間違いではありません。

このコードは以下と同等でしょうか？

同等です

Comments

[March03]

ご回答ありがとうございます。
正しいコーディングですが、あまり使われない書き方ということですね。
勉強になりました。

Answer 2

実務でこのようなコードを書くと、上記のようなミスを見分けられず不便そうに思いました。

こちらについては、if ( ($username = filter_input( INPUT_POST, 'username' )) !== FALSE )のように条件判定を追加することで、（比較演算子の結果を比較するようなことは通常行いませんので）代入を意図的に行っているとコードで表現できます。

Comments

[March03]

ご回答ありがとうございます。
おっしゃるとおり、その記述でしたら意図的な代入だと伝わりますね。
色々な書き方があって面白いです。勉強になりました。

Answer 3

[疑問1について]

条件式で比較演算子「==」「===」を使うべきところ、
「=」にしちゃってたというミスはよくあると思いますが、
意図的に「=」を使うのは一般的な記法でしょうか？
（実務でこのようなコードを書くと、上記のようなミスを見分けられず不便そうに思いました。）

意味が違いますよ。質問にあるコードでの "="は比較演算子としてのイコールではなく、代入演算子としてのイコールです。つまりif ( $username = filter_input( INPUT_POST, 'username' ) )の条件式中に出てくるイコールは「filter_input関数を引数IとしてINPUT_POSTやらなんやらを渡して呼び出し、その結果を$usernameという変数に入れろ」です。

[疑問2について]

このコードは以下と同等でしょうか
(コード)

YES.

Comments

[March03]

ご回答ありがとうございます。
理解できました！