Using a SwingWorker in an Executor class?

Question

Here's some code that works and does what I want but I don't know if its a good idea....
I have a program that may run one of a couple dozen long operations. I want to bring up a progress dialog to show the user how these operations are coming along.
I was using a swing worker every time I ran a long task but its a pain to make changes in so many different places. So I set up the approach below.
import java.util.concurrent.Executor;
import javax.swing.JOptionPane;
import com.sun.java.help.impl.SwingWorker;
* This class takes a long task as a Runnable and
* runs it while showing a progress dialog
public class MyExecutor implements Executor
     public void execute(Runnable r)
          final MySmallProgressWindow dialog;
          dialog = new MySmallProgressWindow();
          final Runnable fr = r;
          dialog.show("Testing Executor ") ;
          final SwingWorker worker = new SwingWorker()
               public Object construct()
                    try
                        fr.run();
                   catch(Exception g)
                         System.err.println(g);
                         JOptionPane.showMessageDialog(null, "Error: " + g.toString(), "Error", JOptionPane.ERROR_MESSAGE);
                   finally
                        dialog.hide();
                        System.out.println("Clean up here in finally");
                   return null;
               public void finished()
                       dialog.hide();
         worker.start();
I call it this way
MyExecutor execute = new MyExecutor();                   
Runnable runnableCopy = new Runnable() {
public void run() {
     someLongTask();
execute.execute(runnableCopy);Comments?
Message was edited by:
legoqueen

Answer 1

import javax.swing.*;
public class MyApplet extends JApplet {
     public void init() {
          //init stuff here
     public void start() {
          //start stuff here
     public static void main(String[] args) {
          JFrame f = new JFrame();
          JApplet a = new MyApplet();
          f.getContentPane().add(a);
          f.setSize(600, 400);
          f.setVisible(true);
          f.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
          a.init();
          a.start();
}Excuse me, but is there a way to do this on Applet only instead of JApplet... I was told we need to use Applet only...