Buttons sollen bei Mausklick einen String schreiben

[waeve]

Hallo ,

ich hab hier in kleines großes Problem. Ich habe eine Grafikoberfläche mit den Buchstaben a - z. Jeder dieser Buchstabe wurde als JPanel-Objekt angelegt und in einer ArrayList gespeichert.
Nun möchte ich jeden dieser Buttons per Mausklick dazu bringen den Buchstaben, zu dem er gehört, in eine Stringvariable zu schreiben.
Es scheitert aber auch schon daran einen einzelnen Button aus der Liste anzusprechen.
Trotz langer Rechere finde ich keine Lösung :bahnhof:

import javax.swing.*;
import java.awt.*;
import java.util.*;
import java.awt.event.*;

/**
 * Write a description of class GUI here.
 * 
 * @author (your name) 
 * @version (a version number or a date)
 */
public class GUI extends JFrame
{
    //Deklarieren der Attribute
     private JButton button;
     private JPanel hangmanPanel;
     private JPanel center1, center2, center3;
     private ArrayList<JLabel> hangmanArray;
     private ArrayList<JLabel> buchstabenArray;
     private JPanel centerPanel;
     private String eingabe;
     
     private int anzahlStriche;
    /**
     * Constructor for objects of class GUI
     */
    public GUI()
    {
       super("Fenster");
       setLocation(300,300);
       setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);

       hangmanPanel= new JPanel();
       hangmanPanel.setLayout(new BorderLayout(5,5));
       centerPanel=new JPanel();
       centerPanel.setLayout(new BorderLayout(5,5));
       center1 = new JPanel();
       centerPanel.add(center1, BorderLayout.NORTH);
       center2 = new JPanel();
       centerPanel.add(center2, BorderLayout.CENTER);
       center3 = new JPanel();
       centerPanel.add(center3, BorderLayout.SOUTH);
       
       // ansatz anzahl der striche muss aus wortpool ermittelt werden
       anzahlStriche = 6;
       for(int i = 0; i<anzahlStriche; i++)
       {
            center1.add(new JLabel(new ImageIcon("img/strich.png")));
        }
       
       buchstabenArray = new ArrayList<JLabel>(26);
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/A.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/B.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/C.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/D.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/E.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/F.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/G.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/H.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/I.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/J.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/K.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/L.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/M.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/N.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/O.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/P.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/Q.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/R.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/S.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/T.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/U.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/V.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/W.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/X.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/Y.png")));
       buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/Z.png")));

Ich hoffe ihr könnt mir helfen
Vielen Dank im Vorraus

 

[Timothy Truckle]

Nun möchte ich jeden dieser Buttons per Mausklick dazu bringen den Buchstaben, zu dem er gehört, in eine Stringvariable zu schreiben.
Es scheitert aber auch schon daran einen einzelnen Button aus der Liste anzusprechen.

Der erste Schritt wäre ja, [JAPI]JButton[/JAPI] statt [STRIKE][JAPI]JLabel[/JAPI][/STRIKE] zu verwenden...

[EDIT]Wenn dann noch jeder Button seinen eigenen[JAPI]ActionListener[/JAPI] bekommt bist Du schon fast am Ziel.[/EDIT]

bye
TT

 

[waeve]

Danke für die schnelle Antwort

Wie sieht das denn ungefähr aus mit den ActionListener? Ich kann mir das nicht vorstellen.
Ich hab die Liste so bekommen. Kann ich das einfach verändern oder hat das Einfluss auf den Rest des Codes?

 

[Timothy Truckle]

Wie sieht das denn ungefähr aus mit den ActionListener?
Ich kann mir das nicht vorstellen.

buttonA.addActionListener(new ActionListener(){
  public void actionPerformed(ActionEvent ae){
    gesamtText.append("A"); // gesamtText ist Variable in Deinem Objekt von Typ StringBuilder.
    textFeld.setText(geamtText.toString());
  }
})

Ich hab die Liste so bekommen. Kann ich das einfach verändern oder hat das Einfluss auf den Rest des Codes?

Das schöne an OOP ist, dass wenn man's richtig macht selbst größere Änderungen nur wenig umgebenden Code beeinflussen. So auch hier: nur die Generierung der Buttons ist betroffen...

bye
TT

 

[L-ectron-X]

Nur so noch als Einwurf: Was hältst du von [JAPI]Action[/JAPI]s bzw. [JAPI]AbstractAction[/JAPI] s?
Ist wahrscheinlich noch ein wenig übersichtlicher, als ne Hand voll anonymer Klassen. Und dabei wesentlich flexibler.

Könnte so in etwa aussehen:

JButton button = new JButton(new ExitAction("Beenden", exitIcon));

oder

JMenuItem exitItem = new JMenuItem(new ExitAction("Beenden", exitIcon));

oder...

Und schließlich die Action:

public class ExitAction extends AbstractAction {
   public ExitAction(String title, Icon icon) {
      super(title, icon);
   }

