重新设计方法以将Java 8 Map.computeIfAbsent（）与抛出的异常一起使用

dabadaba 发表于 Dev

达巴达巴

我正在尝试使用Java 8重做我的一些方法，以使其更加简洁，我正在尝试逐渐吸收其新功能。

这是一种将a添加value到的方法Map<Key, Set<Value>>。存在三种可能性：

密钥不存在：已添加密钥，并且已将包含该值的新集合关联到该密钥。
密钥存在：将值添加到现有集中。请注意，集合永远不会是null，因为我有一些先决条件要处理。
该键已存在，并且该值已包含在集合中：IllegalArgumentException引发了一个。

实现此行为的代码如下，并且不使用Java 8功能：

public void addValue(Key key, Value value) {
    // irrelevant preconditions...

    Set<Value> valuesForKey = myMap.get(key);
    if (valuesForKey != null && valuesForKey.contains(value))
        throw new IllegalArgumentException("Association exists already");

    if (valuesForKey == null)
        myMap.put(key, new HashSet<Value>(Arrays.asList(value)));
    else
        valuesForKey.add(value);
}

我想使用Java 8方法（例如computeIfAbsent）来缩短此代码。

我可以总结出最后一个if-else块，但是key在进行前提条件检查时，我无法克服映射到已被检索到的值集的冗余性。

public void addValue(Key key, Value value) {
    // irrelevant preconditions...

    Set<Value> valuesForKey = myMap.get(key);
    if (valuesForKey != null && valuesForKey.contains(value))
        throw new IllegalArgumentException("Association exists already");

    myMap.computeIfAbsent(key, v -> new HashSet<Value>()).add(value);
}

无论如何，我可以将所有内容合并到一条指令中吗？

图那基

您可以利用以下事实：如果集合中不包含指定的元素，则该add(element)方法将返回true。如果此调用返回false，则表示未添加该元素，因为该元素已经存在。因此，您可以使用：

public void addValue(Key key, Value value) {
    boolean added = myMap.computeIfAbsent(key, k -> new HashSet<>()).add(value);
    if (!added) {
        throw new IllegalArgumentException("Association exists already");
    }
}

本文收集自互联网，转载请注明来源。

如有侵权，请联系[email protected] 删除。