Warten auf Rückgabewert im EventDispatchingThread

[ChristianB]

Hallo,

ich habe folgendes Code Besispiel:

Die folgende Methode wird im Zuge eines ActionPerfomed Events aufgerufen:

/**
*@return Returns if save operation is done correctly
*/
public boolean saveData(){
    boolean bRet = true;
    showSaveStateInfoDialog();     //NonModal Frame
    
    for(int xy = 0 ; xy < ITEMXYZCOUNT ; xy ++){
        bRet = bRet && saveItem(xy);
        setSaveStateInfoText("Saveing item " +  xy);
    }
    return bRet;
}

Das Problem ist, das die Swing Oberfläche jetzt nicht neu gezeichnet wird, da der EventDispatcher ja vor verlassen der Methode garnicht dazu veranlasst wird.

Ich habe auch schon einen Gedanken daran verschwendet die Sache in einen SwingWorker bzw. auch einen normalen Thread auszulagen, jedoch habe ich bisher keine Lösung wie ich in dieser Methode dann noch auf einen Rückgabewert warten kann ohne Swing gleich wieder auszubremsen.


Würde mich über gute Vorschläge freuen
Christian

 

[Marco13]

"Warten"....? Welcher Thread soll "warten"? Hilft vielleicht die "publish"-Methode aus dem SwingWorker? Der SwingWorker workt in einem eigenen Thread, und veröffentlicht seine Zwischenergebnisse auf WUnsch im EDT....

 

[ChristianB]

Hi,

ich glaube um mein Problem verständlich rüber zu bringen poste ich hier mal einen Code schnipsel, der zwar funktional zeigt was ich will jedoch so nicht funktioniert.

/**
*This Method is an action Event Handler called by the EDT
*/
public void btnExitActionPerformed(Action a){
    if( saveData() ) exitProgram();
}

/**
*Return if save operation was successfull
*/
public boolean saveData(){
    Thread thSaver = new Thread(new Runnable(){
        public void run(){
            //-Do save oprations
            //-Update Gui with InvokeLaiter() (InvokeAndWait() end up in a DeadLock)
        }
    });
    thSaver.start();
    //Wait for completition
    while(thSaver.isAlive()){}
    return bResultFromThread;
}

Wie hier nun zu sehen ist benötige ich hier in einer Methode, die aus dem EDT aufgerufen wird und einen Rückgabeparameter liefert. Dieser Thread bzw. der SwingWorker wären alle beide super, wenn ich da nicht ein resultat benötigen würde. Ich habe leider keine andere Möglichkeit , da ich die Methode saveData() aus einem Interface implementieren muss. Die ActionPerformed Methode implementiere ich nicht selber.

Gruß Christian

 

[SlaterB]

die Abfrage
> if( saveData() ) exitProgram();
könnte an das Ende der run()-Methode?
dann musst du gar nicht den Rückgabewert erhalten usw. (edit: ok, ist vorgegeben)

----

statt
> while(thSaver.isAlive()){}
gibts noch
thSaver.join();

wenn aber die actionPerformed-Methode so lange warten muss, dann bringt ein Thread gar nix

---

hilfreich wäre da vielleicht nur, wenn generell ein Thread parallel zur GUI läuft und jederzeit bzw. zu jeder freien Zeit das save() soweit wie möglich vorbereitet hat,
dann kann bei einem save-Klick die save-Aufgabe möglichst schnell abgeschlossen werden

-----

bei einem Button, der das Programm recht bald beendet, ist Nebenläufigkeit eh weniger wichtig,
wer will schon nach dem Klick auf Exit noch irgendeine andere Aktion starten?

auch saveData() allgemein klingt eher nach etwas, wo maximal ein 'Bitte warten' zu sehen ist,
aber auch da sollten keine anderen Aktionen möglich sein müssen,

tja, hast du wohl auch schon, bis auf Änderung dieser Nachricht,
ich sehe jedenfalls keine Möglichkeit, wenn die ActionPerformed so ungünstig vorgegeben ist

oder ist es nicht so, dass bei Anzeige eines Dialoges der ActionListener unterbrochen wird?
wenn man darin etwas aus einer ComboBox auswählt, wird doch auch neu gezeichnet..,

teste mal Starten eines Thread + (danach!) modaler Dialog

 

[SlaterB]

muss ich nun auch mal selber wissen :

public class TestGUI
    extends JFrame
{
    public TestGUI()
    {
        JButton b = new JButton("Start");
        b.addActionListener(new ActionListener()
            {

                public void actionPerformed(ActionEvent e)
                {
                    if (saveData())
                    {
                        System.exit(0);
                    }
                }

                private boolean saveData()
                {
                    final JTextField tf = new JTextField();
                    tf.setText("-1");

                    final JDialog d = new JDialog(TestGUI.this, true);
                    d.setSize(300, 300);
                    d.setLocation(300, 300);
                    d.add(tf);
                    
                    Thread thSaver = new Thread(new Runnable()
                        {
                            public void run()
                            {
                                for (int i = 0; i < 5; i++)
                                {
                                    try
                                    {
                                        Thread.sleep(1000);
                                    }
                                    catch (InterruptedException e)
                                    {
                                        e.printStackTrace();
                                    }
                                    tf.setText("" + i);
                                }
                                d.dispose();
                            }
                        });
                    thSaver.start();

                    
                    d.setVisible(true);
                    System.out.println("a");
                    return true;
                }
            });

        add(b);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(200, 200);
        setVisible(true);
    }

    public static void main(String[] args)
        throws Exception
    {
        new TestGUI();
    }
}

 

[ChristianB]

Hi SlaterB,


deine Version läuft auch bei mir. Aber müssten nicht eigentlich deine *.setText("") aufrufe in einem InvokeLater Aufruf stehen?
Was damit dann leider nicht mehr funktioniert.

(Es neigt sich alles dem Guten)

Gruß Christian

 

[Verjigorm]

http://java.sun.com/j2se/1.4.2/docs/api/javax/swing/text/JTextComponent.html#setText(java.lang.String)

This method is thread safe, although most Swing methods are not. Please see Threads and Swing for more information.

 

[ChristianB]

Dann ist ja alles super!

Das ist eigentlich genau die Lösung die ich dann gebraucht habe!

Vielen Dank sann mal an alle

Gruß Christian

 

[SlaterB]

invokeLater geht bei mir übrigens auch, was zum 'repaint ist noch möglich'-Gedanken passt,

scheiterst du vielleicht nur daran, dass die Variable i nicht final ist?