JButton identifizieren

[Pearls]

Guten Morgen

Bei meinem Kartenspiel werden die Karten per Funktion als JButtons dargestellt. Nun habe ich ein ActionEvent eingebaut, welches mir mit System.out.println(e.getSource()); folgendes ausgibt:

javax.swing.JButton[,14,14,101x100,alignmentX=0.0,alignmentY=0.5,border=javax.swing.border.EmptyBorder@6bade9,flags=296,maximumSize=,minimumSize=,preferredSize=,defaultIcon=javax.swing.ImageIcon@128f6ee,disabledIcon=,disabledSelectedIcon=,margin=javax.swing.plaf.InsetsUIResource[top=2,left=14,bottom=2,right=14],paintBorder=true,paintFocus=true,pressedIcon=,rolloverEnabled=true,rolloverIcon=,rolloverSelectedIcon=,selectedIcon=,text=,defaultCapable=true]

Wie kann ich das JButton nun identifizieren um zu wissen, welche der Karten angeklickt wurde?

 

[jgh]

durch abfragen der entsprechenden Variablennamen.
wenn du deine Karten allerdings irgendwie in dieser Art initialisierst...:

for (int i=0; i< kartenanzahl; i++){
      kartenPanel.add(new JButton(kartenListe[i]);
     }

könnte das m.E. ein Problem sein?!

Wenn du allerdings eindeutige Variablen verwendest, sollte dir

if (e.getSource==jButtonHerzKoenig){
   //...
   }

true zurückgeben und entsprechend "handlebar" sein

 

[Pearls]

Die Buttons werden so erstellt:

    private void Bilder() {
        ArrayList list = new ArrayList();
        for (int i = 0; i < ANZAHL / 2; i++) {
            list.add(i);
            list.add(i);
        }
        Collections.shuffle(list);
        Object lista[] = list.toArray();   //get array

        for (int i = 0; i < lista.length; i++) {
            int img = ((Integer) lista[i]);
            filename = "/images/" + img + ".jpg";
            try {
                bilder[i] = ImageIO.read(this.getClass().getResource(filename));
                cards[i] = new JButton(new ImageIcon(bilder[i]));
                cards[i].setBorder(BorderFactory.createEmptyBorder());
                cards[i].addActionListener(this);
                this.add(cards[i], i);
//        System.out.println(lista);
                for (int j = 0; j < lista.length; j++) {
                    System.out.print(" " + ((Integer) lista[j]));
                }
                System.out.println();

            } catch (IOException ex) {
                System.out.println(ex.getMessage() + " (" + filename + ")");
            }
        }

    }

    public void actionPerformed(ActionEvent e) {
        System.out.println(e.getSource());
    }
}

Nun möchte ich ablesen können, welcher angeklickt wurde.

 

[KrokoDiehl]

Du kannst mit

setName()

dem Button auch einen Namen geben (das ist nicht

setText()

) oder auch ein Action-Command via

setActionCommand()

.
Das sind beides Strings, die du dann im ActionListener nachprüfen kannst.

 

[jgh]

 public void actionPerformed(ActionEvent e) {
        for (int i=0; i< cards.length; i++){
            if (e.getSource()==cards[i]){
                System.out.println("Ausgewählte Karte: " +i ;
               }
        }
    }

so sollte das klappen...auch wenn es nicht schön ist^^

 

[Pearls]

Super! Vielen Dank!

 

[Marco13]

Das mit dem getName gefällt mir irgendwie nicht (ist aber eher so was subjektives... weil dort so "untransparente" Verbindungen hergestellt werden, die anhand der jeweiligen Codestücke nicht ersichtlich sind... von Tippfehlern oder versehentlichen Umbenennungen mal abgesehen). Schau dir auf jeden Fall auch mal http://www.java-forum.org/bilder-gui-damit-zusammenhaengt/49255-componenten-listener.html an...

 

[Marcinek]

Hallo,

bevor ich irgentwas in der Richtung machen würde, was die beiden vorredner hier von sich geben, würde ich es so machen

MeineKarte extends JButton {

    String kartenName = "";

...

}

Dann benutz du statt JButton MeineKarte und kannst dieser Karte einen Namen oder eine Komplexe Struktur einer Karte geben.

Die Sache mit dem Array durchsuchen finde ich gräßlich und habe dies noch in keinem richtigen Projekt gesehen.

Die Buttons haben Namen, damit man darüber sehen kann, welcher Button gedrückt worden ist. Also sollte man das auch nutzen. Das Argument mit "tippfehler" kann ich überhaupt nicht nachvollziehen, wer nicht schreiben kann, benutzt Konstanten.



Alternativ kann man das, wie bereits in einem anderen Thrad durch ActionEvents machen.

 

[Marco13]

Hm. Das mit dem Tippfehler war kein wirkliches Argument. Da jetzt was mit Vererbung zu machen, und statt set/getName nun set/getKartenName zu verwenden verbessert das IMHO nicht wirklich. Ähnliches habe ich zwar schon in Projekten gesehen, aber ob die schwer objektiv zu fassende Bedingung, "richtig" zu sein, auf diese zutreffen, sei mal dahingestellt. Auf Basis des bisher geposteten Codes maße ich mir mal ganz frech an, zu behaupten, dass da noch nicht viel ... ... Engineering oder allgemein "Struktur" drin steckt, deswegen würde das Argument "Eine Karte ist kein Knopf" hier vielleicht auch nicht greifen, aber ... im allgemeinen schon.