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)
![;)](wcf/images/smilies/wink.png.pagespeed.ce.L9LRa_F2a5.png)
)
- einheitliche Namensgebung (d.h. nicht einmal großbuchstaben zu Funktionsnamenbeginn, bei nächsten dann kleinbuchstaben
![;)](wcf/images/smilies/wink.png.pagespeed.ce.L9LRa_F2a5.png)
)
- std::sort statt sort
- Stell mal höchste Warnungsstufe ein, könnte sein das dem noch anderes auffällt als mir
![;)](wcf/images/smilies/wink.png.pagespeed.ce.L9LRa_F2a5.png)
- 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
![:D](wcf/images/smilies/biggrin.png.pagespeed.ce.UPFXer8Yu2.png)
Aber sonst schöne Sache