Re: New Swing Window Not Drawn
Hal Vaughan wrote:
I've created a small class that's not too different from a JOptionPane. The
purpose is to use it when I need to put up a "Please Wait" type message or
something similar. I'm using a separate thread to show and hide the
window. The other actions that need to happen while this window appears
and later disappears are happening, but the window is not actually
rendered.
I know I need to use a separate thread when I have actions going on while
new Swing components are rendered or changed, and I'm doing that, but that
doesn't seem to make a difference.
Here's the method that is supposed to show the window and check periodically
to see if the window should be closed:
public void activate() {
//flagActive is set to false when the window should disappear
flagActive = true;
new Thread(new Runnable() {
public void run() {
System.out.println("-----Opening Wait Window.");
//jSelf is the JFrame class for the window
jSelf.setVisible(true);
while (true) {
try {Thread.sleep(50);} catch (Exception e) {
System.out.println("Insomnia");
}
if (!flagActive) break;
}
System.out.println("-----Closing Wait Window.");
jSelf.setVisible(false);
}
}).start();
return;
}
I've had cases where I've used different threads like this before, but it
was always the new or internal thread that did the actions and the Swing
events were done by the "regular" thread.
Why is the window not being rendered here, even though it has its own thread
and Swing should be able to have time to render it?
Thanks!
Hal
All interaction with Swing components should be done on the Event
Dispatch Thread.
Look at the class SwingUtilities
<http://java.sun.com/j2se/1.5.0/docs/api/javax/swing/SwingUtilities.html>.
specifically the invokeLater() method
Instead of setting aFlag, you should simple fire an event on the EDT.
Even if you wanted to wait for the flag to change, you need to use
synchronization. You also shouldn't be using a busy wait (even if you
sleep for 50), but instead do something like this:
public void sleepWhileActive() {
synchronize(myLock) {
while (active) {
try {
myLock.wait()
} catch (InterruptedException e);
System.out.println("Insomnia");
}
}
}
}
public void setActive(boolean active) {
synchronize(myLock) {
this.active = active;
myLock.notifyAll();
}
}
But, like I said, you should be using the EDT instead...
public void activate() {
assert !SwingUtilities.isEventDispatchThread() : "This would cause
a deadlock"
SwingUtilities.invokeLater(new Runnable() {
System.out.println("-----Opening Wait Window.");
//jSelf is the JFrame class for the window
jSelf.setVisible(true);
});
doExpensiveOperation();
SwingUtilities.invokeLater(new Runnable() {
System.out.println("-----Closing Wait Window.");
jSelf.setVisible(false);
});
}
Well, hope this helps.
Daniel.