根据条件比较数组列表中的对象字段值，如果条件为真则删除答案

【问题标题】：Compare object field values in array list based on conditions and remove if condition is true根据条件比较数组列表中的对象字段值，如果条件为真则删除
【发布时间】：2018-12-06 21:47:26
【问题描述】：

假设这些是 java bean 类-

class Camp {
      int number;
      User userId;
      Date createDt;
 }

class User {
  private String id;
  private String name;
}

我想从Camp 类型的arrayList 中删除那些具有相同userId 但较小createDt 的记录。

假设这是列表-

// ... the list is already populated
List<Camp> usersList = populatedListOfUser();

这就是方法-

private List<Camp> skipByComparison(List<Camp> usersList){
    List<Camp> mainList = new ArrayList<Camp>();
    Set<Camp> s = new TreeSet<Camp>(new Comparator<Camp>() {

        @Override
        public int compare(Camp o1, Camp o2) {
            if(o1.getUserId().getId().equals(o2.getUserId().getId())) {
                if(o1.getCreatedTs().compareTo(o2.getCreatedTs()) < 0) {
                    usersList.remove(o1);
                    return -1;
                } else {
                    return 1;
                }
            } else {
                return 0;
            }
        }
    });
    s.addAll(usersList);
    mainList.addAll(s);
}

所以当我打电话时 -

usersList = skipByComparison(usersList);

我没有从 Camp 类型的 arrayList 中获取记录，这些记录具有相同的 userId 但较小的 createDt。

例如。如果我有 2 条记录具有相同的 Camp.getUser().getId()，但 record 1 的 CreateDt 大于 record 2，那么我想删除 record 2。

【问题讨论】：

这是什么UserId userId;？ yiu 的意思是User userID 吗？
@john- 是的。基本上我只想在 Camp.getUser().getId() 相同时删除那些没有最新 CreateDt 的记录。例如。如果我有 2 条具有相同 Camp.getUser().getId() 的记录，但记录 1 的 CreateDt 大于记录 2，那么我想删除记录 2。
skipByComparison()中没有return吗？

标签： java java-8 comparator

【解决方案1】：

您可以将代码更新为：

private List<Camp> skipByComparison(List<Camp> usersList){
    Set<Camp> s = new TreeSet<>((o1, o2) -> o1.getUserId().getId().equals(o2.getUserId().getId()) ? 
            o1.getCreateDt().compareTo(o2.getCreateDt()) : 0);
    s.addAll(usersList);
    return new ArrayList<>(s);
}

或者更好的是你可以使用Collectors 和流来解决它：

private List<Camp> skipByComparisonStream(List<Camp> usersList) {
    return new ArrayList<>(usersList.stream()
            .collect(Collectors.toMap(c -> c.getUserId().getId(),
                    Function.identity(),
                    (a, b) -> a.getCreateDt().after(b.getCreateDt()) ? a : b))
            .values());
}

说明：当两个日期可能是equal 的情况下，您的Comparator 目前没有返回任何值，您可能应该只依赖@987654328 的compareTo 实现@ 类：

Set<Camp> s = new TreeSet<>(new Comparator<Camp>() {
    @Override
    public int compare(Camp o1, Camp o2) {
        return o1.getUserId().getId().equals(o2.getUserId().getId()) 
                ? o1.getCreateDt().compareTo(o2.getCreateDt()) : 0;
    }
});

完成此操作后，您可以使用 lambda 表达式将其简化为：

Set<Camp> s = new TreeSet<>((o1, o2) -> o1.getUserId().getId().equals(o2.getUserId().getId())
        ? o1.getCreateDt().compareTo(o2.getCreateDt()) : 0);

我所做的其余修改是简化执行顺序并清理多余的初始化。

旁白：我刚刚在另一个问题中得到了更正，并建议您使用java.time.LocalDate 代替createDt 属性。

【讨论】：

你们对最新的 java8 知识非常了解。太棒了！

【解决方案2】：

您可以使用toMap 收集器：

Collection<Camp> values = usersList.stream()
                .collect(toMap(c -> c.getUser().getId(),
                        Function.identity(),
                        BinaryOperator.maxBy(Comparator.comparing(Camp::getCreateDt))))
                .values();

see this 回答有关 toMap 如何工作的解释

【讨论】：