Swing Daten von JPanel zu JFrame senden

[McDonnough]

Ich bin momentan wieder an einem kleinen Chat am Arbeiten und brauche (wieder einmal) eure Hilfe.

Momentan besteht meine GUI aus drei Komponenten:

• Loginpanel und
• Chatpanel
• Chatframe

Die ersten zwei sind Ableitungen von JPanel und werden schlussendlich im Chatframe (JFrame) übereinander angezeigt. Wie das ganze dann aussieht, könnt ihr im Anhang sehen.
Zu erwähnen ist noch, dass der Chatframe ein Objekt (ChatClient) besitzt, in welchem Host, Port und Timeout gespeichert werden und über welches die Methoden connect(), reconnect(), disconnect() und sendMessage() aufgerufen werden können.

Jetzt möchte ich dem User die Möglichkeit geben, dass er einfach einen neuen Host eintippen und 'connect' drücken kann und er wird automatisch ausgeloggt und neu eingeloggt.
Kein Problem, wenn ich es in der Loginpanel-Klasse mache, aber eigentlich will ich diese Arbeit im Chatframe erledigen und das Loginpanel soll nur eine Nachricht mit den neuen Daten an den Frame schicken, wenn auf 'connect' gedrückt wird.

Ich habe gedacht, ich könne das irgendwie mit einem Listener lösen, aber ich stehe momentan auf dem Schlauch und habe keine Ahnung, wie ich das machen soll.

Wäre schon, wenn mir jemand von euch erklären könnte, wie man das am besten löst.

 

[vanny]

also entweder referenzierst du die benötigten Komponenten untereinander und per getter/setter/eigenen -Methoden greifst du dann zu.

oder tfa hat da mal was vorbereitet:

Snippet Eventbus

oder oder oder ....

ohne Code ists schwer, da was zu zu sagen

 

[McDonnough]

Vielen Dank für den Link! Das sieht ziemlich genau nach dem aus, was ich gesucht habe.

Wenn noch von Interesse, findet ihr den Sourcecode hier: crumbchat - Revision 32: /trunk/src/org/crumbleworks/crumbchat/gui/test

 

[McDonnough]

Habe soeben auf stackoverflow diesen Eintrag gefunden und muss dem Autor irgendwie recht geben. Vor allem dieser Teil ist für mich sehr überzeugend:

"It's just the popular Observer (Listener) pattern made 'globally' [...] it breaks the Encapsulation principle (each object knows about everything) and Single Responsibility principle [...]"

Ich würde aber gerne auch noch Meinungen von euch zu diesem Thema (pro oder contra EventBus) hören.

 

[vanny]

jain.

nur weil die events im Bus veröffentlicht werden springen ja nicht gleich alle Lauscher auf jedes event an^^.

Is auch nur eine Idee gewesen, dein Problem mit einem kleinen link zu lösen.

Gruß Vanny

 

[McDonnough]

Ich werde heute Abend das Ganze mal mit dem EventBus umsetzen, da ich bisher keine wirklich gute Alternative gefunden habe, die ähnlich wenig Aufwand erfordert.
Bin aber selbstverständlich weiterhin offen für andere Ansätze und/oder Lösungsvorschläge.

 

[Michael...]

da ich bisher keine wirklich gute Alternative gefunden habe, die ähnlich wenig Aufwand erfordert.

Standard OOP? Ganz normal mit Listenern? Bin mir nicht sicher ob hier eine so lose Kopplung notwendig oder gar sinnvoll ist. Zumal hier doch ein recht enger Zusammenhang notwendig scheint.

Aufgrund der Fragestellung vermute ich hier ein grundsätzliches Verständnisproblem, das sich m.M. auch mit dem EventBus nicht vollständig lösen lässt.
Das Konzept würde ich nochmal überdenken. Interessant und für solche Vorhaben empfehlenswert: MVC Entwurfsmuster

 

[McDonnough]

Ich habe mal ein Klassendiagramm erstellt, welches zeigt, wie ich den Chat (Clientseitig) bisher aufgebaut habe.

Meine Idee ist jetzt, dass wenn der User im 'LoginPanel' auf den 'connectButton' drückt, der Wert (i.e. Serveradresse) aus dem 'hostField' gelesen wird und dem 'ChatFrame' bekanntgemacht wird.
Dieser ändert den Wert 'host' in seiner Referenz auf 'ChatClient', führt 'disconnect()' und danach wieder 'connect()' aus, um auf den neuen Server zu verbinden.

Natürlich könnte ich die Referenz auf 'ChatClient' dem 'LoginPanel' nach unten reichen und das ganze so machen, aber eigentlich möchte ich das Ganze nicht so eng aneinanderknüpfen und auch keine grosse Logik im 'LoginPanel' einbauen.

