Aus Hintergrundthread UI-Thread anpassen

Hallo,

ich mal wieder.

Ich möchte gern im Hintergrund Daten herunterladen (genau genommen muss ich das tun... +knurr+).
Während des Herunterladens der Daten soll ein AlertView schön den Fortschritt des Downloads anzeigen.
(Das können 20 oder mehr MB sein, weshalb so ein Fortschrittsbalken definitiv eine gute Idee ist.)

Ich habe es aktuell so gelöst, dass ich mir eine eigene Klasse DownloadManager gebastelt habe.
Dieser bekommt eine Instanz des Interfaces DownloadManagerDelegate mit und an den kritischen Punkten sendet er seinem Delegate ein paar Informationen.
Die kritischen Punkte wären die Folgenden im Interface definierten Punkte:

public interface DownloadManagerDelegate {
    public void managerDidBeginDownload(DownloadManager downloader, long expectedBytes);
    public void managerDidReceiveData(DownloadManager downloader, byte[] data, long downloadedBytes, long expectedBytes);
    public void managerDidFinishDownload(DownloadManager downloader, long downloadedBytes);
    public void managerDidFailWithError(DownloadManager downloader, Error downloadError);
}

Nun ja, aus irgendwelchen Gründen befindet sich meine als Delegate übergebene Activity nicht im UI-Thread, so dass ich beispielsweise in der Implementierung der managerDidReceiveData(DownloadManager downloader, byte[] data, long downloadedBytes, long expectedBytes); in etwa wie folgt arbeite:

uiThread.post(new Runnable() {
            @Override
            public void run() {
                ProgressBar fileProgressBar = (ProgressBar)downloadDialog.findViewById(R.id.progress);
                TextView fileProgressPercent = (TextView)downloadDialog.findViewById(R.id.progress_percent);
                TextView fileProgressNumber = (TextView)downloadDialog.findViewById(R.id.progress_number);
                fileProgressBar.setProgress(downloaded);


                int percent;
                if(expected <= 0)
                {
                    if(fileProgressBar.getMax() <= 0)
                    {
                        percent = -1;
                    }
                    else
                    {
                        percent = (int)(((downloaded)/(fileProgressBar.getMax()*1.0))*100);
                    }
                }
                else
                {
                    percent = (int)((downloaded/(fileProgressBar.getMax()*1.0))*100);
                }
                String percentString;
                if(percent == -1)
                {
                    percentString = "%nan%";
                }
                else
                {
                    percentString = percent+"%";
                }
                fileProgressPercent.setText(percentString);
                fileProgressNumber.setText(downloaded+"/"+fileProgressBar.getMax());
            }
        });

Wo ich das so einfüge sehe ich, dass ich vielleicht mit dem HolderPattern noch etwas Performance rausholen kann.
Wie dem auch sei, mir ist das alles zu langsam. Der Fortschrittbalken bleibt signifikant lange bei einem Wert, sagen wir '16%' stehen und hüpft dann quasi zu '53%'. Das sieht doof aus, zumal die Delegate-Methode alle 1024 Byte aufgerufen wird. Irgendwo hakt bei mir also die Performance.

Wie ist das unter Android übliche Vorgehen, das UI gemäß Daten aus einem anderen Thread anzupassen?
Mein von iOS bekannter Ansatz des 'Delegating' läuft hier offenbar träge bis gar nicht.
(Das 'Delegating' unter Mac OS / iOS bedient mindestens das Command Muster.)

Zitat von killphil75

Warum nutzt du keinen AsyncTask ??? Dafür sind die doch gebaut worden.

Weil ich just in diesem Moment von deren Existenz erfahre.
Mein Büchlein[1] schweigt sich darüber aus.

Dann bastel ich mal, danke für den Tipp.

(Erst mal soll es funktional sein, Code Cleaning kommt dann im Anschluss, wenn ich mich mit dem Thema einigermaßen auskenne.
Und ich finde, dass ich in den letzten zwei Monaten doch ausreichend gelernt habe, um jetzt damit anfangen zu können.)

--
[1] Thomas Künneth: Android 3 Apps entwickeln mit dem Android SDK, Galileo Computing, 1. Auflage 2011

[killphil]
Danke für den Tipp mit den Views.
Ich werde vermutlich dort mit dem ViewHolder arbeiten, da teste ich sowieso immer vorher, ob die Views noch existieren.
Erschwerend hinzu kommt, dass diese Activity während des Downloads nicht verlassen werden kann.

Zitat von killphil75

Stimmt ich habe das Buch auch hier, der redet nur von Threads und den Rest lässt er unter den Tisch fallen.
sehr unerfreulich.

Insgesamt habe ich das Gefühl, dass der Herr Künneth für Android so ähnlich ist wie der Herr Hinzberger für iOS. Hilfreiche Dinge auslassen, mit Implementierungsdetails verwirren und schlimmstenfalls noch Falsches einbauen. +sigh+

Zitat von ChampS

ich denke der asynctask ist hier in dem fall auch die beste wahl, meiner meinung nach auch nicht so "anspruchsvoll" wie die ios variante

Der Vorteil bei der iOS Variante unter iOS ist einfach, dass du stumpf alles komplett getrennt hältst.
Jetzt habe ich noch locker zwei zusätzliche private Klassen in meiner .java rumfliegen, so dass die Activity selbst unübersichtlicher wird.
Von den gefühlt 30 Imports reden wir gar nicht erst.

Insgesamt finde ich die strukturelle Trennung in Java recht ungünstig gelöst. Das ViewHolder Pattern gibt es in iOS Beispielsweise gar nicht, weil der Controller (ein Objekt) Referenzen zur Speicheradresse aller wichtigen Views aus der View-Hierarchie (ein Objektgraph) hat.
Man könnte stilistisch behaupten, mein findViewById() würde entweder direkt zur Compilezeit fehlschlagen oder niemals. Das ist ja bei Activities nicht so gegeben...

Insofern ist das Delegating dort State-of-the-Art. Die Dereferenzierung des Delegateobjektes und anschließend der einzelnen Views passiert nahezu in notime.
Der Hauptgrund, warum ich es erst versuche (ist ein bekanntes Anwendungsmuster) und dann immer auf die Nase falle. (auf Grund der gegensätzlichen Implementierung von Android läuft das hier nicht.)

[Nur ein bissl blabla, falls mal wer auf die Idee kommt für iOS entwickeln zu wollen. Genauso, wie mir mein iOS Wissen hier nix bringt wird das mit dem Android-Wissen dort sein.]

Dafür mache ich jetzt aber keinen neuen Thread auf...

Also meine DownloadAsyncTask().execute(URL... params); erwartet offenbar eine Liste von mehreren URL.
Jetzt baue ich mir diese mehreren URL an Hand eines Keys aus einem Set zusammen.

Wie bekomme ich dann jetzt diese mehreren URL als Parameterliste übergeben?

Sagen wir der Einfachheit halber:

List<URL> fileUrls = new ArrayList<URL>();
        for(String key: data.keySet())
        {
            String urlString = "http://www.example.com/"+key+"/info.rss";
            URL currentURL = null;
            try {
                currentURL =new URL(urlString);
            }
            catch (MalformedURLException e)
            {
                Log.e("startDownloadOfData", "URL "+urlString+" is malformed");
            }


            if(currentURL != null)
            {
                fileUrls.add(currentURL);
            }
        }


        // Was muss ich nun mit fileURLs machen, damit ich sie als Parameterliste an das Execute übergeben kann?
        new DownloadAsyncTask().execute( <?fileURLs.getItemsAsParameterList()?> );