javascript - 我如何向其中添加选择器更改事件？

Question

这是我的 jquery，效果很好

$(document).ready(function(){
    $('#new_average, #max_occupancy').keyup(function(){
        factor = $(".industry_picker option:checked").attr("title");
        average = $('#new_average').val();
        if($('#new_average').val().indexOf("$") != -1){
            average = average.substring(1);                                                     
        }
        if($("#max_occupancy").val() != ""){
            max_occupancy = $("#max_occupancy").val();
            max_total = ((average * factor) * max_occupancy) * 30;
            if(factor != ""){
                $("#new_calc").val("$" + max_total + ".00");
            }

        }                                                   
    });
});

但问题是我需要它在按键和启动时触发

 $(".industry_picker") 

改变了......我知道我可以复制这个确切的功能并拥有一个

 $(".industry_picker").change 

但我觉得这是一个黑客......有没有更好的方法来拥有

$('#new_average, #max_occupancy").keyup 

和

$(".industry_picker").change  

在同一个选择器中或实现此目的的更好方法，而无需再次复制整个函数

Answer 1

不要使用匿名函数。

function myCallback(){
    factor = $(".industry_picker option:checked").attr("title");
    average = $('#new_average').val();
    if($('#new_average').val().indexOf("$") != -1){
        average = average.substring(1);                                                     
    }
    if($("#max_occupancy").val() != ""){
        max_occupancy = $("#max_occupancy").val();
        max_total = ((average * factor) * max_occupancy) * 30;
        if(factor != ""){
            $("#new_calc").val("$" + max_total + ".00");
        }

    }
}

$('#new_average, #max_occupancy').keyup(myCallback);
$('.industry_picker').change(myCallback);

旁注:您没有在任何地方使用var，这意味着只有一个副本每个变量的值:factor、average、max_ocupancy 和 max_total。引入全局状态是草率的编码，并使程序更难以维护和调试。 使用 var 声明变量。

推荐阅读:MDC var docs .