回答編集履歴
6
低評価理由をコメント願います。
test
CHANGED
|
File without changes
|
5
グループ化
test
CHANGED
|
@@ -5,6 +5,8 @@
|
|
|
5
5
|
※再現可能な情報をご提示いただけてないため、的確な回答ではなくコードレビューの結果です
|
|
6
6
|
|
|
7
7
|
|
|
8
|
+
|
|
9
|
+
---
|
|
8
10
|
|
|
9
11
|
- shop_idで絞り込むなら1件しか出力されないはずなので回す必要はないのでは
|
|
10
12
|
|
|
@@ -44,6 +46,8 @@
|
|
|
44
46
|
|
|
45
47
|
```
|
|
46
48
|
|
|
49
|
+
---
|
|
50
|
+
|
|
47
51
|
|
|
48
52
|
|
|
49
53
|
- $shop_idがちゃんときているか見ている方に分からないので何とも言えないところ、というかここはfetchAllでいい
|
|
@@ -81,6 +85,8 @@
|
|
|
81
85
|
}
|
|
82
86
|
|
|
83
87
|
```
|
|
88
|
+
|
|
89
|
+
---
|
|
84
90
|
|
|
85
91
|
|
|
86
92
|
|
|
@@ -148,6 +154,8 @@
|
|
|
148
154
|
|
|
149
155
|
```
|
|
150
156
|
|
|
157
|
+
---
|
|
158
|
+
|
|
151
159
|
|
|
152
160
|
|
|
153
161
|
- ということで、`$result`に入っているのは`select SQL_CALC_FOUND_ROWS * from reserve order by reserve_time desc limit :limit offset :offset`の結果。
|
4
修正
test
CHANGED
|
File without changes
|
3
修正
test
CHANGED
|
@@ -172,8 +172,10 @@
|
|
|
172
172
|
|
|
173
173
|
|
|
174
174
|
|
|
175
|
+
----
|
|
176
|
+
|
|
177
|
+
|
|
178
|
+
|
|
175
179
|
もし同じプロジェクトのメンバーがこのコード持って来たら全部書き直させます。
|
|
176
180
|
|
|
177
|
-
それか全部引き取って自分で組みます。
|
|
181
|
+
それか全部引き取って自分で組みます。
|
|
178
|
-
|
|
179
|
-
それぐらい、ひどいコード。
|
2
修正
test
CHANGED
|
@@ -2,13 +2,13 @@
|
|
|
2
2
|
|
|
3
3
|
不要な処理、使ってない変数の整理は必要。
|
|
4
4
|
|
|
5
|
-
※的確な回答ではなくコードレビューの結果です
|
|
5
|
+
※再現可能な情報をご提示いただけてないため、的確な回答ではなくコードレビューの結果です
|
|
6
6
|
|
|
7
7
|
|
|
8
8
|
|
|
9
9
|
- shop_idで絞り込むなら1件しか出力されないはずなので回す必要はないのでは
|
|
10
10
|
|
|
11
|
-
- $_SESSION['user']でshop_id
|
|
11
|
+
- $_SESSION['user']という名称でshop_idに当てているのは謎。ショップIDでログインするの？intならログイン時に数字を打っていることになるけど、
|
|
12
12
|
|
|
13
13
|
ここは全体仕様とログイン処理含めて提示されないと確かめようがない
|
|
14
14
|
|
1
修正
test
CHANGED
|
@@ -1,6 +1,8 @@
|
|
|
1
1
|
ごちゃっとしているので作り直した方がいいです。
|
|
2
2
|
|
|
3
3
|
不要な処理、使ってない変数の整理は必要。
|
|
4
|
+
|
|
5
|
+
※的確な回答ではなくコードレビューの結果です
|
|
4
6
|
|
|
5
7
|
|
|
6
8
|
|
|
@@ -167,3 +169,11 @@
|
|
|
167
169
|
<?php endforeach; ?>
|
|
168
170
|
|
|
169
171
|
```
|
|
172
|
+
|
|
173
|
+
|
|
174
|
+
|
|
175
|
+
もし同じプロジェクトのメンバーがこのコード持って来たら全部書き直させます。
|
|
176
|
+
|
|
177
|
+
それか全部引き取って自分で組みます。そのメンバーにはテストだけさせる。
|
|
178
|
+
|
|
179
|
+
それぐらい、ひどいコード。
|