GUI außerhalb GUI-Thread updaten - GUI friert ein

qwert · 5. Okt 2014

Hallo 🙂,

Ich möchte meine GUI außerhalb des Main-Threads aktualisieren. Im Netz stieß ich dabei auf folgende Variante:

Java:

new Thread() {
  @Override
  public void run() {
    for (;; ) {
      try {
        Thread.sleep(5000);
      } catch (InterruptedException ix) {
        return;
      }
      SwingUtilities.invokeLater(() -> {
        if (fillListFlag) {
          fillListFlag = false;
          Test.fillList();
        }
      });
    }
  }
}.start();

Hierbei wird "fillListFlag" von außen gesetzt. "fillList()" entfernt zunächst alle GUI-Elemente und fügt wieder welche hinzu.

Nun kann ich genau während den 5 Sekunden Wartezeit die GUI-Elemente bedienen, danach friert alles ein.

Woran liegt das und wie kann ich es ändern?
Falls ihr noch mehr Code braucht, bitte mitteilen, denn das Projekt ist ziemlich umfangreich, ich muss also zwangsläufig kürzen.

Vielen Dank für eure Hilfe! 🙂

njans · 5. Okt 2014

Hast du denn mal debugged, ob deine fillList Methode überhaupt zuende läuft?

qwert · 5. Okt 2014

Ja, die läuft auf jeden Fall durch.

qwert · 6. Okt 2014

Hier mal die fillList-Methode:

Java:

private synchronized void fillList() {
    getContentPane().removeAll();
    setLayout(new GridLayout(5,1));
    Vector<String> chatParticipantNames = db.getChatParticipantNames();
    chatbuttons_enter = new JButton[chatParticipantNames.size()];
    chatbuttons_remove = new JButton[chatParticipantNames.size()];
    chatbuttons_add  = new JButton[chatParticipantNames.size()];
    for(int i = 0; i < chatParticipantNames.size(); i++) {
        //System.out.println("cPN = "+chatParticipantNames.get(i));
        chatbuttons_enter[i] = new JButton();
        chatbuttons_remove[i] = new JButton("Diesen Chat aus der Liste verbergen");
        chatbuttons_add[i] = new JButton("Teilnehmer aus Adressbuch hinzufügen");
        if(chatParticipantNames.get(i) != null) {
            chatbuttons_enter[i].setText(chatParticipantNames.get(i));
        }
        chatbuttons_enter[i].addActionListener(this);
        chatbuttons_remove[i].addActionListener(this);
        chatbuttons_add[i].addActionListener(this);
        JPanel outer = new JPanel(new GridLayout(3,1));
        JPanel p = new JPanel(new GridLayout(1,2));
        outer.add(chatbuttons_enter[i]);
        p.add(chatbuttons_remove[i]);
        p.add(chatbuttons_add[i]);
        outer.add(p);
        add(outer);            
    }
}

knilch · 6. Okt 2014

Hi,
fillList() wird nur ausgeführt wenn fillListFlag true ist. Im Thread setzt du fillListFlag auf false, dann wird fillList() ausgeführt.
Dann schreibst du, dass nach 5 sec nichts mehr passiert.
Du solltest mal analysieren, wann fillListFlag wieder auf true gesetzt, damit fillList() wieder ausgeführt werden kann.

qwert · 6. Okt 2014

Hi,

danke für deine Antwort. Aber fillListFlag wird von einem anderen Thread auf true gesetzt (und ist auch als volatile deklariert). Wenn ich eine Ausgabe in die Methode fillList() schreibe, sehe ich, dass diese auch immer wieder durchlaufen wird.

knilch · 6. Okt 2014

Hi,
Ok, dann wird die Methode also regelmässig ausgeführt.
Hast du es auch schon mit:

Java:

revalidate();
repaint();

probiert?

qwert · 6. Okt 2014

Hier ein kleines compilierbares Beispiel:

Java:

import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;
import java.util.Vector;

import javax.swing.JButton;
import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.SwingUtilities;

