フォームのチェックについて

質問

今、フォームの送信ボタンを押すとチェックする
関数を作成しております。
function restChar() {
if(10000 < document.F1.ご質問内容.value.length) window.alert("ご質問内容は全角5,000字以内でお願いします");
}
function check(){
var flag = 0;
if(document.F1.name.value == ""){ // 「お名前」の入力をチェック
flag = 1;
}
else if(document.F1.kana.value == ""){ // 「パスワード」の入力をチェック
flag = 2;
}
else if(document.F1.mail.value == ""){ // 「コメント」の入力をチェック
flag = 3;
}
　　　　　else if(document.F1.ご質問内容.value == ""){ // 「コメント」の入力をチェック
flag = 4;
}
if(flag == 1){

window.alert('お名前を入力してください'); // 入力漏れがあれば警告ダイアログを表示
return false; // 送信を中止
}
if(flag == 2){
window.alert('フリガナを入力してください'); // 入力漏れがあれば警告ダイアログを表示
return false; // 送信を中止
}
if(flag == 3){
window.alert('メールアドレスを入力してください'); // 入力漏れがあれば警告ダイアログを表示
return false; // 送信を中止
}
　　　　　if(flag == 4){
window.alert('ご質問内容を入力して下さい'); // 入力漏れがあれば警告ダイアログを表示
return false; // 送信を中止
}

fOBJ = document.F1.mail.value;
check = /.+@.+\..+/;
if (!fOBJ.match(check)){
window.alert("メールアドレスが正しくありません");
return false; // 送信を中止
}
fOBJ = document.F1.ご質問内容.value.length;
if(10000 < fOBJ){
window.alert("ご質問内容は全角5,000字以内でお願いします");
return false; // 送信を中止
}
return true; // 送信を実行
}
こんな感じです。
1-4のチェックは問題ないのですが、
アドレスと文字数のチェックは、一度は
ダイアログがでますが、連続で押すと通ってしまいます。
何がわるいのでしょうか？

回答No.2

googoo131 · ベストアンサー

もうひとつですが、flagの判定は不要だと思います。
あと、フォームチェック関数作成してみました。

function restChar() {
if(navigator.appName == "Netscape"){// ブラウザ名取得
Num = document.F1.ご質問内容.value.length / 2; // NetScape では全角１文字を２文字として数える
if ((document.F1.ご質問内容.value.length % 2) != 0) {
Num += 1;
}
}
else
{
Num = document.F1.ご質問内容.value.length; // IE では全角１文字を１文字として数える
}

if(5000 < Num)){
window.alert("ご質問内容は全角5,000字以内でお願いします");
}
}

function check(){
var flag = 0;
if(document.F1.name.value == ""){ // 「お名前」の入力をチェック
window.alert('お名前を入力してください'); // 入力漏れがあれば警告ダイアログを表示
return false; // 送信を中止

}
else if(document.F1.kana.value == ""){ // 「パスワード」の入力をチェック
window.alert('フリガナを入力してください'); // 入力漏れがあれば警告ダイアログを表示
return false; // 送信を中止
}
else if(document.F1.mail.value == ""){ // 「コメント」の入力をチェック
window.alert('メールアドレスを入力してください'); // 入力漏れがあれば警告ダイアログを表示
return false; // 送信を中止
}
else if(document.F1.ご質問内容.value == ""){ // 「コメント」の入力をチェック
window.alert('ご質問内容を入力して下さい'); // 入力漏れがあれば警告ダイアログを表示
return false; // 送信を中止
}

check = /.+@.+\..+/;
if (!document.F1.mail.value.match(check)){
window.alert("メールアドレスが正しくありません");
return false; // 送信を中止
}

if(navigator.appName == "Netscape"){// ブラウザ名取得
Num = document.F1.ご質問内容.value.length / 2; // NetScape では全角１文字を２文字として数える
if ((document.F1.ご質問内容.value.length % 2) != 0) {
Num += 1;
}
}
else
{
Num = document.F1.ご質問内容.value.length; // IE では全角１文字を１文字として数える
}

if(5000 < Num)){
window.alert("ご質問内容は全角5,000字以内でお願いします");
return false; // 送信を中止
}

return true; // 送信を実行
}

これでどうですか？

回答No.1

googoo131

はじめまして。

(1)　入力チェックにおいては以下のサイトを参考にしてみた方がいいかと思います。

　http://f32.aaa.livedoor.jp/~azusa/index.php?p=formcheck&t=js

(2)　解析してみたところ、ご質問内容の入力数判定においてはまずブラウザの判定が必要です。
NetScape では全角１文字を２文字、IEでは全角１文字を１文字として数えるようになっております。

とすると、restChar 関数、check 関数ともご質問内容入力判定部分のコードは以下のようにしなければなりません。

fOBJ = document.F1.ご質問内容.value.length;
BROW = navigator.appName; // ブラウザ名取得
if(BROW == "Netscape"){
fOBJ /= 2; // NetScape では全角１文字を２文字として数える
}
if(5000 < fOBJ)){
window.alert("ご質問内容は全角5,000字以内でお願いします");
return false; // 送信を中止
}
return true; // 送信を実行
}

参考ＵＲＬはこちらです。
http://www.umechando.com/tips/10.htm