Gleiches gilt für den 'ChatPanel'. Wenn der User auf den 'disconnectButton' drückt, soll dies dem 'ChatFrame' bekanntgemacht werden und dieser soll über 'client' die 'disconnect()'-Methode aufrufen.

Und ich habe im Moment wirklich keine Ahnung, wie ich das gescheit umsetzen kann/soll (ausser EventBus). Listener habe ich mir auch schon überlegt, habe ich aber (leider) noch nie selber gemacht und habe dementsprechend keine blassen Dunst, wie ich das genau einbauen muss.

Anmerkung: Das Attribut 'client' in der Klasse 'ChatClient' rührt daher, dass ich die Kryonet-Library verwende.

 

[McDonnough]

So, ich habe das ganze mal umgesetzt, wobei ich vor allem im ChatClient ein Änderungen gemacht habe.

ChatClient:

package org.crumbleworks.crumbchat.client;

import java.io.IOException;

import com.esotericsoftware.kryonet.Client;

public class ChatClient {
    private int tcpPort, udpPort, timeout;
    private String host;
    private Client client;
    
    private final int DEFAULT_TCPPORT = 44444;
    private final int DEFAULT_UDPPORT = 44444;
    private final int DEFAULT_TIMEOUT = 5000;
    
    public ChatClient() {
        this.tcpPort = DEFAULT_TCPPORT;
        this.udpPort = DEFAULT_UDPPORT;
        this.timeout = DEFAULT_TIMEOUT;
        
        init();
    }
    
    public ChatClient(String host) {
        this.host = host;
        this.tcpPort = DEFAULT_TCPPORT;
        this.udpPort = DEFAULT_UDPPORT;
        this.timeout = DEFAULT_TIMEOUT;
        
        init();
    }
    
    public ChatClient(String host, int tcpPort) {
        this.host = host;
        this.tcpPort = tcpPort;
        this.udpPort = DEFAULT_UDPPORT;
        this.timeout = DEFAULT_TIMEOUT;
        
        init();
    }
    
    public ChatClient(String host, int tcpPort, int udpPort) {
        this.host = host;
        this.tcpPort = tcpPort;
        this.udpPort = udpPort;
        this.timeout = DEFAULT_TIMEOUT;
        
        init();
    }
    
    public ChatClient(String host, int tcpPort, int udpPort, int timeout) {
        this.host = host;
        this.tcpPort = tcpPort;
        this.udpPort = udpPort;
        this.timeout = timeout;
        
        init();
    }
    
    private void init() {
        client = new Client();
        client.start();
    }
    
    public void connect() throws IOException {
        client.connect(timeout, host, tcpPort, udpPort);
    }
    
    public void connect(String host, int port) throws IOException {
        this.host = host;
        this.tcpPort = port;
        this.udpPort = port;
        
        connect();
    }
    
    public void reconnect() throws IOException {
        client.reconnect();
    }
    
    public void disconnect() {
        client.close();
    }
    
    public void sendMessage(String message) {
        client.sendTCP(message);
        client.sendUDP(message);
    }
    
    public String getHost() {
        return host;
    }
    
    public void setHost(String host) {
        this.host = host;
    }
    
    public int getTcpPort() {
        return tcpPort;
    }
    
    public void setTcpPort(int tcpPort) {
        this.tcpPort = tcpPort;
    }
    
    public int getUdpPort() {
        return udpPort;
    }
    
    public void setUdpPort(int udpPort) {
        this.udpPort = udpPort;
    }
}

ChatFrame:

package org.crumbleworks.crumbchat.gui.test;

import java.awt.Dimension;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.Insets;
import java.awt.Toolkit;

import javax.swing.JFrame;
import javax.swing.JSeparator;
import javax.swing.UIManager;

import org.crumbleworks.crumbchat.client.ChatClient;
import org.crumbleworks.crumbchat.gui.GeneralWindowListener;

public class ChatFrame extends JFrame {
    private static final long serialVersionUID = -5850291837870435892L;
    
    private ChatClient client;
    
    private ChatPanel chatPanel;
    private LoginPanel loginPanel;
    
    private JSeparator horizontalSeparator;

    public ChatFrame() {
        client = new ChatClient();
        
        setLookAndFeel();
        
        setTitle("crumbchat");
        setLayout(new GridBagLayout());
        addWindowListener(new GeneralWindowListener());
        
        initComponents();
        placeComponents();
        
        setPosition();
        pack();
        setVisible(true);
    }
    
    private void setLookAndFeel() {
        try {
            UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
        }
        catch(Exception e) {
            e.printStackTrace();
        }
    }
    
