Dokładny sposób sprawdzenia, czy zestaw nie zawiera wartości pustej

Question

Mam metodę, która ma Set obiektów. Metoda, którą deleguje, wymaga, aby Set nie zawierał żadnych elementów zerowych. Chciałbym check the precondition, że Set nie zawiera żadnych elementów zerowych wcześniej, w metodzie przed delegacją. Oczywistym kod rób tak to:

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 
} 

Ale to jest błędne, ponieważ Set.contains() może rzucić NullPointerException jeśli realizacja samaSet nie zezwala na elementy null. Catching następnie ignorowanie NullPointerException w tym przypadku będzie działać but would be inelegant. Czy istnieje dobry sposób na sprawdzenie tego warunku?

---

Czy w interfejsie Set występuje błąd projektowy? Jeśli implementacja Set nigdy nie może zawierać wartości null, dlaczego zamiast tego należy wymagać , aby zawsze zwracać false? Czy masz predykat isNullElementPermitted()?

Answer 1

Najprostszym sposobem jest wyliczenie zestawu i sprawdzenie wartości zerowych.

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

Tworzenie HashSet z plugIns i sprawdzić istnienie 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

Wystarczy złapać NullPointerException jeśli rzucony i ignorować go:

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 
} 

Stwarza to jedynie niewielki narzut jeśli rzucone, ale jeśli nie korzystasz z wyjątku (zwłaszcza śledzenia strackowego), to jest całkiem lekki.