回答編集履歴
2
見直し
test
CHANGED
|
@@ -2,7 +2,7 @@
|
|
|
2
2
|
|
|
3
3
|
初期化漏れがあるとちょっと怖い。
|
|
4
4
|
|
|
5
|
-
is_uploaded_file()やmove_uploadfile()のelse条件の処理が雑なので、
|
|
5
|
+
**is_uploaded_file()やmove_uploadfile()のelse条件の処理が雑**なので、
|
|
6
6
|
|
|
7
7
|
直後のINSERT INTO文を実行しちゃってるね、きっと。
|
|
8
8
|
|
|
@@ -11,6 +11,12 @@
|
|
|
11
11
|
使いまわししている変数に1のときの処理結果が残っちゃっているから、
|
|
12
12
|
|
|
13
13
|
それを元にINSERT INTO文が実行されているのだと思う。
|
|
14
|
+
|
|
15
|
+
|
|
16
|
+
|
|
17
|
+
is_uploaded_file()やmove_uploadfile()で条件が成立していないとき、
|
|
18
|
+
|
|
19
|
+
どういうINSERT INTO文がいいのかをもう一度考えて実装を見直してみては。
|
|
14
20
|
|
|
15
21
|
|
|
16
22
|
|
1
見直し
test
CHANGED
|
@@ -1,6 +1,16 @@
|
|
|
1
1
|
No1と2と3で変数名の使い回しがあるため、
|
|
2
2
|
|
|
3
3
|
初期化漏れがあるとちょっと怖い。
|
|
4
|
+
|
|
5
|
+
is_uploaded_file()やmove_uploadfile()のelse条件の処理が雑なので、
|
|
6
|
+
|
|
7
|
+
直後のINSERT INTO文を実行しちゃってるね、きっと。
|
|
8
|
+
|
|
9
|
+
1だけ指定して2や3がカラのとき、
|
|
10
|
+
|
|
11
|
+
使いまわししている変数に1のときの処理結果が残っちゃっているから、
|
|
12
|
+
|
|
13
|
+
それを元にINSERT INTO文が実行されているのだと思う。
|
|
4
14
|
|
|
5
15
|
|
|
6
16
|
|