Swing Problem im ActionListener

[partsch]

Hallo,
Ich habe mir ein Programm geschrieben, bei welchem der Benutzer eine beliebige Anzahl an Farben auswählen kann (welche in eine JList(DefaulListModel) eingefügt werden) und eine Zeit in Millisekunden festsetzen kann. Danach wird ein Full - Screen erzeugt und mit dem Click auf den "Starten" - Button werden alle Farben nacheinander als Hintergrund angezeigt. Natürlich soll jede Farbe genau so lang wie der Benutzer es am JSlider im ersten Frame eingestellt hat angezeigt werden!

Mein Problem:
Es funktioniert fast alles wie gewünscht. Die Zeit und die vom Benutzer ausgewählten Farben werden übergeben und der Full - Screen mit den verschiedenen Buttons wird auch erzeugt. Nur leider wird der Code im ActionListener des FullScreens zwar ausgeführt jedoch werden weder die Buttons ausgeblendet oder noch die verschiedenen Farben als Hintergrund gesetzt.
Ich habe mir zuerst gedacht es hat etwas mit der Methode

SwingUtilities.invokeLater(new Runnable())

zu tun und habe auch mit "normalen" Threads es probiert, jedoch funktioniert es noch immer nicht.

Hier ist mein Code, ich hoffe jemand von euch weiß mir zu helfen:

Main - Klasse

import javax.swing.SwingUtilities;


public class Main {
	public static void main(String[] args) {
		SwingUtilities.invokeLater(new Administrator());
	}
}

Administrator: Hier kann der User die Farben und die Zeit einrichten:

import java.awt.Color;
import java.awt.event.ActionListener;
import java.awt.event.ActionEvent;
import java.util.ArrayList;
import java.util.HashMap;

import javax.swing.BorderFactory;
import javax.swing.DefaultListModel;
import javax.swing.ImageIcon;
import javax.swing.JButton;
import javax.swing.JColorChooser;
import javax.swing.JFrame;
import javax.swing.JLabel;
import javax.swing.JList;
import javax.swing.JScrollPane;
import javax.swing.JSlider;
import javax.swing.ListSelectionModel;
import javax.swing.SwingUtilities;
import javax.swing.event.ChangeEvent;
import javax.swing.event.ChangeListener;


@SuppressWarnings("serial")
public class Administrator extends JFrame implements ActionListener, Runnable {
	
	/**
	 * Standart - Farben welche in der SELECTION_LIST angezeigt werden.
	 */
	private static final HashMap<java.lang.String, java.awt.Color> COLOR_MAP =
			new HashMap<String, Color>();
	static {
			/* Create Default Colors */
		COLOR_MAP.put("Rot", Color.RED); 		COLOR_MAP.put("Pink", Color.PINK);
		COLOR_MAP.put("Grün", Color.GREEN); 	COLOR_MAP.put("Blau", Color.BLUE);
		COLOR_MAP.put("Gelb", Color.YELLOW); 	COLOR_MAP.put("Orange", Color.ORANGE);
		COLOR_MAP.put("Schwarz", Color.BLACK);  COLOR_MAP.put("Grau", Color.DARK_GRAY);
		COLOR_MAP.put("Cyan", Color.CYAN); 		COLOR_MAP.put("Magenta", Color.MAGENTA);
	}
	
	/**
	 * Hier wird die Zeit festgelegt, welche pro Farbe anhalten soll.
	 */
	private JSlider slider;
	private JLabel SLIDER_LABEL;
	
	/**
	 * Liste für Standart Farben der Color Klasse
	 */
	private JList SELECTION_LIST;
	private DefaultListModel SELECTION_MODEL;
	
	/**
	 * Liste der vom Benutzer gewählten Farben
	 */
	private JList USER_LIST;
	private DefaultListModel USER_MODEL;
	
	private final JFrame OBSERVER_BINDING = this;
	
	/**
	 * Konstruktor
	 * Erzeugt die Klassen Variablen
	 */
	public Administrator() {
		super("Screen - Tester");
		setLayout(null);
		
		SELECTION_MODEL = new DefaultListModel();
		SELECTION_MODEL = addAll(COLOR_MAP.keySet().toArray(), SELECTION_MODEL);
		SELECTION_LIST = new JList(SELECTION_MODEL);
		SELECTION_LIST.setSelectionMode(ListSelectionModel.MULTIPLE_INTERVAL_SELECTION);
		
		USER_MODEL = new DefaultListModel();
		USER_LIST = new JList(USER_MODEL);
		USER_LIST.setSelectionMode(ListSelectionModel.MULTIPLE_INTERVAL_SELECTION);
		
		slider = new JSlider();
		slider.setBounds(10, 450, 280, 25);
		slider.setBackground(Color.WHITE);
		slider.addChangeListener(new ChangeListener() {
			public @Override void stateChanged(ChangeEvent arg0) {
				SLIDER_LABEL.setText(String.format("Zeit pro Bild per Milli - Sekunde: %d", slider.getValue() * 100));
			}
		});
		
		SLIDER_LABEL = new JLabel(String.format("Zeit pro Bild per Milli - Sekunde: %d", slider.getValue() * 100));
		SLIDER_LABEL.setBounds(12, 420, 280, 25);
	}
	
