优化 jquery 复选框脚本

Question

我正在努力提高使用 Jquery 的能力，因此希望得到有关以下优化我的脚本的方法的反馈。

为了简要描述功能，我有一个“checkall”复选框和一个用于对选中元素执行操作的按钮。如果没有检查任何元素，我的按钮应该被禁用并且类也被禁用。如果只检查了一个元素，则不应禁用该按钮，也不应禁用该类。

提前致谢

$(document).ready(function(){

$('#checkall').click(function () {
    $(this).parents('.table_form:eq(0)').find(':checkbox').attr('checked', this.checked);

    if($(this).parents('.table_form:eq(0)').find(':checkbox').is(':checked')) {
        $("#delete_selected").attr("disabled");
        $("#delete_selected").removeClass("disabled");
    } else {
        $("#delete_selected").removeAttr("disabled").addClass("disabled");
    }
});

$("#blog_posts tbody :checkbox").click(checked_status);

});

function checked_status() {
    var n = $("input:checked").length;

    if(n > 0) {
        $("#delete_selected").attr("disabled");
        $("#delete_selected").removeClass("disabled");
    } else {
        $("#delete_selected").removeAttr("disabled").addClass("disabled");
    }
}

Answer 1

关于优化这里我没有太多要说的，但大概你正在尝试将其设置为禁用：

$("#delete_selected").attr("disabled"); // won't work, simply returns true or false

应该是：

$("#delete_selected").attr("disabled", true); // or "disabled"

你可以使用链接：

$("#delete_selected").attr("disabled", true)
                     .removeClass("disabled");

见http://api.jquery.com/attr/

另外，这个条件：

if($(this).parents('.table_form:eq(0)').find(':checkbox').is(':checked')) {

可以改成稍微简洁一点的：

if($(this).closest('.table_form').find(':checkbox:checked').length) {

但这确实是一种偏好。