0

我无法让单选按钮验证正常工作。复选框和文本字段都很好，但是单选按钮没有被选中，所以页面没有继续到成功页面。

<script>
        window.onload =function()
        {
            document.getElementById("pie_form").onsubmit = validateForm;
        }

        function validateForm()
        {
            var validName = validateTextBox("pie_name", "error_pie_name");
            var validFlavor = validateFlavor("flavor", "error_flavor");
            var validIceCream = validateCheckBox("ice_cream", "error_ice_cream");

            //if all fields validate go to next page
            return validName && validFlavor && validIceCream;
        }

        function validateTextBox(fieldId, errorId)
        {
            var text = document.getElementById(fieldId).value;
            var errorSpan = document.getElementById(errorId);

            if(text == "")
            {
                errorSpan.innerHTML = "* blank";
                return false;   //stay on this page
            }
            else
            {
                errorSpan.innerHTML = "";   //clear the error
                return true;    //go to success page
            }
        }


        function validateFlavor()
        {
            var flavor = document.getElementById("pie_form").flavor;
            var errorSpan = document.getElementById("error_flavor");
            errorSpan.innerHTML = "";

            if(!flavor.checked)
            {
                errorSpan.innerHTML = "* You must pick a flavor.";
                return false;
            }

            return true;
        }

        function validateCheckBox(fieldId, errorId)
        {
            var checkbox = document.getElementById(fieldId);
            var errorSpan = document.getElementById(errorId);
            errorSpan.innerHTML = "";

            //if you didn't check the checkbox show error
            if(!checkbox.checked)
            {
                errorSpan.innerHTML = "* You didn't agree to have Ice Cream?";
                return false;
            }

            //if you checked return true to say its valid
            return true;
        }

    </script>
</head>
<body>
    <h1>Pie Festival!</h1>

    <form id="pie_form" action="pie_success.html">
        <p>
            <label>Pie Name:
                <input type="text" id="pie_name" name="pie_name">
            </label>
            <span id="error_pie_name" class="error"></span>
        </p>            
        <p>
            Flavor:
            <span id="error_flavor" class="error"></span><br>
            <label><input type="radio" name="flavor" value="apple">Apple</label><br>
            <label><input type="radio" name="flavor" value="blueberry">Blueberry</label><br>
            <label><input type="radio" name="flavor" value="cherry">Cherry</label><br>
        </p>
        <p>
            <label>
                <input type="checkbox" id="ice_cream" name="ice_cream">
                Do you want Ice Cream?
            </label>
            <span id="error_ice_cream" class="error"></span>
        </p>

        <input type="submit" name="continue" value="Continue">
    </form>
4

4 回答 4

1

document.getElementById("pie_form").flavor返回一个数组。您需要创建一个变量，例如isChecked设置为 false，循环遍历数组，如果选中其中一个单选按钮，则将变量设置为 true。然后继续编写您的脚本。

此代码有效：

function validateFlavor() {
    var flavor = document.getElementById("pie_form").flavor,
        errorSpan = document.getElementById("error_flavor"),
        isChecked = false,
        i;

    errorSpan.innerHTML = "";

    for (i = 0; i < flavor.length; i += 1) {
        if (flavor[i].checked) {
            isChecked = true;
            break;
        }
    }

    if (!isChecked) {
        errorSpan.innerHTML = "* You must pick a flavor.";
        return false;
    }

    return true;
}

这是一个小提琴。

于 2013-05-19T22:24:40.233 回答
0

我建议你尝试使用一些 jQuery 并使用这样的:checked选择器：

$('form').submit(function(e){

    e.preventDefault()

    if($('input[name=flavor]:checked').size() < 1){

        ... do ya thing...

    }else{

        $(this).submit()

    }

你真的应该使用 jQuery，因为它干净且易于调试。

这里是工作小提琴

编辑

选择器:checked基本上是一个 CSS 选择器，因此可用于通过 jQuery 选择器选择选中的元素。您可以在此处了解有关:checked选择器的更多信息

于 2013-05-19T22:19:36.003 回答
0

您没有为单选元素提供任何 ID，因此您不能使用 getelementbyid 并且单选组提供了一个列表，因此您必须循环通过以下修改后的 validateflavor 函数：

function validateFlavor(fieldName, errorId) {
    var errorSpan = document.getElementById(errorId);
    errorSpan.innerHTML = "";
    var flavorgroups = document.getElementsByName(fieldName)

            for (var i = 0; i < flavorgroups.length; i++) {
                if (flavorgroups[i].checked) {
                return true; // checked
            }
            };

            // not checked, show error
            errorSpan.innerHTML = '* You must pick a flavor.';
            return false;
        }
于 2013-05-19T22:36:44.167 回答
0

默认情况下始终选中一个单选按钮是个好主意，但有时这并不适合。您可以将表单控件引用为表单的属性，因此将返回名称为flavorform.flavor的控件的 HTMLCollection 。然后，您可以遍历它们以查看是否已检查一个，例如

function validate(formId) {
  var form = document.getElementById(formId);
  var radios = form.flavor;
  var oneChecked = false;

  for (var i=0, iLen=radios.length; i<iLen && !oneChecked; i++) {
    oneChecked = radios[i].checked
  }

  return oneChecked;
}
于 2013-05-19T22:40:23.637 回答