多选删除功能异常：仅删除表格首行数据求助

我来帮你搞定这个批量删除的问题，你的代码里有两个核心原因导致不管选多少行，只会删第一条数据：

1. HTML元素id重复：所有复选框都用了id="del"，但HTML里id是全局唯一的，document.getElementById("del")只会拿到DOM中第一个匹配的复选框值，所以永远只会传第一条的patientId给后端。
2. 前后端都只处理单个id：前端删除函数只传递一个pid，后端也只接收单个id做删除，完全没做批量处理的逻辑。

下面是具体的修改方案：

1. 前端代码修改（Masterpage.jsp） #

先把复选框的id改成class（class可以重复使用），然后修改删除函数，收集所有选中的复选框id，拼成字符串传给后端：

<script type="text/javascript">
function addNewPatient(){
    debugger;
    var addNewPat = "true";
    document.form.method = "POST";
    document.form.action = "addPatient?newPage="+addNewPat;
    document.form.submit();
}

function deletePatient(){
    debugger;
    // 收集所有被选中的患者ID
    var selectedIds = [];
    // 通过class选择所有选中的复选框
    var checkboxes = document.querySelectorAll('.del-checkbox:checked');
    checkboxes.forEach(checkbox => {
        selectedIds.push(checkbox.value);
    });
    
    // 先判断有没有选中项
    if(selectedIds.length === 0){
        alert("请先选择要删除的患者哦");
        return;
    }
    
    var delpatient = "true";
    document.form.method = "POST";
    // 把选中的ID用逗号拼接成字符串传递
    document.form.action = `deletePatient?delPage=${delpatient}&pids=${selectedIds.join(',')}`;
    document.form.submit();
}
</script>

<c:forEach var="user" items="${patients}">
<tr>
    <td>
        <!-- 将id改为class，避免重复 -->
        <input type="checkbox" class="del-checkbox" value="<c:out value="${user.patientId}"/>" >
    </td>
    <td>
        <input type="hidden" id="delpatient"><c:out value="${user.patientId}"/>
    </td>
    <td><c:out value="${user.patientName}" /></td>
    <td><c:out value="${user.patientAddress}" /></td>
    <td><c:out value="${user.patientPhone}" /></td>
</tr>
</c:forEach>

<!-- 删除按钮不需要传参数，函数内部自己收集选中项 -->
<div class="ui small button" onClick="deletePatient()">
    <i class="trash alternate icon"></i> Delete
</div>

2. 后端代码修改（DeletePatientServlet.java） #

修改后端逻辑，接收逗号分隔的ID字符串，分割后批量处理删除：

public class DeletePatientServlet extends HttpServlet {
    private static final long serialVersionUID = 1L;
    private PatientDAO patientDao = null;

    protected void doPost(HttpServletRequest request, HttpServletResponse response)throws ServletException, IOException {
        // 获取前端传递的选中ID字符串
        String pidsStr = request.getParameter("pids");
        
        // 校验参数是否为空
        if(pidsStr == null || pidsStr.trim().isEmpty()){
            System.out.println("未选择任何要删除的患者");
            RequestDispatcher rd = request.getRequestDispatcher("/Pages/MasterPage.jsp");
            rd.forward(request, response);
            return;
        }
        
        // 分割成ID数组
        String[] pidsArray = pidsStr.split(",");
        patientDao = new PatientDAO();
        
        boolean allDeletedSuccessfully = true;
        
        // 循环处理每个ID的删除
        for(String pidStr : pidsArray){
            try{
                int patientId = Integer.parseInt(pidStr);
                boolean deleteResult = patientDao.isPatientDeleted(patientId);
                
                if(!deleteResult){
                    allDeletedSuccessfully = false;
                    System.out.println("删除患者ID: " + patientId + " 失败");
                }
            }catch(NumberFormatException e){
                allDeletedSuccessfully = false;
                System.out.println("无效的患者ID: " + pidStr);
                e.printStackTrace();
            }
        }
        
        // 输出结果日志
        if(allDeletedSuccessfully){
            System.out.println("所有选中的患者都删除成功啦！");
        }else{
            System.out.println("部分患者删除失败，请查看日志排查问题");
        }
        
        // 跳转回主页面
        RequestDispatcher rd = request.getRequestDispatcher("/Pages/MasterPage.jsp");
        rd.forward(request, response);
    }
}

优化建议：DAO层批量删除 #

上面的循环单删虽然能解决问题，但效率不高，建议在PatientDAO里实现批量删除的方法，用SQL的IN关键字一次性删除，示例如下：

// PatientDAO中的批量删除方法
public boolean batchDeletePatients(List<Integer> patientIds) {
    if(patientIds == null || patientIds.isEmpty()){
        return false;
    }
    
    // 构建批量删除的SQL语句
    StringBuilder sqlBuilder = new StringBuilder("DELETE FROM patients WHERE patient_id IN (");
    for(int i = 0; i < patientIds.size(); i++){
        sqlBuilder.append("?");
        if(i != patientIds.size() - 1){
            sqlBuilder.append(",");
        }
    }
    sqlBuilder.append(")");
    
    try(Connection conn = getConnection()){ // 假设你有获取连接的方法
        PreparedStatement ps = conn.prepareStatement(sqlBuilder.toString());
        
        // 设置参数
        for(int i = 0; i < patientIds.size(); i++){
            ps.setInt(i + 1, patientIds.get(i));
        }
        
        // 执行删除，返回受影响的行数
        int affectedRows = ps.executeUpdate();
        return affectedRows == patientIds.size();
    }catch(SQLException e){
        e.printStackTrace();
        return false;
    }
}

然后在Servlet中调用这个批量方法，替换循环单删的逻辑：

// 在DeletePatientServlet的doPost方法中替换循环部分
List<Integer> patientIdList = new ArrayList<>();
for(String pidStr : pidsArray){
    try{
        patientIdList.add(Integer.parseInt(pidStr));
    }catch(NumberFormatException e){
        allDeletedSuccessfully = false;
        System.out.println("无效的患者ID: " + pidStr);
        e.printStackTrace();
    }
}

if(!patientIdList.isEmpty()){
    boolean deleteResult = patientDao.batchDeletePatients(patientIdList);
    allDeletedSuccessfully = deleteResult;
}

这样就能更高效地完成批量删除了。

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