Nicht benannte Buttons - Wenn gedrückt > Action

[Guest]

Hi,
ich habe hier viele Buttons (für einen Taschenrechner -> als kleine Übung), doch diese Buttons haben ja keinen Namen! Wie verleihe ich ihnen dann Action wenn ich zB.: auf 1 drücke?

void createGuiAndShow()
    {
        setTitle("Taschenrechner");
        setSize(200, 240);
        setResizable(false);
        add(display, BorderLayout.NORTH);
        add(nummernblock);
        
        nummernblock.add(new Button("%"));
        nummernblock.add(new Button("/"));
        nummernblock.add(new Button("*"));
        nummernblock.add(new Button("Exit"));
        
        nummernblock.add(new Button("7"));
        nummernblock.add(new Button("8"));
        nummernblock.add(new Button("9"));
        nummernblock.add(new Button("C"));
        
        nummernblock.add(new Button("4"));
        nummernblock.add(new Button("5"));
        nummernblock.add(new Button("6"));
        nummernblock.add(new Button("+"));
        
        nummernblock.add(new Button("1"));
        nummernblock.add(new Button("2"));
        nummernblock.add(new Button("3"));
        nummernblock.add(new Button("-"));
        
        nummernblock.add(new Button("."));
        nummernblock.add(new Button("0"));
        nummernblock.add(new Button("CE"));
        nummernblock.add(new Button("Enter"));
        
        display.setEditable(false);
        display.setText(anzeige);
        
        setVisible(true);
    }

Danke
MfG

 

[Escorter]

Warum speicherst du die Buttons nicht in Variablen?

Als alternative könntest du JButton vererben:

class Taste extends JButton{
 public Taste(){
  addActionListener(new myListener(this));
 }
}
class myListener() implements ActionListener{

String buttonText;

public myListner(JButton b){
 buttonText = b.getText();
}

public void actionPerformed(){
if(b.equals("+")){
add();
}
....
}

}

Gruß
Esco

 

[Guest]

Weil ich mir gedacht habe das es doch auch so irgendwie gehen sollte oder täusche ich mich da?

 

[Escorter]

Wie gesagt vererbe JButton und setzte einen Listener.

Dann musst du halt wie oben beschrieben, den Text des Buttons auslesen damit der Listener weiß was er tun muss.

Gruß,
Esco

 

[Guest]

Kann man ihn nicht gleich nach dem Text suchen lassen anstatt ihn zu sagen welcher Button den Text haben könnte/hat?

 

[Escorter]

wie willst du auf den text zu greifen?
der actionlistener ist ne eigene klasse.
Also hast du ein paar möglichkeiten.

1. beim konstruktor entscheiden welche actionlistener benutzt werden soll - schlechte lösung
2. beim konstruktor den button text mitgeben - funzt solange wie sich der button text nicht zur laufzeit ändert
3. den ganzen button an den actionlustener weitergeben.

Gruß,
Esco

 

[Maeher]

Wenn du unbedingt Text haben willst, kannst du auch (AbstractButton).setActionCommand verwenden. Diesen Text kannst du aus dem ActionEvent auslesen, dann würde dir ein Listener für alle reichen.

 

[Escorter]

Bei meiner Lösung reicht auch ein Listener.

Aber irgendwie muss man erkennen, welcher button gedrückt wurde - und wie, wenn nicht über den text?

Gruß,
Esco

 

[Guest]

Ich habe mich jetzt für diese Lösung entschieden:

void createGuiAndShow()
    {
        setTitle("Taschenrechner");
        setSize(265, 300);
        setResizable(true);
        add(display, BorderLayout.NORTH);
        add(nummernblock);
        
        button[0] = new JButton("%");
        button[1] = new JButton("/");
        button[2] = new JButton("*");
        button[3] = new JButton("Exit");
        button[4] = new JButton("7");
        button[5] = new JButton("8");
        button[6] = new JButton("9");
        button[7] = new JButton("C");
        button[8] = new JButton("4");
        button[9] = new JButton("5");
        button[10] = new JButton("6");
        button[11] = new JButton("+");
        button[12] = new JButton("1");
        button[13] = new JButton("2");
        button[14] = new JButton("3");
        button[15] = new JButton("-");
        button[16] = new JButton(".");
        button[17] = new JButton("0");
        button[18] = new JButton("CE");
        button[19] = new JButton("Enter");
        
        for(int i = 0; i < 20; i++)
            nummernblock.add(button[i]);
        
        display.setEditable(false);
        paint();
        setVisible(true);
    }

 

