spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

• Besonders wenn du anderen dein Framework zur Verfügung stellen willst solltest du deinen Quellcode ausführlicher kommentieren. Erläuterungen zu den Funktionen und den Parametern, die erwartet werden etc. sind nicht bzw. nur minimal zu finden. Das macht es schwer bzw. viel Aufwand sich in dein Framework einzuarbeiten.
• Du solltest dir angewöhnen deinen Funktionen einen Rückgabewert zu verpassen. Was wenn zum Beispiel Window.Create() schief läuft? Du schreibst zwar eine Meldung in ein Log-File, aber die Funktion hat keinen Rückgabewert, wie willst du also innerhalb deines Programms prüfen ob alles glatt ging?! Als Rückgabewert könntest du dir einen "Datentyp" bzw. eine Aufzählung basteln, die verschiedene, eigene Resultate definiert.
• Es gibt - wie du vielleicht schon gesehen hast - einen gar nicht all zu alten Thread zum Thema "Ungarische Notation" oder ähnliches. Die verursacht bei dir ein bisschen Verständnis-Probleme an manchen Stellen. Beispiel: Warum übergibst du der Funktion Create() der Klasse CWindow ein ... , HINSTANCE pInstance, ... ? Was soll das "p" am Beginn des Variablennamen, deutet das nicht auf einen Zeiger hin? Aber es ist doch gar kein Zeiger!?

Zitat von »SaRu«

Du solltest dir angewöhnen deinen Funktionen einen Rückgabewert zu verpassen. Was wenn zum Beispiel Window.Create() schief läuft? Du schreibst zwar eine Meldung in ein Log-File, aber die Funktion hat keinen Rückgabewert, wie willst du also innerhalb deines Programms prüfen ob alles glatt ging?! Als Rückgabewert könntest du dir einen "Datentyp" bzw. eine Aufzählung basteln, die verschiedene, eigene Resultate definiert.

Zitat von »PCShadow«

Für sowas sollte man besser Exceptions verwenden, die sind nämlich dafür gedacht

	C-/C++-Quelltext
1 2 3 4 5 6 7 8 9 10	enum Result { ERR_OK, ERR_CANCELED, ERR_STOP, ERR_OUT_OF_MEMORY, ERR_NULL_POINTER, ERR_NOT_FOUND, ERR_EMPTY, ERR_INVALID, ERR_FAILED };

Zitat von »BurningWave«

Zitat von »PCShadow«
Für sowas sollte man besser Exceptions verwenden, die sind nämlich dafür gedacht


Am besten beides, so kann der User selbst entscheiden, was ihm besser gefällt.

Zitat

Zitat

Zitat von »PCShadow«
Für sowas sollte man besser Exceptions verwenden, die sind nämlich dafür gedacht


Am besten beides, so kann der User selbst entscheiden, was ihm besser gefällt.

Nein, wieso? Es reicht wenn Fehlerbehandlung in einer Form konsequent durchgezogen wird!

	C-/C++-Quelltext
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25	int RenderDevice:Create(...) { hr = CreateDevice(...); if(hr) { // hier Aufräumcode return -1; } hr = CreateVertexBuffer(...); if(hr) { // hier Aufräumcode return -1; } // ... hr = CreateIndexBuffer(...); if(hr) { // hier Aufräumcode return -1; } }

	C-/C++-Quelltext
1 2 3 4 5 6 7 8 9 10 11 12 13 14	void RenderDevice:Create(...) { try { CreateDevice(...); CreateVertexBuffer(...); // ... CreateIndexBuffer(...); } catch(...) { // Error - Behandlung oder Weiterleitung } }