检查 Set 是否不包含 null 的巧妙方法-解网

问：

我有一个给定对象的方法。它委托给的方法要求不包含任何 null 元素。我想在委托之前的方法中尽早检查不包含空元素的前提条件。这样做的明显代码是这样的：SetSetSet

public void scan(Set<PlugIn> plugIns) {
   if (plugIns == null) {
      throw new NullPointerException("plugIns");
   } else if (plugIns.contains(null)) {
      throw new NullPointerException("plugIns null element");
   }
   // Body
 }

但这是不正确的，因为如果实现本身不允许 null 元素，Set.contains（） 可能会抛出一个。在这种情况下，捕捉然后忽略会起作用，但会不优雅。有没有一种巧妙的方法来检查这个前提条件？NullPointerExceptionSetNullPointerException

界面是否存在设计缺陷？如果实现可能永远不会包含 null，为什么不改为 require to always return 呢？还是有谓语？SetSetSet.contains(null)falseisNullElementPermitted()

java 空集 nullpointerexception

我同意这非常烦人。您允许使用泛型创建类，但希望确保它不包含类合约中的 s。然后，如果有人真的通过了不允许 s 的 s，您的安全检查会抛出 NPE，因为它在规范中。恕我直言，这是设计错误，因为调用者对此无能为力（即，无法询问 Set 是否允许 null），更不用说在这种情况下抛出 NPE 而不是仅仅返回似乎是愚蠢的。SetnullSetnullcontainsfalse

答：

5赞 Dan Hardiker 1/9/2012 #1

最简单的方法是枚举 Set 并检查 null。

public void scan(Set<PlugIn> plugIns) {
  if (plugIns == null) throw new NullPointerException("plugIns");
  for (PlugIn plugIn : plugIns) {
    if (plugIn == null) throw new NullPointerException("plugIns null element");
  }
}

我担心的不是过早的优化。一个简洁的解决方案应该是通用的，并且相当有效。每当有人在没有首先衡量性能的情况下使用，并获得快速的好处时，他们就不会进行过早的优化。programmers.stackexchange.com/questions/79946/....HashSetSet.contains()

3赞 Matt 1/9/2012 #2

创建一个 from 并检查是否存在HashSetplugInsnull

public void scan(Set<PlugIn> plugIns) {
  if (plugIns == null) throw new NullPointerException("plugIns");
  Set<PlugIn> copy = new HashSet<PlugIn>(plugIns);
  if (copy.contains(null)) {
      throw new NullPointerException("null is not a valid plugin");
  }
}

好的，我明白你的意思了。但是，让我们假设这是实际 API 的一部分：在这种情况下，它取决于调用方和被调用方之间的接口/合约。如果允许作为集合的一部分，你可以用一个简单的（因为我们不知道如何处理 Null-Plugins）来处理它。如果不允许在集合中，则根本不需要检查，除非您怀疑 API 有问题。恕我直言，这与个人品味有很大关系，所以没有对错之分。希望能;)null//bodyifnullnull

2赞 Bohemian 1/9/2012 #3

只需捕获抛出的 NullPointerException 并忽略它：

public void scan(Set<PlugIn> plugIns) {
    if (plugIns == null) {
        throw new NullPointerException("plugIns");
    }

    NullPointerException e = null;
    try {
        if (plugIns.contains(null)) {
            // If thrown here, the catch would catch this NPE, so just create it
            e = new NullPointerException("plugIns null element");
        }
    } catch (NullPointerException ignore) { }

    if (e != null) {
        throw e;
    }
    // Body
}

如果抛出，这只会产生很小的开销，但如果你不使用异常（尤其是跟踪跟踪），它实际上是相当轻量级的。

检查 Set 是否不包含 null 的巧妙方法

Neat way to check whether a Set does not contain a null

评论

评论

评论

评论