任意の関数でエラーが発生します。

以下のようなコードを書いたときに、
”Variable '$data1' is undefined”と$data1の所が黄色いアンダー線で強調されエラーになります。
処理事体はできているのですが、何故このエラーになるのかご教示していただきたいです。
よろしくお願い致します。

<?php //繰り返し処理を行いランダムな数値を配列に格納 for($i=0;$i<4;$i++){ $data1[]=mt_rand(0,6); //関数を利用して0～6のランダム数値を1つ取得 } $data2=$data1; //配列データをコピー rsort($data2); //関数を利用して降順に並び替え ?> <html> <head> <meta http-equiv="Content-Type"content="text/html;charset=UTF-8"> </head> <body> ランダムデータ…並び替えたデータ<br> <?php for($i=0;$i<count($data1);$i++){ //関数を利用して配列要素分だけ繰り返す echo"$data1[{$i}]=".$data1[$i]."---$data2[{$i}]=".$data2[$i]."<br>\n"; } ?> </body> </html>

m.ts10806

2018/10/23 14:20

コードはマークダウンのcode機能を利用してください。エラーメッセージはなるべくそのまますべてコピペしてください（ファイルの行数も出ますが、もしセキュリティ的にまずい部分あれば、ファイルパスの前半部分をマスクかけるなどしていただいて構いませんが、そのまま全文コピペが原則です）

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

回答2件

ベストアンサー

黄色いアンダー線で強調されエラーになります。

ご自身が何でこの情報を得たか書かないと見ている人にはわかりません。
おそらくEclipse上でコードを組まれているかどうかかとは思いますが、
全ての人がEclipseを使っているとは限りませんし、Eclipseであってもデフォルトテーマから変更していたりカスタマイズされていると何のことかわかりません。
同じEclipseでもバージョンまで明記（できればPHPも）してください。

さて、これはエラーではなくどちらかというと警告です。
おそらく下記の場所などに出ている警告かと思いますが、

php
1for($i=0;$i<4;$i++){
2  $data1[]=mt_rand(0,6); 
3}

この$data1 という変数は初期宣言なしに配列としていきなり使われています。
PHPプログラム側はforが確実に回るかどうかは知らないため、文法的には所定の警告が出ます。

本来は利用前に下記のように初期で「配列の変数である」ことを宣言してあげるべきです。

php
1$data1 = [];

そもそも「配列である前提」だから提示のように組まれたのかと思いますが、プログラムは指示した通りにしか動かないので、配列かどうか宣言されていないのにいきなり配列として使おうとしているため、警告がでています。
型なども緩い言語ではありますが、明示することでより確固たるコードができますので、できる限り明示されたほうが後々考えると良いです。

投稿2018/10/23 14:32

編集2018/10/24 00:36

m.ts10806

総合スコア80850

aru_2048

2018/10/25 00:51

わかりやすい回答、本当にありがとうございます。まだ始めたばかりで、学校のテキストをそのままコーディングしていたらこのような警告が発生しました。開発環境はお察しの通りeclipseです。バージョン等の表記は次回から記載するよう心がけます。

m.ts10806

2018/10/25 01:12

学校で習ったことがないのでどのようなテキストかはわかりませんが、あまり問題は起きない部分ではありますので省略されたのかな？と思います。 IDEとしては未定義の変数は結構厳しく見ていますが、問題は起きにくいところとはいえ、用途を明示するのは非常に大事なところですので、きちんとおさえていただければと思います。質問は編集できるので適宜ご対応くださいね

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