回答編集履歴

3

修正

2019/11/30 03:23

投稿

m.ts10806
m.ts10806

スコア80850

test CHANGED
@@ -32,4 +32,4 @@
32
32
 
33
33
 
34
34
 
35
- あとtry-catchでDB関連の処理を全て囲って[execption](https://www.php.net/manual/ja/class.mysqli-sql-exception.php)を捕捉する仕組みにすべきです。
35
+ あとtry-catchでDB関連の処理を全て囲って[execption](https://agohack.com/mysqli-report-throw-exceptions/)を捕捉する仕組みにすべきです。

2

修正

2019/11/30 03:23

投稿

m.ts10806
m.ts10806

スコア80850

test CHANGED
@@ -29,3 +29,7 @@
29
29
 
30
30
 
31
31
  取得結果を別のDBに入れるなど特別な要件がない限り、接続先を替えるってないと思います。今回は接続した直後に替えにいっているので、謎の実装になっています。
32
+
33
+
34
+
35
+ あとtry-catchでDB関連の処理を全て囲って[execption](https://www.php.net/manual/ja/class.mysqli-sql-exception.php)を捕捉する仕組みにすべきです。

1

修正

2019/11/30 03:22

投稿

m.ts10806
m.ts10806

スコア80850

test CHANGED
@@ -16,7 +16,7 @@
16
16
 
17
17
 
18
18
 
19
- というか、下記の第4引数で既にDBは選択されて接続が確立されています。直後に選択しなおすくらいなら接続時に設定良いのではないでしょうか?
19
+ というか、下記の第4引数で既にDBは選択されて接続が確立されています。直後に選択しなおすくらいなら接続時に設定するので良いのではないでしょうか?
20
20
 
21
21
  そのほうが先に問題に気づけます。
22
22