Große Klasse - was fällt euch so ins Auge? Kritik bitte!

[dyn]

Hi, habe in den Code-Snippets eine große Klasse von mir gepostet, die ein Glücksradspiel simuliert.
Hier www.java-forum.org/de/viewtopic.php?t=23928
findet ihr sie.

Ich bitte euch einmal den Code durchzugehen und mal zu schreiben, was euch so auffällt.
Ich bitte um konstruktive Kritik am Code, damit ich ihn verbessern bzw. verkleinern kann.

Zeile 478 - 514 z.b. finde ich ziemlich lang. Würde es sinn das ganze in eine for-Schleife zu packen?

290 - 351: Die Methode soll die Kontostände der drei Spieler vergleichen und den größten rausfinden.
Das wäre kein Problem gewesen. Jetzt muss man ja aber noch berücksichtigen, was passieren soll, wenn 2 oder mehr Spieler den gleichen Kontostand haben. Ich hab das irgendwie nicht hinbekommen, war aber auch schon spät.
Bitte um nen kleinen Denkanstoß, aber bitte keine fertige Lösung, wills mir nicht zu einfach machen!

Danke für eure Aufmerksamkeit!

 

[bygones]

Zeile 478 - 514 z.b. finde ich ziemlich lang. Würde es sinn das ganze in eine for-Schleife zu packen?

sorry, aber alleine diese Zeile hier schreckte mich davon ab, den Code mir genauer anzuschauen. über 500 Zeilen für eine Klasse find ich schon ziemlich hart. Mein erster Tipp, versuch deinen Code besser zu trennen. Lieber ein, zwei Klassen mehr, die dafür übersichtlicher und somit verständlicher....

 

[DreamArtist]

Würde generell versuchen das ganze ins MVC - Konzept zu gliedern.
Verschafft einen besseren Überblick und bringt eine Struktur in das Ganze.

 

[Mag1c]

Moin,

genau, und jetzt das ganze auf OOP

bei diesem Projekt läßt sich sehr schön das Konzept der OOP umsetzen. Eine Klasse für den Spieler, eine Klasse für das Glücksrad und eine Klasse für die GUI. (evtl. noch eine Klasse für den Spielablauf). Damit sollte das ganze wesentlich übersichtlicher werden und du kannst vor allem in nullkommanix 20 Spieler mitspielen lassen.

Gruß
Mag1c

 

[Bleiglanz]

478-514: mach lieber eine Lookuptable (Map) draus

Spieler1 Spieler2 Spieler3: sowas ist ganz furchtbar, dein Programm wäre wahrscheinlich kürzer und besser wenn du gleich n Spieler unterstützen würdest...

leider sind fast alle Kommentare "sinnlos", erklärt wird meistens nichts

                // Buttons werden wieder aktiviert.
                gluecksradinteraktion.aktiviereButtons();

Stichwort: kommentiere WIE etwas gemacht wird, nicht WAS

Hauptkritik: das ganze ist einfach zu lang, man soll nicht alles in eine einzige Klasse packen! Auch die Methoden sind teilweise zu lang.

IMHO wär das ganze jetzt eine gute Gelegenheit, die Refactoring Funktionen deiner IDE zu lernen

 

[Sky]

Also, ein Glücksrad sollte aus mehr als einer Klasse bestehen:

z.B.:
- eine grafische Komponente
- eine Klasse "Spieler"
- eine Klasse für Frage/Antwort
- Eine Klasse, die Spieler und/oder Runden verwaltet

Und schon wird's ein bißchen übersichtlicher

 

[dyn]

habe ja schon mehrere klassen. das problem ist aber dass es gerade durch mehrere klassen unübersichtlicher wird.
ich habe bereits 12 klassen insgesamt...naja ich guck nomma durch was ich so ändern kann..

 

[Sky]

habe ja schon mehrere klassen. das problem ist aber dass es gerade durch mehrere klassen unübersichtlicher wird.
ich habe bereits 12 klassen insgesamt...naja ich guck nomma durch was ich so ändern kann..

Wieso dass denn ? Werd mal bitte konkreter...

 

[bygones]

habe ja schon mehrere klassen. das problem ist aber dass es gerade durch mehrere klassen unübersichtlicher wird.
ich habe bereits 12 klassen insgesamt...naja ich guck nomma durch was ich so ändern kann..

wenn du es schaffst die Klassen logisch zu trennen sollte eben dies nicht geschehen....

Einfach so einen große Klasse zu zerstückeln ist natürlich kontraproduktiv...

was natürlich verständlich ist, wenn du ohne IDE arbeitest und dann 12 Files offen hast - aber das hat nichts mit der Übersicht der struktur zu tun....

 

[dyn]

ja...ich habe bereits eine spieler-klasse. dann habe ich für jedes panel das in der gui angezeigt wird eine eigene klasse angelegt.
ausserdem habe ich eine klasse spielabfolge in der die spieler initialisiert werden und die methode drin ist die bestimmt wer dran ist usw.
ich hatte das schonmal aufgeteilt in RATEN und PRUEFEN.
habs dann aber wieder rückgängig gemacht.

Einfach so einen große Klasse zu zerstückeln ist natürlich kontraproduktiv...

da geb ich dir recht. war auch ein fehler. es sei mir aber nach meinem ersten java-semester verziehen, wo ich mir aufgrund der mangelnden materialien der vorlesung so ziemlich alles selbst beibringen musste.

 

[bygones]

Einfach so einen große Klasse zu zerstückeln ist natürlich kontraproduktiv...

da geb ich dir recht. war auch ein fehler. es sei mir aber nach meinem ersten java-semester verziehen, wo ich mir aufgrund der mangelnden materialien der vorlesung so ziemlich alles selbst beibringen musste.

glaub mir am anfang ist das logische Trennung von Klassen verdammt schwer. Meine ersten Programme bestanden auch aus 1000Zeilen Klassen... man lernt nach ner Zeit wie man was am besten trennt....

so keep on going