GUI Update /EDT

[Turing0001]

Hallo Leute,

ich stehe gerade auf dem Schlauch bei einem Problem mit dem Update der Oberfläche eines
kleinen Servers (Tipp des Tages). Dabei versuche ich in einer Schleife ein JTextField und eine
JTextArea zu aktualisieren (siehe Code). Obwohl das natürlich keine sehr gute Idee ist, würde
ich das Ganze gerne so zum Laufen bringen. Natürlich gibt es dabei das Problem der Blockade
des EDT. Ich habe bereits versucht, das Problem zu beheben mittels eines Timers bzw eines
separaten Threads (siehe Kommentare), aber nichts funktioniert. Es sollen in einem JTextField
die IP-Adresse des Servers und in der JTextArea Ip-Adresse und Portnummer aller verbundenen
Clients eingetragen werden. Da ich schon einige Jahre aus der Java-Programmierung raus bin,
wäre ich für jede Hilfe dankbar.
Hier der Code des Servers, der über eine GUI aufgerufen wird und Referenzen zu
jTextField und JTextArea mitbekommt. Ich hoffe, es ist okay wenn ich den Code hier einfach anhänge.
Da ich noch nie in einem Forum gepostet habe, besitze ich keinerlei Erfahrung. Sollte der modus
operandi ein anderer sein, bite ich um entsprechende Hinweise:
[CODE lang="java" title="Einfacher Server"]package mypackage;


import java.awt.Font;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.awt.event.*;
import java.io.*;
import java.net.*;


import javax.swing.JButton;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JOptionPane;
import javax.swing.JPanel;
import javax.swing.JTextArea;
import javax.swing.JTextField;
import javax.swing.SwingUtilities;
import javax.swing.border.EmptyBorder;
import javax.swing.Timer;

public class server {



InetAddress ip;
String hostname;
int port;
Socket sock;

String[] tippListe={"Weniger Süßigkeiten essen!","Kaufen Sie sich was schönes!","Sagen Sie heute die Wahrheit","Mehr Programmieren Üben!","Knuddeln Sie Ihren Partner!"};





public void los(int port,JTextField tfip,JTextArea tacl)
{

try
{
ServerSocket servsock = new ServerSocket(port); // Verbindungsschnittstelle auf Port 4243 erstellen


while(true)
{

System.out.println("Hallo hier ist der Server auf Port " + port);
sock = servsock.accept(); // Warten auf "Anruf" von einem Client. accept() blockiert bis ein Client "anruft"
PrintWriter writer = new PrintWriter(sock.getOutputStream()); // Ausgabedatenleitung zum Clientsocket aufbauen...
String tipp = getTipp();
writer.println(tipp); // ... und die Daten an den Client senden

ip = InetAddress.getLocalHost(); // IP Adresse des Servers ermitteln
hostname = ip.getHostName();
// timer = new javax.swing.Timer(5,new ActionListener(){
// public void actionPerformed(ActionEvent ae){
// tfip.setText(ip.toString());
// tacl.append(" Clientadresse: " + sock.getInetAddress().toString() + " Portnummer: "+ sock.getPort() + System.lineSeparator());
// }
// });
// if (!timer.isRunning())
// {
// timer.start();
// System.out.println("Timer gestartet");
// }
//----------------------------------------------------------------
// new Thread(new Runnable()
// {
// public void run()
// {
//
// SwingUtilities.invokeLater(new Runnable()
// {
//
// public void run()
// {
// tfip.setText(ip.toString());
// tacl.append(" Clientadresse: " + sock.getInetAddress().toString() + " Portnummer: "+ sock.getPort() + System.lineSeparator());
// System.out.println("Komponenten aktualisiert!");
// }
// });
//
// }
// }).start();
tfip.setText(ip.toString());
tacl.append(" Clientadresse: " + sock.getInetAddress().toString() + " Portnummer: "+ sock.getPort() + System.lineSeparator());


System.out.println( "Text aus Textfeld und Textarea: " + tfip.getText() + " " + tacl.getText());

//System.out.println(ip + "\t" + hostname);
//System.out.println("Clientadresse: " + sock.getInetAddress() + " Portnummer: "+ sock.getPort());

writer.close(); // Ausgabedatenleitung wieder schließen
//servsock.close();
sock.close();
System.out.println(tipp);
}

}
catch (IOException ex){ex.printStackTrace();}
}

private String getTipp()
{
int zufall = (int)(Math.random()*tippListe.length);
return tippListe[zufall];
}




}

