Übersprungen 130 Frames, zu viel Arbeit auf dem Hauptthread

Question

Ich verstehe, dass Netzwerkaufrufe und SQLite Zugriffe am besten durch Worker-Threads getan werden. In meinem Fall werden meine Netzwerkanrufe durch Okhttp erledigt und die SQLite wird durch einen einzelnen Arbeitsthread erledigt.

Also hier ist mein Problem. Ich habe eine Hauptaktivität, die alle Aktien enthält, die der Benutzer in einer Listenansicht gekauft hat. Jedes Mal, wenn diese neue Aktivität erstellt wird, durchläuft die Methode onCreate die Bestände in der Datenbank und aktualisiert sie einzeln und aktualisiert dann die Listenansicht.

Dies belastet natürlich den Hauptfaden zu sehr. Aber mein Problem ist, wie kann ich das besser gestalten? Im Moment habe ich so etwas wie dies im onCreate:

Thread t1 = new Thread(new Runnable{ 
    @Override 
    public void run(){ 
     // access sqlite 
     positionsArray = retrieveStocks(); 
    }  
}); 
t1.start(); 
t1.join(); //try catch omitted 

Thread t2 = new Thread(new Runnable{ 
    @Override 
    public void run(){ 
     // use Okhttp to get data 
     positionsArray = refreshStocks(); 
    }  
}); 
t1.start(); 
t2.join(); //try catch omitted 

adapter = new PositionsAdapter(this, positionsArray); 
listview.setAdapter(adapter); 

Das ist nicht das Problem weggehen nicht machen, weil der Haupt-Thread noch für t1 und t2 warten. Irgendwelche Designvorschläge, wie ich dieses Problem beheben kann?

Answer 1

Sie sollten nie thread.join() im ui-Thread aufrufen, da dies die App pausiert. Sie könnten eine Asynchron-Aufgabe verwenden diese AsyncTask Android example

ProgressDialog progress; 
private class DoNetworking extends AsyncTask<Void, Void, PosArray> { 

    @Override 
    protected PosArray doInBackground() { 
     positionsArray = retrieveStocks(); 
     positionsArray = refreshStocks(); 
     PosArray positionsArray; 
    } 

    @Override 
    protected void onPostExecute(PosArray positionsArray) { 
     adapter = new PositionsAdapter(this, positionsArray); 
     listview.setAdapter(adapter); 
     progress.dismiss(); 
    } 
} 
progress = ProgressDialog.show(this, "dialog title","dialog message", true); 
new DoNetworking().execute()