eigene Methoden für Buttons?

[rorton]

im unterricht haben wir gehört es sei sinnvoll, für die erzeugung von buttons eigene methoden für die erstellung zu schreiben!
ist das wirklich praxisnahe?
was ist wenn ich 30 buttons habe, dann müßte ich für jeden button eine eigene methode schreiben.. zumindest wenn es nach unserem lehrer geht!

was sagt ihr dazu?

 

[XHelp]

Naja, wenn du 30 Buttons auf einem Frame hast, dann hast du meist eh was falsch gemacht
Vermutlich meint der Lehrer etwas ähnliches wie:

class MyFrame extends JFrame {
  public MyFrame() {
    super("Frame");
    initGui();
  }
  private void initGui() {
    //in dieser Methode eben alles Initialisieren und auf dein Frame packen
  }
}

 

[rorton]

nö, di init-methode meine ich nicht..

schaut eher so aus:

public JButton getButton_ok() {
     if (ok_button == null) {
           ok_button = new JButton("OK");
     }
     return ok_button;
}

Bei unserem Beispiel hatten wir 5 Buttons, die wir alle mit dieser Methode erzeugt und zum frame hinzugefügt haben..
das ganze auch mit Labels, TextFields..

ja, 30 buttons sind vielleicht etwas übertrieben, aber ingesamt 30 komponenten auf einem frame ist wohl eher nicht so ungewöhnlich, zb. bei einem formular, oder? wie würdest du das machen?

 

[XHelp]

Aso. Ja, sowas habe ich schon mal gesehen, vor allem in den GUIs die mit einem Builder erstellt wurden. Aber ich persönlich habe es nie verwendet. (zumal ich auch nicht oft GUIs baue)

 

[Marco13]

Als pauschale Regel ist das ... zu pauschal Das, was du da al Beispiel geschrieben hast, mit dem "if (ok_button == null)" kommt wirklich eher von GUI-Buildern: Dort wird oft so eine Methode erstellt, die dann überall in dem Stil verwendet wird

getOKButton().setEnabled(true);
getOKButton().setBackground(Color.GREEN);
...

Dabei ersetzt die Methode praktisch die Instanzvariable. Das finde ich auch eher unschön...

Aber grundsätzlich kann es schon sinnvoll sein, das Erstellen von Buttons in eine Methode auszulagern. Z.B. wenn man in der Methode noch irgendwelche Initialisierungen des Buttons macht, oder insbesondere, wenn man auch noch gleich einen anonymen Listener an den Button hängt:

private JButton createOKButton()
{
    JButton button = new JButton("OK");
    button.addActionListener(new ActionListener()
    {
        public void actionPerformed(ActionEvent e)
        {
            okPressed();
        }
    });
    button.setEnabled(false);
    button.setBackground(Color.YELLOW);
    ....
    return button;
}

Dann in der init-Metode sowas zu haben wie

somePanel.add(createOKButton());
somePanel.add(createCancelButton());
somePanel.add(createRetryButton());
somePanel.add(createIgnoreButton());

ist schon übersichtlicher, als wenn dort alles "inline" steht. Aber das ist der Punkt: Wenn der Code dadurch übersichtlicher wird, kann man das schon machen.

 

[Antoras]

Ja, ich würde empfehlen das so zu machen. Der Grund ist, dass du so einfach einzelne Komponenten deaktivieren und die Komponentenhierarchie einfach überblicken kannst. Und dabei entstehen schon mal Klassen mit 1000+ Zeilen Code, das kann man nun mal nicht ändern.

Den kompletten Code in eine Methode zu schreiben bringt dir gar nichts, da er nicht wesentlich kürzer wird und ohne GUI-Builder praktisch nicht mehr zu warten ist.

 

[rorton]

so ist das,verstehe! Danke euch für die ausführliche beschreibung!
das ganze macht dann von der übersichtlichkeit her schon sinn, auch wenn die lines of code etwas ansteigen!

edit: könnt ihr mir einen tipp geben, wie ich so ein formular am besten hinbekomme... GroupLayout? oder mit mehreren Panels und BoxLayout, GridLayout und co.. ich weiß schon, es gibt keine Musterlösung, aber ein denkanstoß wäre sehr hilfreich!

 

