使我的功能可重用

Question

我有一个可以正常工作的函数，但是我想使其可重用，以便以后在其他项目中使用。 当我尝试修改代码以接受参数而不是在函数中全部声明时，什么也没有发生。 这是我当前的工作代码……

var textColours = ["red", "orange", "green", "blue", "purple"];

var textToColour = document.getElementsByClassName("menuItem");

var randomColour = "";

var coloursUsed = [];

function randomize(array) {
 return array[Math.floor(Math.random() * array.length)];
}


function changeColour() {
for(var i = 0; i < textToColour.length; i++) {
randomColour = randomize(textColours);
if(!coloursUsed.includes(randomColour)){
  console.log("dingaling aling!");
  textToColour[i].style.color =  randomColour;
  coloursUsed.push(textToColour[i].style.color);
} else {
  i--
  console.log("duplicate");
}
console.log(coloursUsed);
console.log(coloursUsed.includes(randomColour));
 }
}


window.addEventListener("load", changeColour);

这是我要做的使我的代码可重用的事情.....

function changeColourDesired(element, colourArray) {
for(var i = 0; i < element.length; i++) {
randomColour = randomize(colourArray);
if(!coloursUsed.includes(randomColour)){
  console.log("dingaling aling!");
  element[i].style.color =  randomColour;
  coloursUsed.push(element[i].style.color);
} else {
  i--
  console.log("duplicate");
}
console.log(coloursUsed);
console.log(coloursUsed.includes(randomColour));
}
}

window.addEventListener("load", changeColourDesired(textToColour,  textColours));

对于第二段代码，我没有任何反应，也没有console.logs，文本仍然保持黑色。 我相信这确实很简单，但我看不到。

Answer 1

您正在将一个函数的结果而不是一个函数传递给addEventListener ：

window.addEventListener("load", function() { changeColourDesired(textToColour,  textColours); });