Java 8 从 HashMap 中提取非 null 和非空值

Question

让我们考虑下面的HashMap

HashMap<String, String> map = new HashMap<String, String>();

我在地图中有值，例如

map.put("model", "test");

目前，如果我想从我正在做的地图中获得价值

if(map!=null){
 if(map.get("model")!=null && !map.get("model").isEmpty()){
   //some logic
 }
}

在 Java 8 中有没有更好的方法通过使用Optional 或 Lambdas 来实现上述条件？

Answer 1

首先，您的地图不应为空。绝不。它可能为空，但没有理由为空。这样就消除了第一次空检查。

现在很遗憾，Java没有这样的实用方法，但是几个常用的lib（apache commons、Guava等）都有，也可以自己写，所以变成了：

String model = map.get("model");
if (!Strings.isEmptyOrNull(model)) {
    // do somthing
}

使用 Optional 将可空值包装为逻辑的一部分被视为反模式。 Optional 旨在用作返回类型。所以我不建议在这里使用它。

还请注意，感觉就像您正在使用映射来存储对象的属性 如果是这样，那么请考虑定义一个具有类型属性的真实类，而不是使用映射。

Answer 2

不知道为什么在创建地图后检查它是否为空，但这里是：

Optional.ofNullable(map)
    .map(m -> m.getOrDefault("model", "")) // Use an empty String if not present
    .filter(s -> !s.isEmpty())             // Filter all empty values
    .ifPresent(valueString -> {            // Check if value is present
        // Logic here
});

或者在一行中：

Optional.ofNullable(map).map(m -> m.getOrDefault("model", "")).filter(s -> !s.isEmpty()).ifPresent(valueString -> {
        // Logic here
});

如果您想返回某些东西，请将ifPresent 更改为map；即，无论您计算什么都是可选的。

Answer 3

如果您对Optional 方法感兴趣，

您可以将map.get("model") 值包装到Optional.ofNullable 中，并通过Predicate<String> value -> !value.isEmpty() 进行过滤工作：

if (isNull(map)) { // import static java.util.Objects.isNull;
    return;        // to minimise nesting
}

Optional.ofNullable(map.get("model"))
        .filter(value -> !value.isEmpty())
        .ifPresent(value -> { ... });

Answer 4

如果您在示例代码中声明了map，则它不会是null，您不需要检查它。如果你想确定然后添加一个断言：

assert map != null;

鉴于您正在测试空字符串，如果键不存在，一种可能的方法是使用空字符串作为默认值：

if (!map.getOrDefault("model", "").isEmpty()) {
    ...
}

如果密钥不存在，我认为没有添加到 Map 的方法返回 Optional 而不是 null 是一种耻辱。比如：

map.getOptional("model").filter(v -> !v.isEmpty()).ifPresent(v -> {
    ...
}

虽然添加了可选项，但旧 API 几乎没有重新设计以弃用返回 null 表示“不存在”的方法。

Answer 5

你也可以试试 containsKey 方法。例如：

HashMap<String, String> map = new HashMap<String, String>();
map.put("model", "test");

 if(map.containsKey("model")){
   //some logic
 }