    private void setPosition() {
        Dimension screen = Toolkit.getDefaultToolkit().getScreenSize();
        this.setLocation(((int) screen.getWidth() / 2) - this.getWidth(), ((int) screen.getHeight() / 2) - this.getHeight());
    }
    
    private void initComponents() {
        chatPanel = new ChatPanel();
        loginPanel = new LoginPanel();
        
        horizontalSeparator = new JSeparator(JSeparator.HORIZONTAL);
    }

    private void placeComponents() {
        GridBagConstraints gbc = new GridBagConstraints();
        gbc.fill = GridBagConstraints.BOTH;
        gbc.insets = new Insets(5, 5, 5, 5);
        
        //LoginPanel
        gbc.gridx = 0;
        gbc.gridy = 0;
        gbc.gridwidth = 1;
        gbc.gridheight = 1;
        add(loginPanel, gbc);
        
        //Separator
        gbc.gridx = 0;
        gbc.gridy = 1;
        gbc.gridwidth = 1;
        gbc.gridheight = 1;
        add(horizontalSeparator, gbc);
        
        //ChatPanel
        gbc.gridx = 0;
        gbc.gridy = 2;
        gbc.gridwidth = 1;
        gbc.gridheight = 1;
        add(chatPanel, gbc);
    }
    
    /**
     * @param args
     */
    public static void main(String[] args) {
        new ChatFrame();
    }
}

ChatPanel:

package org.crumbleworks.crumbchat.gui.test;

import java.awt.Color;
import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.Insets;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JPanel;
import javax.swing.JTextArea;
import javax.swing.JTextField;

public class ChatPanel extends JPanel implements ActionListener {
    private static final long serialVersionUID = -2073883748397706101L;

    private JTextArea messageArea;
    private JTextField messageField;
    private JButton sendButton, disconnectButton;
    
    public ChatPanel() {
        setLayout(new GridBagLayout());
        
        initComponents();
        placeComponents();
        
        setVisible(true);
    }
    
    private void initComponents() {
        messageArea = new JTextArea(15, 0);
        messageArea.setEditable(false);
        messageArea.setBorder(BorderFactory.createLineBorder(Color.LIGHT_GRAY));
        
        messageField = new JTextField(20);
        messageField.addActionListener(this);
        
        sendButton = new JButton("send");
        sendButton.addActionListener(this);
        
        disconnectButton = new JButton("disconnect");
        disconnectButton.addActionListener(this);
    }

    private void placeComponents() {
        GridBagConstraints gbc = new GridBagConstraints();
        gbc.fill = GridBagConstraints.BOTH;
        gbc.insets = new Insets(5, 5, 5, 5);
        
        //MessageArea
        gbc.gridx = 0;
        gbc.gridy = 0;
        gbc.gridwidth = 3;
        gbc.gridheight = 1;
        add(messageArea, gbc);
        
        //MessageField
        gbc.gridx = 0;
        gbc.gridy = 1;
        gbc.gridwidth = 1;
        gbc.gridheight = 1;
        add(messageField, gbc);
        
        //SendButton
        gbc.gridx = 1;
        gbc.gridy = 1;
        gbc.gridwidth = 1;
        gbc.gridheight = 1;
        add(sendButton, gbc);
        
        //LogoutButton
        gbc.gridx = 2;
        gbc.gridy = 1;
        gbc.gridwidth = 1;
        gbc.gridheight = 1;
        add(disconnectButton, gbc);
    }
    
    @Override
    public void actionPerformed(ActionEvent e) {
        if(e.getSource().equals(sendButton) && !messageField.getText().isEmpty() || e.getSource().equals(messageField) && !messageField.getText().isEmpty()) {
            //Nachricht senden
            System.out.println("message: " + messageField.getText());
            messageField.setText("");
        }
        else if(e.getSource().equals(disconnectButton)) {
            //Disconnect durchfuehren
            System.out.println("disconnect");
        }
    }
}

LoginPanel:

package org.crumbleworks.crumbchat.gui.test;

import java.awt.GridBagConstraints;
import java.awt.GridBagLayout;
import java.awt.Insets;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.JButton;
import javax.swing.JLabel;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.JTextField;

public class LoginPanel extends JPanel implements ActionListener {
    private static final long serialVersionUID = -1137766211210278587L;
    
    private JLabel usernameFieldLabel, hostFieldLabel;
    private JTextField usernameField, hostField;
    private JButton connectButton;
    
    public LoginPanel() {
        setLayout(new GridBagLayout());
        
        initComponents();
        placeComponents();
        
        setVisible(true);
    }
    
