回答編集履歴
1
テキスト修正
test
CHANGED
@@ -31,3 +31,51 @@
|
|
31
31
|
|
32
32
|
|
33
33
|
にしてみるといかがでしょう?
|
34
|
+
|
35
|
+
|
36
|
+
|
37
|
+
|
38
|
+
|
39
|
+
### 補足
|
40
|
+
|
41
|
+
|
42
|
+
|
43
|
+
このような `input_check()` を書く場合には、まず初めに
|
44
|
+
|
45
|
+
|
46
|
+
|
47
|
+
```javascript
|
48
|
+
|
49
|
+
function input_check(){
|
50
|
+
|
51
|
+
var result = true;
|
52
|
+
|
53
|
+
|
54
|
+
|
55
|
+
// TODO: ここで入力値のバリデーションを行う
|
56
|
+
|
57
|
+
|
58
|
+
|
59
|
+
return result;
|
60
|
+
|
61
|
+
}
|
62
|
+
|
63
|
+
```
|
64
|
+
|
65
|
+
と書いてしまって、上記の状態で登録ボタンをクリックすればフォームがsubmit されて、サーブレットにリクエストが投げられることを、まず確認します。その後、 `// TODO` のところに `result` に `false` を入れるロジックを追加していくという手順で進めていけば、よいかと思います。
|
66
|
+
|
67
|
+
|
68
|
+
|
69
|
+
それと、入力エラーがあったときは、「CSSクラスに `inp_error` を持つ要素が1つ以上ある」という条件が成り立つならば、 ご質問にあるコードから `result` を削除してしまって、最後の `return` を
|
70
|
+
|
71
|
+
|
72
|
+
|
73
|
+
```javascript
|
74
|
+
|
75
|
+
return $('.inp_error').length === 0;
|
76
|
+
|
77
|
+
```
|
78
|
+
|
79
|
+
|
80
|
+
|
81
|
+
とするのも検討してみて頂きたいリファクタ案です。 `result` のような`true`か`false`の入っている変数のことをフラグといったりしますが、フラグを一つ増やすたびに、コードを追ってロジックを把握するのが大変になってきます。
|