Zitat:
Zitat von tuete771
kannst du das spezifizieren? man will ja stets dazulernen...
|
Moderne Programmiersprachen (wie eben Java) bieten genug Möglichkeiten, um "sauberen" Code zu schreiben. Du verwendest aber keine einzige davon.
Dein Code scheint dann vielleicht irgendwie zu funktionieren, aber es ist keine Struktur zu erkennen!
1. Dein Konstruktor baut ein UI auf! Das ist schon mal ziemlicher Murks.
Konstruktoren sollen Dein Objekt nur instanzieren.
Methoden sind dazu da, um das Objekt agieren zu lassen, aber nicht Konstruktoren.
2. Deine Code-Kommentare ("Button", "Fenster", "Label") sind nutzlos.
Da kannst Du auch gleich vor jede Variable "Variable", vor jede Klasse "Klasse" und vor jede Methode "Methode" schreiben.
3. Du definierst globale Variablen, die Du nur lokal verwendest.
4. Du definierst Null-Variablen und keinen einzigen Null-Check.
5. Du verwendest konstante Werte, definierst diese aber nicht als Konstanten
6. Deine if-Abfragen sind nicht performant sondern überprüfen die EventSource doppelt
7. String-Vergleich mit "==" statt equals
8. Keine catch/try beim Parsen
9. Kein Extrahieren mehrfach verwendeter Objekt-Eigenschaften (.getSource(), .getText())
10. Unnütze Verwendung/Instanzierung neuer Variable (Euro, Dollar kannst Du auch mit einer Double-Variable errechnen)
11. Keine Verwendung von Methode sondern alles in einem Block (hier auch noch der Konstruktor)
12. Package-Name = Klassen-Name
13. Euro, Dollar, Button sind public
und und und...