teratail header banner
teratail header banner
質問するログイン新規登録

回答編集履歴

2

加筆

2021/02/06 05:49

投稿

退会済みユーザー
answer CHANGED
@@ -30,4 +30,35 @@
30
30
  「created BETWEEN DATE('2021-02-01') AND DATE('2021-02-28')」
31
31
  になるように
32
32
  「created BETWEEN DATE(:gessho) AND DATE(:getsumatsu)」
33
- みたいにしてこれもbindValue()で与えればいいわけで。
33
+ みたいにしてこれもbindValue()で与えればいいわけで。
34
+
35
+ ---
36
+
37
+ もう一つ。
38
+ 構造の話になってくるのですが、
39
+ 質問者さんは二次元配列(週単位の配列)を組んで実装していますが、
40
+ 日曜日から週を始めるためには二次元配列を組まなくてもいい方法があります。
41
+
42
+ 日付を管理する配列だけのシンプルさにして、
43
+ 曜日を判定して土曜日であればtr要素を出力して週を改めるってだけで実現できます。
44
+ 日付は常に+1方向で増えていくだけなので、単純な一つのfor()ループだけです。
45
+ その日付のデータを出力する前や後に、曜日の判定処理を加えるのです。
46
+
47
+ tr出力
48
+
49
+ 当月1日の曜日ギャップを調整するfor()ループでtd出力
50
+
51
+ 曜日ごとデータ出力for()ループ開始
52
+
53
+ その日付データを出力(曜日ごとの色替えや今日の色替えなどもここで)
54
+
55
+ もしも土曜日ならtr出力して週を改める
56
+
57
+ ループ終了、繰り返し。
58
+
59
+ 末日以降の曜日ギャップを調整するfor()ループでtd出力
60
+
61
+ 忘れずに閉じtr出力して終了
62
+
63
+ という流れで、組めるはず。
64
+ 曜日の繰り返しなんてしなくても組めます。

1

見直し

2021/02/06 05:49

投稿

退会済みユーザー
answer CHANGED
@@ -1,7 +1,7 @@
1
1
  データベースからの読み出しの箇所、なぜ2度に分けて読み出しているのか。
2
2
  これ、一度にできますよね。
3
- それに、もしもデータが複数年に渡った場合に全部のデータを抽出すると邪魔なので、
3
+ それに、もしもデータが当月分だけじゃなく複数に渡った場合に全部のデータを抽出すると邪魔なので、
4
- 一ヶ月分のカレンダーを作ろうとしているように見えたので、念の為にcreatedを`$year`と`$month`で区切ってみるのも加えてみたり。
4
+ 一ヶ月分のカレンダーを作ろうとしているのであれば、念の為にcreatedを`$year`と`$month`で区切ってみるのも加えてみたり。
5
5
  それに、ORDER BY句がないので、抽出される順序が不定になってるので補ってみたり。
6
6
 
7
7
  ```php
@@ -21,4 +21,13 @@
21
21
 
22
22
  これなら、日付の「日」と投稿データをセットで得られますよ。
23
23
  MySQL前提でDAYOFMONTH()を使いましたが、1日目がゼロではじまるため、+1しています。
24
- MySQL 5.6のマニュアルでは→[12.7 日付および時間関数](https://dev.mysql.com/doc/refman/5.6/ja/date-and-time-functions.html)
24
+ MySQL 5.6のマニュアルでは→[12.7 日付および時間関数](https://dev.mysql.com/doc/refman/5.6/ja/date-and-time-functions.html)
25
+
26
+ なお、安直に「year(created) = :year and month(created) = :month」とした箇所は、
27
+ いちいち関数による演算を加えると遅くなる傾向にあるため、
28
+ より負担なく高速にするためには
29
+ 当月の初日の文字列と最終日の文字列を用意してBETWEENするのが良いかと。
30
+ 「created BETWEEN DATE('2021-02-01') AND DATE('2021-02-28')」
31
+ になるように
32
+ 「created BETWEEN DATE(:gessho) AND DATE(:getsumatsu)」
33
+ みたいにしてこれもbindValue()で与えればいいわけで。