@SuppressWarnings("serial")
public class Main extends JFrame implements ActionListener {
    JButton chatbuttons_enter[], chatbuttons_add[], chatbuttons_remove[];
    ArrayList<Adresseintrag> eintraege = new ArrayList<Adresseintrag>();
    String[] mBook;
    int [] mBookIDs;
    JButton[] ok;
    JDialog dialog;
    private volatile boolean fillListFlag = true;
    public Main() {
        
        loadAddressbook();
        fillList();
        setSize(500,500);
        setVisible(true);

        new Thread(){
            public void run() {
                for(;; ){
                    try { 
                        sleep(2000);
                    } catch(InterruptedException ix){ 
                        return;
                    }
                    SwingUtilities.invokeLater(new Runnable() {
                        public void run() {
                            if(fillListFlag) {
                                fillListFlag = false;
                                fillList();
                            }
                        }
                    });
                }
            }
       }.start();
    }
    public void actionPerformed(ActionEvent e) {
        Object o = e.getSource();
        for(int i=0;i<chatbuttons_enter.length;i++) {
            if(chatbuttons_enter[i] == o) {
                
            }
        }
    }
    public void loadAddressbook() {
        eintraege.clear();
        eintraege.add(new Adresseintrag("3","host"));
        mBook = new String[eintraege.size()];
        mBookIDs = new int[eintraege.size()];
        chatbuttons_enter = new JButton[eintraege.size()];
        for(int i=0;i<eintraege.size();i++) {
            mBook[i] = eintraege.get(i).name+" ("+eintraege.get(i).nummer+")";
            try {
                mBookIDs[i] = Integer.parseInt(eintraege.get(i).nummer);
            } catch(NumberFormatException err) {
                System.err.println("Fehler im Adressbuch!");
                err.printStackTrace();
            }
        }        
    }
    private synchronized void fillList() {
//        System.out.println("db.getChatParticipantNames().size() = "+db.getChatParticipantNames().size());
        getContentPane().removeAll();
        setLayout(new GridLayout(5,1));
        Vector<String> chatParticipantNames = getChatParticipantNames();
        chatbuttons_remove = new JButton[chatParticipantNames.size()];
        chatbuttons_add  = new JButton[chatParticipantNames.size()];
        chatbuttons_enter = new JButton[chatParticipantNames.size()];
        for(int i = 0; i < chatParticipantNames.size(); i++) {
            //System.out.println("cPN = "+chatParticipantNames.get(i));
            chatbuttons_enter[i] = new JButton();
            chatbuttons_remove[i] = new JButton("Diesen Chat aus der Liste verbergen");
            chatbuttons_add[i] = new JButton("Teilnehmer aus Adressbuch hinzufügen");
            if(chatParticipantNames.get(i) != null) {
                chatbuttons_enter[i].setText(chatParticipantNames.get(i));
            }
            chatbuttons_enter[i].addActionListener(this);
            chatbuttons_remove[i].addActionListener(this);
            chatbuttons_add[i].addActionListener(this);

            JPanel outer = new JPanel(new GridLayout(3,1));
            JPanel p = new JPanel(new GridLayout(1,2));
            outer.add(chatbuttons_enter[i]);
            p.add(chatbuttons_remove[i]);
            p.add(chatbuttons_add[i]);
            outer.add(p);
            add(outer);            
        }
    }
    public Vector<String> getChatParticipantNames() {
        Vector<String> cpn = new Vector<String>();
        cpn.add("Hallo");
        cpn.add("test");
        return cpn;
    }
    private class Adresseintrag {
        public String nummer;
        public String name; //Intern verwendeter Name (Alias)
        public Adresseintrag(String id, String host) {
            this.nummer = id;
        }
    }
    public static void main(String[] args) {
        new Main();
    }
}

knilch · 6. Okt 2014

Hi,
Du deklarierst

Java:

private volatile boolean fillListFlag = true;

. private. Aber im Beispiel wird fillListFlag nur beim Start true sein. Dann immer false. Somit wird nur 1 mal updated. du sagst, dass fillListFlag von einem anderen Thread auf true gesetzt wird. Wenn fillListFlag private ist, dann geht das nur innerhalb der selben Klasse...

qwert · 6. Okt 2014

Ja, schon. Ich habe meine Klasse ja gekürzt, damit das Beispiel möglichst kurz ist.
Im Original gibt es noch diese Methode:

Code:

	@Override
	public void updateChatRooms() {
		fillListFlag = true;
	}

knilch · 6. Okt 2014

Hi,
Ich hab dein Beispiel mal genommen und etwas abgeändert.
Damit wird nun immer ein neuer user hinzugefügt und das Gui dann laufend updated. Vielleicht hilft dir das weiter:

Java:

import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.ArrayList;
import java.util.Vector;

import javax.swing.JButton;
import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.SwingUtilities;

public class Main extends JFrame implements ActionListener {
    private static final long serialVersionUID = -711820085243592196L;
    
    JButton chatbuttons_enter[], chatbuttons_add[], chatbuttons_remove[];
    ArrayList<Adresseintrag> eintraege = new ArrayList<Adresseintrag>();
    String[] mBook;
    int [] mBookIDs;
    JButton[] ok;
    JDialog dialog;
    private volatile static boolean fillListFlag = true;
    private static Vector<String> cpn = new Vector<String>();

    public Main() {
        loadAddressbook();
        fillList();
        setSize(500,500);
        setVisible(true);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        start();
    }
    
    public void actionPerformed(ActionEvent e) {
        Object o = e.getSource();
        for(int i=0;i<chatbuttons_enter.length;i++) {
            if(chatbuttons_enter[i] == o) {
                System.out.println(chatbuttons_enter[i].getText() +  ": bla");
            }
        }
    }
    
    private void start() {
        new Thread(){
            public void run() {
                for(;; ){
                    try {
                        sleep(2000);
                    } catch(InterruptedException ix){
                        return;
                    }
                    SwingUtilities.invokeLater(new Runnable() {
                        public void run() {
                            if(fillListFlag) {
                                System.out.println("update");
                                fillListFlag = false;
                                fillList();
                            }
                        }
                    });
                }
            }
        }.start();
    }
    
    
    public void loadAddressbook() {
        eintraege.clear();
        eintraege.add(new Adresseintrag("3","host"));
        mBook = new String[eintraege.size()];
        mBookIDs = new int[eintraege.size()];
        chatbuttons_enter = new JButton[eintraege.size()];
        for(int i=0;i<eintraege.size();i++) {
            mBook[i] = eintraege.get(i).name+" ("+eintraege.get(i).nummer+")";
            try {
                mBookIDs[i] = Integer.parseInt(eintraege.get(i).nummer);
            } catch(NumberFormatException err) {
                System.err.println("Fehler im Adressbuch!");
                err.printStackTrace();
            }
        }
    }
    
    private synchronized void fillList() {
        // System.out.println("db.getChatParticipantNames().size() = "+db.getChatParticipantNames().size());
        getContentPane().removeAll();
        setLayout(new GridLayout(5,1));
        Vector<String> chatParticipantNames = cpn;
        chatbuttons_remove = new JButton[chatParticipantNames.size()];
        chatbuttons_add = new JButton[chatParticipantNames.size()];
        chatbuttons_enter = new JButton[chatParticipantNames.size()];
        for(int i = 0; i < chatParticipantNames.size(); i++) {
            //System.out.println("cPN = "+chatParticipantNames.get(i));
            chatbuttons_enter[i] = new JButton();
            chatbuttons_remove[i] = new JButton("Diesen Chat aus der Liste verbergen");
            chatbuttons_add[i] = new JButton("Teilnehmer aus Adressbuch hinzufügen");
            if(chatParticipantNames.get(i) != null) {
                chatbuttons_enter[i].setText(chatParticipantNames.get(i));
            }
            chatbuttons_enter[i].addActionListener(this);
            chatbuttons_remove[i].addActionListener(this);
            chatbuttons_add[i].addActionListener(this);

            JPanel outer = new JPanel(new GridLayout(3,1));
            JPanel p = new JPanel(new GridLayout(1,2));
            outer.add(chatbuttons_enter[i]);
            p.add(chatbuttons_remove[i]);
            p.add(chatbuttons_add[i]);
            outer.add(p);
            add(outer);
            revalidate();
        }
    }
    
    
    public static void setChatParticipantNames(String name) {
        cpn.add(name);
    }
    
    private class Adresseintrag {
        public String nummer;
        public String name; //Intern verwendeter Name (Alias)
        public Adresseintrag(String id, String host) {
            this.nummer = id;
        }
    }
    
    public static void updateChatRooms() {
        fillListFlag = true;
    }
    
    public static void main(String[] args) {
        new Main();
        setChatParticipantNames("User 1");
        setChatParticipantNames("User 2");
        new Thread(){
            public void run() {
                for(;; ){
                    try {
                        updateChatRooms();
                        sleep(2000);
                        setChatParticipantNames("User ");
                    } catch(InterruptedException ix){
                        return;
                    }
                }
            }
        }.start();
        
    }
}

X5-599 · 6. Okt 2014

Ok, hier mal mein Erklärungsversuch. Bitte beachten: Dieses Wissen stammt von meinen eigenen Erfahrungen und kann evtl. völlig falsch sein. Zumindest habe ich es mir so zusammengereimt.

Alles was mittels SwingUtilities.invokeLater() ausgeführt wird, wird auf dem EDT ausgeführt. Somit sorgt eine zeilich lange Operation für ein "Einfrieren" der Oberfläche.

Für mich ergibt sich daraus: Da die Oberfläche immer vom EDT aus verändert werden sollte, sollten diese Operationen so kurz wie möglich sein. Wie sich das in deinem Fall umsetzen lässt kann ich dir leider nicht sagen...

Thallius · 6. Okt 2014

Soweit richtig. Du must deine lange Berechnung nicht mit invokeLater starten sondern du must einen Swingworker nehmen. Tutorials und Anleitungen dazu gibt es jede Mange im Netz.

Gruß

Claus

qwert · 6. Okt 2014

Ok. Danke!! 🙂

qwert · 7. Okt 2014

Hab's mal versucht, will aber auch nicht klappen 🙁 :

Java:

import java.awt.GridLayout;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.Vector;
import java.util.concurrent.ExecutionException;

import javax.swing.JButton;
import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.JScrollPane;
import javax.swing.SwingWorker;
 
@SuppressWarnings("serial")
public class Main extends JFrame implements ActionListener {
    JButton chatbuttons_enter[], chatbuttons_add[], chatbuttons_remove[];
    JButton[] ok;
    JDialog dialog;
    public Main() { 
        setSize(500,500);
        setVisible(true);
 
        new Thread(){
            public void run() {
                for(;; ){
                    try { 
                        sleep(2000);
                    } catch(InterruptedException ix){ 
                        return;
                    }
                    SwingWorker<Boolean, Void> worker = new SwingWorker<Boolean, Void>() {
                        Vector<String> chatParticipantNames = new Vector<String>();
                        @Override
                        public Boolean doInBackground() {
                            chatParticipantNames.add("Name1");
                            chatParticipantNames.add("Name2");
                            chatbuttons_remove = new JButton[chatParticipantNames.size()];
                            chatbuttons_add  = new JButton[chatParticipantNames.size()];
                            chatbuttons_enter = new JButton[chatParticipantNames.size()];
                            for(int i = 0; i < chatParticipantNames.size(); i++) {
//                                System.out.println("cPN = "+chatParticipantNames.get(i));
                                chatbuttons_enter[i] = new JButton();
                                chatbuttons_remove[i] = new JButton("Diesen Chat aus der Liste verbergen");
                                chatbuttons_add[i] = new JButton("Teilnehmer aus Adressbuch hinzufügen");
                                if(chatParticipantNames.get(i) != null) {
                                    chatbuttons_enter[i].setText(chatParticipantNames.get(i));
                                }
                                chatbuttons_enter[i].addActionListener(Main.this);
                                chatbuttons_remove[i].addActionListener(Main.this);
                                chatbuttons_add[i].addActionListener(Main.this);
                            }
                            System.out.println("doInBackground() => "+chatbuttons_add.length);
                            return true;
                        }

                        @Override
                        public void done() {
                            Main.this.getContentPane().removeAll();
                            Main.this.setLayout(new GridLayout(1,1));
                            JPanel panel = new JPanel(new GridLayout(chatParticipantNames.size(),1));
                            JScrollPane sp = new JScrollPane(panel);
                            try {
                                get();
                            } catch (InterruptedException | ExecutionException e) {
                                e.printStackTrace();
                            }
                            for(int i = 0; i < chatParticipantNames.size(); i++) {                
                                JPanel outer = new JPanel(new GridLayout(3,1));
                                JPanel p = new JPanel(new GridLayout(1,2));
                                outer.add(chatbuttons_enter[i]);
                                p.add(chatbuttons_remove[i]);
                                p.add(chatbuttons_add[i]);
                                outer.add(p);
                                panel.add(outer);    
                            }
                            Main.this.add(sp);
                            System.out.println("sp added");
                        }
                    };
                    worker.execute();
                }
            }
       }.start();
    }
    public void actionPerformed(ActionEvent e) {
    }
    public static void main(String[] args) {
        new Main();
    }
}