    private void initComponents() {
        usernameFieldLabel = new JLabel("Username:");
        hostFieldLabel = new JLabel("Host:");
        
        usernameField = new JTextField(20);
        hostField = new JTextField(10);
        
        connectButton = new JButton("connect");
        connectButton.addActionListener(this);
    }

    private void placeComponents() {
        GridBagConstraints gbc = new GridBagConstraints();
        gbc.fill = GridBagConstraints.BOTH;
        gbc.insets = new Insets(5, 5, 5, 5);
        
        //Username
        gbc.gridx = 0;
        gbc.gridy = 2;
        gbc.gridwidth = 1;
        gbc.gridheight = 1;
        add(usernameFieldLabel, gbc);
        
        gbc.gridx = 1;
        gbc.gridy = 2;
        gbc.gridwidth = 1;
        gbc.gridheight = 1;
        add(usernameField, gbc);

        //Host
        gbc.gridx = 0;
        gbc.gridy = 3;
        gbc.gridwidth = 1;
        gbc.gridheight = 1;
        add(hostFieldLabel, gbc);
        
        gbc.gridx = 1;
        gbc.gridy = 3;
        gbc.gridwidth = 1;
        gbc.gridheight = 1;
        add(hostField, gbc);
        
        //LoginButton
        gbc.gridx = 0;
        gbc.gridy = 4;
        gbc.gridwidth = 1;
        gbc.gridheight = 1;
        add(connectButton, gbc);
    }
    
    @Override
    public void actionPerformed(ActionEvent e) {
        if(e.getSource().equals(connectButton)) {
            if(usernameField.getText().isEmpty()) {
                JOptionPane.showMessageDialog(this, "Bitte geben sie einen Usernamen an.", "Usernamen angeben", JOptionPane.INFORMATION_MESSAGE);
            }
            else if(hostField.getText().isEmpty()) {
                JOptionPane.showMessageDialog(this, "Bitte geben sie einen Host an.", "Host angeben", JOptionPane.INFORMATION_MESSAGE);
            }
            else {
                //Connect durchfuehren
                System.out.println("connect");
            }
        }
    }
}

Dort, wo eine Aktion ausgeführt werden muss, die über den ChatClient läuft, habe ich es hingeschrieben (Nachricht senden, Disconnect, Connect).

Ich hoffe, es kann mir jemand auf Basis dieses Codes erklären, wie man mein Vorhaben am einfachsten umsetzen kann.

 

[McDonnough]

Ich habe zwischenzeitlich auch mal EventBus eingebaut und muss sagen, dass das ganze ziemlich einfach ist und wunderbar funktioniert.

Im LoginPanel sieht's jetzt so aus:

else {
    //Connect durchfuehren
    System.out.println("connect");
    EventBus.publish(new ConnectEvent(hostField.getText()));
}

ConnectEvent:

package org.crumbleworks.crumbchat.client.gui.event;

public class ConnectEvent {
	private String host;

	public ConnectEvent(String host) {
		this.host = host;
	}
	
	public String getHost() {
		return host;
	}
}

Und die Methode im ChatFrame:

@EventSubscriber(eventClass = ConnectEvent.class)
public void onConnect(ConnectEvent ce) {
    client.setHost(ce.getHost());

    try {
        client.connect();
    } catch (IOException e) {
        System.err.println("couldn't connect to server");
    }
}

Selbiges habe ich auch für ein MessageEvent und ein DisconnectEvent gemacht und finde es eigentlich eine schöne Lösung für das Ganze.

 

[GUI-Programmer]

Ich beziehe mich jetzt mal auf den Titel. In deinem Fall, glaube ich benötigst du gar kein EventBus. Es würde völlig reichen, wenn due eine Instanz von ChatFrame an ChatPanel per Konstruktor weiter gibts (evtl. auch weiter an LoginPanel). Dann kann dein ChatPanel die Methoden deines Frames aufrufen.

 

[McDonnough]

Das ist natürlich auch eine Möglichkeit.
Ich wollte das Ganze (auch als Übung für mich) aber eigentlich nicht so aneinanderbinden, sprich das LoginPanel soll nichts vom ChatFrame wissen, sondern einfach Nachrichten (bzw. Events) "veröffentlichen" und der ChatFrame spring dann darauf an.

 

[GUI-Programmer]

Das ist natürlich auch eine Möglichkeit.
Ich wollte das Ganze (auch als Übung für mich) aber eigentlich nicht so aneinanderbinden, sprich das LoginPanel soll nichts vom ChatFrame wissen, sondern einfach Nachrichten (bzw. Events) "veröffentlichen" und der ChatFrame spring dann darauf an.

OK, dann bietet sich EventBus natürlich schon an.

Thema erledigt???

 

[McDonnough]

Ja, ist erledigt.

Danke für eure Hilfe!