收集可能为空的值

Question

I have the following code:

private static <T> Map<String, ?> getDifference(final T a, final T b, final Map<String, Function<T, Object>> fields) {
    return fields.entrySet().stream()
            .map(e -> {
                final String name = e.getKey();
                final Function<T, Object> getter = e.getValue();
                final Object pairKey = getter.apply(a);
                final Object pairValue = getter.apply(b);
                if (Objects.equals(pairKey, pairValue)) {
                    return null;
                } else {
                    return Pair.of(name, pairValue);
                }
            })
            .filter(Objects::nonNull)
            .collect(Collectors.toMap(Pair::getKey, Pair::getValue));
}

现在，pairValue可以为null。为了避免像这里描述的NPE，在“collect”时，我希望确保只发送那些非null的值。如果为null，则要发送“”。

所以，我尝试将最后一行替换为以下内容：

.collect(Collectors.toMap(Pair::getKey,Optional.ofNullable(Pair::getValue).orElse(""));

其他修改如下：

.collect(Collectors.toMap(pair -> pair.getKey(), Optional.ofNullable(pair -> pair.getValue()).orElse(""));

代码无法编译。我不确定需要什么。有什么帮助吗？

- user2116243

---

你能详细说明一下哪里出了问题吗？电脑着火了吗？ - Chetan Kinger

Java 8中Collectors.toMap出现NullPointerException https://dev59.com/tWAf5IYBdhLWcg3wcyer 这是同样的问题吗？ - Chota Bheem

很不幸，目前我无法做太多事情。我已经投票重新开放该问题并将其标记为需要管理员干预，所以我们会看看会发生什么。 - Jacob G.

@user2116243 看起来它被移除了！ - Jacob G.

[mcve]带有测试数据会很有帮助。 - c0der

Answer 1

您可以将内容收集到HashMap中，它允许使用null值而无需使用Optional：

private static <T> Map<String, Object> getDifference(
        final T a, final T b, final Map<String, Function<T, Object>> fields) {
    return fields.entrySet().stream()
        .map(e -> {
            final Function<T, Object> getter = e.getValue();
            final Object value = getter.apply(b);
            return Objects.equals(getter.apply(a),value)? null: Pair.of(e.getKey(), value);
        })
        .filter(Objects::nonNull)
        .collect(HashMap::new, (m,p) -> m.put(p.getKey(),p.getValue()), Map::putAll);
}

顺便提一下，在返回类型中使用通配符是不被鼓励的，它们会使调用者的生活变得毫无必要的困难。

为了对比，这里是没有使用Stream进行相同操作的代码：

private static <T> Map<String, Object> getDifference(
        final T a, final T b, final Map<String, Function<T, Object>> fields) {
    HashMap<String, Object> result = new HashMap<>();
    fields.forEach((key, getter) -> {
        final Object value = getter.apply(b);
        if(!Objects.equals(getter.apply(a), value)) result.put(key, value);
    });
    return result;
}

当然，这也适用于可选项:

private static <T> Map<String, Optional<Object>> getDifference(
        final T a, final T b, final Map<String, Function<T, Object>> fields) {
    HashMap<String, Optional<Object>> result = new HashMap<>();
    fields.forEach((key, getter) -> {
        final Object value = getter.apply(b);
        if(!Objects.equals(getter.apply(a), value))
            result.put(key, Optional.ofNullable(value));
    });
    return result;
}

---

但是，如果你只想用空字符串替换null，你不需要使用Optional：

private static <T> Map<String, Object> getDifference(
        final T a, final T b, final Map<String, Function<T, Object>> fields) {
    HashMap<String, Object> result = new HashMap<>();
    fields.forEach((key,getter) -> {
            final Object value = getter.apply(b);
            if(!Objects.equals(getter.apply(a), value))
                result.put(key, value==null? "": value);
        });
    return result;
}

好的，如果您只是在map函数中进行替换，而不是在收集器中进行替换，那么这个替换也可以直接与您的原始代码配合使用：

private static <T> Map<String, ?> getDifference(final T a, final T b, final Map<String, Function<T, Object>> fields) {
    return fields.entrySet().stream()
        .map(e -> {
            final String name = e.getKey();
            final Function<T, Object> getter = e.getValue();
            final Object pairKey = getter.apply(a);
            final Object pairValue = getter.apply(b);
            if (Objects.equals(pairKey, pairValue)) {
                return null;
            } else {
                return Pair.of(name, pairValue==null? "": pairValue);
            }
        })
        .filter(Objects::nonNull)
        .collect(Collectors.toMap(Pair::getKey, Pair::getValue));
}

或者

private static <T> Map<String, Object> getDifference(
        final T a, final T b, final Map<String, Function<T, Object>> fields) {
    return fields.entrySet().stream()
        .map(e -> {
            final Function<T, Object> getter = e.getValue();
            final Object pairValue = getter.apply(b);
            return Objects.equals(getter.apply(a), pairValue)? null:
                Pair.of(e.getKey(), pairValue==null? "": pairValue);
        })
        .filter(Objects::nonNull)
        .collect(Collectors.toMap(Pair::getKey, Pair::getValue));
}

- Holger

Answer 2

您的语法有误。 toMap() 的第二个参数必须是 lambda 表达式，因此请修改代码。

.collect(Collectors.toMap(
             pair -> pair.getKey(),
             pair -> Optional.ofNullable(pair.getValue()).orElse("")
));

或者

您可以按照以下方式修改map()部分

return Pair.of(name, Optional.ofNullable(pairValue).orElse(""));

并使用您的原始collect()

- Nikolai Shevchenko