Swing Button in anderer Klassen ausführen

[Øtzi]

Hallo Zusammen,

schon eine Weile sitze ich an diesem Problem und habe unterschiedliche Ansätze bei google und hier im Forum gefunden, die zur Lösung beitragen könnten. Aber es war noch nichts passendes dabei.

Folgendes Problem:
Ich möchte meine Progammlogik und die GUI mit Hilfe eines Controllers sauber voneinander trennen. Dabei soll die Logik und die GUI nichts von diesem wissen (sie dürfen keine Methoden vom Controller aufrufen).
Wenn ich in der GUI eine Button habe, möchte ich im Controller Dinge ausführen, wenn dieser angeklickt wird.

Hier mein Programm, welches so nicht funktioniert:

package testbutton;

public class TestButton {

    public static void main(String[] args) {
        final Control control = new Control();
        control.start();
    }
   
}

Das ist meine Main, wo nur die Control-Klasse initialisiert und gestartet wird.

package testbutton;

import java.awt.event.*;

public class Control {
    final View view = new View();
    final Logic logic = new Logic();
   
    public void start() {
        view.view();
    }
   
    private void actionButton() {
        view.getButton().addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                view.show(logic.getText());
            }
        });
    }
   
}

In der Control-Klasse werden View und Logic erstellt und die View wird gestartet.

package testbutton;

import javax.swing.*;

public class View extends JFrame {
    Panel panel = new Panel();
       
    public void view() {
        add(panel);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        pack();
        setTitle("Mensch Ärgere Dich Nicht");
        setLocationRelativeTo(null);
        setVisible(true);
    }
   
    public JButton getButton() {
        return panel.getButton();
    }
   
    public void show(String text) {
        System.out.println(text);
    }
   
}

Die Klasse-View ist quasi mein Hauptfenster. In der meine Klasse-Panel aufgerufen und dargestellt wird.

package testbutton;

import javax.swing.*;

public class Panel extends JPanel {
    public JButton button = new JButton("Gib Text");
   
    public Panel() {
        add(button);
    }
   
    public JButton getButton() {
        return button;
    }
   
}

In der Klasse Panel wird der Button erstellt.

Zum Problem zurück und zu meiner Hauptfrage:
Wie ich bereits oben geschrieben habe, funktioniert dieses Beispiel so nicht. Das Wort "Hallo" wird nicht angezeigt. Aber wenn ich die Klasse Panel weg lasse und den Button direkt in die Klasse View packe und diesen von private auf public umstelle, kann ich in der Klasse Control wie folgt auf ihn zugreifen:

view.button.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                view.show(logic.getText());
            }
        });

Die Frage ist jetzt: Wieso?

Ich hoffe ihr könnt mir helfen. Vielen Dank schon einmal im Voraus.

 

[Robat]

Wo rufst du die Methode actionButton denn auf, um den Listener hinzuzufügen?

 

[Øtzi]

Danke für den Hinweis. Die Methode wird in der Klasse Control aufgerufen:

public void start() {
    view.view();
    actionButton();
}

Jetzt bin ich allerdings noch verwirrter, denn nun funktioniert zwar mein kleines Beispielprogramm, aber nicht mein eigentliches Programm, wo ich genau das Gleiche tu. Verdammt, ich kann doch hier nicht alles hochladen.

 

[Robat]

Naja da lässt sich jetzt nicht so viel sagen. Entweder machst du ne abgespeckte Version deines Programms und lädst es hoch oder du schaust einfach noch mal alleine, Schritt für Schritt durch.

 

[Øtzi]

Ok, dann hier die 3 betroffenen Klassen aus meinem Hauptprogramm:
(Wie beschrieben funktioniert der Button im Panel "PanelDice" nicht, aber der im mainWindow "bt".)

package DoNotGetAngry;

import DoNotGetAngry.View.*;
import DoNotGetAngry.GameLogic.Game;
import java.awt.event.*;

public class Control {
    final Game game;
    final MainWindow mainWindow;
   
    public Control() {
        game = new Game();
        mainWindow = new MainWindow();
        clickButton();
    }
   
    public void start() {
        StartDialog startDialog = new StartDialog(Control.this);
        startDialog.view();
        mainWindow.setActivePlayer(
                game.getActivePlayer(), game.getNameActivePlayer());
        mainWindow.view();
    }
   
