user402642
user402642

Reputation:

Java Threads -- Perpetual Loop in Swing Event Dispatch Thread

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.

Upvotes: 2

Views: 513

Answers (2)

Erick Robertson
Erick Robertson

Reputation: 33082

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.

Upvotes: 3

John Gardner
John Gardner

Reputation: 25146

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?

Upvotes: 0

Related Questions