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

首先，我向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());

            }

Answer 2

您应该保留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没有任何文档时，您将在空的get()上调用Optional，这将导致此异常。

您需要确定这种情况是否会出现在真实数据中，如果是，那么您要如何对没有文档的ClientInfo进行排序。如果您所有的createdDate都是过去的，则只需将“无创建日期”替换为“现在”：

        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()并根据需要进行处理。

具有一对多关系的比较器

2 个答案: