Verbesserungsvorschläge?! Kleines AWT Programm

[JavaRookieSteffen]

Hallo!

ich bin ein Java (und Foren)-Neuling. Habe mich nun seit einiger Zeit mit Java beschäftigt und nun ein kleines Programm geschrieben, das eine leichte Matheaufgabe stellt und mit den Ergebnissen eben umgehen muss. Mir geht es hauptsächlich um den AWT-Part.
Da ich, wie gesagt, keine Erfahrung habe und versuche mich zu verbessern, würde ich den Code gerne reinstellen und mal hören, was Ihr meint: wo etwas schlecht ist, was eleganter wäre usw. Die Mainklasse schenke ich mir, die ruft nur die Fenster-Klasse auf. Bin über Anregungen sehr dankbar. Auch über Tipps, wie man nach den ersten Schritten weitermacht, um sich zu verbessern und "professioneller" zu werden.

Gruß

import java.awt.*;
import java.awt.event.*;

@SuppressWarnings("serial")
public class Fenster extends Frame {
    private Panel panel1, panel2;
    private Button button1, button2, button3;
    static public TextField eingabe;
    private Label label1;
    private Label label2;
    private int a, b;

    Fenster() {
        //Initalisieren des Fensters
        super();
        this.setLayout(new GridLayout(3, 1));
        this.setSize(400, 400);
        this.setLocation(700, 400);

        //Aufbau des Fensters
        //Erstes Panel mit Text und Aufgabe       
        panel1 = new Panel(new GridLayout(2, 1));
        label1 = new Label("Matheaufgaben", Label.CENTER);
        label2 = new Label("Zum Starten ´Neue Aufgabe´ drücken", Label.CENTER);
        panel1.add(label1);
        panel1.add(label2);
       
        //EingabeFeld
        eingabe = new TextField(23);
       
        //Zweites Panel (drittes Element) mit den 3 Buttons
        panel2 = new Panel();
        button1 = new Button("NeueAufgabe");
        button1.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                neueAufgabe();
            }
        });
        /*Abdeckung der 3 Möglichkeiten:
         * 1. Falsche Eingabe
         * 2. korrekte Lösung
         * 3. falsche Lösung aber korrekte Eingabe
         * als Actionhandling wird immer das AntwortFenster aufgerufen in einer
         * anonymen Klasse
         */
        button2 = new Button("Lösung");
        button2.addActionListener(new ActionListener() {
            int c;
            @Override
            public void actionPerformed(ActionEvent e) {
                try {
                    c = Integer.parseInt(eingabe.getText());
                } catch (NumberFormatException keineZahl) {
                    AntwortFenster antfenster = new AntwortFenster();
                    antfenster.antButton.setLabel("Nochmal");
                    antfenster.antLabel.setText("Falsche Eingabe");
                    antfenster.setVisible(true);
                    return;
                }
                if ((a + b) == c) {
                    AntwortFenster antfenster = new AntwortFenster();
                    dispose();
                    antfenster.antButton.setLabel("Nächste Aufgabe");
                    antfenster.antLabel.setText("Korrekt");
                    antfenster.setVisible(true);

                } else {
                    AntwortFenster antfenster = new AntwortFenster();
                    antfenster.antButton.setLabel("Nochmal");
                    antfenster.antLabel.setText("Falsch");
                    antfenster.setVisible(true);
                }
            }
        });
        button3 = new Button("Ende");
        button3.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent arg0) {
                System.exit(0);
            }
        });
   
        panel2.add(button1);
        panel2.add(button2);
        panel2.add(button3);
       
        //Hinzufügen ins Frame
        this.add(panel1);
        this.add(eingabe);
        this.add(panel2);
   

        this.addWindowListener(new WindowAdapter() {
            @Override
            public void windowClosing(WindowEvent e) {
                System.exit(0);
            }
        });

    }
    /*Erzeugt die Zufallszahlen, fügt sie in das Label ein
     *und leert das Textfeld
     */
    public void neueAufgabe() {
        a = (int) (Math.round(Math.random() * 10) + Math
                .round(Math.random() * 10));
        b = (int) (Math.round(Math.random() * 10) + Math
                .round(Math.random() * 10));
        String s = (a + " + " + b);
        label2.setText(s);
        eingabe.setText("");

    }

}

import java.awt.*;
import java.awt.event.*;

@SuppressWarnings("serial")
public class AntwortFenster extends Frame {

    Button antButton;
    Label antLabel;

