切换显示/隐藏DIV功能出现故障，求助排查代码问题

看起来你的切换功能出了问题，大概率是因为判断元素显示状态的方式不准确，再加上代码有不完整的地方，咱们一步步拆解解决：

先梳理现有代码的核心问题 #

1. 依赖element.style.display判断状态不可靠
   你用tbl.style.display == ""来判断元素是否显示，但style.display只能读取内联样式里的display值。如果元素的显示是通过CSS类、默认样式（比如块级元素默认display:block）设置的，那tbl.style.display会是空字符串，这时候判断逻辑会彻底混乱——比如元素明明是显示的，但判断条件却成立，导致点击行为完全不符合预期。

2. 代码逻辑不完整
   你的else分支里e.innerHTML = ...没有写完，这会导致元素从隐藏切换回显示时，图标无法同步切换成减号，功能缺失。

3. 修改原生原型的潜在风险
   直接给HTMLCollection.prototype和NodeList.prototype赋值Array.prototype.forEach虽然能生效，但可能和其他脚本产生冲突，不是最佳实践。

修复后的完整可运行代码 #

// 推荐用Array.from或者扩展运算符处理集合，避免修改原生原型
// 示例：Array.from(document.querySelectorAll('.your-target-class')).forEach(...)

function HideShow(e, itm_id) {
  const tbl = document.getElementById(itm_id);
  if (!tbl) return; // 先判断元素是否存在，避免控制台报错

  // 获取元素实际的显示状态（包含CSS类、默认样式的影响）
  const isVisible = window.getComputedStyle(tbl).display !== 'none';

  if (isVisible) {
    // 隐藏元素，切换为加号图标
    e.innerHTML = "<i class='fa fa-plus' aria-hidden='true'></i>";
    tbl.style.display = 'none';
  } else {
    // 显示元素，切换为减号图标
    e.innerHTML = "<i class='fa fa-minus' aria-hidden='true'></i>";
    tbl.style.display = ''; // 恢复默认显示规则，比硬设block更灵活
  }
}

额外优化建议（更优雅的实现方式） #

用CSS类控制显示隐藏是更专业的做法，样式和逻辑分离，后期维护更方便：

/* 定义隐藏类 */
.hidden {
  display: none !important;
}

对应的JS代码：

function HideShow(e, itm_id) {
  const tbl = document.getElementById(itm_id);
  if (!tbl) return;

  const isHidden = tbl.classList.contains('hidden');
  if (isHidden) {
    tbl.classList.remove('hidden');
    e.innerHTML = "<i class='fa fa-minus' aria-hidden='true'></i>";
  } else {
    tbl.classList.add('hidden');
    e.innerHTML = "<i class='fa fa-plus' aria-hidden='true'></i>";
  }
}

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