Hi...
.. das ist so schwer zu sagen.
Der Code ist - um es kurz und knapp zu sagen - ziemlich schlecht.
Schlecht, da zu viele Abhängigkeiten existieren und nahe zu jede Zeile gegen OOP-Grundsätze verstößt.
Andererseits kannst du von diesen OOP-Grundsätzen in der Schule mit Sicherheit nie etwas gehört haben. Ich bin mir auch sicher, dass euer Lehrer keine Ahnung hat, was OOP, OOAD etc. wirklich bedeuten.
Du solltest Dir mal die Designfragen durch den Kopf gehen:
- Warum hat dein Umrechner ein Fenster? Der Währungsumrechner soll Währungen umrechnen! Wozu braucht er ein Fenster?
- Warum kann dein Währungsumrechner KEINE Währung umrechnen? Genau das ist doch seine Aufgabe... der Umrechner hat aber lediglich die Methode "currentName" und "fuehreAus"?
- Wo kann man dem Währungsrechner denn mitteilen, welche Währung und welcher Betrag umgerechnet werden soll?
Das macht so doch keinen Sinn.
Des Weiteren ist der Code an vielen Stellen kopiert und nicht in Methoden ausgelagert.
|