回答編集履歴
3
修正
test
CHANGED
|
@@ -22,6 +22,8 @@
|
|
|
22
22
|
|
|
23
23
|
でないとデータが増えれば増えるほど無用なループが回り、パフォーマンスがどんどん悪くなります。
|
|
24
24
|
|
|
25
|
+
|
|
26
|
+
|
|
25
|
-
月のカレンダーに時間を表示させる
|
|
27
|
+
月のカレンダーに時間を表示させる必要があるかないかも検討が必要ですし、通常は表示させません。「その日のデータがあるかないか（または件数）」のみ分かればいいので、時間まで込みのデータは不要と思います。
|
|
26
28
|
|
|
27
29
|
group byで年月日で絞って日付毎のデータ件数だけ取っては。
|
2
修正
test
CHANGED
|
@@ -1,8 +1,8 @@
|
|
|
1
1
|
2回回っているわけではありません。
|
|
2
2
|
|
|
3
|
-
$current_reserved_dateに
|
|
3
|
+
$current_reserved_dateにデータが2本あってどの日でもその分だけ回しているだけです。
|
|
4
4
|
|
|
5
|
-
日でしか比較していないため全てelseに入りますね。
|
|
5
|
+
そして日でしか比較していないため10日以外は全てelseに入りますね。今の作りだと年月関係なく。
|
|
6
6
|
|
|
7
7
|
|
|
8
8
|
|
|
@@ -16,6 +16,12 @@
|
|
|
16
16
|
|
|
17
17
|
|
|
18
18
|
|
|
19
|
+
もっと言えば`Reservation::get(['reservation_date']);`も。
|
|
19
20
|
|
|
21
|
+
カレント年月のカレンダーしか表示させないのでしたら年月で絞ったデータのみを取得するべきです。
|
|
20
22
|
|
|
23
|
+
でないとデータが増えれば増えるほど無用なループが回り、パフォーマンスがどんどん悪くなります。
|
|
24
|
+
|
|
25
|
+
月のカレンダーに時間を表示させるのであれば、「その日のデータがあるかないか（または件数）」のみ分かればいいので、時間まで込みのデータは不要と思います。
|
|
26
|
+
|
|
21
|
-
|
|
27
|
+
group byで年月日で絞って日付毎のデータ件数だけ取っては。
|
1
修正
test
CHANGED
|
File without changes
|