Reverse Engineering der NLT II

[siebenstreich]

Meine Meinung:

Ja, ich finde das generell total sinnvoll.
Magic numbers sollten eigentlich alle irgendwann mal durch was Verständliches ersetzt werden. (Abgesehen von Fällen, wo es wahrscheinlich keinen Sinn macht, wie in get_ttx(<magic number>), das dürfte ausufern.) Das ist einer der Bereiche, wo ich einiges gemacht habe, aber fertig ist es noch lange nicht.

Anstelle von #define haben wir immer enums angelegt, meist in der common.h. Es gibt Ausnahmefälle, z.B ist im Rahmen meines oben diskutierten Bugfix das enum TEVENT_POSITION_ entstanden, das ich direkt in die seg094.cpp reingesetzt habe. Denn diese Bezeichner werden nur lokal benötigt und sind auch nicht von höherem Interesse.

Anstelle von GRAM_ würde ich GRAMMAR_ schreiben. Dann ist sofort klar, was gemeint ist, und die drei zusätzlichen Buchstaben tun weder heutigen Monitoren noch Festplatten weh.

Mit Henne habe ich mal besprochen, dass Bezeichner und Kommentare generell auf Englisch gemacht werden. Ausnahme: Bezeichner für Dinge, die wörtlich im Spiel auftreten, also z.B. ITEM_OCHSENHERDE (und nicht ITEM_OXEN_HERD; es handelt sich um einen dreifach-Morgenstern, also eine recht martialische Waffe).

Deine Bezeichner sind irgendwie ein Grenzfall: Wie albern ist es, Eigenschaften der deutschen Grammatik mit englischen Bezeichnern zu versehen?

Wahrscheinlich würde ich trotzdem Dinge wie
GRAMMAR_SINGULAR
GRAMMAR_PLURAL
GRAMMAR_NO_ARTICLE
GRAMMAR_DEFINITE_ARTICLE
verwenden.