JavaScriptの記述を添削してください。

質問

アンケートフォームを作成して送信ボタンを押してから
送信確認メッセージを出るようにいたしました。

今までは送信確認メッセージは出さずに
必須項目の未入力部分のみの確認で行っていましたが
送信ボタンを押した後に送信確認メッセージを出してから
OKなら送信、キャンセルなら元に戻るというようにしてみました。

自分で動作確認したところでは問題ないようですが、
JavaScriptは超初心者の為まったく自信がありません。
お手すきの方がいましたら
間違っている箇所などをご指摘くださいますでしょうか。

//入力確認
//
function kakunin() {
if (document.F2.名前.value == "") {
alert("名前が入力されていません。");
document.F2.名前.focus();
return(false);
}
if (document.F2.ふりがな.value == "") {
alert("ふりがなが入力されていません。");
document.F2.ふりがな.focus();
return(false);
}
　if (document.F2.メール.value == "") {
alert("メールアドレスが入力されていません。");
document.F2.メール.focus();
return(false);
}
//
//↓以下を付け足しました。
//送信確認
//
myRet = confirm("送信します。よろしいですか？");
if ( myRet == false ) return(false);
return(true);
}

<FORM name="F2" method="POST" action="***/mail.cgi" onsubmit="return kakunin()"><INPUT type="hidden" name="_to_" value="***@***.ne.jp"><INPUT type="hidden" name="_subject_" value="***"><INPUT type="hidden" name="_location_" value="***.htm"><BR>

<INPUT type="submit" value="送信ボタン">

上部、入力確認は今まで問題無く動作していました。
よろしくお願いします。

回答No.1

s_doc · ベストアンサー

文法的にはこれでいいと思います。

が、オブジェクト名（タグの名前）には全角文字は使うべきではありません。
今動いていたとしても将来まで保証されないからです。
何しろ海外生まれのブラウザがほとんどですからね。
半角英文字または「_」で始まり、
半角英数字または「_」が続く名前を付ける習慣を付けましょう。
ローマ字で全く構いません。
Namae、Furigana、とかね。

それから最後の

myRet = confirm("送信します。よろしいですか？");
if ( myRet == false ) return(false);
return(true);

は、

confirm("送信します。よろしいですか？");

これだけにしたほうがすっきりします。

完璧な検証作業をしたわけではないので自信は「なし」にしておきます。
ご免。

お礼

ありがとうございます。

>オブジェクト名（タグの名前）には全角文字は使うべきではありません。

薄々感じてはいたんですが
やっぱりそうかと認識いたしました。

>confirm("送信します。よろしいですか？");
>これだけにしたほうがすっきりします。

confirm("送信します。よろしいですか？");
上だけにするとキャンセルした場合でも送信してしまいましたので
トホホで「confirm」を検索し調べたところ
if(confirm("送信します。よろしいですか？")) {
　return true;
}
else {
　return false;
}}
こんな感じにすればよいことが分かりました。
confirmの意味さえよく分からなかった自分ですから
勉強になりました。

回答No.2

s_doc

ポカミスしてしまいました。

> confirm("送信します。よろしいですか？");
> 上だけにするとキャンセルした場合でも送信してしまいましたので

仰る通りです。
正しいのはこっちです。

return(confirm("送信します。よろしいですか？"));

confirm()はキャンセルボタンをクリックするとfalseを返しますので、
return(false)が実行されます。