为什么这个简单的函数不起作用

我将重写checkOptions和checkAccomp函数，并使用以下模式删除所有这些if语句:创建一个存储每个复选框的id及其相应价格的对象，然后使用具有键值查找的for-in循环，如下所示:

var OptionPricing = {     
    'pack11049': 1049,
    'pack21199': 1199,
    'pack31199': 1199,
    'pack41299': 1299,
    'pack61499': 1499
};
var AccompPricing = {
    0: 0,
    1: 129,
    2: 258,
    3: 1057,
    4: 1856 
};
function checkOptions() {
    var Price = 0;
    for (Packs in OptionPricing) {
        if ($('#' + Packs).is(':checked')) {           
            Price += OptionPricing[Packs];
        }
    }
    return Price;
}
function checkAccomp() {
    var Accomp = parseInt($('#HowMany').val(), 10);
    return AccompPricing[Accomp];
}
function updateTotal() {
    var ThePrice = checkOptions() + checkAccomp();
    $('#TotalPrice').text(ThePrice);
}
$(function () { $('.DoPricing').click(updateTotal); });

这是工作的jsFiddle。我没有将所有id和相应的价格添加到OptionPricing对象中，但是您可以理解。此外，如果价格发生变化，或者添加了新的价格，则该模式应该更容易维护，更不用说代码将大大减少到仅几行(如果您喜欢简洁的语法，甚至可以进一步减少)。我使用jQuery(你在问题中有标签)，但你可以很容易地修改它，如果需要的话，使用纯js。

我可能在这里说的很明显，但是第一个函数(updateTotal)缺少右括号。如果您从源代码复制和粘贴，那么这可能是您的问题。当你盯着它看的时候很容易忽略;-)

我会试试这个http://jsfiddle.net/EFWf5/1:

$("form#order :input[type=radio]").each(function(){
    $(this).bind('change', function() {
        checkOptions();
    });
});
var checkOptions = function () {
    var total = 0;
    var pack = parseInt($('input[name=pack]:checked', '#order').val());
    var accomp = parseInt($('input[name=accomp]:checked', '#order').val());
    if (!isNaN(pack)) total += pack;
    if (!isNaN(accomp)) total += accomp;
    $('#total').text(total);
    $('#title').text($('input[name=pack]:checked', '#order').attr('title'));
};

<form id="order">
    <div class="col">
        <strong>Package</strong><br /><hr />
        <input type="radio" name="pack" value="1049" />Pack 1<br />
        <input type="radio" name="pack" value="1199" title="MVP Package"/>Pack 2<br />
        <input type="radio" name="pack" value="1199" title="Oceanview" />Pack 3<br />
        <input type="radio" name="pack" value="1299" />Pack 4<br />
        <input type="radio" name="pack" value="1499" />Pack 5</div>
    <div class="col">
        <strong>Accomp</strong><br /><hr />
        <input type="radio" name="accomp" value="129" />1<br />
        <input type="radio" name="accomp" value="258" />2<br />
        <input type="radio" name="accomp" value="1057" />3<br />
        <input type="radio" name="accomp" value="1856" />4
    </div>
</form>
<div class="clear"></div>
<hr id="bar" />
<div class="clear"></div>
<div>
    <div class="col">
        <strong>Total: </strong><span id="total">0</span>
    </div>
    <div class="col" style="width:200px">
       <strong>Title: </strong><span id="title"></span> 
    </div>
</div>