   public void actionPerformed(ActionEvent e) {
      //do something
   }
}

 

[waeve]

OK super Danke
Nun noch die Frage wie ich die einzelnen Buttons ansprechen kann um ihnen ihre "ToDo-Liste" zuzuweisen, denn sie gehen ja alle in diese ArrayList.
Und eine Verständisfrage hab ich noch.

(ActionEvent ae)

Wofür steht das ae?

 

[Solor]

Naja, das "ae" steht vielleicht für ActionEvent ????

 

[Timothy Truckle]

OK super Danke
Nun noch die Frage wie ich die einzelnen Buttons ansprechen kann um ihnen ihre "ToDo-Liste" zuzuweisen, denn sie gehen ja alle in diese ArrayList.

Das hat L-ectron-X doch gezeigt. Mach eine eigene Spezialisierung von AbstactAction und übergib die dem Konstruktor des Buttons.

Und eine Verständisfrage hab ich noch.

(ActionEvent ae)

Wofür steht das ae?

Wie bei allen Methoden-Parametern ist dass der Name. OK, der ist nicht gut aber üblich...

bye
TT

 

[waeve]

Ich kann allerdings keinen beliebigen Button erzeugen sonder muss auf die implementierten Grafiken zugreifen, da diese die Buttons sind. Sie zeigen die Buchstaben A-Z. Wenn ich das Programm starte sehe ich diese. Nun möchte ich sie anklicken können. Dazu muss ich doch jedem eine Funktion zuweisen damit der dahinter verborgene Buchstabe erscheint.
Dazu muss ich jeder Zeile in meinem Code eine Zuweisung geben, damit sie weiß was sie tun soll. D. h. ich möchte jede Zeile dazu bringen, dass sie per Mausklick den Buchstaben ausgibt. Nun werden aber alle Grafiken mit folgender Zeile implementiert:

buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/H.png")));

Wie kann nun jeder dieser Zeile gesagt werden was sie tun soll?

Ich verstehe was Timothy Truckle machen will aber ich weis nicht, wie ich das für jeden Button machen kann. Weil wenn ich ButtonA eingebe wird mir Undefind wiedergegeben da es vorher nicht definiert wird.

Ich hoffe das mir auch zu dieser späten Stunde jemand antwortet

 

[Timothy Truckle]

Ich kann allerdings keinen beliebigen Button erzeugen sonder muss auf die implementierten Grafiken zugreifen,

Das verstehe ich nicht, Du kanst Doch der AbstactAction ein beliebiges Icon übergeben:
AbstractAction (Java Platform SE 7 )

da diese die Buttons sind. Sie zeigen die Buchstaben A-Z.Wenn ich das Programm starte sehe ich diese.

Nein, es sind nur Labels.

Nun möchte ich sie anklicken können.

Dazu must Du statt der Labels Buttonm nehemen, dass sgten wir ja bereits.

Dazu muss ich doch jedem eine Funktion zuweisen damit der dahinter verborgene Buchstabe erscheint.
Dazu muss ich jeder Zeile in meinem Code eine Zuweisung geben, damit sie weiß was sie tun soll. D. h. ich möchte jede Zeile dazu bringen, dass sie per Mausklick den Buchstaben ausgibt. Nun werden aber alle Grafiken mit folgender Zeile implementiert:

buchstabenArray.add(new JLabel(new ImageIcon("img/Alphabet/H.png")));

Wie kann nun jeder dieser Zeile gesagt werden was sie tun soll?

buchstabenArray.add(new JLabel(new JButton(new AbstractAction("A", exitIcon)){
 @Override public void actionPerformed(ActionEvent e){
    // "A" zum Text hinzu
 }
});

[EDIT]Aus der anonymen AbstackAction-Klasse machst du nann noch eine innere, die das Icon und den String (hier "A") als Konstruktorparameter bekommt...[/EDIT]

bye
TT

 

[waeve]

Was haltet ihr denn von dieser Variante?
Das ist mir heute eingefallen.
Ist sicher nicht die schönste aber wenn man das für jeden Buchstaben macht funktioniert das auch.

    JLabel lblB = new JLabel("");
        lblB.addMouseListener(new MouseAdapter() {
                public void mouseClicked(MouseEvent e) {
                    System.out.println("mouseClicked lblB");
                }
            });
        lblB.setIcon(new ImageIcon(HangingBird.class.getResource("B.png")));
        lblB.setHorizontalAlignment(SwingConstants.CENTER);
        lblB.setBounds(86, 150, 66, 57);
        contentPane.add(lblB);

Bei dem anderen wusste ich nicht, wo ich ansetzen sollte um das zu verändern.

LG

 

[Timothy Truckle]

Was haltet ihr denn von dieser Variante?
Das ist mir heute eingefallen.

Zumindest ein Schritt in die richtige Richtung.

Wenn Du aber weiterhin die zur Verfügung gestellten Codeschnipsel ignorierst kann ich Dir auch nicht mehr helfen...

bye
TT