Swing Threading de Java

Question

Según tengo entendido, si inicio otro hilo para realizar algunas acciones, necesitaría SwingUtilities.invokeAndWait o SwingUtilities.invokeLater para actualizar la GUI mientras estoy en dicho hilo. Por favor corrígeme si estoy equivocado.

Lo que intento lograr es relativamente sencillo: cuando el usuario hace clic en enviar, quiero (antes de realizar cualquier acción) deshabilitar el botón Enviar, realizar la acción y, al final de la acción, volver a habilitar el botón. Mi método para realizar la acción actualiza la GUI directamente (muestra resultados) cuando recupera los resultados.

Esta acción básicamente consulta un servidor y obtiene algunos resultados.

Lo que tengo hasta ahora es:

boolean isRunning = false; 

synchronized handleButtonClick() { 
    if (isRunning == false) { 
    button.setEnabled(false); 
    isRunning = true; 
    doAction(); 
    } 
} 

doAction() { 
    new Thread() { 
    try { 
     performAction(); // Concern A 
    } catch (...) { 
     displayStackTrace(...); // Concern B 
    } finally { 
     SwingUtilities.invokeLater (/* simple Runnable to enable button */); 
     isRunning = false; 
    } 
    } 
} 

Para ambos de mis preocupaciones anteriores, no tendría que utilizar SwingUtilities.invokeAndWait ya que ambos van a actualizar la interfaz gráfica de usuario? Todas las actualizaciones GUI giran en torno a la actualización JTextPane. ¿Debo verificar mi hilo si estoy en EDT y, de ser así, puedo llamar a mi código (independientemente de si actualiza la GUI o no) y NO usar SwingUtilities.invokeAndWait?

EDIT: Esto es lo que estoy haciendo ahora:

handleButtonClick() { 
    if (isRunning == true) 
    return; 
    disable button; 
    SwingWorker task = new MyTask(); 
    task.execute(); 
} 

...inside MyTask 
doInBackground() { 
    return performAction(); 
} 

done() { 
    result = get(); 
    enable button; 
    isRunning = false; 
    interpret result (do most of the GUI updates here); 
} 

Mientras performAction() hace algunos cambios GUI, he envuelto aquellos en:

if (SwingUtil.isEDT()) 
    doGUIupdate() 
else 
    SwingUtil.invokeLater(new Runnable() { 
    run() { 
     doGUIupdate(); 
    } 
    }); 

Esperemos que este es un paso en la dirección correcta , por favor coméntenos si cree que hay mejores formas de manejar mi situación.

Answer 1

En mi opinión, casi nunca deberías usar invokeAndWait(). Si algo va a tomar un tiempo que bloqueará su UI.

Utilice un SwingWorker para este tipo de cosas. Eche un vistazo al Improve Application Performance With SwingWorker in Java SE 6.

Answer 2

Debería considerar el uso de SwingWorker ya que no bloqueará el subproceso de interfaz de usuario, mientras que ambos métodos SwingUtilities se ejecutarán en el subproceso EDT, lo que bloquea la interfaz de usuario.

Answer 3

me quedo con la sencilla Thread dentro EventQueue.invokeLater(...) y que funcionó sin problemas ...

java.awt.EventQueue.invokeLater(new Runnable() { 
    public void run(){ 

     new Thread(new Runnable(){ 
      public void run(){ 

       try{ 
        EdgeProgress progress = EdgeProgress.getEdgeProgress(); 
        System.out.println("now in traceProgressMonitor..."); 
        while(true){ 
         // here the swing update 
         if(monitor.getState() == ProgressMonitor.STATE_BUSY){ 
          System.out.println(monitor.getPercentDone()/2); 
          progress.setProgress(monitor.getPercentDone()/2); 
         }else{ 
          break; 
         } 
         Thread.sleep(5); 
        } 
       }catch(InterruptedException ie){} 

      } 
     }).start(); 

    } 
});