JDialog und SwingUtilities

[Sanix]

Hi,
Ich habe eine Klasse welche von einem JDialog erbt. Um die Komponenten darin zu aktualisieren verwende ich immer einen Thread der mit SwingUtilities.invokeLater aufgerufen wird. Diesen JDialog instanzieren ich von einem JFrame aus, wo ich auch die einzelnen Methoden benutze. Es funktioniert alles gut, ausser wenn ich den Dialog auf modal setze. Und ich verstehe nicht warum. Sowieso dieses ganze nervige invokeLater, dann hat man schlussendlich 100 Threads, nur damit das GUI sauber upgedatet wird und nicht einfriert.

 

[javimka]

Das invokeLater startet keinen neuen Thread, wenn du das nicht selber im Runnable geschrieben hast. Das wäre wohl auch eher verkehrt. Das invokeLater übergibt der EventQueue einen weiteren Job, der ausgeführt wird, sobald alle vorherigen Jobs abgearbeitet sind.
Was genau passiert denn, wenn du den JDialog modal setzt? Kannst du ein Beispiel posten, wo dieser Fehler reproduzierbar ist?

 

[Sanix]

Das invokeLater erwartet aber ein Runnable, welches als Thread gedacht ist.

public class ProgressWindow extends JDialog {

	/** unique ID for serialization */
	private static final long serialVersionUID = 255284650387076779L;
	
	/** Showing the progress */
	private JProgressBar progress;

	/** Label showing additional information */
	private JLabel jlStatus;

	/** Box showing buttons */
	private Box buttonBox, progressBarBox, labelBox;
	
	/**
	 * Member implicating if file correction is on hold
	 */
	private boolean paused = false;
	
	/**
	 * Member implicating if file correction must be cancelled
	 */
	private boolean running = true;
	/**
	 * Status before pause is stored here
	 */
	private String lastStatus;
	
	/**
	 * Progress window
	 *
	 * @param owner {@link JFrame} 
	 * @param title {@link String} 
	 */
	public ProgressWindow(JFrame owner, String title) {
		super(owner, title, false);

		//Init GUI
		setSize(500, 140);
		setLocationRelativeTo(owner);
		setDefaultCloseOperation(JFrame.DISPOSE_ON_CLOSE);
		setLayout(new BoxLayout(getContentPane(), BoxLayout.Y_AXIS));

		//Current file
		jlStatus = new JLabel();
		labelBox = Box.createHorizontalBox();
		labelBox.add(jlStatus, Box.CENTER_ALIGNMENT);
		add(labelBox);

		//Progress bar
		progress = new JProgressBar();
		progress.setBorder(BorderFactory.createEmptyBorder(20, 20, 20, 20));
		progress.setValue(0);
		progress.setStringPainted(true);
		progressBarBox = Box.createHorizontalBox();
		progressBarBox.add(progress, Box.CENTER_ALIGNMENT);
		add(progressBarBox);



		//Buttons
		JButton cancel = new JButton("Cancel");
		cancel.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent e) {
				running = false;
			}
		});
		final JButton pause = new JButton("Pause");
		pause.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent e) {
				if (e.getActionCommand().equals("Pause")) {
					pause.setText("Continue");
					setPaused(true);
				} else {	// Continue
					pause.setText("Pause");
					setPaused(false);
				}
			}
		});
		buttonBox = Box.createHorizontalBox();
		buttonBox.add(pause, Box.RIGHT_ALIGNMENT);
		buttonBox.add(cancel, Box.LEFT_ALIGNMENT);
		add(buttonBox);

		setVisible(true);		
	}
	
	/**
	 * Shows that action has been finished
	 */
	public void showFinish() {
		//Show finish
		buttonBox.removeAll();
		JButton finish = new JButton("Finished");
		finish.addActionListener(new ActionListener() {
			@Override
			public void actionPerformed(ActionEvent e) {
				ProgressWindow.this.dispose();
			}
		});
		buttonBox.add(finish);
		//Force refresh
		setSize(500, 141);
	}
	
	/**
	 * Set value for progress bar
	 * @param value 0-100
	 */
	public void showProgress(final int value) {
		try {
			SwingUtilities.invokeLater(new Runnable() {
				@Override
				public void run() {
					progress.setValue(value);
				}
			});
		} catch (Exception e) {
			// do nothing
		}
	}

	
	/**
	 * Calculates the progress in a range from 0-100
	 * @param current current value
	 * @param all total
	 */
	public void showProgress(int current, int all) {
		showProgress((int) ((double) (current + 1) / all * 100d));
	}
	
	/**
	 * Set status
	 * @param status String
	 */
	public void setStatus(final String status) {
		try {
			SwingUtilities.invokeLater(new Runnable() {
				@Override
				public void run() {
					jlStatus.setText(status);
				}
			});
		} catch (Exception e) {
			// do nothing
		}
	}
	
	/**
	 * Pause the current thread with busy waiting
	 */
	public void doPauseIfNeeded() {
		//Paused?
		while (paused) {
			try {
				Thread.sleep(1000);
			} catch (InterruptedException e1) {
				//do nothing
			}
		}
	}
	
	/**
	 * Getter for paused
	 * @return true if paused
	 */
	public boolean isPaused() {
		return paused;
	}

	/**
	 * Setter for paused
	 * @param paused true if paused
	 */
	public void setPaused(boolean paused) {
		if(paused) {
			lastStatus = jlStatus.getText();
			setStatus("Paused");
		} else {
			setStatus(lastStatus);
		}
		this.paused = paused;
	}

	/**
	 * Getter for running
	 * @return false if must be canceled
	 */
	public boolean isRunning() {
		return running;
	}

	/**
	 * Setter for running
	 * @param running true, if running
	 */
	public void setRunning(boolean running) {
		this.running = running;
	}	
}

