Validating Radio buttons problem

[prakus]

I have the following code for validating Radio buttons:
When I select the radio button and submit, it gives me a message
'Please select a Section and press "Submit"'.
I donot have any problem when there are more than one radio buttons to choose from.

I am not able to figure out what the problem is. Any help will be appreciated.

Thank you.
PKS

function checkSecRadioAlert() {
var el = document.forms[0].elements;
for(var i = 0 ; i < el.length ; ++i) {
if(el.type == "radio") {
var radiogroup = el[el.name]; // get the whole set of radio buttons.
var itemchecked = false;
for(var j = 0 ; j < radiogroup.length ; ++j) {
if(radiogroup[j].checked) {
itemchecked = true;
break;
}
}
if(!itemchecked) {
alert("Please select a Section and press \"\Submit\"\.");
if(el.focus)
el.focus();
return false;
}
}
}
return true;
}

<Form name='RearrSecForm' Method='POST' Action='RearrangeVarFrame.asp' onsubmit='return checkSecRadioAlert(this);'>

<table>
<tr>
<td><input type='radio' value='1872' name='SecName'></td>
<td>Endowment</td> <tr>
<tr>
<td></td>
<td><input type='submit' value='Submit' name='Cnt'></td>
</tr>
</table>
</Form>

 

[jemminger]

you need to check to see if you have more than one radio button first, using the existence of the "length" property of the radiobutton:

function checkSecRadioAlert() {
    var el = document.forms[0].elements;
        for(var i = 0 ; i < el.length ; ++i) {
            if(el[i].type == "radio") {
                var radiogroup = el[el[i].name]; // get the whole set of radio buttons.
                var itemchecked = false;
                
                if (!radiogroup.length) {
                  itemchecked = radiogroup.checked;
                }
                else {
                  for(var j = 0 ; j < radiogroup.length ; ++j) {
                      if(radiogroup[j].checked) {
                          itemchecked = true;    
                          break;                    
                      }
                  }
                }
                if(!itemchecked) {
                    alert("Please select a Section and press \"\Submit\"\.");
                    if(el[i].focus)
                    el[i].focus();
                    return false;
                }                
            }
        }
    return true;
}

=========================================================
-jeff

http://www.jeffemminger.com

try { succeed(); } catch(E) { tryAgain(); } finally { rtfm(); }

 

[prakus]

Thank you Jeff. I should have noticed that.

Thank you very much.
PKS.