[slawaweis]

schaut eher so aus:

public JButton getButton_ok() {
     if (ok_button == null) {
           ok_button = new JButton("OK");
     }
     return ok_button;
}

Bei unserem Beispiel hatten wir 5 Buttons, die wir alle mit dieser Methode erzeugt und zum frame hinzugefügt haben..
das ganze auch mit Labels, TextFields..

ja, 30 buttons sind vielleicht etwas übertrieben, aber ingesamt 30 komponenten auf einem frame ist wohl eher nicht so ungewöhnlich, zb. bei einem formular, oder? wie würdest du das machen?

es ist das Lazy-Creation-Pattern, d.h. ein Objekt wird erst dann erzeugt, wenn es benötigt wird. Es ist ein nützliches Pattern, aber meiner Meinung nach nicht an dieser Stelle. Ich arbeite eher mit dem Factory-Pattern. Hierbei ist der Vorteil, dass die gesamte Logik einer konkreten (Teil-)GUI an einer Stelle zu finden ist (und nicht über die Java-Datei verteilt ist), aber die reine Konstruktion von Objekten ausgelagert wird. Beispiel für Logik:

UIFactory.createButton(new Action(){
// ...
});
UIFactory.createButton("name1", "Title", new ActionListener() {
// ...
});
UIFactory.createButton("name2", "Title", "actionCommand");
UIFactory.createButton("name3", "Title", icon);
UIFactory.createButton("name4", "Title", icon, Color.white, Color.black); // Hintergrund, Vordergrund

der erste Parameter ist übrigens die ID einer Komponenten, die man über getName() bekommen kann. Diese Funktionalität war von Anfang an drin, wird aber kaum genutzt. Doch die ist sehr nützlich. Man kann sich so spezielle universelle Methoden zulegen, mit welchen man Komponenten aus einer GUI bekommt, ohne sich Klassenvariablen für jede Komponente anzulegen. Dann kann man solche schönen Sachen schreiben:

ContainerUtils.getAs(frame, "name4", JButton.class).setEnabled(true);

Slawa

 

[Antoras]

Das Factory-Pattern in GUIs einzusetzen hat mich noch nicht so richtig überzeugt.

UIFactory.createButton("name1", "Title", new ActionListener() {
// ...
});

Was wenn du dem Button hier neben dem Action- noch einen Focus- und MouseListener hinzufügen möchtest? Das ist doch nicht mehr übersichtlich. Zumal es ja noch hundert andere Sachen gibt mit denen man seine Komponenten dekorieren möchte. Ich will mir doch nicht die Position von jedem Parameter merken...

Und wie merkt man sich die Namen der Komponenten? Da ich den mehrmals benötige (beim Erstellen, Aufrufen, Konfigurieren...) müssten Konstanten angelegt werden. Ansonsten wäre das nicht mehr wartbar. Und dann gäbe es wieder keinen Vorteil, da Klassenvariablen durch Konstanten ausgetauscht werden würden.

Den Vorteil zusammengehörenden Code an einer Stelle wiederzufinden ist mit lazy creation auch gegeben. Die Methode zur Erzeugung der Komponente wird im besten Fall genau einmal aufgerufen - nämlich in der add-Methode der darunterliegenden Komponente.

 

[Gast2]

Also ich benutze auch eine WidgetFactory für einige Komponenten damit der initial Wert wie Highlighter, Farben, StandardAction gleich gefüllt wird. So dass die gleichen Komponenten ein einheitliches Verhalten haben.
Aber dabei wird die Instanz zurückgeliefert, so dass man einfach noch zusätzlich benötigte Listener anfügen kann.

Als Layout würde ich das JGoodies FormLayout runterladen.

 

[Raziell]

Hi,

Eventuell könntest du dir neben dem JGoodies FormLayout auch mal das MiGLayout
für deinen Zweck anschauen.

Gruß

 

[rorton]

Wow, danke für die vielen Antworten... Werd mich gleich mal ins Zeug legen! DANKE an ALLE für die ausführlichen Infos!