How do I check if a radio button is checked using JavaScript?

Question

When I select a radio button then click the download button, nothing happens. No alert shows up or anything.

Here is my code right now:

HTML:

<form id="Resume_Select">
<input type="radio" name="resume_type" value="Economics" >Economics
<br />
<input type="radio" name="resume_type" value="MIS" >Management Information Systems
<br />
<button onClick="resume_select()"> Download</button>
</form>

JavaScript:

<script>
    function resume_select(){
    var radios = document.getElementByName('resume_type');
    for(var i = 0, length = radios.length; i<length;i++){

        if(radios[i].checked){
            var resume = radios[i].value;
            alert(radios[i].value);
            break;
        }
    }

</script>

@TimRijckaert elaborate please. I took the loop from an answer I found for someone with the same question and that is the loop they used. — ddrossi93
– ddrossi93, Commented Mar 25, 2015 at 0:32
@TimRijckaert I'm assuming you mean the ", length..." Part? Because I was confused with that too. I never wrote a for loop that used a comma. But I just took the guys word. I'll change that back to a semicolon and then get the length variable before the loop starts. That would be better, correct? — ddrossi93
– ddrossi93, Commented Mar 25, 2015 at 0:36
The initialisation part of the for loop is valid. It's caching the length of the HTMLCollection rather than calculating it each iteration. — Phylogenesis
– Phylogenesis, Commented Mar 25, 2015 at 0:41

atomCode · Accepted Answer · 2015-03-25 00:29:29Z

4

It should be getElementsByName plural not getElementByName. First thing that jumped at me.

answered Mar 25, 2015 at 0:29

atomCode

9327 silver badges17 bronze badges

Sign up to request clarification or add additional context in comments.

8 Comments

atomCode Over a year ago

You can simplify your loop like so: for(i = 0; i<radios.length; i++){ ... }

ddrossi93 Over a year ago

Okay, I will give that a try. Sounds like a simple enough fix.

ddrossi93 Over a year ago

will do! Just got home, so I am testing it now

atomCode Over a year ago

That's because you have everything inside form tags. So two options remove them or create a input type button with the on click event attached to it. I noticed that issue when I tested your code on jsfiddle so instead of using a button use an input of type button

ddrossi93 Over a year ago

@atomCode took out the form tags. Still nothing. No alert.

|

Collectives™ on Stack Overflow

How do I check if a radio button is checked using JavaScript?

1 Answer 1

8 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

8 Comments

Your Answer

Sign up or log in

Post as a guest

Related