	/**
	 * Fügt und erzeugt die restlichen Komponenten in das Frame ein.
	 */
	public void initComponents() {
		JScrollPane selection_observer = new JScrollPane(SELECTION_LIST);
		selection_observer.setBounds(30, 20, 100, 200);
		add(selection_observer);
		
		JScrollPane user_observer = new JScrollPane(USER_LIST);
		user_observer.setBounds(170, 20, 100, 200);
		add(user_observer);
		
		add(slider); add(SLIDER_LABEL);
		
		JButton commit_button = new JButton("Commit", new ImageIcon("Recources/ImageRecources/pfeil.jpg"));
		commit_button.addActionListener(this);
		commit_button.setBounds(getWidth()/2  - 75, 230, 150, 35);
		commit_button.setBackground(Color.WHITE);
		commit_button.setBorder(BorderFactory.createLineBorder(Color.BLACK));
		add(commit_button);
		
		JButton delete_button = new JButton("Delete", new ImageIcon("Recources/ImageRecources/kreuz.jpg"));
		delete_button.addActionListener(this);
		delete_button.setBounds(getWidth()/2 - 75, 270, 150, 35);
		delete_button.setBackground(Color.WHITE);
		delete_button.setBorder(BorderFactory.createLineBorder(Color.BLACK));
		add(delete_button);
		
		JButton create_color_button = new JButton("Choose Color", new ImageIcon("Recources/ImageRecources/choose.jpg"));
		create_color_button.addActionListener(this);
		create_color_button.setBounds(getWidth()/2 - 75, 310, 150, 35);
		create_color_button.setBackground(Color.WHITE);
		create_color_button.setBorder(BorderFactory.createLineBorder(Color.BLACK));
		add(create_color_button);
		
		/* Start Full Screen */
		JButton run_button = new JButton("Run Full Screen");
		run_button.addActionListener(this);
		run_button.setBounds(getWidth()/2 - 75, 350, 150, 35);
		run_button.setBackground(Color.WHITE);
		run_button.setBorder(BorderFactory.createLineBorder(Color.BLACK));
		add(run_button);
	}
	
	
	public @Override void run() {
		setDefaultCloseOperation(EXIT_ON_CLOSE);
		setSize(300, 520);
		setLocationRelativeTo(null);
		setResizable(false);
		initComponents();
		getContentPane().setBackground(Color.WHITE);
		
		setVisible(true);
	}
	
	public @Override void actionPerformed(ActionEvent event) {
		String command = event.getActionCommand();
		if(command.equals("Commit")) {
			for(Object obj : SELECTION_LIST.getSelectedValues()) {
				USER_MODEL.addElement(obj);
			}
		} else if(command.equals("Delete")) {
			Object[] values_to_delete = USER_LIST.getSelectedValues();
			for(Object obj : values_to_delete) {
				USER_MODEL.removeElement(obj);
			}
		} else if(command.equals("Choose Color")) {
			Color color = JColorChooser.showDialog(this, "Choose a Color", Color.WHITE);
			if(color != null) {
				String code = String.format("[%d - %d - %d]", color.getRed(), color.getGreen(), color.getBlue());
				COLOR_MAP.put(code, color);
				USER_MODEL.addElement(code);
			}
		} else if(command.equals("Run Full Screen")) {
			// startet den Full - Screen, welcher die Farben abspielt
			SwingUtilities.invokeLater(new Runnable() {
				public @Override void run() {
					setVisible(false);
					ArrayList<Color>colors = new ArrayList<Color>();
					for(int model_index = 0; model_index < USER_MODEL.size(); model_index++) {
						String key = (String)USER_MODEL.get(model_index);
						if(COLOR_MAP.containsKey(key)) {
							colors.add(COLOR_MAP.get(key));
						}
					}
					
					FullScreen screen = new FullScreen(colors, slider.getValue() * 100, OBSERVER_BINDING);
					SwingUtilities.invokeLater(screen);
					
					while(screen.isVisible()) {
						try { Thread.sleep(20); } catch(InterruptedException exception) { /*EMPTY*/ }					}
					
					System.gc();
					USER_MODEL.removeAllElements();
					slider.setValue(slider.getMaximum()/2);
					setVisible(true);
				}
			});
		}
	}
	
	/**
	 * Gügt einen Object[] in ein DefaultListModel ein und gibt es anschließlich
	 * wirder zurück.
	 * @param args
	 * @param model
	 * @return
	 */
	private DefaultListModel addAll(Object[]args, DefaultListModel model) {
		for(Object obj : args) {
			model.addElement(obj);
		} return model;
	}
}

FullScreen: Hier sollten die Farben nacheinander als Hintergrund gesetzt werden