[Gast]

Nur wie mache ich hier nun etwas wenn ich einen Button klicke?

 

[Escorter]

du musst einen action Listener dem Button zuweisen...

for(int i = 0;i<button.length;i++){
button[i].addActionListener(button[i]);
}

public myListner(JButton b){
buttonText = b.getText();
}

public void actionPerformed(){
if(b.equals("+")){
 // Deine Action
}
....
}

Gruß,
Esco

 

[Marco13]

Von JButton erben, nur um ihm einen ActionListener zuzuweisen ist ... :autsch: suboptimal.

Du kannst z.B. mal hier nachsehen: http://www.java-forum.org/de/viewtopic.php?p=310844
Statt eines direkten Vergleiches wie

if (event.getSource() == buttonSoUndSo) {...}

kann man ggf. auch das ActionCommand abfragen, wie es schon angedeutet wurde.

if (event.getActionCommand().equals("+")) {...}

aber da muss man ein bißchen aufpassen, dass es nicht zu unübersichtlich wird. Du solltest auch in Erwägung ziehen, die Buttons zu gruppieren:

JButton numberButtons[] = new JButton[10]; // 0 ... 9
JButton operatorButtons[] = new JButton[5]; // +, -, * ...
JButton functionButtons[] = new JButton[5]; // C, CE, ...

Dann kann man ggf. für jede Gruppe denselben ActionListener verwenden, und es ist einfacher, zu entscheiden, was jeweils gemacht werden soll. (Z.B. muss man für alle Numbers nur das ActionCommand in einen int umwandeln, und bei den operatoren nur das erste Zeichen des ActionCommands lesen, und kann dann ein switch über '+', '-', '*' usw machen...)

 

[Escorter]

Von JButton erben, nur um ihm einen ActionListener zuzuweisen ist ... :autsch: suboptimal.

Warum?

Gruß,
Esco

 

[Marco13]

Naja ... übertrieben konsequent weitergeführt klingt das irgendwie nach sowas wie

class ListWithNames extends ArrayList
{
    public ListWithNames()
    {
        add("Alice");        
        add("Bob");
    }
}

... :? Das würde ja auch kein Mensch machen. (halt: Das sollte ja kein Mensch machen :wink: )

Und ganz allgemein, konzeptionell... welche Idee soll denn dahinter stecken? Der Button wird weder um echte Eigenschaften, noch um Methoden erweitert.

Es KÖNNTE zwar manchmal vielleicht Sinn machen, von einer Klasse abzuleiten, um sie als eigenständigen Typ (zwar nicht mit zusätzlichen, aber mit bestimmten Eigenschaften) vorliegen zu haben (insbesondere wenn z.B. abzusehen oder geplant ist, dass in Zukunft noch Eigenschaften oder Methoden dazukommen) aber in diesem speziellen Fall sehe ich da keinen Grund... :?

 

[Verjigorm]

Naja ... übertrieben konsequent weitergeführt klingt das irgendwie nach sowas wie

class ListWithNames extends ArrayList
{
    public ListWithNames()
    {
        add("Alice");        
        add("Bob");
    }
}

... :? Das würde ja auch kein Mensch machen. (halt: Das sollte ja kein Mensch machen :wink: )

Und ganz allgemein, konzeptionell... welche Idee soll denn dahinter stecken? Der Button wird weder um echte Eigenschaften, noch um Methoden erweitert.

Es KÖNNTE zwar manchmal vielleicht Sinn machen, von einer Klasse abzuleiten, um sie als eigenständigen Typ (zwar nicht mit zusätzlichen, aber mit bestimmten Eigenschaften) vorliegen zu haben (insbesondere wenn z.B. abzusehen oder geplant ist, dass in Zukunft noch Eigenschaften oder Methoden dazukommen) aber in diesem speziellen Fall sehe ich da keinen Grund... :?

hey, schöne Idee zur Initalisierung einer Arraylist, muss ich mir merken!

 

