simple javascript radio button doubt

[Jas Oberai]

Hi Friends,
I had a code,that produced 3 radio buttons,and used this javascript code to check if any one of them is selected or not

var checkedFlag = false;
for (i=0; i<document.forms[0].book.length; i++){
if(document.forms[0].book[i].checked == true){
checkedFlag= true;
}
}

This code produced the radio buttons before:

<logic:iterate id="bookClassList" name="addBookForm" property="bookClassList">
 <tr>
  <td>
    <html:radio idName="bookClassList" property="book" value="value"/>
  </td>
  <td>
  <logic:equal name="bookClassList" property="value" value="A">
       //print something
  </logic:equal>
 
  <logic:equal name="bookClassList" property="value" value="B">
       //print something
  </logic:equal>
 
  <logic:equal name="bookClassList" property="value" value="C">
       //print something
  </logic:equal> 
 
  </td>
  <td>
       <bean:write name="bookClassList" property="label"/>
  </td>
 </tr>
</logic:iterate>
 

But now I changed the code,and I am jus producing one radio button at a time,but when I select that button and click submit,that javascript function still says -"select a box".So can someone please tell me what to change in the javascript function???
[ April 26, 2006: Message edited by: jas oberai ]

[Weerawit Maneepongsawat]

try this

 
var checkedFlag = false;
if (document.forms[0].book.length) {
   for (i=0; i<document.forms[0].book.length; i++){
      if(document.forms[0].book[i].checked == true){
         checkedFlag= true;
      }
   }
} else {
   if (document.forms[0].book.checked == true) {
      checkedFlag= true;
   }
}

i didn't test it yet, but it should work

hope this help