Manera aseada de comprobar si un conjunto no contiene un nulo

Question

Tengo un método que tiene un Set de objetos. Un método al que delega requiere que el Set no contenga ningún elemento nulo. Me gustaría check the precondition que el Set no contiene elementos nulos temprano, en el método antes de la delegación. El código obvia no hacerlo es la siguiente:

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 
} 

Pero esto es incorrecto, porque Set.contains() puede lanzar una NullPointerException si el Set aplicación en sí no permite elementos nulos. La captura luego ignorando el NullPointerException en ese caso funcionaría but would be inelegant. ¿Hay alguna manera clara de verificar esta condición previa?

---

¿Existe un error de diseño en la interfaz Set? Si una implementación Set nunca puede contener un valor nulo, ¿por qué no necesita para devolver siempre false? ¿O tiene un predicado isNullElementPermitted()?

Answer 1

La manera más simple sería enumerar el conjunto y verificar nulos.

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"); 
    } 
} 

Answer 2

Crear un HashSet de plugIns y comprobar la existencia de null

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"); 
    } 
} 

Answer 3

Sólo coger el NullPointerException si se tiran e ignorarlo:

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 
} 

Esto crea sólo una sobrecarga menor si lanzados, pero si no usas la excepción (especialmente el rastro), en realidad es bastante liviano.