Aufruf hier:

		new Thread() {
			@Override
			public void run() {
				//Show progress
				ProgressWindow pWindow = new ProgressWindow(ReportCleaner.this, "Scanning files...");
				pWindow.setStatus("Crawling files...");
			

...
				if (!variableContent) {
					for (int i = 0; i < size && pWindow.isRunning(); i++) {
						//File and progress
						File f = fileList.get(i);
						pWindow.showProgress(i, size);
						
						// adds all the variables
						variableList.addAll(reportFileReader.getReport(f).getVariables());

						//force the garbage collector to start. This keeps the memory usage low
						// start every 100 files
						if (counter++ > 100) {
							counter = 0;
							addToUnsortedList(false);
							Runtime.getRuntime().gc();
						}
						
						//Paused?
						pWindow.doPauseIfNeeded();
					}
					
					//this method adds all the data contained in the variableList.
					try {
					SwingUtilities.invokeAndWait(new Runnable() {
						
						@Override
						public void run() {	
							addToUnsortedList(true);
						}
					});
					} catch(Exception ex) {
						//do nothing
					}

				} else {
					int x = 0;
					for (int i = 0; i < size && pWindow.isRunning(); i++) {
						//File and progress
						File f = fileList.get(i);
						pWindow.showProgress(i, size);
...

					//this method adds all the data contained in the variableList.
					try {
					SwingUtilities.invokeAndWait(new Runnable() {
						
						@Override
						public void run() {	
							//add all the variables as keys into the modelMap, set mapped RuleTreeModels to null
							for (String s : variableMap.keySet()) {
								modelMap.put(s, null);
							}

							//add data for suggest field
							sfm.addAll(variableMap.keySet());
							suggestField.setSuggestData(sfm);
						}
					});
					} catch(Exception ex) {
						//do nothing
					}
					
				}
				
				//Show that action has been finished -> Close window
				pWindow.dispose();
			}
		}.start();

 

[javimka]

Das Runnable ist ein Objekt, das lauffähig ist. Es wird nicht als Thread gedacht.

 

[Ein Keks]

Runnable sieht ungefähr so aus:

public interface Runnable{
  public abstract void run();
}

da is nix zu sehn von nem Thread^^

 

[Sanix]

ok, ich habe die Javadoc falsch interpretiert. Meine Frage steht aber noch immer im Raum...

 

[Ebenius]

ok, ich habe die Javadoc falsch interpretiert. Meine Frage steht aber noch immer im Raum...

Diese auch: ;-)

Was genau passiert denn, wenn du den JDialog modal setzt? Kannst du ein Beispiel posten, wo dieser Fehler reproduzierbar ist?

Am besten ein hübsches, kleines, abgespecktes, ausführbares Beispiel...

Ebenius