Swing Problem mit Button und ActionListener

[Stefan42]

Hallo zusammen,

ich habe einen für mich unerklärlichen Fehler in meinem Programm, der nachfolgende Code zeigt den Fehler auch. Denn wenn ich load über einen Button mit ActionListener aufrufe (1. load) ändert sich die Farbe des Anzeigebuttons (button) nicht, wird allerdings load direkt im Programm verwendet (2. load) funktioniert es einwandfrei. Daher die Frage an euch, wie ich das auch über den Button erreichen könnte. An fehlenden repaints sollte es nicht liegen, da es auch ohne diese funktioniert.

Für eure Hilfe bin schon jetzt sehr dankbar!

Gruß
Stefan

import java.awt.Color;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.*;

public class Test_GUI {

    static JFrame frame;
    static JButton button = new JButton();
   
    public static void main(String[] args) {
        frame = new JFrame("NNVT - Neuronales Netz Visualisierungs Tool");
        frame.setSize(800, 800);
        frame.setDefaultCloseOperation(javax.swing.JFrame.EXIT_ON_CLOSE);
        frame.setLayout(null);
        JButton test = new JButton("Test");
        test.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent e)
            {
                load(); // 1. load
                frame.repaint();
            }
        });
        test.setSize(100,50);
        test.setLocation(300, 350);
        button.setSize(100,50);
        button.setLocation(300, 300);
        frame.add(test);
        frame.add(button);
        frame.setVisible(true);
        load(); // 2. load
    }
   
    public static void load()
    {   
        for(int i=0;i<101;i++)
        {
            if(i%2==0)
                button.setBackground(Color.BLUE);
            else
                button.setBackground(Color.LIGHT_GRAY);
            button.repaint();
            try {
                Thread.sleep(100);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
        frame.repaint();
    }

}

 

[Enceladus271]

Der Unterschied liegt den ausführenden Threads: Beim Klick auf den Button wird die Methode actionPerformed im Event Dispatch Thread ausgeführt. Wenn du nun hier Thread.sleep() ausführst wird der Event Dispatch Thread die ganze Zeit über blockiert. Dadurch kann die GUI nicht aktualisiert werden.

 

[sascha-sphw]

Der Unterschied liegt den ausführenden Threads: Beim Klick auf den Button wird die Methode actionPerformed im Event Dispatch Thread ausgeführt. Wenn du nun hier Thread.sleep() ausführst wird der Event Dispatch Thread die ganze Zeit über blockiert. Dadurch kann die GUI nicht aktualisiert werden.

Aslo das mit dem blocken des GUI Threads kann ich so bestätigen, aber, dass es einen eigenen Event Dispatch Thread gibt, wäre mir neu. Wenn es wirklich einen eigenen Event Dispatch Thread geben würde, würde der GUI Thread ja nicht blockieren... => Es läuft alles im GUI Thread.

@Stefan42 Du müsstest Deine Schleife in einen extra Thread packen, damit Du die Farbänderung sehen kannst. Und ich würde Dir noch raten das so umzubauen, dass Du keine static Member mehr hast.

 

[Stefan42]

Vielen Dank Sascha hab jetzt das ganze mit einem Thread gelöst und es funktioniert perfekt!

import java.awt.Color;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;

import javax.swing.*;

public class Test_GUI {
  
    public static void main(String[] args) {
      
        JButton test = new JButton("Test");
        JButton button = new JButton();
      
        Thread thread = new Thread()
        {
            @Override
            public void run()
            {
                load(button);
            }
        };
      
        JFrame frame = new JFrame("NNVT - Neuronales Netz Visualisierungs Tool");
        frame.setSize(800, 800);
        frame.setDefaultCloseOperation(javax.swing.JFrame.EXIT_ON_CLOSE);
        frame.setLayout(null);
        test.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent e)
            {
                thread.start();
            }
        });
        test.setSize(100,50);
        test.setLocation(300, 350);
        button.setSize(100,50);
        button.setLocation(300, 300);
        frame.add(test);
        frame.add(button);
        frame.setVisible(true);
        //thread.start();
    }
      
    public static void load(JButton button)
    {  
        for(int i=0;i<101;i++)
        {
            if(i%2==0)
                button.setBackground(Color.BLUE);
            else
                button.setBackground(Color.LIGHT_GRAY);
            button.repaint();
            try {
                Thread.sleep(100);
            } catch (InterruptedException e) {
                e.printStackTrace();
            }
        }
    }

}

 

[Enceladus271]

aber, dass es einen eigenen Event Dispatch Thread gibt, wäre mir neu

https://docs.oracle.com/javase/tutorial/uiswing/concurrency/dispatch.html

Ist wahrscheinlich das was du als GUI Thread bezeichnest. Wir meinen das Gleiche.

 

[sascha-sphw]

https://docs.oracle.com/javase/tutorial/uiswing/concurrency/dispatch.html

Ist wahrscheinlich das was du als GUI Thread bezeichnest. Wir meinen das Gleiche.

Wir meinen das Gleiche.