回答編集履歴
3
追記
answer
CHANGED
|
@@ -44,9 +44,12 @@
|
|
|
44
44
|
SQLにPOSTされた情報をそのまま入れるのはNGです。必ずエスケープしてください。
|
|
45
45
|
できればPDO使った方がすっきり書けます。
|
|
46
46
|
|
|
47
|
-
もう1点。
|
|
47
|
+
もう1点。(むしろこっちのほうが大事)
|
|
48
48
|
既に指摘があるとおり全て配列に突っ込んでいては処理が明確にわけられません。
|
|
49
49
|
nameにはきちんと役割をそれぞれ明確に与えた名前をつけるべきです。（もしフィールド名も同じようにマジックナンバーなどつけていれば改めるべき）
|
|
50
50
|
何1ついいことはないので、今からでも改善してください。
|
|
51
51
|
おそらくこのコードから離れて数ヶ月たったときこのコードを見たときにどこが何をやっているか全て見直さなければ理解できない事態が確実に起きます。
|
|
52
|
-
メンテナンス性・可読性の低いコードは低ければ低いほど、不具合が起きる可能性が高くなります。
|
|
52
|
+
メンテナンス性・可読性の低いコードは低ければ低いほど、不具合が起きる可能性が高くなります。
|
|
53
|
+
- [まずコードの可読性を最適化しよう](https://postd.cc/optimize-for-readability-first/)
|
|
54
|
+
- [初心者が保守性の高いプログラムを書くために注意するべきこと](http://programming-study.com/trouble/things-to-keep-in-mind-when-writing-programs/)
|
|
55
|
+
- [保守性の高いコードを書くためにはどうしたらよいか](https://qiita.com/syguer/items/f5d9d02d8405d63d30a4)
|
2
修正
answer
CHANGED
|
@@ -42,4 +42,11 @@
|
|
|
42
42
|
```
|
|
43
43
|
|
|
44
44
|
SQLにPOSTされた情報をそのまま入れるのはNGです。必ずエスケープしてください。
|
|
45
|
-
できればPDO使った方がすっきり書けます。
|
|
45
|
+
できればPDO使った方がすっきり書けます。
|
|
46
|
+
|
|
47
|
+
もう1点。
|
|
48
|
+
既に指摘があるとおり全て配列に突っ込んでいては処理が明確にわけられません。
|
|
49
|
+
nameにはきちんと役割をそれぞれ明確に与えた名前をつけるべきです。（もしフィールド名も同じようにマジックナンバーなどつけていれば改めるべき）
|
|
50
|
+
何1ついいことはないので、今からでも改善してください。
|
|
51
|
+
おそらくこのコードから離れて数ヶ月たったときこのコードを見たときにどこが何をやっているか全て見直さなければ理解できない事態が確実に起きます。
|
|
52
|
+
メンテナンス性・可読性の低いコードは低ければ低いほど、不具合が起きる可能性が高くなります。
|
1
修正
answer
CHANGED
|
@@ -20,7 +20,7 @@
|
|
|
20
20
|
> テーブルにAUTO_INCREMENTのフィールドがあります。
|
|
21
21
|
間違いないのは配列の数もテーブルのフィールドの数も同じです。
|
|
22
22
|
|
|
23
|
-
であれば、`AUTO_INCREMENT
|
|
23
|
+
であれば、`AUTO_INCREMENT（および初期値指定がある）フィールドを除いた全フィールドを指定したINSERT文`を作成しないといけません。
|
|
24
24
|
|
|
25
25
|
フィールド名分からないですが、例えばこんな感じ。
|
|
26
26
|
|