Swingworker rückgabewert verwenden

[athikka]

Hallo zusammen

ich lade in einem Swingworker einige dateien auf einen FTP-server hoch und will dabei die fortschritt anzeigen (deshalb swingworker - mit anderen wie invokelater geht es nicht)

nun möchte ich aber eine liste der hochgeladenen dateien zurückgeliefert bekommen:

public String[] fileUpload(final File[] f, final boolean compress, final String ftpPath){
       

final String[] ee; //für return der fileupload methode

        SwingWorker<String[], Void> worker = new SwingWorker<String[], Void>() {
    
                
                public String[] doInBackground() {
    
                   
                    
                    LinkedList<String> paths = new LinkedList<String>(); //soll zurückgegeben werden in der methode wenn fertig
                
                    //...FTP verbindung erstellen und verbinden...
                    
                    for(final File ff : files){ //alle selektierten dateien hochladen
                        if(ff.isDirectory()) {
                            try {
                               ftp.makeDirectory(ff.getName());
                            } catch (IOException e) {
                                e.printStackTrace();
                            }
                            
                            paths.add("DIR:" + directory + "/" + ff.getName());
                            
                            storeFiles(ff, ftp, ff.getName() + "/");
                           } 
                        } 
                    

    
                  //  ...FTPLogout...;

    
                    return paths.toArray(new String[0]);
                    
                }
    
    
    
                protected void done() {
                   
                    try {
                        ee = get(); [B][COLOR="Red"]//hier meckert es[/COLOR][/B] egal was ich versuche
//get() kann ich ja hier wegen dem void nicht returnen...
                    } catch (InterruptedException e) {

                    } catch (ExecutionException e) {

                    }
                    
                }
            };
            
            worker.execute();

        return ee;

ich bekomme auch kein ergebnis wenn ich statt dem "return ee" sage: "return worker.get()"...


ich krieg das string array einfach nicht aus dem swingworker heraus... kann mir wer helfen?

egal wie, ich will einfach das String[] aus dem worker herausbekommen wenn er fertig ist damit ich damit weiterarbeiten kann...

 

[Tomate_Salat]

Das weiterarbeiten wird eher schwierig, da du oft nicht weist, WANN der worker fertig ist. Wenn du weiterarbeiten willst, dann entweder in einer neuen Methode oder direkt in der done()-methode. Der SwingWorker wäre wohl auch sinnlos, wenn du nach [c]worker.execute();[/c] darauf wartest, dass er fertig wird ;-).

MFG

Tomate_Salat

 

[eRaaaa]

Das weiterarbeiten wird eher schwierig, da du oft nicht weist, WANN der worker fertig ist. Wenn du weiterarbeiten willst, dann entweder in einer neuen Methode oder direkt in der done()-methode. Der SwingWorker wäre wohl auch sinnlos, wenn du nach [c]worker.execute();[/c] darauf wartest, dass er fertig wird ;-).

MFG

Tomate_Salat

Mhm? Verstehe ich nicht, meine Antwort wäre gewesen:
"Du musst doch get() nicht unbedingt in done() aufrufen. Kannst ja auch außerhalb, es wird dann einfach der Thread blockiert, bis doInBackground fertig ist." :autsch:

 

[Tomate_Salat]

Das man get(); außerhalb aufrufen kann, habe ich noch garnicht gesehen. Ich habe das so noch nicht verwendet, aber spontan würde ich sagen, du erreichst damit genau dass Gegenteil von dem was du willst: du machst aus asynchron synchron. Da kannst du dir auch gleich den Worker sparen.

Edit Hier nochmal zur verdeutlichung:

public class WorkerDemo
{
    public static void main(String[] args)
    {
        SwingWorker<String[], Void> worker    = new SwingWorker<String[], Void>()
        {
            @Override
            protected String[] doInBackground() throws Exception
            {
                try {
                    Thread.sleep(2000);
                } catch(InterruptedException e) {
                    e.printStackTrace();
                }
                                
                return new String[] {"a","b","c"};
            }
        };
        
        long current    = System.currentTimeMillis();
        worker.execute();
        
        try {
            worker.get();
        } catch(InterruptedException e) {
            e.printStackTrace();
        } catch (ExecutionException e)  {
            e.printStackTrace();
        }

        System.out.println("Dauer: " + ((System.currentTimeMillis() - current)/1000) + "s" );
    }
}

Ergebnis:

Dauer: 2s

 

[athikka]

also werd ich um eine eigene version des swingworkers nicht rumkommen?

blöd dass ich hier codeverdoppeln muss nur weil ich einen rückgabewert haben möchte (in done weiterarbeiten)...

 

[eRaaaa]

