Cách gọn gàng để kiểm tra xem Tập hợp có chứa một số không

Question

Tôi có một phương thức được gán cho một đối tượng Set. Phương thức mà đại biểu yêu cầu rằng Set không chứa bất kỳ phần tử null nào. Tôi muốn check the precondition rằng Set không chứa các phần tử null sớm, theo phương pháp trước khi phái đoàn. Mã rõ ràng làm làm như vậy là thế này:

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 
} 

Nhưng điều này là không chính xác, bởi vì Set.contains() có thể ném một NullPointerException nếu Set thực hiện tự không cho phép các yếu tố null. Khi đó, việc bỏ qua NullPointerException trong trường hợp đó sẽ hoạt động but would be inelegant. Có cách nào gọn gàng để kiểm tra điều kiện tiên quyết này không?

---

Có lỗ hổng thiết kế trong giao diện Set không? Nếu triển khai Set có thể không bao giờ chứa giá trị rỗng, tại sao thay vào đó không yêu cầu để luôn trả về false? Hoặc có một vị từ isNullElementPermitted()?

Answer 1

Cách đơn giản nhất là liệt kê Tập hợp và kiểm tra các giá trị rỗng.

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

Tạo một HashSet từ plugIns và kiểm tra sự tồn tại của 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

Chỉ cần bắt NullPointerException nếu ném và bỏ qua nó:

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 
} 

Điều này tạo ra chỉ là một chi phí nhỏ nếu ném, nhưng nếu bạn không sử dụng ngoại lệ (đặc biệt là dấu vết strack), nó thực sự khá nhẹ.