spieleprogrammierer.de - Forum und Wiki zur Spieleprogrammierung und Spieleentwicklung

Zitat von »Glocke«

z.B. prinzipiell Rawpointer statt Smartpointer

Aber raw-pointer sind nicht obsolet. Sie sind super praktisch, wenn ein Objekt ein anderes nur benutzt und garantiert ist, dass das benutze Objekt länger lebt als das benutzende. Was sehr häufig vorkommt.
Der Vorteil daran ist nicht nur, dass raw-pointer keinen Overhead erzeugen, sondern dass die Semantik auch viel klarer wird. Man sieht einfach schon an der Deklaration, wie das Objekt benutzt wird.

Zum Code: Ich werde mal drüber gucken, wenn ich Zeit habe.

Ich habe mal drüber geguckt. Soweit ich es richtig gesehen habe, ist es gar nicht so viel Code, ich habe 3 .cpp und ~8 .hpp Dateien gesehen.
Anzumerken habe ich recht wenig, für mich sieht der Code ansich ganz hübsch aus. Wenn ich damit arbeiten müsste, würde ich mich vermutlich weniger gequält fühlen, als mit so manch anderen Code, den ich in letzter Zeit zu Gesicht bekommen habe

Was mich am meisten gestört hat, war die Klammersetzung, erinnerte mich stark an Java und ich mag das überhaupt nicht. Aber das ist eine Stilfrage, und notfalls gibt es automatische Tools um das zu ändern. Ansonsten hast du manchmal in Methoden ein this->var=var; geschrieben. Natürlich ist es eine Stilfrage, wie man seine Variablen benennt (bei mir fangen Klassen groß an, Variablen klein und Member mit m_), aber wenn sich Variablen aufgrund Namensgleichheit überdecken, sehe ich das schon kritischer. Es kann halt leicht zu Verwirrung führen, und so etwas möchte man sicherlich nicht. Natürlich ist es bei so kleinen Funktionen irrelevant, aber vielleicht hat man ja irgendwann mal etwas komplexeres. Du könntest also darüber nachdenken, ein Namensschema einzuführen, dass doppelte Namen versucht zu vermeiden.
Ansonsten fand ich den Kommentarstil ganz passen. Doxygen-Kommentare in den Headern und kurze Kommentare im eigentlichen Code, die Dinge erklären, die nicht direkt offensichtlich sind. Ich war jetzt nicht darauf angewiesen, den Code komplett zu verstehen, aber wäre ich es gewesen, wäre es mir glaube ich nicht sonderlich schwer gefallen. Und ich hab halt viele Kleinigkeiten gesehen, die für mich modernen Code ausmachen, deine Aussage, dich um modernen Code zu bemühen, nimmt man dir also durchaus ab.


Als Tipp noch: Immer dran bleiben. Man findet immer noch irgendetwas, was man lernen kann. Auf Stackoverflow wird z.B: oft Diskutiert, wie man irgendwelche Kleinigkeiten am elegantesten Lösen kann, wodurch man oft nochmal neue Einsichten bekommt. Außerdem gibt es natürlich Videos von Konferenzen, in denen einige der Prominentesten C++ Programmierer Erklärungen und Hintergrundinformationen zu neuen Features geben.

Mir fällt die inkonsistente Benamsung der Methoden auf. Logger ist so ein Beispiel. Hier hast Du "getLast" und "isEmpty" aber "begin" und "end". Dann die Verwendung von "const" ist auch nur halb durchdacht. Außerdem muss man nicht zwingend immer "auto" verwenden, damit es C++11 Code ist. An manchen Stellen wirkt es bei Dir etwas arg gewollt.
Dann kann man noch über die einzelnen Implementierungen diskutieren. Gerade für Logger gibt es da draußen wesentlich bessere. In Deinem Fall müßte ich in jeder Klasse, wo ich den verwenden will, eine Instanz erstellen.
Aber wir wollen das mal nicht zu schlecht reden da Du ja noch Student bist. Das meine ich jetzt nicht böse. Aber man sieht es am Code.

Zitat von »Glocke«

Im Grunde verwende ich auto für folgende Szenarien:
für Iteratoren: auto i = std::find(...);
für Ergebnisse von Typecasts: auto fpos = static_cast<sf::Vector2f>(pos);
und sonst überall wenn der Rückgabetyp relativ lang ist, er aber aus dem Kontext klar ist, z.B. auto& rect = view.getViewport();

Die Benutzung von auto geht so ziemlich auf den persönlichen Stil zurück. Ich finde deinen Code etwas mühsam zu lesen, da ich immer zwei Mal überlegen muss, was man da nun für einen Typ erhält/damit arbeitet.

Herb Sutter z.B. verfolgt sein "auto by default" Prinzip, wie zusammengefasst in dieser StackOverflow Antwort lesen kann oder in seinem relativ langen GotW #94.
Im Gegensatz zu Herb Sutters AAA-Style steht z.B. der Artikel von Joseph Mansfield.

Schlussendlich ist es halt wie mit vielem in C++, man muss sich eine Art/einen Weg aussuchen und diesen dann konsequent durchziehen.

Vll noch etwas als Empfehlung.

Du kannst dir ja mal den http://google-styleguide.googlecode.com/svn/trunk/cppguide.html oder den http://llvm.org/docs/CodingStandards.html anschauen.
Beides kannst du super mit http://clang.llvm.org/extra/clang-modernize.html und den http://clang.llvm.org/docs/ClangFormat.html nutzen und dein Code schön dadurch jagen und dann ist es schon mal einigermaßen "modernisiert" und formatiert.

Ansosnten gibts es noch einige hübsche Talks zu C++11/14 bei der CppCon 2014 und GoingNative 2012.