[siriuswhite]

Denk an die ActionCommands

//button wäre der Name vom Button
button.setActionCommand("button");
//musst natürlich "button durch dein Action Command(aber mit "") ersetzen


//später im Listener
public void actionPerformed(ActionEvent evt)
{
  if(evt.getActionCommand.equals("button"))
   //mach was der button machen soll
}

So mach ichs immer
(kann aber sein dass es nur mit JButton geht,weiß ich nicht so genau)

 

[Escorter]

Wo ist der Vorteil wenn ich action command statt den text benutze?

Das einzige was mir spontan einfällt das ich die gleiche Aktion von verschiedenen Buttons aus ausführen kann.

Gruß,
Esco

 

[Marco13]

... oder die gleiche Aktion von verschiedenen Buttons :wink: Oder dass man einem Button eine Aktion zuweisen kann, der keinen Text, sondern nur ein Icon enthält, oder einem Button eine feste Aktion geben kann, obwohl die Aufschrift des Buttons sich ändert, und dass man sich evtl. das Leben erleichtert, wenn man mal eine Internationalisierte Anwendung schreiben will... Aber meistens besteht trotzdem kein direkter Grund, warum man "immer" das ActionCommand per Hand setzen sollte... :?
Für einige dieser Sachen wären vielleicht auch Actions günstiger als spezielle ActionListener, aber das muss man von Fall zu Fall entscheiden...

 

[Escorter]

Ok überzeugt.

Aber ich denke ich blaib für mich dabei für jede Action einen eigenen ActionListener zu schreiben, denn dann ist es mir egal welcher Button die Action auslößt...


Gruß,
Esco

 

[Maeher]

Aber ich denke ich blaib für mich dabei für jede Action einen eigenen ActionListener zu schreiben, denn dann ist es mir egal welcher Button die Action auslößt...

Geht natürlich wunderbar, nur erzeugst du damit jede Menge Hilfsklassen (auch wenn man nicht optimieren soll: Die kosten doch immer irgendwie Speicherplatz, bis zu mehreren 100Byte :lol: und machen meiner Meinung nach den Code mit den vielen Unterklassen unübersichtlicher als wenn man an einer Stelle sämtliche Events mit if-Abfragen beieinander hat).
Das ist wahrscheinlich Geschmackssache, die meisten automatischen GUI-Designer machen es übrigens wie du :lol: .

 

[Escorter]

GUI Designer sind doch scheiße, da kennt man seinen eigenen code nicht. Ne GUI von Hand schreiben ist nicht so schwer wie man denkt, wenn man sich etwas eingearbeitet hat.

Das mit dem Listenern ist, wie du schon gesagt hast, geschmacks Sache und naja der Speicherplatz...

Was mich noch interessieren würde was denn die performantere Lösung ist?

Gruß,
Esco

 

[Guest]

Action Command hat auch den Vorteil dass du die Knöpfe unterscheiden kannst,aber nur einen Listener pro Klasse brauchst

 

[siriuswhite]

Gast war ich,hab vergessen mich einzuloggen

 

[Guest]

Hi, danke für die vielen Anregungen/Lösungen.

Habe nur eine Frage zur JAVA FAQ in dem Forum, warum erstellt man dort eine eigene Methode (private JPanel createButtonsPanel()) und schreibt das ganze nicht einfach in den Konstruktor?

Danke

 

[Marco13]

Sowas wie

public Application()
{
    add(createControlPanel());
    add(createMainPanel());
}

private JPanel createControlPanel()
{
    JPanel controlPanel = new JPanel();
    controlPanel.add(createButtonPanel());
    controlPanel.add(createSliderPanel());
    controlPanel.add(createOptionsPanel());
    return controlPanel;
}

private JPanel createOptionsPanel()
{
    JPanel optionsPanel = new JPanel();
    optionsPanel .add(...);
    ...
    return optionsPanel;
}
...

ist doch deutlich übersichtlicher, aufgeräumter und leichter wartbar(!) als ein 5642-Zeilen-Konstruktor, wo man 56 ineinander verschachtelte Panels mit 324 Buttons und anonymen Listenern und 10 verschiedenen Layouts zusammenpfuscht..... Aber es geht natürlich beides... :roll: