Question

I have a SwingWorker with the implementation:

public Void doInBackground() throws InterruptedException
{
     while (true)
     {
            while(_update)
            {

            }
     }
}

If _update is set to false, is it safe to let the EDT perpetually go on a loop? Or should I let it sleep every second?

-- EDIT

It looks like I'm not using SwingWorker as it's supposed to be used. Thanks all. I'll stick to creating SwingWorkers as I need them.

Was it helpful?

Solution

It is not safe to let the Event Dispatch Thread go in a loop.

The event dispatch thread is needed to provide user interaction with the application. If you want an event to go off every few seconds, you should put it, or another copy of it, back on the event queue each time it has completed.

OTHER TIPS

Why wouldn't you just start another thread and let it run/sleep as needed, and have it use SwingUtilities.InvokeLater or whatever to do UI work as necessary?

Without any other context it looks like you might be misusing SwingWorker?

Licensed under: CC-BY-SA with attribution
Not affiliated with StackOverflow
scroll top