PHPで変数を初期化したほうが良い場合の具体例

わからないこと

PHPで変数を初期化したほうがよいケース

環境

PHP 5.4.16

具体例で例えば...

こういうコードの場合の初期化は不要でしょうか?

php
1...
2
3         while (($instance = $class->method()))
4         {
5             // ファイル名をセット
6-            $fileName    = '';
7             $fileName    = $instance->fileName;
8 
9             // 説明をセット
10-            $description = '';
11             $description = $instance->description;
12 
13             // 状態をセット
14-            $status      = '';
15             $status      = $runRecordInstance->get($fileName);
16
17...

上記コードの初期化の意図

変数を利用する上で型を明示する意味合いと、while文で繰り返し処理で使用する変数のため、
2回目以降のループで各値をセットする前に初期化したほうが上書きするより適切?と感じた

なぜ上記のような意図を考えたのか

PHPを勉強していく中ででさまざまな書籍やネット上の記事を参考にした過程で
「変数や配列は値をセットする前に初期化したほうがよい」という認識に至りました。

回答について

①上記具体例についての正否等
②そもそも「初期化」についての考え方や具体例

についてご教示いただければ幸いです。
お手数ですが宜しくお願いいたします。

行動規範の内容に同意します

回答1件

ベストアンサー

こういうコードの場合の初期化は不要でしょうか?

不要です。

ただ $fileName などの変数を while の後でも使うことがあるなら while の前で初期化しておいたほうが良いです。

php
1// 初期化
2$fileName = null;
3
4while (($instance = $class->method())) {
5    // ファイル名をセット
6    $fileName    = $instance->fileName;
7
8    :
9}
10
11if (isset($fileName)) {
12    // 最後のループの $fileName を使って何かする
13    :
14}

PHP でこのような初期化が必要と言われた最も大きな理由は register_globals という設定が原因です。
この設定が On だと、ブラウザから送られたパラメータがそのままグローバル変数になっていました。
例えば、次のようなリクエストだと、

http://example.com/?fileName=/path/to/secret

次のようなグローバル変数が自動的に作成されます。

php
1$fileName = "/path/to/secret";

したがって、上のコードで while の外で初期化していな場合、

php
1while (($instance = $class->method())) {
2    // ファイル名をセット
3    $fileName    = $instance->fileName;
4
5    :
6}
7
8if (isset($fileName)) {
9    // 最後のループの $fileName で何かする
10    :
11}

もし、while ループが１回も回らなかったら（条件がはじめから false だったら）、その次の if では $fileName は "/path/to/secret" になってしまいます。

もっとも register_globals は PHP 5.4 だとなくなっているので、PHP 5.4 では関係ありません。

が、register_globals とは無関係に、次のようなケースでは初期化しておいた方が良いだろうと思います。

php
1// 初期化
2$fileName = null;
3
4while (($instance = $class->method())) {
5    // ファイル名をセット
6    $fileName    = $instance->fileName;
7
8    :
9}
10
11if (isset($fileName)) {
12    // 最後のループの $fileName を使って何かする
13    :
14}