Java列表删除项而不执行任何操作

Question

private void updateCommentIfNeeded() {
    List<AccessRequest> accessRequestList = getAllRequest();
    List<AccessRequest> commentsList = getCommentsListProcessedManually(accessRequestList);
}

public List<AccessRequest> getCommentsListProcessedManually(List<AccessRequest> accessRequests) {

    accessRequests.removeIf(ar -> !ar.getComment().equals("To be processed manually"));

    if (accessRequests.size() != 0) {
        SQLServerConnection sqlServerConnection = new SQLServerConnection(sqlServerUrl);
        accessRequests.removeIf(ar -> !sqlServerConnection.emailExists(ar.getEmail()));
    }
    return accessRequests;
}

• 我的第一个列表中有3个对象，但只有一个包含必需的注释
• 两个列表都以包含包含注释的唯一对象结尾

如果有人有主意的话，我就有点迷路了！

Answer 1

您的方法getCommentsListProcessedManual修改您正在传递的列表。我相信您的操作假设是将列表作为参数传递，以某种方式创建了列表的副本，而实际发生的是通过值传递对列表的引用。

有几种方法可以解决这个问题，但最简单的方法是在方法开始时创建输入列表的副本：

public List<AccessRequest> getCommentsListProcessedManually(List<AccessRequest> input) {
    List<AccessRequest> accessRequests = new ArrayList<>(input);

    accessRequests.removeIf(ar -> !ar.getComment().equals("To be processed manually"));

    if (accessRequests.size() != 0) {
        SQLServerConnection sqlServerConnection = new SQLServerConnection(sqlServerUrl);
        accessRequests.removeIf(ar -> !sqlServerConnection.emailExists(ar.getEmail()));
    }
    return accessRequests;
}

您还可以为此使用Stream API（使用filter操作），但在这种情况下，这会比较棘手。