1

我有这个简单的脚本。我正在尝试获取检查的值并将它们添加到禁用输入框中的运行总计中。我知道它正在检查选项，但它没有更新到输入框，我不知道为什么。谁能帮我？

<html>
<head>
<script type="text/javascript">
    function updateForm()
    {
        var type = document.pizzaForm.pizzaType;
        var toppings = document.pizzaForm.toppings; 
        var pizzaType;
        var toppings;

        for(var i = 0; i <= type.length; i++)
        {
            if(type[i].checked)
            {
                total = type[i].value;
            }
        }

        for(var i = 0; i <= toppings.length; i++)
        {
            if(toppings[i].checked)
            {
                toppings += toppings[i].value;
            }
        }

        var total = pizzaType + toppings;

        pizzaForm.total.value = total;
    }
</script>
</head>
<body>
    <h1>Order Pizza Here:</h1>
    <form action="" method="get" name="pizzaForm">
        What Type of Pizza Would You Like? <br />
        <input type="radio" name="pizzaType" value="10.00" onchange="updateForm()" />Vegetarian<br />
        <input type="radio" name="pizzaType" value="20.00" onchange="updateForm()" />Meat Lovers<br />
            <br />
            <br />
        Extra Toppings: <br />
        <input type="checkbox" name="toppings" value="2.00" onchange="updateForm()" />Extra Cheese <br />
        <input type="checkbox" name="toppings" value="3.00" onchange="updateForm()" />Mushrooms <br />
        <input type="checkbox" name="toppings" value="4.00" onchange="updateForm()" />Anchovies <br />
            <br />
        Total <input type="text" disabled="disabled" name="total" />
    </form>
</body>
</html>
4

2 回答 2

5

您有一些基本的 Javascript 错误：

  1. 你的 for 循环看起来像：

    for(var i = 0; i <= type.length; i++)

    这意味着它们将从 0 变为长度（包括长度）= 长度 + 1
    它应该是：

    for(var i = 0; i < type.length; i++)

    看到区别了吗？<=现在是<。（因一个错误而关闭？）

  2. 您正在使用toppings变量两次。（javascript对此真的很糟糕，让你在脚下开枪。）你也应该初始化所有的值。

    var type = document.pizzaForm.pizzaType;
var toppings = document.pizzaForm.toppings; 
var pizzaTypeValue = 0;
var toppingsValue = 0;

    我还添加Value了保存数字而不是元素的变量。其他人可能会为此或某些此类约定添加前缀，以记住它包含一个值而不是元素列表。

  3. 标记中的值是parseFloat(用于将它们转换为浮点数的字符串：

    pizzaTypeValue += parseFloat(type[i].value);

    还要注意+=手段：把这个加给我。相当于pizzaTypeValue = pizzaTypeValue + ...。

  4. 没有真正需要总变量。如果您想记住它是总数，只需添加评论。

请参阅此jsFiddle：http://jsfiddle.net/F53ae/以查看它的实际效果。

于 2011-09-29T02:43:39.810 回答
1

这是工作代码。我把它放在 jsfiddle 给你。主要问题是您有两个名为 toppings 的变量。此外，在第一个循环中，当您打算设置另一个总数时，您正在设置“总计”变量。看看这个。 http://jsfiddle.net/eXPAj/1/

于 2011-09-29T02:44:29.053 回答