Objects.requireNonNull (T obj) anstelle von Null-Checks und manuell geworfen IllegalArgumentException?

Question

Jedes Mal, wenn ich, wenn die übergebenen Parameter auf ein Verfahren zu überprüfen, hatte nicht null sind, habe ich eine Null-Scheck zu schreiben und eine IllegalArgumentException wenn die Nullprüfung nicht werfen:

if (user == null) { 
     throw new IllegalArgumentException("User can't be null."); 
    } 

jedoch durch den Quellcode Lesen von Einige Java 8 Klassen wie ArrayList, ich fand heraus, dass Oracle Objects.requireNonNull verwenden, um einen Parameter gegen einen Nullwert zu überprüfen, und dann, wenn der Test fehlschlägt, wird NullPointerException geworfen.

Auf diese Weise, die früheren Code-Schnipsel wie dies durch die Annahme dieses Ansatzes aussehen sollen:

Objects.requireNonNull(user, "User can't be null."); 

Kleiner und besser lesbar.

Unter der Annahme, dass ich die Kontrolle über die gesamte Ausnahmebehandlung des Systems haben, (auch, dass ich nicht, manchmal ist es Teil des Geschäfts ist es, diese ungeprüften Ausnahmen zu behandeln), sollte ich meine IllegalArgumentExceptions mit NullPointerException und Nutzung zu ersetzen Objects.requireNonNull anstatt meine eigene Nullprüfung und Ausnahmewerfen zu schreiben?

Answer 1

Mit Objects.requireNonNull(c) können Sie sehr elegant überprüfen, ob das Element nicht null ist. Aber es gibt eine interessante Diskussion darüber, ob man NullPointerException oder IllegalArgumentException ->IllegalArgumentException or NullPointerException for a null parameter? wählt. So werfen NullPointerException ist der Java-Weg, um auszudrücken, dass eine Referenz null ist.

Andernfalls können Sie Ihre eigene Methode requireNotNull() machen. Es ist ganz einfach:

public static <T> T requireNonNull(T obj) { 
     if (obj == null) 
      throw new NullPointerException(); 
     return obj; 
    } 

und Sie können die Ausnahme NullPointerException von IllegalArgumentException ändern.

Answer 2

Es gibt eine Diskussion darüber, welche Art von Ausnahme ausgelöst werden soll, wenn eine Methode einen Nullwert empfängt, den sie nicht erwartet. Einige Leute argumentieren für NullPointerException, einige Leute argumentieren für IllegalArgumentException. Der JDK-Weg scheint in solchen Fällen NullPointerException zu werfen, weshalb der Objects.requireNonNull es wirft.

Aber ich würde den vorhandenen Code nicht nur wegen dieser Methode ändern, obwohl Sie möglicherweise die Verwendung von Objects.requireNonNull in neuem Code in Betracht ziehen möchten. (Wenn Sie es verwenden, wird der Code normalerweise lesbarer als auf Null überprüft und eine Ausnahme manuell ausgelöst.)