jQuery Selector预计

Question

我的问题与我的jQuery代码在哪里我的Firebug给我警告：选择器的预期。

下面是代码：

$("img[id$='_tick']").each(function() { 

    $(this).click(function() { 

     var tickIDFinder = $(this).attr("id"); 
     var tickSection = tickIDFinder.slice(0,1); 

     var checkboxID = "\"input[id^='" + tickSection + "_checkbox_']\""; 
     var airTableID = "#" + tickSection + "_airline_table tr"; 

     $(checkboxID).each(function() { 

      if ($(this).is('checked')) { 
       alert("Checkbox is Checked."); 
      } 

      $(this).attr('checked','checked'); 

     }); 

    }); 

}); 

我试图做的是写一个jQuery允许用户点击一个链接（如图像）。当用户点击这个图像时，它会'检查'指定的所有复选框。然后我会从Firebug获得Selector预期的警告。

jQuery本身正在按照我的预期工作。我只是想尝试解决警告。

任何人都可以帮忙吗？我真的很感激。

感谢您的想法堆。

Answer 1

除了其他的答案，这可以进一步简化，像这样：

$("img[id$='_tick']").click(function() { 
    var tickSection = this.id.slice(0,1); 

    var checkboxID = "input[id^='" + tickSection + "_checkbox_']"; 
    var airTableID = "#" + tickSection + "_airline_table tr"; 

    $(checkboxID).each(function() { 
     if (this.checked) { 
      alert("Checkbox is Checked."); 
     } 
     $(this).attr('checked', true); 
    }); 
}); 

这里有几件事：

• 你不需要.each()，只需.click()，它会绑定到他们所有的人。
• 在可用时使用DOM属性，例如， this.id和this.checked
• 的checkboxID有额外的引号的选择，删除它们

可能找不到调试，这是更简单，如.attr()作品上的多个元素，以及：

$("img[id$='_tick']").click(function() { 
    $("input[id^='" + this.id.slice(0,1) + "_checkbox_']").attr('checked', true); 
}); 

Answer 2

更换

if ($(this).is('checked')) 

与

if ($(this).is(':checked')) 

见:checked

您可以使用this.id这将快于$(this).attr("id")

var tickIDFinder = this.id; 

Answer 3

更换

var checkboxID = "\"input[id^='" + tickSection + "_checkbox_']\""; 

与

var checkboxID = "input[id^='" + tickSection + "_checkbox_']"; 

（除了什么@rahul说:)