Java中使用compareTo排序对象自身列表属性的实现是否正确？

• 首先，Comparable 接口定义的是类的自然默认排序，是全局生效的排序规则，不应该和某一个特定业务场景的排序需求绑定。你现在把compareTo硬编码成年龄降序，相当于所有涉及Person对象排序的场景默认都会按年龄从大到小排，后续如果出现需要按年龄升序、按姓名排序等其他需求时，这个默认规则会直接冲突，非常容易引出隐蔽bug。而且从通用认知来说，年龄的自然排序通常是升序，你写死降序也不符合其他开发者的使用预期。
• 其次，你直接对getChildren()返回的原列表执行Collections.sort，会永久修改原children列表的元素顺序，那些不需要排序的场景下次读取children时，拿到的也是被篡改过顺序的列表，完全违背你“部分场景需要排序、部分场景不需要”的初始需求，属于典型的意外副作用。
• 最后，你担心“每次使用都新建Comparator实例浪费性能”属于完全没必要的过度优化。无状态的Comparator完全可以定义成静态常量全局复用，根本不需要每次调用都新建实例。

不要为了单个场景的排序需求强行给Person实现Comparable接口，直接在Person类中定义一个全局复用的静态比较器常量即可：

public class Person {
    // 全局唯一的年龄降序比较器，类加载时初始化一次，所有场景共用
    public static final Comparator<Person> AGE_DESC = Comparator.comparingInt(Person::getAge).reversed();

    private int age;
    private List<Person> children = new ArrayList<>();

    // 省略其他属性、构造方法、普通getter/setter
    public int getAge() {
        return age;
    }

    public List<Person> getChildren() {
        return children;
    }
}

根据你是否需要保留原children列表的顺序，选择对应的调用方式即可：

• 如果children本身没有固定存储顺序的要求，允许直接修改原列表顺序，直接复用静态比较器排序即可，不会产生额外的Comparator实例：

person.getChildren().sort(Person.AGE_DESC);
for (Person child : person.getChildren()) {
    // 执行业务逻辑
}

• 如果需要保留原children列表的顺序，避免影响其他不需要排序的场景，先复制一份列表副本，对副本排序后再遍历：

List<Person> sortedChildren = new ArrayList<>(person.getChildren());
sortedChildren.sort(Person.AGE_DESC);
for (Person child : sortedChildren) {
    // 执行业务逻辑
}

补充说明：你原来写的比较逻辑用==判断int值虽然不会出错，但如果后续age改成Integer包装类型，会出现引用比较错误、空指针等问题，用JDK提供的Comparator.comparingInt或者Integer.compare()写法更健壮，能规避这类问题。

内容的提问来源于stack exchange，提问作者MrMadium