将Java比较器逻辑移出实体？

Question

我目前有一个javaEntities：Cat和Kitten

下面是包含当前 comparator 逻辑的 Cat 实体，用于对 Cat 拥有最老小猫的 Cat 实体进行排序

public class Cat implements Comparable<Cat>
{

    //other fields and methods

    @OneToMany(mappedBy = "cat",fetch = FetchType.EAGER)
    private List<Kitten> kittenList= new ArrayList<Kitten>();

     public DateTime getOldestBornValueForKittenInList(){

        return kittenList.stream().min(Comparator.comparing(Kitten::getBorn))
                .get().getBorn();
    }

    public int compareTo(Cat c) {

        //fist compare on the oldest Created DateTime Value
        int i = getOldestBornValueForKittenInList().compareTo(c.getOldestBornValueForKittenInList());
        if (i != 0){
            return i;
        }

    }

}

我不再希望我的 Cat 实体实现 comparable 接口。我可以在我的应用程序中调用比较器吗？

即如果我想要 ordered Map 的猫和它们的小猫列表，其中最老的小猫是地图中的第一个条目：

Map<Cat, List<Kitten>> mapOfCatsAndKittens = new HashMap<Cat, List<Kitten>>();

Answer 1

请注意，您的代码包含几个不必要的重复项。在

public DateTime getOldestBornValueForKittenInList() {
    return kittenList.stream().min(Comparator.comparing(Kitten::getBorn)).get().getBorn();
}

您让比较器获取属性getBorn，尽管这是您唯一感兴趣的属性。因此您可以将操作简化为

public DateTime getOldestBornValueForKittenInList() {
    return kittenList.stream().map(Kitten::getBorn).min(Comparator.naturalOrder()).get();
}

通过首先映射到属性，您可以获得DateTime 元素流，您可以自然地比较这些元素，而无需在每次比较时重新获取属性。结果已经是最小值DateTime。

请记住，无论哪种情况，如果列表为空，操作都会失败。

但是Comparator.comparing 工厂仍然占有一席之地。当您想通过属性比较 Cat 对象时：

Comparator<Cat> oldestKitten=Comparator.comparing(Cat::getOldestBornValueForKittenInList);

您可以使用这样的比较器来创建按该属性排序的地图

Map<Cat, List<Kitten>> mapOfCatsAndKittens = new TreeMap<>(oldestKitten);

或对Cats 的列表进行排序：

List<Cat> list= … ; // should be a mutable list implementation, e.g. ArrayList
// perhaps add (some more) Cats
list.sort(oldestKitten);