回答編集履歴

2

修正

2020/02/04 06:45

投稿

m.ts10806
m.ts10806

スコア80852

test CHANGED
@@ -1,4 +1,10 @@
1
+ エラーからすると[PDO::query](https://www.php.net/manual/ja/pdo.query.php)の返却値がfalseなんでしょう
2
+
3
+ > PDO::query() は、PDOStatement オブジェクトを返します。 失敗した場合は FALSE を返します。
4
+
5
+
6
+
1
- 直に実行して通るSQLかどうかは確認してください。
7
+ 直に実行して通るSQLかどうかは必ず確認してください。phpMyAdminからも実行できます。
2
8
 
3
9
  直に実行して想定通りの情報が得られなければ当然PHPから実行しても
4
10
 

1

修正

2020/02/04 06:45

投稿

m.ts10806
m.ts10806

スコア80852

test CHANGED
@@ -1,16 +1,32 @@
1
1
  直に実行して通るSQLかどうかは確認してください。
2
+
3
+ 直に実行して想定通りの情報が得られなければ当然PHPから実行しても
4
+
5
+ 想定の結果は得られません。
6
+
7
+
2
8
 
3
9
  シングルクォートはSQLにおいては「文字列」を表します。
4
10
 
5
11
  テーブル名を「文字列」として扱ってませんか?
6
12
 
7
- テーブル名やカラム名は通常は何も囲まないか丁寧にするなら````(バッククォート2つ)で囲みます。
13
+ テーブル名やカラム名は通常は何も囲まないか丁寧にするなら
14
+
15
+ ```
16
+
17
+ `table_nawe`
18
+
19
+ ```
20
+
21
+ のように(バッククォート2つ)で囲みます。
8
22
 
9
23
  ※全角名称採用は考慮しません
10
24
 
11
25
 
12
26
 
13
- また、直に$_POSTを参照するのではなく[filter_input](https://www.php.net/manual/ja/function.filter-input.php)を採用することで、`if(@$_POST["Name"] != ""){ `というエラーを意図的にスキップする実装を避けることができます。
27
+ また、直に$_POSTを参照するのではなく[filter_input](https://www.php.net/manual/ja/function.filter-input.php)を採用することで、
28
+
29
+ `if(@$_POST["Name"] != ""){ `というエラーを意図的にスキップする実装を避けることができます。
14
30
 
15
31
  エラーはむしろ出すべきで、提示のようにスキップする実装は避けるべきです。
16
32
 
@@ -22,7 +38,9 @@
22
38
 
23
39
  また、try-catchでPDOExeptionを捕捉する仕組みは入れましょう。
24
40
 
41
+ DBはPHPからすると外部のアプリケーションです。
42
+
25
- DBはPHPからすると外部のアプリケーションです。ちゃんと拾ってあげる仕組みにしないと何が起きているのか把握することはできません。
43
+ ちゃんと拾ってあげる仕組みにしないと何が起きているのか把握することはできません。
26
44
 
27
45
 
28
46