2013-01-23 102 views
0

目标:修复下面的代码,并理解为什么专注于单选按钮无法像专注于文本输入框一样工作。专注于单选按钮

问:为什么这个不行:

function isValidRadio(elem, helperMsg) { 
    var valid = false; 
    for (var i = 0; i < elem.length; i++) { 
     if (elem[i].checked) { 
      return true; 
     } 
    } 
    alert(notice); 
    setTimeout("focusElement('" + elem.form.name + "', '" + elem.name + "')", 0); 
    return false; 
} 

...当这样做:

function isValidText(elem, notice) { 
    var str = elem.value; 
    var re = /.+/; 
    if(!str.match(re)) { 
     alert(notice);   
     setTimeout("focusElement('" + elem.form.name + "', '" + elem.name + "')", 0); 
     return false; 
    } else { 
     return true; 
    } 
} 

示例代码:

function focusElement(formName, elemName) { 
    var elem = document.forms[formName].elements[elemName]; 
    elem.focus(); 
    elem.select(); 
} 
function validateForm(form) { 
    if (isValidText(form.phone, "Fail.")) { 
     if (isValidRadio(form.yesNo, "Fail.")) { 
      return true; 
     } 
     return false; 
    } 
} 
<form id='myForm' action='/formPage/collectinfo/' method='post' accept-charset='UTF-8' name="myForm"> 
    <input name="phone" type="text" maxlength="14" size="14" /> 
    <input type="radio" name="yesNo" value="Yes"> Yes <input type="radio" name="yesNo" value="No" > No 
<input type="submit" value=" Submit " name="Submit" onclick="return validateForm(myForm)" /> 

详细信息:我使用的是php/javascript/html,到目前为止我只在Chrome中测试过这件作品。该表单工作得很好,验证工作,直到它击中isValidRadio。警报运行,但随后JavaScript中断。代码修复会很好,但我不会介意'钓鱼一辈子......'

+0

请不要评估你的setTimeout回调。 – AlienWebguy

+0

@AlienWebguy:我开始讨论这些问题,因为他们被认为是解决这类问题的一种可能的解决方案(显然他们没有工作)。我很乐意将它们加入,但希望能详细了解它们的问题。 (除了它们是不必要的,我认为)。 – lostphilosopher

+1

它只是使用'setTimeout(“myfunction()”,0);''与'setTimeout(function(){myfunction();},0)的语法。 ' – AlienWebguy

回答

2

好的,我找到了解决方案,AlienWebguy把我放在思考阵列的正确轨道上。我只需要指定要关注哪个单选按钮而不是哪个单选按钮。这里是工作代码:

// Validate that the user has checked one of the radio buttons 
function isValidRadio(radio, helperMsg) { 
    var valid = false; 
    for (var i = 0; i < radio.length; i++) { 
     if (radio[i].checked) { 
      return true; 
     } 
    } 
    alert(helperMsg); 
    radio[0].focus(); 
    return false; 
} 

调用通过:isValidRadio(form.yesNo, "Error.")感谢您的协助。