アンケート送信のチェックについて

質問

初心者で本を見ながらアンケートを作っています。
質問事項はラジオボタンで選ぶようになっていて、
送信時にチェックされていない項目があったら「すべての項目をチェックしてください」とメッセージを出したいです。
ある本に「フォームに空欄がないか確認する」ソースがあったのでそれを応用したいのですが上手くできません。
＝＝＝＝＝ソースは下記です。＝＝＝＝＝
<script language="JavaScript">
<!--
function check(){
for( i=0; i<document.form1.length; i++ ){
if(document.form1.elements[i].type =="radio"){
if(document.form1.elements[i].value ==""){
alert("すべての項目にチェックをしてください");
return false;
}
}
}
return true;
}
//-->
</script>
～～～～
～～～～
<FORM NAME="form1" action="****/*****.pl" method="POST" onSubmit="return check()">
～～～
～～～
＝＝＝＝＝ここまで

本にはtype =="radio"のところが"text"になっていたのですが、エレメントの種類には"radio"もあったので
これをいれればチェック出来ると思ったのですが、チェックしないで送信ができてしまいます。

どなたかご教授お願いします。

回答No.3

mousengoke · ベストアンサー

function check(){
var befname = "";
var namehantei = true;
var nameradio;

for( i=0; i<document.form1.length; i++ ){
if(document.form1.elements[i].type =="radio"){
nameradio = document.form1.elements[i].name;
if(befname == nameradio){
thishantei = document.form1.elements[i].checked;
if(thishantei){
namehantei = true;
}
}else{
if(namehantei){
befname = nameradio;
namehantei = document.form1.elements[i].checked;
}else{
alert("すべての項目にチェックをしてください");
return false;
break;
}
}
}
}
if(!namehantei){
alert("すべての項目にチェックをしてください");
}
return namehantei;
}

お礼

ご回答ありがとうございました。頂いたソースをそのまま使ってみました。お陰様でチェックができました。ソースに書かれている意味がまだわからず苦労していますが少しずつ勉強したいと思っています。
本当に助かりました。ありがとうございました！

回答No.2

Y_eRu

こんにちわ(o^-^o)
えっとちょっと気になったのでお邪魔します。
チェックは「全質問に各々ラジオボタンがチェックされているか？」を見るのですよね？
書かれているif文が「ラジオボタンが空つまりチェックされていなかったらアラート表示」なので
これだとfor文で順番に見ていることから
『一番目のラジオボタンが空だったら』
（つまり絶対1番目の答えにチェックがないと抜けない）
の条件になってしまっているのでは？
と思ったもので・・。
なのでif文を
if(document.form1.elements[i].checked){
　return　true;
}
にして、
trueが返らずにfor文を抜けてしまったらアラートを表示
のようにした方がいいのではと思います。

見当違い＆大きなお世話でしたらすみません(汗)
失礼しましたm(_ _)m

お礼

ご回答ありがとうございました。
すみません。。頂いたif文を何処の部分にはめ込んだらよいのかわかりません。。いろいろ試してみたのですが・・もっと勉強しなくちゃダメですね。
コメント頂いているようにチェックは「全質問に各々ラジオボタンがチェックされているか？」を確認したいと思っています。

回答No.1

takataka111

お疲れ様です。

if(document.form1.elements[i].value ==""){
の部分を
if(document.form1.elements[i].checked == true){
に変更してください。

radioはtextと違いvalueで判断するのではなくchecked がtrueかfalseかで判断します。

true:チェック有り
false:チェック無し

お礼

早々のご回答ありがとうございました。
上記部分だけを修正してみたのですが、送信されてしまいました・・