回答編集履歴
6
低評価理由をコメント願います。
answer
CHANGED
|
File without changes
|
5
グループ化
answer
CHANGED
|
@@ -2,6 +2,7 @@
|
|
|
2
2
|
不要な処理、使ってない変数の整理は必要。
|
|
3
3
|
※再現可能な情報をご提示いただけてないため、的確な回答ではなくコードレビューの結果です
|
|
4
4
|
|
|
5
|
+
---
|
|
5
6
|
- shop_idで絞り込むなら1件しか出力されないはずなので回す必要はないのでは
|
|
6
7
|
- $_SESSION['user']という名称でshop_idに当てているのは謎。ショップIDでログインするの？intならログイン時に数字を打っていることになるけど、
|
|
7
8
|
ここは全体仕様とログイン処理含めて提示されないと確かめようがない
|
|
@@ -21,6 +22,7 @@
|
|
|
21
22
|
}
|
|
22
23
|
|
|
23
24
|
```
|
|
25
|
+
---
|
|
24
26
|
|
|
25
27
|
- $shop_idがちゃんときているか見ている方に分からないので何とも言えないところ、というかここはfetchAllでいい
|
|
26
28
|
```php
|
|
@@ -40,6 +42,7 @@
|
|
|
40
42
|
];
|
|
41
43
|
}
|
|
42
44
|
```
|
|
45
|
+
---
|
|
43
46
|
|
|
44
47
|
- 今回の無駄と思われる処理。$whereSqlをnullで定義して終わっている。
|
|
45
48
|
`$where[] = "name like :name";`としているけど使われていない。reserveテーブルにnameというカラムがある？あったとしても先の処理で一緒にできるのでは
|
|
@@ -73,6 +76,7 @@
|
|
|
73
76
|
$sql->bindValue(":offset", $offset, PDO::PARAM_INT);
|
|
74
77
|
$sql->bindValue(":limit", PAGE_LIMIT, PDO::PARAM_INT);
|
|
75
78
|
```
|
|
79
|
+
---
|
|
76
80
|
|
|
77
81
|
- ということで、`$result`に入っているのは`select SQL_CALC_FOUND_ROWS * from reserve order by reserve_time desc limit :limit offset :offset`の結果。
|
|
78
82
|
```php
|
4
修正
answer
CHANGED
|
File without changes
|
3
修正
answer
CHANGED
|
@@ -85,6 +85,7 @@
|
|
|
85
85
|
<?php endforeach; ?>
|
|
86
86
|
```
|
|
87
87
|
|
|
88
|
+
----
|
|
89
|
+
|
|
88
90
|
もし同じプロジェクトのメンバーがこのコード持って来たら全部書き直させます。
|
|
89
|
-
それか全部引き取って自分で組みます。
|
|
91
|
+
それか全部引き取って自分で組みます。
|
|
90
|
-
それぐらい、ひどいコード。
|
2
修正
answer
CHANGED
|
@@ -1,9 +1,9 @@
|
|
|
1
1
|
ごちゃっとしているので作り直した方がいいです。
|
|
2
2
|
不要な処理、使ってない変数の整理は必要。
|
|
3
|
-
※的確な回答ではなくコードレビューの結果です
|
|
3
|
+
※再現可能な情報をご提示いただけてないため、的確な回答ではなくコードレビューの結果です
|
|
4
4
|
|
|
5
5
|
- shop_idで絞り込むなら1件しか出力されないはずなので回す必要はないのでは
|
|
6
|
-
- $_SESSION['user']
|
|
6
|
+
- $_SESSION['user']という名称でshop_idに当てているのは謎。ショップIDでログインするの？intならログイン時に数字を打っていることになるけど、
|
|
7
7
|
ここは全体仕様とログイン処理含めて提示されないと確かめようがない
|
|
8
8
|
- というよりも「ログインしたショップの」という前提なら毎回全部とるのはそれだけ何度もDBにアクセスしまくることになるので無駄。
|
|
9
9
|
ログイン時にセッションに保存しておいてセッションから引き出した方がいい
|
1
修正
answer
CHANGED
|
@@ -1,5 +1,6 @@
|
|
|
1
1
|
ごちゃっとしているので作り直した方がいいです。
|
|
2
2
|
不要な処理、使ってない変数の整理は必要。
|
|
3
|
+
※的確な回答ではなくコードレビューの結果です
|
|
3
4
|
|
|
4
5
|
- shop_idで絞り込むなら1件しか出力されないはずなので回す必要はないのでは
|
|
5
6
|
- $_SESSION['user']でshop_idというのは謎。ショップIDでログインするの？
|
|
@@ -82,4 +83,8 @@
|
|
|
82
83
|
<td><?= htmlspecialchars($value['time'], ENT_QUOTES, 'UTF-8'); ?></td>
|
|
83
84
|
<tr>
|
|
84
85
|
<?php endforeach; ?>
|
|
85
|
-
```
|
|
86
|
+
```
|
|
87
|
+
|
|
88
|
+
もし同じプロジェクトのメンバーがこのコード持って来たら全部書き直させます。
|
|
89
|
+
それか全部引き取って自分で組みます。そのメンバーにはテストだけさせる。
|
|
90
|
+
それぐらい、ひどいコード。
|