    public void createPlayer(String[] name, boolean[] human) {
        game.createPlayer(name, human);
    }
   
    private void clickButton() {
       
        // click Dice Button
        mainWindow.getButtonDice().addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                System.out.println("Funzt");
            }
        });
       
        // bt
        mainWindow.bt.addActionListener(new ActionListener() {
            @Override
            public void actionPerformed(ActionEvent e) {
                System.out.println("Funzt auch");
            }
        });
       
    }
   
}

package DoNotGetAngry.View;

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

public class MainWindow extends JFrame {
    final JPanel panelMain = new JPanel();
    final PanelBoard panelBoard = new PanelBoard();
    final PanelDice panelDice = new PanelDice();
    final PanelActivePlayer panelActivePlayer = new PanelActivePlayer();
    final Insets set = new Insets(5, 5, 5, 5);
    final GridBagConstraints grid = new GridBagConstraints();
   
    public JButton bt = new JButton("toll");
   
    public MainWindow() {
        createLayout();
    }
   
    public void view() {
        add(panelMain);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        pack();
        setTitle("Mensch Ärgere Dich Nicht");
        setLocationRelativeTo(null);
        setVisible(true);
    }
   
    // modify ------------------------------------------------------------------
   
    // set ---------------------------------------------------------------------
   
    public void setActivePlayer(int activePlayer, String name) {
        panelActivePlayer.setActivePlayer(activePlayer, name);
    }
   
    // get ---------------------------------------------------------------------
   
    public JButton getButtonDice() {
        return panelDice.getButtonDice();
    }
   
    // private -----------------------------------------------------------------
   
    private void createLayout() {
        panelMain.setLayout(new GridBagLayout());
        panelMain.setBorder(BorderFactory.createEmptyBorder(5, 5, 5, 5));
        grid.anchor = GridBagConstraints.CENTER;
        grid.insets = set;
        grid.gridx = 0;
        grid.gridy = 0;
        panelMain.add(new PanelBoard(), grid);
        grid.gridx = 1;
        panelMain.add(new PanelDice(), grid);
        grid.gridy = 2;
        panelMain.add(panelActivePlayer, grid);
       
        // temp
        grid.gridy = 3;
        panelMain.add(bt, grid);
    }
   
}

package DoNotGetAngry.View;

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

public class PanelDice extends JPanel {
    final JButton buttonDice = new JButton("Würfeln");
    final JLabel[] labelResultDice = new JLabel[2];
    final JLabel[] labelTitleDice = new JLabel[2];
   
    final Insets set = new Insets(5, 5, 5, 5);;
    final GridBagConstraints grid = new GridBagConstraints();
   
    public PanelDice() {
        createLabel();
        createLayout();
    }
   
    // modify ------------------------------------------------------------------
   
    // set ---------------------------------------------------------------------
   
    public void setDice(int[] resultDice) {
        for(int i=0; i<2; i++) {
            labelResultDice[i].setText(String.valueOf(resultDice[i]));
        }
    }
   
    // get ---------------------------------------------------------------------
   
    public JButton getButtonDice() {
        return buttonDice;
    }
   
    // private -----------------------------------------------------------------
   
    private void createLabel() {
        for(int i=0; i<2; i++) {
            int number = i + 1;
            labelResultDice[i] = new JLabel();
            labelTitleDice[i] = new JLabel("Würfel " + number + ": ");
        }
    }
   
    private void createLayout() {
        setLayout(new GridBagLayout());
        grid.insets = set;
        for(int i=0; i<2; i++) {
            grid.anchor = GridBagConstraints.LINE_START;
            grid.gridx = 0;
            grid.gridy = i;
            add(labelTitleDice[i], grid);
            grid.gridx = 1;
            grid.anchor = GridBagConstraints.LINE_END;
            add(labelResultDice[i], grid);
        }
        grid.gridwidth = 2;
        grid.gridx = 0;
        grid.gridy = 2;
        buttonDice.setPreferredSize(new Dimension(100, 40));
        add(buttonDice, grid);
    }
   
}

 

[Robat]

panelMain.add(new PanelBoard(), grid);
grid.gridx = 1;
panelMain.add(new PanelDice(), grid);

Hier kein neues PanelBoard / PanelDice Objekt erstellen, sondern die Objekte nehmen, die du oben deklariert hast.

 

[Øtzi]

Hab tausend Dank Robat. Das ist der Fehler gewesen.