import java.awt.Color;
import java.awt.Dimension;
import java.awt.Toolkit;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import java.util.List;

import javax.swing.BorderFactory;
import javax.swing.JButton;
import javax.swing.JDialog;
import javax.swing.JFrame;
import javax.swing.JOptionPane;
import javax.swing.SwingUtilities;

@SuppressWarnings("serial")
public class FullScreen extends JDialog implements Runnable, ActionListener {
	private List<Color> colors;
	private long time;

	private JButton[] buttons = new JButton[4];

	private final JDialog OBSERVER_BINDING = this;

	public FullScreen(List<Color> colors, long time, JFrame observer) {
		super(observer);
		setTitle("Full Screen");

		this.colors = colors;
		this.time = time;

		setLayout(null);
	}

	public void initComponents() {
		JButton start_normal = new JButton("Starten");
		start_normal.setBounds(40, 30, 150, 50);
		start_normal.setBackground(Color.WHITE);
		start_normal.setBorder(BorderFactory.createLineBorder(Color.BLACK));
		start_normal.setActionCommand("normal");
		start_normal.addActionListener(this);
		add(start_normal);
		buttons[0] = start_normal;

		JButton speed_button = new JButton("Schnell Starten");
		speed_button.setBounds(40, 90, 150, 50);
		speed_button.setBackground(Color.WHITE);
		speed_button.setBorder(BorderFactory.createLineBorder(Color.BLACK));
		speed_button.setActionCommand("speed");
		speed_button.addActionListener(this);
		add(speed_button);
		buttons[1] = speed_button;

		JButton crazy_button = new JButton("Psycho Durchgang");
		crazy_button.setBounds(40, 150, 150, 50);
		crazy_button.setBackground(Color.WHITE);
		crazy_button.setBorder(BorderFactory.createLineBorder(Color.BLACK));
		crazy_button.setActionCommand("psycho");
		crazy_button.addActionListener(this);
		add(crazy_button);
		buttons[2] = crazy_button;

		JButton exit_button = new JButton("Exit");
		exit_button.setBounds(40, 210, 150, 50);
		exit_button.setBackground(Color.WHITE);
		exit_button.setBorder(BorderFactory.createLineBorder(Color.BLACK));
		exit_button.setActionCommand("dispose");
		exit_button.addActionListener(this);
		add(exit_button);
		buttons[3] = exit_button;
	}

	public @Override
	void run() {
		setDefaultCloseOperation(DISPOSE_ON_CLOSE);
		setUndecorated(true);
		setResizable(false);

		Dimension screen_size = Toolkit.getDefaultToolkit().getScreenSize();
		setSize(screen_size);
		setLocation(0, 0);

		initComponents();
		getContentPane().setBackground(Color.WHITE);
		setVisible(true);
	}

	public @Override
	void actionPerformed(final ActionEvent event) {
		String command = event.getActionCommand();
		if (command.equals("dispose")) {
			setVisible(false);
			dispose();
			return;
		}

		for (JButton button : buttons) {
			button.setVisible(false);
		}

		if (command.equals("normal")) {
			for (Color color : colors) {
				OBSERVER_BINDING.getContentPane().setBackground(color);
				try {
					Thread.sleep(time);
				} catch (InterruptedException exception) {
					showErrorMessage();
				}
			}

		} else if (command.equals("speed")) {
			SwingUtilities.invokeLater(new Runnable() {
				public @Override
				void run() {
					for (Color color : colors) {
						getContentPane().setBackground(color);
						try {
							Thread.sleep(time / 10);
						} catch (InterruptedException exception) {
							showErrorMessage();
						}
					}
				}
			});
		} else if (command.equals("psycho")) {
			SwingUtilities.invokeLater(new Runnable() {
				public @Override
				void run() {
					for (Color color : colors) {
						getContentPane().setBackground(color);
						try {
							Thread.sleep(16);
						} catch (InterruptedException exception) {
							showErrorMessage();
						}
					}
				}
			});
		}

		for (JButton button : buttons)
			button.setVisible(true);
		getContentPane().setBackground(Color.WHITE);
	}

	private void showErrorMessage() {
		JOptionPane.showMessageDialog(this,
				"Es trat ein Fehler beim abspielen der Farben auf!", "Fehler",
				JOptionPane.ERROR_MESSAGE);
	}
}

 

[eRaaaa]

Mhm also actionPerformed wird eh auf dem EDT aufgerufen daher brauchst du eigentlich da kein invokeLater. Was aber viel schlimmer ist, ist dass du Thread.sleep aufrufst, das gilt in diesem Falle nämlich dann dem EDT, d.h. der legt sich schlafen.

Ich denke du müsstest einen neuen Thread starten in der actionPerformed. Dieser ruft dann invokeLater auf, wenn entsprechende Änderungen an GUI-Objekten gemacht werden soll und dieser legt sich dann immer für [c]time[/c] ms schlafen.

 

[partsch]

Danke eRaaaa für die Hilfe funktioniert prima!