Also jetzt kann ich dir überhaupt nicht mehr folgen...
----
Mhm. Nö, ich habe eine lange Aktion die ich abarbeite(das passiert in einem (Worker)Thread != EDT (doInBackground()), zwischendurch möchte ich den Fortschritt anzeigen (process() -> EDT).Ganz am Ende könnte ich das Gesamtergebnis wieder in meiner GUI anzeigen(das hätte man dann in der done machen können(auch EDT)) oder eben in meinem anderen aktuellen Thread(!= EDT) weiter bearbeiten...
Natürlich hast du recht, wenn man keine Zwischenergebnisse in der GUI veröffentlichen will(ich habe ihn aber so verstanden dass er das machen will (wobei, dann wäre das Void falsch ;/), würde das so nicht wirklich Sinn machen.
Schau dir das zweite Beispiel in SwingWorker (Java Platform SE 6) an, das ist ähnlich!

 

[Tomate_Salat]

also werd ich um eine eigene version des swingworkers nicht rumkommen?

Nein, erklär erstmal wie du das weiter behandeln willst. Vllt geht es ja auch so mit dem [c]get()[/c] (ob das immernoch hilft, dass die GUI nicht blockiert kann ich dir so ohne selbst zu testen nicht sagen). Wenn dein Thread also warten kann, dann mache es so. Wenn es blockiert, kannst du es immernoch so machen, aber arbeitest aus der [c]done();[/c]-Methode heraus, so wie ich es oben angesprochen hatte.

Edit
@eRaaaa: Setzt vorraus, dass man die Anwendung korregt im EDT startet. Und wie ich in diesem Post zusätzlich angemerkt habe: ich weis nicht, ob das blockieren der GUI verhindert wird. Habe bisher versucht immer zu vermeiden, auf den SwingWorker zurückgreifen zu müssen.

 

[athikka]

neinnein, entshculdige falls ich undeutlich war

ich habe eine schöne uploadfunktion gebaut (eben dieser swingworker oben)
den habe ich fürs forum nur ordentlich gekürzt, also da stecken noch viele feinheiten drinnen (FTP verbindungen usw)

ich hätte jtzt gehofft dass ich diese fileUpload einfach mit nem rückgabewert versehen kann der mir sagt wo die dateien am server jetzt genau liegen (weil ein zufallsordner erstellt wird beim hochladen).

damit ich diese hochgeladenen dateien gleich als links weiterverwenden kann im programm

(eine art rapidshare vom system: ich selektiere dateien, diese werden hochgeladen und ich krieg nen link zurück wo genau sie liegen)

ich hoffe ihr könnt mir folgen

 

[Tomate_Salat]

Ja ich war wohl zu sehr auf das Asynchrone fixiert. Versuche es mal über das [c]get()[/c] wie es eRaaaa hier erwähnt hat:

import java.awt.Dimension;
import java.util.List;
import java.util.concurrent.ExecutionException;

import javax.swing.JFrame;
import javax.swing.JProgressBar;
import javax.swing.SwingUtilities;
import javax.swing.SwingWorker;


public class WorkerDemo
{
    public static void main(String[] args)
    {
        final JFrame frame    = new JFrame();
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        
        final JProgressBar bar    = new JProgressBar();
        bar.setMaximum(10);
        bar.setValue(0);
        bar.setPreferredSize(new Dimension(200, 20));
        
        frame.add(bar);
        
        SwingUtilities.invokeLater(new Runnable()
        {
            @Override
            public void run()
            {
                frame.pack();
                frame.setLocationRelativeTo(null);
                frame.setVisible(true);
            }
        });
        
        SwingWorker<String[], Void> worker    = new SwingWorker<String[], Void>()
        {
            
            @Override
            protected String[] doInBackground() throws Exception
            {
                for(int i=1; i<=10;i++) {
                    try {
                        publish();
                        Thread.sleep(1000);
                    } catch(InterruptedException e) {
                        e.printStackTrace();
                    }
                }
                
                return new String[] {"c:\\asdf.jpg","d:\\jkl.png","c:\\neuer ornder\\doat.java"};
            }
            
            @Override
            protected void process(List<Void> chunks)
            {
                bar.setValue(bar.getValue()+1);
            }
        };
        
        worker.execute();
        
        try {
            String[] data = worker.get();
            for(String d : data)
                System.out.println("UPLOADED FILE: " + d);
        } catch(InterruptedException e) {
            e.printStackTrace();
        } catch (ExecutionException e) {
            e.printStackTrace();
        }        
    }
}

 

[eRaaaa]

ich hätte jtzt gehofft dass ich diese fileUpload einfach mit nem rückgabewert versehen kann der mir sagt wo die dateien am server jetzt genau liegen (weil ein zufallsordner erstellt wird beim hochladen).

damit ich diese hochgeladenen dateien gleich als links weiterverwenden kann im programm

Ja, dann brauchst du ja auch nicht die done-Methode überschreiben?! ;/ rufe nach execute, einfach

return get();

auf anstelle von

return ee;

und gut ists.

fileUpload

darf dann natürlich nicht vom EDT ausgeführt werden, sondern von einem anderen Thread !