Javascript Syntax Error?

Arrgghhhh... I must have read this code 20 times. The old "2nd set of eyes" adage. Thanks.

Make sure that the Form name is exactly "mySurvey"

Try using
document.mySurvey.elements[0].checked
document.mySurvey.elements[1].checked
instead of document.mySurvey.R1.checked
let me know ...

I fixed the quotes and added the "outside" parenthesis and that fixed the problem.

Awesome. Please close the question if you are done.

-- attempt to answer future question :)

What I always do is label the radio or checkboxes with the same name.
then in my javascript i can loop through them.

<script>
function chksurvey()
{
   pass = false;
   for(i=0;i<document.mySurvey.question1.length;i++)
   {
      if(document.mySurvey.question1[i].checked == true)
         pass = true;
   }
   if(!pass)
   {
      alert("Please answer Question 1");
      return false;
   }
   return true;
}
</script>

<form name='mySurvey' onsubmit='return chksurvey();'>
Question 1: ...
<input type='radio' name='question1' value='A'> A
<input type='radio' name='question1' value='B'> B
<input type='radio' name='question1' value='C'> C
</form>

Yup. I tried that. I'm developing in VS2003 and when I type "document.mySurvey." the drop-down doesn't list "question1" as an object I can select. It lists the values from the ID tags (R1, R2) in the radio buttons, but not the NAME tag(q1). Wierd. I even tried just ignoring Intellisense and typed "alert(document.mySurvey.q1[0].value);" and got a missing object error.  Here's the HTML:

<input type="radio" name="q1" value ="1" id="R1" onClick="showQs(1,1)"> 1-3 times / week</input>
<input type="radio" name="q1" value ="2" id="R2" onClick="showQs(1,2)"> 3 or more times / week</input>
<input type="radio" name="q1" value ="3" id="R3" onClick="showQs(1,3)"> Do not use cafeteria services</input>

So I should be able to iterate through for(i=0;i<document.mySurvey.q1.length;i++), but like I said above, when I get to the Intellisense drop-down for mySurvey it doesn't list "q1" anywhere.

Any thoughts

I dont use VS2003... im a dreamweaver person.
I used VS a lil while ago and I know what you're talking about.
Some of the fields would show up in the auto popup thingie when your typing script, and some wont.
I basically ignored it and developed the page the way it should and it still seemed to work.

So I guess I don't know why VS2003 would be ignoring some of your fields.
But your alert of q1[0].value should still work.. heh wierd.

what if you alert(document.mySurvey.q1.length)? what do you get?

I dont use VS2003... im a dreamweaver person.
I used VS a lil while ago and I know what you're talking about.
Some of the fields would show up in the auto popup thingie when your typing script, and some wont.
I basically ignored it and developed the page the way it should and it still seemed to work.

So I guess I don't know why VS2003 would be ignoring some of your fields.
But your alert of q1[0].value should still work.. heh wierd.

what if you alert(document.mySurvey.q1.length)? what do you get?

"undefined"

when you check the length of a field, it will give you a "undefined" result if there is only one field with that name.
so, if question 1 only has one field, q1.length wont return 1, it will return undefined.
question 1 needs atleast 2 fields with the same name for q1.length to return atleast 2.

that may also be why q1[0].value wouldnt work for you before.
im sure if you add another field with the same name, different value... you'll be able to reference them as an array.

HTML as posted above. Shouldn't this result in an array with the name "q1"?

<input type="radio" name="q1" value ="1" id="R1" onClick="showQs(1,1)"> 1-3 times / week</input>
<input type="radio" name="q1" value ="2" id="R2" onClick="showQs(1,2)"> 3 or more times / week</input>
<input type="radio" name="q1" value ="3" id="R3" onClick="showQs(1,3)"> Do not use cafeteria services</input>