javascript - 以下是否敲响了警钟？ - $ ("body div div span #foo:eq(0)")

Question

我知道这是使用(我假设)jQuery 选择节点并选择 ID 的一种非常具体的方式可能会更高效。但除此之外，它是一个长选择器，还有什么在这里敲响“警钟”？

我问这个问题是因为我现在已经在一些工作网站上看到了这个问题，我只是想了解这是否比表面看起来更重要，以及它是否应该在面试中出现，我会明白如何回应。

在这里看到问题:http://www.reed.co.uk/jobs/senior-web-developer/29234531

Answer 1

是的。

• 太具体了
• 它对 HTML 结构的变化非常脆弱
• 它使用了一个id(意味着每个页面是唯一的)但是用其他选择器来限定它，这应该是不必要的。当一个 JS 文件在多个页面上运行时，有时这是必要的，但这通常是一个糟糕的设计。
• 它使用 :eq(0) 这会降低性能，因为选择器不能交给 document.querySelectorAll()
• 由于使用了 id，返回的元素应该只是一个元素。

它应该只是 $('#foo')，它将直接传递给 document.getElementById()(除非 Sizzle 自从我上次阅读源)或简单的原生 DOM API。