函数已定义却提示未使用？拖拽功能脚本报错求助

我帮你梳理下代码里的两个关键问题：

1. "函数已定义但未使用"的核心原因 #

你确实写了addDnDHandlers函数，但整个代码里没有任何地方调用它！JS引擎识别到你定义了这个函数，但没看到它被执行，所以会抛出这个提示。

解决方法很直接：在函数定义的外部加上调用语句，最好确保DOM加载完成后再执行（避免找不到页面元素的问题），比如：

document.addEventListener('DOMContentLoaded', function() {
  addDnDHandlers();
});

如果你的脚本是放在HTML的</body>标签前，直接写addDnDHandlers();也能正常工作。

2. 循环逻辑的致命bug #

你的循环条件for (var i = 0; i <= coffeeimages; i++)完全错误：

• coffeeimages是getElementsByClassName返回的HTMLCollection对象，不是数字，直接和i比较会得到false，导致循环一次都不会执行，所有拖放事件都没绑定上。

正确的写法应该用集合的长度属性来遍历：

for (var i = 0; i < coffeeimages.length; i++) {
  // 事件绑定代码
}

修正后的完整代码 #

function addDnDHandlers() { 
  var coffeeimages = document.getElementsByClassName("productarticlewide"); 
  var shoppingcartDropZone = document.getElementById("shoppingcart"); 
  var shoppingcart = document.querySelectorAll("#shoppingcart ul")[0]; 

  // 修复循环条件
  for (var i = 0; i < coffeeimages.length; i++) { 
    coffeeimages[i].addEventListener("dragstart", function (ev) { 
      ev.dataTransfer.effectAllowed = 'copy'; 
      ev.dataTransfer.setData("Text", this.getAttribute("id")); 
    }, false); 
  } 

  shoppingcartDropZone.addEventListener("dragover", function (ev) { 
    if (ev.preventDefault) ev.preventDefault(); 
    ev.dataTransfer.dropEffect = "copy"; 
    return false; 
  }, false); 

  shoppingcartDropZone.addEventListener("drop", function (ev) { 
    if (ev.stopPropagation) ev.stopPropagation(); 
    var coffeeId = ev.dataTransfer.getData("Text"); 
    var element = document.getElementById(coffeeId); 
    addCoffeeToShoppingCart(element, coffeeId); 
    ev.stopPropagation(); 
    return false; 
  }, false); 

  function addCoffeeToShoppingCart(item, id) { 
    var html = id + " " + item.getAttribute("data-price"); 
    var liElement = document.createElement('li'); 
    liElement.innerHTML = html; 
    shoppingcart.appendChild(liElement); 
  } 
}

// 调用初始化函数
document.addEventListener('DOMContentLoaded', function() {
  addDnDHandlers();
});

这样修改后，不仅能消除“函数未使用”的提示，拖放功能也能正常工作了。

内容的提问来源于stack exchange，提问作者munzi Syed