[/CODE]

 

[Robert Zenz]

Ich rate mal und sage das "los" auf dem Swing-Thread aufgerufen wird? Damit wirst du immer dne Thread blockieren, es nuetzt dann auch nicht so viel darin dann einen Thread fuer das Update aufzureissen, "los" muss bereits in einem eigenen Thread laufen. Also einen neuen Thread erzeugen, und darin "los" ausfuehren.

 

[Turing0001]

Ich rate mal und sage das "los" auf dem Swing-Thread aufgerufen wird? Damit wirst du immer dne Thread blockieren, es nuetzt dann auch nicht so viel darin dann einen Thread fuer das Update aufzureissen, "los" muss bereits in einem eigenen Thread laufen. Also einen neuen Thread erzeugen, und darin "los" ausfuehren.

Hallo Robert,

Du bist mein Held!! Na klar, das war ein prinzipieller Denkfehler. Einfach den Aufruf in einen Thread gepackt und schon läuft es wie es soll.
Vielen, vielen Dank, da habe ich gestern ewig rumgetan. Der Sonntag ist gerettet.

 

[Robert Zenz]

Vergiss nicht dass du dich auf den Swing Thread zurueck musst wenn du den Wert setzt. Das hast du ja bereits mit "SwingUtilities.invokeLater" gemacht. Das ist deswegen notwendig weil Aenderungen an Swing Controls nur vom Swing Thread aus passieren sollte, das betrifft auch neue Werte fuer die Anzeige. Eventuell ist hier dann "SwingUtilities.invokeAndWait" angebrachter, weil die Funktion erst zurueck kehrt wenn Swing den neuen Wert wirklich angezeigt hat (es bringt ja nichts 300 Werte zu holen wenn nur einer davon angezeigt wird).

 

[Turing0001]

Vergiss nicht dass du dich auf den Swing Thread zurueck musst wenn du den Wert setzt. Das hast du ja bereits mit "SwingUtilities.invokeLater" gemacht. Das ist deswegen notwendig weil Aenderungen an Swing Controls nur vom Swing Thread aus passieren sollte, das betrifft auch neue Werte fuer die Anzeige. Eventuell ist hier dann "SwingUtilities.invokeAndWait" angebrachter, weil die Funktion erst zurueck kehrt wenn Swing den neuen Wert wirklich angezeigt hat (es bringt ja nichts 300 Werte zu holen wenn nur einer davon angezeigt wird).

Hallo Robert,
das habe ich berücksichtigt, obwohl es auch ohne invokeLater/invokeAndWait funktioniert, aber so ist es sauberer. Vielen Dank nochmals, Du hast mir wiklich sehr geholfen.

 

[mihe7]

obwohl es auch ohne invokeLater/invokeAndWait funktioniert

Analog: ich schaue nach links und rechts, bevor ich über die Straße gehe, obwohl es auch ohne funktioniert

 

[Turing0001]

Analog: ich schaue nach links und rechts, bevor ich über die Straße gehe, obwohl es auch ohne funktioniert

Hallo mihe7,

vielen Dank für den netten Hinweis.

 

[mihe7]

Der war eigentlich nur zur Ergänzung für diejenigen gedacht, die später mal über den Thread stolpern. Du verwendest ja invokeLater. Hintergrund ist einfach der, dass die Swing-Komponenten nicht thread-safe verwendet werden können (ein paar Methoden bilden die Ausnahme, wie z. B. repaint), d. h. auch wenn es scheinbar funktioniert, ist es falsch, die Komponenten außerhalb des EDT zu manipulieren. Das kann in 99,99 % der Zeit funktionieren und plötzlich taucht ein scheinbar unerklärbares Phänomen an möglicherweise ganz anderer Stelle auf. Das sind m. E. die übelsten Probleme, die man überhaupt in Software haben kann: selten, nicht reproduzierbar, nicht lokalisierbar.