回答編集履歴
3
修正
test
CHANGED
@@ -150,7 +150,7 @@
|
|
150
150
|
|
151
151
|
[SQLインジェクション](https://ja.wikipedia.org/wiki/SQL%E3%82%A4%E3%83%B3%E3%82%B8%E3%82%A7%E3%82%AF%E3%82%B7%E3%83%A7%E3%83%B3)の概念を学んでください。
|
152
152
|
|
153
|
-
PDOでしたら[bind
|
153
|
+
PDOでしたら[bindValue](http://php.net/manual/ja/pdostatement.bindvalue.php)などを活用しましょう。
|
154
154
|
|
155
155
|
|
156
156
|
|
2
表示調整
test
CHANGED
@@ -130,33 +130,27 @@
|
|
130
130
|
|
131
131
|
|
132
132
|
|
133
|
-
|
133
|
+
>[teratail:PHP、PDOの例外処理について](https://teratail.com/questions/56396)
|
134
134
|
|
135
135
|
|
136
136
|
|
137
137
|
ただ、この処理内容は他にもよくないところが幾つもあります。
|
138
138
|
|
139
|
+
「必ず成功する前提」のプログラムはかなり危険です。何かで失敗することを前提で、その対策を適宜してください。
|
139
140
|
|
140
141
|
|
142
|
+
|
141
|
-
|
143
|
+
0. mb_send_mail()より前に「メールを送りました」と文言を出している
|
142
144
|
|
143
145
|
メール送信成功するかどうか分かりませんよね。せめて返り値(bool)を確認してから画面表示しましょう。
|
144
146
|
|
145
|
-
|
147
|
+
0. ↑と同様、DB登録も成功するかどうか分かりません。正否確認をしてから画面表示しましょう。
|
146
148
|
|
147
|
-
|
148
|
-
|
149
|
-
「必ず成功する前提」のプログラムはかなり危険です。何かで失敗することを前提で、その対策をしてください。
|
150
|
-
|
151
|
-
|
152
|
-
|
153
|
-
|
149
|
+
0. pdo利用しているのに一切エスケープせずにそのままSQL文に突っ込んでいる。
|
154
|
-
|
155
|
-
[bindParam](http://php.net/manual/ja/pdostatement.bindparam.php)などを利用してください
|
156
|
-
|
157
|
-
|
158
150
|
|
159
151
|
[SQLインジェクション](https://ja.wikipedia.org/wiki/SQL%E3%82%A4%E3%83%B3%E3%82%B8%E3%82%A7%E3%82%AF%E3%82%B7%E3%83%A7%E3%83%B3)の概念を学んでください。
|
152
|
+
|
153
|
+
PDOでしたら[bindParam](http://php.net/manual/ja/pdostatement.bindparam.php)などを活用しましょう。
|
160
154
|
|
161
155
|
|
162
156
|
|
1
修正
test
CHANGED
@@ -2,7 +2,7 @@
|
|
2
2
|
|
3
3
|
DBに関する処理を行うときは、言ってしまえば外部の仕組みを使うわけですから、
|
4
4
|
|
5
|
-
try-catchを利用して異常を
|
5
|
+
[try-catch](https://qiita.com/Chelsea/items/59436cfda149a6ac6c91)を利用して異常を検知するようにしてください。
|
6
6
|
|
7
7
|
基本的に、ですがPHPの構文としてまずいことがない限り外部の仕組みのエラーは
|
8
8
|
|
@@ -130,7 +130,11 @@
|
|
130
130
|
|
131
131
|
|
132
132
|
|
133
|
+
- [teratail:PHP、PDOの例外処理について](https://teratail.com/questions/56396)
|
134
|
+
|
135
|
+
|
136
|
+
|
133
|
-
ただ、この処理内容はよくないところが幾つ
|
137
|
+
ただ、この処理内容は他にもよくないところが幾つもあります。
|
134
138
|
|
135
139
|
|
136
140
|
|