javascript getElementsByClassName 和 setAttribute 不起作用

Question

我添加了 4 个“范围”类型输入，并使用它们的值来设置另一个元素的 RGBA 值，但它不起作用。

window.onload = init;

function init(){
document.getElementById("colorR").onchange() = setColors;
document.getElementById("colorG").onchange() = setColors;
document.getElementById("colorB").onchange() = setColors;
document.getElementById("Opacity").onchange() = setColors;  
}
function setColors(){
var r = document.getElementById("colorR").value;
var g = document.getElementById("colorG").value;
var b = document.getElementById("colorB").value;
var a = document.getElementById("Opacity").value;

    //alert(r+", "+g+", "+b+", "+a);

document.getElementsByClassName("previewAreaBox")[0].style.setAttribute(
"background-color", "rgba(",r+", "+g+", "+b+", "+a/100+");");

}

任何帮助将不胜感激。谢谢。

Answer 1

style.setAttribute(

setAttribute 适用于元素，而不是样式。

而background-color 是样式属性，而不是属性。

document.getElementsByClassName("previewAreaBox")[0].style.backgroundColor = ".."

Answer 2

至少，

document.getElementById("colorR").onchange() = setColors;

应该改为

document.getElementById("colorR").onchange = setColors;

如果不进行更改，其余代码将根本无法运行，因为onchange() 调用事件处理程序，而不带括号的onchange 用于分配事件处理程序。

也

正如康纳的有用评论所述，安迪的回答包含部分解决方案，我还注意到您的代码中还有一个错误：

"rgba(",r+", "+g+", "+b+", "+a/100+");"

需要是这样的（加上现在逗号所在的位置，以及算术周围的括号以防止其左操作数过早转换为字符串）：

"rgba(" +r+ ", " +g+ ", " +b+ ", " + (a/100) +");"

因此，将其插入安迪的答案中，setColors 函数的最后一行应该是这样的：

document.getElementsByClassName("previewAreaBox")[0].style.backgroundColor = "rgba(" +r+ ", " +g+ ", " +b+ ", " + (a/100) +");";

Answer 3

好的，这里有几个问题

1.您需要将函数分配给更改事件，而不是调用更改事件。

2。您不要在样式上使用 setAttribute，而是使用它在元素上设置属性。

3。你应该缓存你的元素。

window.onload = init;


var elementR, elementG, elementB, opacityElement;

function init(){
   elementR = document.getElementById("colorR");
   elementG = document.getElementById("colorG");
   elementB = document.getElementById("colorB");
   opacityElement = document.getElementById("Opacity");

   // Set change events
   elementR.onchange = 
   elementG.onchange = 
   elementB.onchange = 
   opacityElement.onchange = setColors;
}
function setColors(){
    var r = elementR.value,
        g = elementG.value,
        b = elementB.value,
        a = opacityElement.value,
        preview = document.getElementsByClassName("previewAreaBox")[0];
        preview.style.backgroundColor =  'rgba(' + r + ',' + g + ',' + b + ',' + a / 100 + ')';
}

---

Element.onchange() = something; 

不起作用，因为onchange(); 实际上触发了事件并且没有将函数分配给它。

你也可以像这样使用Array.join 来表示 rgba

var rgba = [elementR.value, elementG.value, elementB.value, 
            opacityElement.value / 100];
preview.style.backgroundColor =  'rgba(' + rgba.join(',') + ')';