neuen Exception vs Catch-Block werfen

8

Gibt es Verhaltensunterschiede zwischen:

%Vor%

Und:

%Vor%

Beide werfen Ausnahmen des null-Objekts, wenn s null ist. Das erste Beispiel ist besser lesbar, da es genau zeigt, wo der Fehler auftritt (das Ausnahme-Bit befindet sich direkt neben der Zeile, die die Ausnahme verursacht).

Ich habe diesen Code-Stil in verschiedenen Blogs von verschiedenen Programmierern aller Arten von Skill-Levels gesehen, aber warum sollte man nicht einfach die Hauptlogik ausführen, indem man prüft, ob s nicht null ist und somit die Ausnahme davor bewahrt, jemals ausgelöst zu werden? Gibt es einen Nachteil für diesen Ansatz?

Danke

    
dotnetdev 22.09.2009, 22:06
quelle

5 Antworten

18

Nein, Console.WriteLine(null) wird keine Ausnahme auslösen. Es wird nur nichts ausgedruckt. Nun vorausgesetzt, du meintest so etwas wie:

%Vor%

dann macht es Sinn ... und Sie sollten die erste Form verwenden. Ausnahmen sollten auftreten, wenn Sie sie nicht im Voraus mit Ihren aktuellen Informationen vorhersagen können. Wenn Sie leicht herausfinden können, dass etwas nicht stimmt, ist es nicht sinnvoll, eine Operation zu versuchen, die zwangsläufig fehlschlägt. Es führt zu Code, der schwerer zu verstehen ist und schlechter abschneidet.

Also NullReferenceException , ArgumentNullException und ähnliches sollten nicht abgefangen werden, es sei denn, sie sind aufgrund eines nasty API, die manchmal Ausnahmen wirft, die Sie behandeln können, die aber nicht wirklich an erster Stelle geworfen werden sollten . In Code Contracts besteht das Standardverhalten für einen fehlgeschlagenen Vertrag daher darin, eine Ausnahme auszulösen, die nicht kann, außer indem alles abgefangen wird (was normalerweise der Fall ist) irgendwo an der Spitze des Stapels).

    
Jon Skeet 22.09.2009, 22:08
quelle
2

Wie schon von Jon Skeet erwähnt, wird Console.WriteLine (null) keine Ausnahme auslösen.

Außerdem möchte ich sagen, dass Sie "schnell scheitern" sollten. Das bedeutet, dass Sie Guard-Klauseln in Ihre Methoden schreiben müssen und die Argumente überprüfen müssen, die in Ihren Methoden angegeben wurden, wenn sie als gültig angesehen werden können. Dadurch können Sie selbst eine Ausnahme auslösen und eine zusätzliche Nachricht geben, die Ihnen beim Debuggen behilflich sein wird. Die Nachricht kann einen deutlichen Hinweis darauf geben, was falsch war, und das ist viel handlicher, wenn Sie mit einer NullReferenceException konfrontiert werden, die ohne eine gute Information in der Nachrichteneigenschaft der Nachricht ausgelöst wurde.

    
Frederik Gheysels 22.09.2009 22:13
quelle
1

Wenn Sie eine Klassenbibliothek schreiben, kann es vorkommen, dass Sie wissen, dass ein bestimmter Parameter, der einen Nullwert enthält, später Probleme verursachen kann. In diesen Fällen halte ich es für eine gute Idee, eine Exception auszulösen (obwohl ich wahrscheinlich ArgumentNullException für diesen Fall verwenden würde), um den Benutzer der Klassenbibliothek so früh und klar wie möglich darauf aufmerksam zu machen.

Ausnahmen sind nicht immer eine schlechte Sache.

    
Fredrik Mörk 22.09.2009 22:14
quelle
1

Jon Skeet hat recht, aber im Allgemeinen ist alles eine Frage der Semantik.

Wenn die Situation eine anwendbare Bedeutung hat (Zahl außerhalb des Gebietes, Geburtsdatum in der Zukunft, usw.), möchten Sie vielleicht vor jeder Operation einen Test durchführen und eine benutzerdefinierte Ausnahme (mit Bedeutung für Ihre Anwendung) auslösen ).

Wenn die Situation wirklich "außergewöhnlich" ist, schreiben Sie den Code so, als wäre der angegebene Wert korrekt. Sehen Sie, wenn Sie den Test setzen, werden Sie es jedes Mal tun, wissend, dass die VM es trotzdem tun wird, falls es eine Ausnahme auslösen muss. Unter dem Gesichtspunkt der Leistung, wenn der Fehler zufällig ein statistisch kleines Vorkommen hat, macht es keinen Sinn.

    
Julian Aubourg 22.09.2009 22:14
quelle
0

Wenn Sie einen Design-by-Contract-Ansatz für Dinge verwenden, kann ein Codeabschnitt angeben, dass er Ausnahmen auslöst, um seinen Vertrag festzulegen und ihn durchzusetzen. Die andere Hälfte nennt natürlich den Code, der den Vertrag anerkennt und erfüllt.

In diesem Fall würde es bedeuten, dass wenn Sie wissen, dass eine Methode eine Ausnahme auslöst, wenn Sie Null übergeben (d. h. ihr Vertrag besagt, dass Sie Nullen nicht übergeben), sollten Sie vor dem Aufruf überprüfen.

Jon Skeet sagt, dass die Methode sowieso keine Ausnahme auslöst. Das mag wahr sein oder auch nicht, aber das Prinzip der Bewachung für Methodenverträge steht (was ich glaube, war der Punkt Ihrer Frage).

    
Joe 22.09.2009 22:14
quelle

Tags und Links