php エラーはでないが、値が予期したものではなくNULL

Question

SQL文絡みの質問です。

持ち出し禁止なのでソースはありませんが、覚えている範囲で質問します。
あるクラス内で定義されている関数内で更に別の関数を、LEFT JOINに使ってサブクエリを得るというものですが、エラーはでないが予期した値が得られないのでどこまで読み込めているのかわからず困っています。

ちゃんとクエリが返っているかの確認は
そのクラスファイルの最下部に下記を付けたし、WinSCPで該当ディレクトリまでcdしてphp -f Class.phpで確認しています。

内容はともかくとして、これで返るはずの値が返ってこないのです。
一応、クエリ文としては引数があろうがなかろうがNULLのときはNULLで返ってくるのでエラーにはならないのです。

$section = array("1","2","3","4","5","6","7","8","9")
というのが、DBレコードに入っている部門ID的なものなので、それに対して２年分の検索をかけています。

が、なにも返ってこないのでどこから手をつけていいのか悩んでいます。

$section = array("1","2","3","4","5","6","7","8","9")

for($year=2015;$year<=2016;$year++){
	for($mon=1;$mon<=12;$mon++){
		for($day=1;$day<=28;$day++){
			for($i=0;$i<count($section);$i++) {
				
				$year = sprintf('%04d',$year);
				$mon = sprintf('%02d',$mon);
				$day = sprintf('%02d',$day);

				$target = $year . "/" . $mon . "/" . $day;

				echo "------------------------------------------------------------";
				echo PHP_ELO,PHP_ELO;
				echo $target;
				echo PHP_ELO;
				echo "Section_id:" . $section[$i];
				echo PHP_ELO,PHP_ELO;
				$test =Class::saranibetsunokansuu($section[$i],$target);
				var_dump($test);
			}
		}
	}
}

$dayのfor文が28までなのはあまり関係ないです。
該当レコードは２８日以下にもあるので、めんどくさがって28にしているだけです。

Comments

[hyper-drums-ko]

Class::saranibetsunokansuu にてデータベースにアクセスしているものと思われますが、そちらを提示いただくことは可能でしょうか？

Answer 1

とりあえず、forで回している変数を
ループ中で更新するのはだめじゃないスかねえ。
--- 追加 ---

あとは、その関数とパラメータの意識があってるか
の確認くらいかなあ。
--- 追加 ---

これはなぜ駄目なのでしょうか？

自分だったら

                $year = sprintf('%04d',$year);
                $mon = sprintf('%02d',$mon);
                $day = sprintf('%02d',$day);

                $target = $year . "/" . $mon . "/" . $day;

の部分を

                $target = sprintf('%04d/%02d/%02d',$year,$mon,$day);

こうします。
意図してそうしているのならかまいませんが、
そうでないならバグのもとになりがちなものです。
ただ、今回のに関しては明確に誤りだという確信はないので、
正しく動作するのならそのままでもいいのではないでしょうか。

Comments

[earnest_gay]

>とりあえず、forで回している変数を
>ループ中で更新するのはだめじゃないスかねえ。

これはなぜ駄目なのでしょうか？

[Panzer_vor]

横から失礼します。

ループ変数の書き換えが一般的に良くないとされるのは、
厄介な不具合を招きやすく、可読性を著しく低下させるからです。

今回のケースではループ変数を書式変換してるのみなので大事には至っていませんが、ループ内でループ変数自体をいじるとその結果を利用してループが継続されます。

例えば0〜5までループする処理でループ変数に対して1加算すると、
実質的なループ処理は、
0、2、4の3回となります。

その処理を書いた人ならいいですが、
他の人がそのソースを見た時に、
この動きをすぐには想定できないし、
普通の感覚ではループ処理は開始から終了まで1つずつ処理されるというのがほとんどなので、注意深く見ないと見落としかねません。

そういう事情もありループ変数を上書きするのは、
一般的にはナンセンスとされています。

今回のケースでも実害が出ていなくとも、
書式変換後の値を保存する専用変数を定義して使用するのが行儀の良いコーディングとなります。

Answer 2

SQL部分の記載はないので、
あまり踏み込んだ回答は出来そうもないですが^^;

当方が気になる点は2カ所で、
takasima20さんが既に指摘されているループ変数の書き換えが行われている点。
(動作するにしても別変数に入れ直した方が安心できます)

それとループで日付と部門ID？を組み立て点は１つ１つ検索をかけてるように見えますが、
SQLの書き方次第で、
対象データを一気に抽出してこの工程が省けるのではないかなといった点です。
(細かい実装が分からないのでもしかしたらループで1件ずつ回すのが正解な可能性もありますが)

特に2点目については、
要件的にSQL自体を改善できるのであれば、
PHP側でごちゃごちゃする必要性が減るのでコードを簡潔にできる可能性は高いように思えます。

Answer 3

SELECT 日時, 欲しいもの, 他に欲しいもの
 FROM 欲しいものが入ったテーブル
 WHERE
  日時 between '2015-01-01' and '2017-01-01'
  and
  部門ID in (1, 2, 3, 4, 5, 6, 7, 8, 9)
 GROUP BY 日時

と言うようなSQLで取得するようにしてはいかがでしょうか。

Comments

[earnest_gay]

ありがとうございます！
参考にさせていただきます。