回答編集履歴

2

下書きで更新されていてむかつく

2016/09/23 05:25

投稿

pandanoir
pandanoir

スコア72

test CHANGED
@@ -13,3 +13,25 @@
13
13
  追記:
14
14
 
15
15
  いろいろと突っ込みどころが多いですが、長くなるのでつっこみはやめておきます。
16
+
17
+
18
+
19
+ 追々記:
20
+
21
+ なぜか下書きで投稿されていたので追記します。まず、検証すべき点はcondition()が呼ばれているのか、です。呼ばれていてtrueが返されているならそこまではクリアできています。次はサーバサイドです。こちらもアクセスされているのかの検証が必要です。検証はどちらもconsole.logやechoを使えば簡単にできます。
22
+
23
+
24
+
25
+ それと、関係ないのですが、気になった突っ込みどころをつっこみますね。
26
+
27
+
28
+
29
+ まず、どうせjQueryを使っているのですからonsubmitなんて使わずに、 `$('#hoge').on('submit', function(){});` としましょう。というか、使ってもいいことはいいのですが、最低限イベントハンドラはonclick系を使うかどちらかに統一しましょう。両方にぶちまけていたらすぐにスパゲティとなりますよ。オススメはがぜん.on('click', f)ですね。こっちなら複数イベントリスナを貼れますし、JSとHTMLで役割分担がされていていいです。
30
+
31
+
32
+
33
+ 次にwith。見たのは数年振りですが、使っているのは初めて見ました。パフォーマンスも下がりますし、コードを読むときにwithの一部なのか分かりづらくなり、デメリットしかないです。with使うのはやめましょう。本当に百害あって一利なしです。
34
+
35
+
36
+
37
+ それと、nameに[]を使うのは悪いとは言いませんが、不要ですしやめた方がいいです。今まで1度もみたことないです。

1

追記を受けて

2016/09/23 05:25

投稿

pandanoir
pandanoir

スコア72

test CHANGED
@@ -7,3 +7,9 @@
7
7
 
8
8
 
9
9
  とりあえず、コードを載せてもらえると回答しやすくなります。
10
+
11
+
12
+
13
+ 追記:
14
+
15
+ いろいろと突っ込みどころが多いですが、長くなるのでつっこみはやめておきます。