如何动态传递元素id并调用javascript函数？

Question

我有如下所示的 javascript 代码 -

function initPage(){

    // Left Navigation Pane - moverOver effect:
    document.getElementById("imgHowToBuy").onmouseover = leftNavHoverIn;
    document.getElementById("imgNewAddition").onmouseover = leftNavHoverIn; 
    document.getElementById("imgMostPopular").onmouseover = leftNavHoverIn;
    document.getElementById("imgOffer").onmouseover = leftNavHoverIn;   
    document.getElementById("imgRecentlySold").onmouseover = leftNavHoverIn;
.....
}

基本上我的代码可以工作（该函数被调用并执行得很好）。但我不认为我在这里使用了最佳实践的原则。不断调用同一个函数看起来有点奇怪；我可以以某种方式传递元素的 id 作为参数，然后执行该函数，以便整个事情减少到一行代码吗？

我是一个自学的 js 人：D

谢谢！

Answer 1

您不是“调用相同的函数”，您只是将单个函数分配给页面上的许多元素。我认为这里没有什么特别错误的地方。

您可以存储要分配此函数的所有元素的 id，这将减少代码行数，但没有具体原因这样做。

var leftNavElements = ["imgHowToBuy","imgNewAddition"]; // etc....
for(var i=0;i<leftNavElements.length;i++){
    document.getElementById(leftNavElements[i]).onmouseover = leftNavHoverIn;
}

Answer 2

您可以这样做：

function assignMouseOver(elem) {
  document.getElementById(elem).onmouseover = leftNavHoverIn;
}

assignMouseOver("imgHowToBuy"); // etc

您还可以将元素 ID 作为数组传递，并循环遍历函数内的数组，这会将其减少为一行（可见）代码：

function assignMouseOver(elems) {
  if (elems.length>0) {
   for (var i=0; i<elems.length; i++) {
     document.getElementById(elems[i]).onmouseouver = leftNavHoverIn;
   }
  }
}

assignMouseOver(["imgHowtoBuy","etc","etc"]);

最佳实践需要更多检查/验证但以上内容应该有所帮助。

Answer 3

试试这个：

var ids = ["imgHowToBuy", "imgNewAddition", "imgMostPopular", "imgOffer", "imgRecentlySold"]

var setMouseover = function (id) {
    document.getElementById(id).onmouseover = leftNavHoverIn;
};

for(var i=0;i<ids.length;i++) {
    setMouseover(ids[i]);
}

Answer 4

function doTheJob(id)
{
    document.getElementById(id).onmouseover = leftNavHoverIn
}

doTheJob("imgHowToBuy");
doTheJob("imgNewAddition");
doTheJob("imgMostPopular");
doTheJob("imgOffer");
doTheJob("imgRecentlySold");