forEach循环中if与else分支同时执行，需满足条件立即终止循环

问题原因 #

你当前使用forEach循环遍历员工数据，forEach无法中途终止遍历。一旦数组中存在至少一个匹配departmentID的员工，同时还有不匹配的员工，就会先执行if分支显示preventdel模态框，后续不匹配的条目又会执行else分支显示confirmdel，导致两个模态框同时弹出。

解决方案 #

方法1：使用for循环手动控制终止 #

用普通for循环替代forEach，找到匹配项后立即break终止循环，循环结束后根据是否找到匹配项决定显示哪个模态框：

$.ajax({
  url: "php/getall.php",
  type: 'GET',
  dataType: 'json',
  success: function(result) {
    const employees = result['data'];
    console.log(employees);
    
    let hasMatchingDept = false;
    // 用for循环遍历，方便终止
    for (let i = 0; i < employees.length; i++) {
      const employee = employees[i];
      if (employee.departmentID === deptid) {
        hasMatchingDept = true;
        break; // 找到匹配项，立即终止循环
      }
    }
    
    // 根据结果显示对应模态框
    if (hasMatchingDept) {
      $('#preventdel').modal('show');
    } else {
      $('#confirmdel').modal('show');
    }
  }
})

方法2：使用Array.some()方法 #

Array.some()会遍历数组，只要有一个元素满足条件就返回true，并且立即停止遍历，非常适合这种"存在性检查"场景：

$.ajax({
  url: "php/getall.php",
  type: 'GET',
  dataType: 'json',
  success: function(result) {
    const employees = result['data'];
    console.log(employees);
    
    // 检查是否存在匹配departmentID的员工，找到后立即停止遍历
    const hasMatchingDept = employees.some(employee => {
      return employee.departmentID === deptid;
    });
    
    // 显示对应模态框
    if (hasMatchingDept) {
      $('#preventdel').modal('show');
    } else {
      $('#confirmdel').modal('show');
    }
  }
})

关键说明 #

• 两种方法都避免了遍历所有元素，找到匹配项后立即停止，确保只会触发一个模态框。
• 优先推荐Array.some()，代码更简洁，语义更清晰，符合现代JS写法。

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