unchecked Exception

[OliW]

Moin Kollegen ,

eine Frage zum Thema Exceptions.
Weshalb gehört eine "IllegalArgumentException" zu den unchecked Exceptions? Schon klar. Das ist eine Kindklasse von "RuntimeException" Aber:
Annahme: Ich übergebe einen String (z.B. Name, Adresse etc.) aus einer Usereingabe als Parameter an eine Methode, die überprüfen soll, ob der String
!=null ist.

*****************************************
public void setName(String name){
if(name == null){
throw new IllegalArgumentException("Name may not be null.");
}
if(name.trim().isEmpty()){
throw new IllegalArgumentException("Name may not be empty.");
}
this.name = name;
****************************************

Dann ist das doch eine typischer Fall für eine "IllegalArgumentException" ("Feld darf nicht leer sein").
Da dieser Fehler aber nicht aus der Programmlogik, sondern aufgrund einer Usereingabe kommt, müsste das doch aber eine klare "checked Exception" sein.
Ich werfe die Exception an den Aufrufer (Usereingabe) und behandle sie da mit einem try-catch.

Oder sehe ich da was falsch.

v.D. Oli

 

[Xyz1]

Der user darf die (raw) Exception (gleich ob checked oder nich) niemals sehen...

Zweitens, ab Java 11 kannst du .isBlank() machen.

 

[temi]

Die Usereingabe kann doch nur zur Runtime und nicht zur Compiletime auftreten, oder?

 

[kneitzel]

Exception heißt übersetzt Ausnahme. Diese sollten nur auftreten, wenn etwas außergewöhnliches passiert. Etwas das nicht erwartet werden konnte. Also eine wirkliche Ausnahme.

Alle Dinge, die man erwarten kann, sollte man aktiv überprüfen! Also bei einer Benutzereingabe wird diese doch bitte immer validiert! Damit ist das eine Exception, die eigentlich nicht auftreten sollte. Sollte diese doch auftreten, dann ist es eine Sache des Entwicklers. Es macht keinen Sinn, hier den Entwickler mit einer Checked Exception zu zwingen, diese Exception abzufangen, denn es ist Unsinnig, da der Entwickler in der Regel Eingaben verifiziert. (Wenn er es nicht möchte, dann kann er aber auch gerne diese Exception abfangen.

Somit macht es aus meiner Sicht sehr wohl Sinn, dass dies eine unchecked Exception ist. Wenn diese Auftritt, dann war der Entwickler schlicht zu blöd, vernünftigen Code zu schreiben, weil er die Methode mit ungültigen Parametern aufgerufen hat.

 

[OliW]

Exception heißt übersetzt Ausnahme. Diese sollten nur auftreten, wenn etwas außergewöhnliches passiert. Etwas das nicht erwartet werden konnte. Also eine wirkliche Ausnahme.

Alle Dinge, die man erwarten kann, sollte man aktiv überprüfen! Also bei einer Benutzereingabe wird diese doch bitte immer validiert! Damit ist das eine Exception, die eigentlich nicht auftreten sollte. Sollte diese doch auftreten, dann ist es eine Sache des Entwicklers. Es macht keinen Sinn, hier den Entwickler mit einer Checked Exception zu zwingen, diese Exception abzufangen, denn es ist Unsinnig, da der Entwickler in der Regel Eingaben verifiziert. (Wenn er es nicht möchte, dann kann er aber auch gerne diese Exception abfangen.

Somit macht es aus meiner Sicht sehr wohl Sinn, dass dies eine unchecked Exception ist. Wenn diese Auftritt, dann war der Entwickler schlicht zu blöd, vernünftigen Code zu schreiben, weil er die Methode mit ungültigen Parametern aufgerufen hat.

So gesehen macht das Sinn. Ich bin davon ausgegangen, daß die checked Exception die Validierung ist. Aber klar. wenn es dann trotzdem zu einer Exception kommt, war meine vorherige Validierung Müll.