Falls sich jemand den Code tatsächlich anguckt, bitte ich um Verbesserungsvorschläge
Na wenn du wirklich willst
-
|
C-/C++-Quelltext
|
1
|
friend std::ostream& operator<<(std::ostream &mystream, HuffmanTree /*!*/const/*!*/&mytree);
|
- const mal überall setzen wo es "hingehört" (d.h. bei parametern, funktionen, usw.)
- Parameter die nicht primitiven Datentypen sind und nicht verändert werden, per const. Ref und nicht per copy übergeben.
- unsigned nutzen.
- C++-Style cast (static_cast, reinterpret_cast, ..., nicht (int)
)
- einheitliche Namensgebung (d.h. nicht einmal großbuchstaben zu Funktionsnamenbeginn, bei nächsten dann kleinbuchstaben
)
- std::sort statt sort
- Stell mal höchste Warnungsstufe ein, könnte sein das dem noch anderes auffällt als mir
- getMinUnAttachedLink könnte auch const.Ref. zurückgeben, sparst dir paar Kopien, hab nicht alle Funktionen durchgeguckt aber sind bestimmt noch welche dabei, wo de ähnliches machen kannst.
- Link::isGreaterThan, was soll hier der int Rückgabewert? Sind wir bei Microsoft BOOL angekommen?
So reicht jetzt erstmal
Aber sonst schöne Sache