spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

riCo
Also dies ist wieder einer der vielen Punkte wo Code-Stil auf "Code-Eleganz" trifft.

Ich weiß nicht, ob Du jemals Code-Style-Rules aufgesetzt hast (bei mir so gute 4 Seiten groß), wo Du Deinen eigenen Stil einheitlich definierst und an dem Du sowie alle anderen, sich bei dem Projekt zu halten haben.

Beispiel bei mir ist z. B. das Thema Namespaces. Ich habe eine Regel mindestens 3 Zeichen lang, maximal 6 Zeichen. Diabloic ist da etwas zu groß.

Wie Du sicherlich gemerkt hast, bekommt man dadurch ziemlich schnell breiten Code und man muss diesen ggf. Auslagern mit Hilfsvariablen usw, damit man diesen besser lesen kann. Nicht gerade eine schöne Art wenn man ehrlich ist.

Darum sollte man sich überlegen ob man etwas sinnvoll kürzen kann. Beispiel: Engine heißt "Vulcan X", warum wird der Namespace nicht dafür "vx" genannt, anstatt "vulcanx" bzw. "vulcan_x". Bei Dir könnte "dia" für "diabloic" gut kommen und ggf. viel Platz sparen.

Was mir aufgefallen ist. Dein DegreeToRadian ist im Utilities-Bereich. Aber hat das nicht eher was mit Mathematik zu tun? Es sind doch nichts anderes als mathematische Funktionen/Konstanten.

Wobei man sagen muss, das sowas eigentlich etwas "unsinnig" ist. Da dies eine Funktion genau wie min, max, abs usw. ist oder eine Konstante wie epsilon. Sowas sollte global verfügbar sein.

Ich habe ebenfalls beachtet, dass Du keine Wortseperatoren benutzt und C für Klassen. Das solltest Du einheitlich definieren!

Beispiel: Du machst eine Frame Per Second Klasse und nach deinem Code-Stil müsste die wie folgt lauten: "CFPS". Sieht wirklich grauen voll aus. Würdest du alles klein schreiben, C weglassen und als Wortseperator _ benutzen könnte man sie so nennen: "frame_per_second". Sieht doch wirklich ansprechender aus, oder?

Auch DegreeToRadian würde schöner aussehen: degree_to_radian. Kann man doch wirklich leichter lesen. Vorallem bei so Brocken wie: "CalculateFPSTime".

Dazu solltest Du Dir gedanken machen über die ungarische Notation. Wie Du sicherlich bemerkt hast, ist diese in C++ mehr als nur verhasst - vorallem in Zeiten moderner IDEs wie Visual Studio.

Mach Dir am Besten paar Gedanken für Deinen Code-Stil und schreib alles nieder und definiere alles. Dadurch lernt man schnell: "Hey, das ist aber blöd, da fällt mir sicherlich noch was besseres ein!" Und man überlegt sich schnell paar coole Sachen die total in den Code-Stil und in das Design passen. Auch Setzung von Klammern usw gehört da rein.

Dein Code von Dir da oben würde bei mir so aussehen:

Deviloper
Nenne mir die Notationen für:
Objekte
Instanzen
std::basic_string<T>
std::list<T>
std::map<T, X>
Iteratoren
Smart Pointer
size_t
usw.

Die gibt es nicht.

Deviloper
Dann gibt es aber ziemlich schnell Konflikte mit der ungarischen Notation. Und warum? Nur weil einige zu Faul sind mit dem Mauszeiger auf den Namen der Variable zu gehen und den Datentypen angezeigt zu bekommen. Außerdem: Wer einen Standard hat in seinem Code-Design, der weiß genau um was für einen Datentypen sich was handelt, dafür braucht man keine schäbbige ungarische Notation.