2015-03-25 4 views
0

Когда я выбираю переключатель, затем нажмите кнопку загрузки, ничего не произойдет. Не появляется предупреждение или что-то еще.Как проверить, установлен ли переключатель с помощью JavaScript?

Вот мой код прямо сейчас:

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> 
+1

@TimRijckaert тщательно продумайте. Я взял цикл из ответа, который я нашел для кого-то с тем же вопросом, и это цикл, который они использовали. – Ddrossi93

+0

@TimRijckaert Я предполагаю, что вы имеете в виду часть «, длина ...»? Потому что я тоже с этим смутился. Я никогда не писал цикл for, который использовал запятую. Но я просто взял слова ребята. Я вернусь к точке с запятой, а затем получу переменную длины до начала цикла. Это было бы лучше, правильно? – Ddrossi93

+0

Исходная часть цикла 'for' действительна. Он кэширует длину HTMLCollection, а не вычисляет ее на каждой итерации. – Phylogenesis

ответ

4

Это должно быть getElementsByName множественного числа не getElementByName. Первое, что прыгнуло на меня.

+0

Вы можете упростить свой цикл следующим образом: для (i = 0; i atomCode

+0

Хорошо, я попробую. Похоже, достаточно простое исправление. – Ddrossi93

+0

сделаю! Просто вернулся домой, поэтому я тестирую его сейчас. – Ddrossi93

Смежные вопросы