querySelectorAll对指定元素不起作用

Question

我正在尝试使用循环的querySelectorAll选择我想要的所有指定元素，但仍然失败。 始终仅选择第一个元素。 这是我的代码：

HTML：

<div id="text">Text 1</div>
<div id="text">Text 2</div>
<div id="text">Text 3</div>

JS：

function query(selector) {
    var elem = document.querySelectorAll(selector);

    if(elem.length) {
        for(var i = 0; i < elem.length; i++) {
            return elem[i];
        }
    }
}

query("#text").style.background = "red";

我做错了什么？

Answer 1

return语句终止函数执行，即不执行后续代码。 您的函数将在循环的第一次迭代中返回第一个被选择的元素，这就是它的结束。

在这种情况下，无需迭代集合。 如果要获取所有选定的元素，则可以返回querySelectorAll方法的返回值，但是由于返回的集合没有这种属性，因此不能使用style属性。

function query(selector) {
    // convert the NodeList into a regular array
    // https://developer.mozilla.org/en-US/docs/Web/API/NodeList
    return Array.prototype.slice.call(document.querySelectorAll(selector));
}

// since `query` returns an array you can use it's `forEach` method 
// for iterating over the array
query(".text").forEach(function(el) {
   el.style.backgroundColor = "red";
});

请注意，ID必须是唯一的。 您应该改用类。

Answer 2

问题出在return语句中，该语句仅返回第一个元素 ，然后对其应用链。 之后，它不再处理任何其他元素。

---

您可以使用此技巧，并拥有真正的通用涂药器 。

function query(selector, callback) {
  [].forEach.call(document.querySelectorAll(selector), callback)
}

这就是如何使用它。

query('#option', function(el){el.style.backgroundColor = "red"})
// or ES6
query('#option', el => el.style.backgroundColor = "blue")

一次应用多个更改：

query('#option', el => {
  el.style.backgroundColor = "green"
  el.style.color = 'cyan'
  el.style.textDecoration = 'line-through'
})

---

或mimick chaining ，如果您出于任何原因想要讨厌...

function query(selector, callback) {
  [].forEach.call(document.querySelectorAll(selector), callback) 

  return arguments.callee.bind(undefined, selector)
}

用法：

query('#option', el => el.style.backgroundColor = 'red')(el => el.style.color = 'black')(el => el.style.textDecoration = 'underline')

Answer 3

使用class而不是id -ID必须是唯一的，但是可以将类分配给所需的任意多个元素，也可以为每个元素分配尽可能多的类，因此

<div class="mytext">Text 1</div>
<div class="mytext">Text 2</div>
<div class="mytext">Text 3</div>

我建议您改用Jquery，这样就可以将背景颜色设置为

$(".mytext").css({background:"red"});