spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Ich würd sagen, es ist sinnvoll und lesbar - also ok.
Ob es schön ist verkommt dann zur Geschmacksfrage

Ist meiner Meinung nach schwierig, weil zwei verschiedene Fehlerquellen mit einer einzigen Anweisung abgehandelt werden.

Nullpointer erfordern in vielen Fällen eine andere Behandlung, als logische Fehler oder Abbruchbedingung. Das hängt aber stark von der jeweiligen Situation ab.

Etwas wie
empfinde ich schon als schwierig. Den Fall, dass (wieso auch immer) kein Spieler existiert, kann man nur schwerlich genauso behandeln wie wenn er keine Waffe hat. Wie würdest du denn in diesem Fall die Fehlermeldung ausgeben?

Meiner Meinung nach sollte man die Integritätsprüfung einer Variable immer in einem einzelnen und alleinigen Schritt machen und danach alle weiteren Schritte in dem Wissen ausführen, dass die Variable in Ordnung ist.

Na dann sehe ich lieber direkt in der Anweisung/Abfrage, dass die Var gültig sein muss, damit die weiteren Bedingungen geprüft werden können. Generell mag ich Zusammenfassungen von Bedingungen, solange sie nicht zu komplex werden, denn sonst hat man so ein Klammermarathon oder Bedingungen wo man erstmal von vorne nach hinten und dann nochmal von hinten nach vorne liest und es immernoch nicht versteht.

Zitat von »"Das Gurke"«

Wie würdest du denn in diesem Fall die Fehlermeldung ausgeben?

If-Anweisungen gibt es nicht nur um detailliert Fehlerbericht zu erstatten... Ich finde Abfragen á la if ( ptr && ptr->foo() ) sehr sinnvoll, da man zum einen sicher geht das der Zeiger existiert (ist Voraussetzung für die Dereferenzierung) und zum anderen keine unnötig verschachtelten If-Anweisungen hat.

Letzten Endes hängt es natürlich von der Situation und nicht zuletzt vom eigenen Geschmack ab, wie man If-Anweisungen gestaltet und kombiniert! Ich finde aber das nichts gegen ein Konstrukt, wie oben genannt, spricht - Im Gegenteil kann die Abfrage sogar besser abgearbeitet werden!

Hmm, bei mir sieht das meist in etwa so aus:

Erst eine Prüfung ob der Pointer auch tatsächlich derefenziert werden kann. Danach dann alle Aktionen. Mein Anspielen auf die Fehlermeldung daher, weil ich bei Nullpointern explizit eine passende Excecption auswerfe, die selbstverständlich auf die zweite Anweisung nicht passen würde.

Ja, das mit den Exceptions ist ja sowiso so eine Sache... Ich finde das Werfen einer Exception bei sowas total unnötig. Die verschwendeten Resourcen kann man anderswo viel besser einsetzen.

Au ja, wollen wir uns um die Frage der korrekten Verwendung von Exceptions kloppen? *g* Ich hab von "hidden goto" bis "unerlässlich" glaube ich schon so jeden Kommentar mal gehört.

Irgendwie verwunderlich, dass die Diskussion hier noch nicht vom Zaun gebrochen ist. JAVA gegen den Rest, VS gegen den Rest, DirectX Graphics gegen OpenGL etc hatten wir ja alles schon. Zeit für ein neues Schlachtfeld? :badgrin:

Fakt ist das jede Ausnahme die geworfen wird unmengen Resourcen verbrät, die anderswo besser verwendet werden könnten... Das ist einfach so und jeder halbwegs gute Programmierer wird dir raten mit Exceptions vorsichtig um zu gehen und sparsam zu verwenden. Da ist keine Diskussion notwendig, das ist einfach so. Und möglichst viele Exceptions werfen gehört auch nicht zum "guten" Programmdesign, sondern eher zum "ich weiß nicht was ich tue" Design...