Wo ist der Fehler?
Danke nochmal 🙂.

Thallius · 7. Okt 2014

Ich habe mir jetzt erst den Eröffnungsthread durchgelesen.

Wenn Du Änderungen am UI machst, dann ist Swingworker natürlich nicht richtig.
Du kannst aus einem zweiten Thread heraus keine Änderungen am UI machen!
Was eventuell gehen könnte ist, dass du im Hintergrund ein neues JPanel erzeugst und dieses dann im EDT Thread aktivierst. Aber das ist natürlich auch nicht wirklich sinnvoll.

Ein Swing UI ist keine Spieleoberfläche. Wenn Du da mehrfach in kurzer Zeit Änderungen dran vornehmen willst, dann wirst du wahrscheinlich scheitern.

Wie oft wird dieses Fillist denn aufgerufen?

Gruß

Claus

qwert · 7. Okt 2014

Es passiert folgendes: Der Benutzer klickt auf einen Button, dann wird im Hintergrund in eine Datenbank etwas eingetragen. Ein weiterer Hintergrundthread pollt die Datenbank und wenn der Eintrag da ist, wird für jeden Eintrag in der DB ein Button erstellt.
Damit die Verzögerung beim Nutzer möglichst gering ist, soll FillList so oft wie möglich aufgerufen werden. (Es können sich auch auf andere Weise Änderungen in der Datenbank ergeben.) Wenn ich nicht immer gleich die GUI update, kann es sein, dass der Benutzer auf einen Button klickt, der gar keine Funktion mehr hat, weil der entsprechende Eintrag gar nicht mehr existiert.
Ich möchte vermeiden, dass andauernd Fehlermeldungen erscheinen à la "Zwischenzeitlich hat sich was geändert, bitte das Fenster schließen und erneut öffnen".

Thallius · 7. Okt 2014

Warum benutzt du keine JTable um die Daten anzuzeigen. Dort kann man auch mit einem Klick eine Aktion auslösen und da hast du auch kein Problem wenn es mal soviele Daten werden das sie nicht mehr auf das Panel passen. Da wird dann automatisch gescrollt. Den Tabellen-Inhalt kannst Du auch über einen Worker aktualisieren.

Gruß

Claus

qwert · 7. Okt 2014

Ja, auf die gleiche Idee bin ich kurz vor deinem Beitrag auch gekommen und es funktioniert jetzt wunderbar 🙂.

Also danke für deine Unterstützung 🙂.

GUI außerhalb GUI-Thread updaten - GUI friert ein

Aktives Mitglied

Top Contributor

Aktives Mitglied

Aktives Mitglied

Bekanntes Mitglied

Aktives Mitglied

Bekanntes Mitglied

Aktives Mitglied

Bekanntes Mitglied

Aktives Mitglied

Bekanntes Mitglied

Top Contributor

Top Contributor

Aktives Mitglied

Aktives Mitglied

Top Contributor

Aktives Mitglied

Top Contributor

Aktives Mitglied

Ähnliche Themen