具有一对多关系的比较器

Question

我有一个包含文档实体列表的 DTO 对象 ClientInfo。

我使用以下比较方法按 documentCreationDate DESC 对 clientInfo 列表进行排序。

    @Override
        public int compareTo(ClientInfo o) {

            Date firstmaxDate = this.getDocuments().stream().map(d -> d.getCreateDate()).max(Date::compareTo).get();
            Date secondmaxDate = o.getDocuments().stream().map(d -> d.getCreateDate()).max(Date::compareTo).get();
            return firstmaxDate.compareTo(secondmaxDate);

        }
List<ClientInfo> clientInfos= serverReturnedList......;
Collections.sort(clientInfos);

但它得到一个例外，告诉比较规则已被违反。 所以我没有得到预期的结果。 谁能解释一下实现这一目标的方法。

这是异常堆栈跟踪

java.util.NoSuchElementException: No value present
        at java.util.Optional.get(Optional.java:135)
        at com.orsbv.hcs.dto.ClientInfo.compareTo(ClientInfo.java:346)
        at com.orsbv.hcs.dto.ClientInfo.compareTo(ClientInfo.java:24)
        at java.util.ComparableTimSort.countRunAndMakeAscending(ComparableTimSort.java:320)
        at java.util.ComparableTimSort.sort(ComparableTimSort.java:188)
        at java.util.Arrays.sort(Arrays.java:1312)
        at java.util.Arrays.sort(Arrays.java:1506)
        at java.util.ArrayList.sort(ArrayList.java:1462)
        at java.util.Collections.sort(Collections.java:141)

Answer 1

您应该保留compareTo()方法，因为它按照自然顺序（升序）进行排序，这是有道理的。

相反，您应该更改排序，以告诉它以反向（降序）顺序排序。

List<ClientInfo> clientInfoList = ...
Collections.sort(clientInfoList, Comparator.reverseOrder());

根据添加到问题的信息更新：

您的某些ClientInfo没有任何可用于检查创建日期的文档。

当你这样做时：

 Date firstmaxDate = this.getDocuments().stream().map(d -> d.getCreateDate()).max(Date::compareTo)

max()返回一个Optional<Date>允许它在空流的情况下返回Optional.empty() 。 当您有一个没有任何文档的 ClientInfo 时，您会在空的Optional上调用get() ，这会导致此异常。

您需要确定在真实数据中是否会出现这种情况，如果是，您希望如何对没有文档的ClientInfo 进行排序。 如果你所有的 createdDates 都是过去的，你可以简单地用“now”替换“no created date”：

        Date thisDate = this.getDocuments()
                .stream()
                .map(Document::getCreateDate)
                .max(Date::compareTo)
                .orElseGet(Date::new);
        Date otherDate = o.getDocuments()
                .stream()
                .map(Document::getCreateDate)
                .max(Date::compareTo)
                .orElseGet(Date::new);

或者，您可以在get()之前停止，使用Optional<Date>并检查isPresent()并根据需要进行处理。

Answer 2

首先，我向 clientInfo 添加了一个 maxDocuemntCreationDate 字段。

  Date maxCreationDate = clientInfo.getDocuments()
                        .stream()
                        .map(Document :: getCreateDate)
                        .max(Date::compareTo)
                        .orElseGet(Date::new);

然后我以这种方式比较

 @Override
            public int compareTo(ClientInfo o) {


              return 
              this.getMaxCreationDate().compareTo(o.getMaxCreationDate());

            }