    AntwortFenster() {
        super();
        this.setSize(300, 100);
        this.setLocation(750, 400);
        Panel panel = new Panel();
        antButton = new Button();
        antLabel = new Label();
        panel.add(antLabel);
        panel.add(antButton);
        this.add(panel);
        antButton.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent e) {
                if (e.getActionCommand() == "Nächste Aufgabe") {
                    dispose();
                    Fenster fenster = new Fenster();
                    fenster.neueAufgabe();
                    fenster.setVisible(true);
                } else {
                    Fenster.eingabe.setText("");
                    dispose();
                }
            }
        });
    }
}

 

[Joose]

was Ihr meint: wo etwas schlecht ist, was eleganter wäre usw.

Ist es Absicht dass du mit AWT arbeitest anstatt mit Swing? Ich würde dir raten mit Swing weiterzumachen (das baut auf AWT auf und hat mehr Möglichkeiten).

Zu deinem Code:
Du solltest nicht von einer Klasse ableiten wenn du nicht vorhast deren Funktionalität zu erweitern, sondern nur ein Objekt von ihr brauchst. Also anstatt von Frame abzuleiten erstelle einfach ein Frame Objekt und arbeite mit diesem.

Vergib deinen Komponenten sprechendere Namen als "button1" (->newTask), "button2" (->resolve) bzw. "button3" (-> exit). Ob du bei den Namen dann noch "button" bzw. "label" dranhängst ist so eine Sache.
Manche meinen so kann man schneller sehen um was es sich handelt, andere meinen der Name sollte schon sprechende genug sein.

Der Aufruf von super() ist in diesem Fall unnötig. Da der Defaultkonstruktor der Superklasse automatisch aufgerufen wird (gibt es keinen Defaultkonstruktor würdest du einen Fehler angezeigt bekommen).
Da die Ableitung von Frame hinfällig ist, ist auch der Aufruf hinfällig.

Vermeide "static" in deinem Code. Vor allem um auf ein bestimmtes Control aus einer anderen Klasse zuzugreifen ist das der falsche Weg.
Übergib lieber eine Referenz dieses Control an die andere Klasse.

Objekte (String ist auch ein Objekt) werden inhaltlich mit equals verglichen, mit == vergleichst du nur die Referenz!

Du solltest den Zugriff auf die Komponenten des AntwortFenster über Methoden regeln und keinen direkten Zugriff zulassen.
Auch erstellst du immer ein neues Fenster bzw. AntwortFenster. Verwende lieber schon die vorhandenen Objekte.

Beispiel:

public class Fenster extends Frame {
   ....
    private TextField eingabe;
  
   private AntwortFenster antwort;

    Fenster() {
        this.setLayout(new GridLayout(3, 1));
        this.setSize(400, 400);
        this.setLocation(700, 400);

       ......
    
        //EingabeFeld
        eingabe = new TextField(23);
       antwort = new AntwortFenster(this);
      
       ....
      
        button2 = new Button("Lösung");
        button2.addActionListener(new ActionListener() {
            int c;
            @Override
            public void actionPerformed(ActionEvent e) {
                try {
                    c = Integer.parseInt(eingabe.getText());
                } catch (NumberFormatException keineZahl) {
                   antwort.setData("Nochmal", "Falsche Eingabe");
                   antwort.setVisible(true);
                    return;
                }
               .....
            }
        });
       .......
    }
  
   public TextField getEingabe() {
       return eingabe;
   }
}

public class AntwortFenster extends Frame {
    private Button button;
    private Label label;
   private Fenster fenster;
  
    AntwortFenster(Fenster fenster) {
       this.fenster = fenster;
       fenster.setVisible(false);
       ....
        button.addActionListener(new ActionListener() {
            public void actionPerformed(ActionEvent e) {
                if (e.getActionCommand().equals("Nächste Aufgabe")) {
                    setVisible(false);
                   fenster.neueAufgabe();
                    fenster.setVisible(true);
                } else {
                   fenster.getEingabe().setText("");
                    setVisible(false);
                }
            }
        });
    }
  
   public void setData(String buttonText, String labelText) {
       button.setLabel(buttonText);
       label.setText(labelText);
   }
}

 

[JavaRookieSteffen]

Hallo Joose,

vielen Dank für die Mühe und die Anregungen!

Gruß

 

[krgewb]

Zu Beitrag #1:

Bisher setzt du Location manuell. Mit folgender Zeile Code kannst du es mittig positionieren:

this.setLocationRelativeTo(null);

Die Mainklasse schenke ich mir, die ruft nur die Fenster-Klasse auf.

Es muss aber auch noch setVisible(true) deklariert werden.

 

[Joose]

Es muss aber auch noch setVisible(true) deklariert werden.

Das wird ihm sehr wohl bewusst sein, da es ja funktioniert und es keine Probleme gibt