如何简化一个空安全的compareTo（）实现？

Question

我正在compareTo()为这样的简单类实现方法（以便能够使用Collections.sort()和Java平台提供的其他功能）：

public class Metadata implements Comparable<Metadata> {
    private String name;
    private String value;

// Imagine basic constructor and accessors here
// Irrelevant parts omitted
}

我希望这些对象的 自然排序
是：1）按名称排序，以及2）如果名称相同，则按值排序；两种比较均应不区分大小写。对于这两个字段，空值都是完全可以接受的，因此compareTo在这些情况下不得中断。

我想到的解决方案是遵循以下思路的（我在这里使用的是“保护条款”，而其他人可能更喜欢单个返回点，但那一点并不重要）：

// primarily by name, secondarily by value; null-safe; case-insensitive
public int compareTo(Metadata other) {
    if (this.name == null && other.name != null){
        return -1;
    }
    else if (this.name != null && other.name == null){
        return 1;
    }
    else if (this.name != null && other.name != null) {
        int result = this.name.compareToIgnoreCase(other.name);
        if (result != 0){
            return result;
        }
    }

    if (this.value == null) {
        return other.value == null ? 0 : -1;
    }
    if (other.value == null){
        return 1;
    }

    return this.value.compareToIgnoreCase(other.value);
}

这可以完成工作，但是我对这段代码并不完全满意。诚然，它不是 很 复杂，但是非常冗长乏味。

问题是， 如何使它变得不太冗长 （同时保留功能）？如果有帮助，请随时参考Java标准库或Apache
Commons。使这个（稍微）简单一点的唯一选择是实现自己的“ NullSafeStringComparator”并将其应用于比较两个字段吗？

编辑1-3 ：埃迪的权利；修复了上面的“名字都为空”的情况

关于已接受的答案

我早在2009年就在Java 1.6上问了这个问题，当时我首选的
[是Eddie的纯JDK解决方案]

我将实现一个空安全比较器。可能有一个实现，但是实现起来如此简单，以至于我总是自己动手做。

注意：如果两个名称都为空，则上面的比较器甚至不会比较值字段。我认为这不是您想要的。

我将通过以下类似的方式实现此目的：

// primarily by name, secondarily by value; null-safe; case-insensitive
public int compareTo(final Metadata other) {

    if (other == null) {
        throw new NullPointerException();
    }

    int result = nullSafeStringComparator(this.name, other.name);
    if (result != 0) {
        return result;
    }

    return nullSafeStringComparator(this.value, other.value);
}

public static int nullSafeStringComparator(final String one, final String two) {
    if (one == null ^ two == null) {
        return (one == null) ? -1 : 1;
    }

    if (one == null && two == null) {
        return 0;
    }

    return one.compareToIgnoreCase(two);
}

编辑：修复了代码示例中的错别字。这就是我不先测试就得到的！

编辑：将nullSafeStringComparator提升为静态。

。直到现在（2017年），我再也没有回过头来改变它。

现在，我将 Lukasz Wiktor 的干净了公认的答案。如果在Java
8上，那绝对应该是首选，现在，Java 8应该可以用于几乎所有项目。

Answer 1

使用 Java 8 ：

private static Comparator<String> nullSafeStringComparator = Comparator
        .nullsFirst(String::compareToIgnoreCase);

private static Comparator<Metadata> metadataComparator = Comparator
        .comparing(Metadata::getName, nullSafeStringComparator)
        .thenComparing(Metadata::getValue, nullSafeStringComparator);

public int compareTo(Metadata that) {
    return metadataComparator.compare(this, that);
}