spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Doxygen

Habe das Paket mal runtergeladen.

2DVector.h:
-Warum wird im default-ctor nichts initialisiert?
-"_" steht normalerweise für eine Präprozessoranweisung.
-Warum eine Extra-Klasse, wenn du eh keine Operatorenüberladung implementiert hast?
-Sind mit deinem Renderer auch float-Anweisungen möglich? Falls ja, mach aus der Klasse eine Templateklasse.

KonsolenRenderer.cpp:
-Erstmal: Warum Englisch-Deutsch-Mischmasch?
-Warum eine 2 dimensionale Parameterliste, wenn du doch eine Vektorklasse hast?
-Soweit ich weiß ist COORD eine einfache Struktur, also kann man zur Vereinfachung auch COORD = { X, Y }; schreiben.

KonsolenRenderer.h:
-Der typedef ist völlig unnötig sowie nichtssagend.
-Woher soll der spätere Endbenutzer wissen, was KRFunktion ist?
-Wieder sinnlose Kommentare.

LineRenderer.cpp:
-Wieder unschlüssige Variablen.
-Nimm als Parameter eine Instanz deiner Vektorklasse.

LineRenderer.h:
-Selbiges wie in LineRenderer.cpp.

objekt.h:
-Warum kleiner Dateiname?
-Warum hast du hier die Includes für die Standard-Libraries wenn du doch eine "... AllIncludes.h"-Datei hast?
-DeletObjekt brauchst du nicht, es sei denn du möchtest Objekte während der Laufzeit löschen. Ansonsten macht das der dtor.
*EDIT*
-Die beiden Methoden sollten konstant sein, weil nichts an der Instanz verändert wird (und das wahrscheinlich auch so bleiben wird).

Gesamt:
-Versuche abgekapselt zu programmieren.
-Packe deine verschiedenen Codesegmente in Klassen.
-Versuche einheitlich zu bleiben, sprich nicht verschiedene Bezeichner zu benutzen (habe hier und da mal X und x gesehen).
-Vernünftige Kommentare, also nicht beschreiben was da passiert, sondern wie. Kommentare wie "//Die Koordinaten" braucht kein Schwein, weil der Code selbstsprechend ist.
-Nutze Includeguards.

Zitat

-Was sind includeGuards?

Das sind Dinge, die man wissen soll, wenn man eine Bibliothek schreiben will.

http://en.wikipedia.org/wiki/Include_guard
Ps: Google Hilft

Zitat

Warum auch?

Gegenfrage: Warum nicht? Hätte doch sein können das mit float-Werten gerendert werden können.

Zitat

KRFunktion wird durch "void" ersetzt.

Nicht ganz, typedef erlaubt eine komplette Typenneudefinierung, sodass neue Instanzen von diesem Objekt gebaut werden können.

Zitat

Ich hab des deshalb gemacht weil KRFunktion mal ein enum war das mir aber dann später als sinnlos aufgefallen ist.

"KRFunktion" ist genau so sinnlos (;

Zitat

Später werd ichs dann überarbeiten. Andere sachen sind erstmal wichtiger

Sowas würde ich dir nicht raten. Ich würde einfach mal behaupten das du noch zu unfähig bist um vernünftiges Coderefactoring durchzuführen

Zitat

Was sind includeGuards?

Includeguards gewährleisten, dass Headerfiles nicht mehrfach eingebunden werden. Sollte also in keinem Header fehlen.

Zitat

Was meinst du mit abgekapselt Programmieren?

Naja, das du eben so programmieren sollst, dass die verschiedenen Klassen (noch Funktionen) nicht voneinander abhängig sind. Und lieber mehrere kleine Klassen als eine gigantische.

Ich hoffe du siehst das als Hilfe und nicht als Runtermachung.

EDIT:

Zitat

Ok ich geb zu ich bin grad n bisschen neben der Spur.

Nein, du hast nur zu wenig Ahnung und bist daher überfordert. Ich würde dir empfehlen, erst vernünftig C++ zu lernen bevor du so ein Projekt anfängst...