C++ programmieren
-
new und delete braucht man eigentlich recht selten und ganz bestimmt nicht, wenn wie hier die Lebensdauer der Objekte mit dem der Pointer identisch ist. Du machst es auch prompt falsch und hast dir ein paar Speicherlöcher eingefangen. Lass es einfach weg.
Noch ein paar Anmerkungen:
Furble Wurble schrieb:
Betrachte folgenden Link: http://www.c-plusplus.net/forum/304133
Besonders z. Thema "Den Code reduzieren" niemand hat Lust sich 400+ Zeilen durchzuschauen.
QFT.
Man kann es mit Leerzeilen auch übertreiben. Wenn man (fast) überall Leerzeilen setzt, dann geht der trennende Effekt der Leerzeilen verloren.
cstdlib für abs fehlt.
So viel zum technischen und zur Form. Ich habe ja schon beschrieben, dass die Klassen nicht gut designed sind und klarer definierte Einsatzgebiete benötigen. Und habe dabei auch vorhergesagt, dass man sich so nur selber Probleme macht, was dann auch prompt eingetreten ist.
Bau deine Klassen so auf, wie die echte Welt. Begriffe, die die Zusammenhänge zwischen Echtweltobjekten beschreiben, lassen sich oft direkt in C++ übersetzen:
"hat ein" -> Member
"ist eine Ansammlung von X" -> container<X>
"ist ein" -> public-Vererbung
"kann X tun" -> Memberfunktion
"kann man X mit tun" -> freie Funktion
-
SeppJ schrieb:
Man kann es mit Leerzeilen auch übertreiben. Wenn man (fast) überall Leerzeilen setzt, dann geht der trennende Effekt der Leerzeilen verloren.
Habe ich auch gedacht, aber vielleicht ist das eine Eigenschaft des Editors der IDE, dass er die Zeilen durch CR+LF abschließt und das Forum daraus eine Leerzeile bastelt?
Aber auf jeden Fall sollte TE das dann manuell korrigieren.
-
Manchmal folgen aber auch mal zwei Zeilen aufeinander.
