PHP 変数定義の仕方

Question

とあるプロジェクトのとある人が書いたコードで以下のような記述がありました。
（わかりやすくシンプルに書き換えています）

<?php

$res;
$input = '123';
$a = 1;

if ($a === 1) {
    $res = $input;
}

return $res;

$resはifがtrueの時だけ値がセットされます。

そもそも「$res;」という書き方自体は特にエラーや警告など発生していないようですが、意味があるのでしょうか？この場合null?
こういう書き方って割と普通ですか？

Answer 1

おそらくその方は、C言語などの変数宣言が必須なプログラミング言語を使ってきたのでしょうね。
PHP では変数宣言は必須ではないので、おっしゃる通り

$res;

という宣言のみの文は無駄だし、PHP の記述としては普通ではないです。

ただし

$result = null;
  :
if ($a === 1) {
    $result = $input;
}
return $result;

のようにデフォルトの返り値を最初に指定しておいて、特定条件の時のみそれを書き換えていくというスタイルはよく見かけます。この場合は、例え返り値のデフォルトが null だったとしてもそれを明示的に代入しておくことが、プログラムを読んだ時の分かりやすさにつながると思います。

Comments

[dupont_kedama]

大きく違う回答ではないので補足だけ。
質問文の例で仮に$aが1でなかった場合、$resを参照しようとしたときにエラー、ワーニングは出ませんが、Noticeが発生します。

[JinwonKim]

@dupont_kedama さんの補足　

「Notice: Undefined variable～～」
「Notice: Undefined offset～～」

[nasu]

koyhogeさん　回答ありがとうございます。
実はこのプログラム書いたのは、社会人1年生です。C#は多少経験あるらしいですが・・・

彼らの作ったものをチェックしていて、たまたま気づきました。
どちらにしてもわかりづらいので直してもらうつもりではいたのですが、なんて説明すればよいかのベストアンサーがわからずに質問させていただきました。

ダメというよりも、紛らわしいので初期値を定義してもらおうと思います。

[nasu]

dupont_kedamaさん
JinwonKimさん
補足ありがとうございます。そうですね。まさにそこが一番の問題になりますね。
すっきりしました。