如何简化在多个字段上创建null安全比较器？

Question

我有一个简单的比较器，它通过日期和时间字段比较Action对象：

static final Comparator<Action> COMPARATOR = comparing(Action::date, 
nullsLast(naturalOrder())).thenComparing(Action::time, 
nullsLast(naturalOrder()));

结果示例如下所示：

{hour: 01/01/2019, time: 15:55}
{hour: 01/01/2019, time: null}
{hour: 03/01/2019, time: 11:11}
{hour: 08/01/2019, time: 11:11}
{hour: 08/01/2019, time: null}
{hour: null, time: null}
{hour: null, time: null}

比较器需要再包含三个字段。nullsLast(naturalOrder())每次重复都让我无休止。

在不使用第三方库的情况下，如何简化比较器的使用？

Answer 1

听起来您的Action班级有五个字段，并且您想在所有字段上创建一个复合Comparator，这也是null安全的。您可以这样做：

    Comparator<Action> COMPARATOR1 =
        comparing(Action::date, nullsLast(naturalOrder()))
        .thenComparing(Action::time, nullsLast(naturalOrder()))
        .thenComparing(Action::foo, nullsLast(naturalOrder()))
        .thenComparing(Action::bar, nullsLast(naturalOrder()))
        .thenComparing(Action::baz, nullsLast(naturalOrder()));

如果重复nullsLast(naturalOrder())操作令人沮丧，则可以创建一些实用程序来提供帮助。首先，我们观察到naturalOrder()返回了一个Singleton Comparator实例。它适用于任何可比类型；它所做的就是调用compareTo()方法。

该nullsLast()委托给它的包裹比较之前的包装只是增加了空检查。它根本不取决于要比较的实际类型。它不是单例，但是任何nulls-last-自然顺序比较器都与其他任何比较器一样好。因此，我们可以创建一个实例并重用它。我们将其包装为一种方法，以避免在每个使用地点将其强制转换为正确的类型。

static final Comparator<?> NLNO = Comparator.nullsLast(Comparator.naturalOrder());

@SuppressWarnings("unchecked")
static <T extends Comparable<T>> Comparator<T> nlno() {
    return (Comparator<T>)NLNO;
}

这使我们可以执行以下操作：

    Comparator<Action> COMPARATOR2 =
        comparing(Action::date, nlno())
        .thenComparing(Action::time, nlno())
        .thenComparing(Action::foo, nlno())
        .thenComparing(Action::bar, nlno())
        .thenComparing(Action::baz, nlno());

还是太冗长了？我们可以编写一个方法，该方法采用可变数量的访问器方法，并通过减少over来创建组合的比较器thenComposing()。看起来像这样：

@SafeVarargs
@SuppressWarnings("varargs")
static <C extends Comparable<C>> Comparator<Action>
comparatorWith(Function<Action, ? extends C>... extractors) {
    return Arrays.stream(extractors)
                 .map(ex -> comparing(ex, nullsLast(naturalOrder())))
                 .reduce(Comparator::thenComparing)
                 .orElseThrow(() -> new IllegalArgumentException("need at least one extractor"));
}

这使我们能够编写以下内容：

    Comparator<Action> COMPARATOR3 =
        comparatorWith(Action::date, Action::time, Action::foo, Action::bar, Action::baz);

这值得么？helper方法可能比编写单个nullsLast(naturalOrder())调用链更为复杂。但是，如果您需要一堆不同的比较器链，则重用辅助方法的能力可能值得其复杂性。