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