2012-08-06 10 views
22

Mam Asynctask, który pobiera dwa int i jak chcę je przekazać do onPostExecute, aby pokazać je w widoku.
Oto mój kod:Asynctask: przekazanie dwóch lub więcej wartości z doInBackground do onPostExecute

public class QueryServer extends AsyncTask <String, Void, Integer> { 

    protected Integer doInBackground(String... serverAddress) { 
     Log.d("QueryServer", ""+serverAddress[0]); 
     MCQuery mcQuery = new MCQuery("" + serverAddress[0] ,25565); 
     QueryResponse response = mcQuery.basicStat(); 

     int Onlineplayers = response.getOnlinePlayers(); //first vaule 
     int Maxplayers = response.getMaxPlayers(); //second vaule 

     Log.d("MCQuery", "" + Onlineplayers + " OnlinePlayers"); 
     return Onlineplayers; 

    } 

    protected void onPostExecute(Integer Onlineplayers){ 

     TextView onlinePlayersView = (TextView) findViewById(R.id.online_players); 

     onlinePlayersView.setText(""+Onlineplayers+"/"+ Maxplayers); //i need to pass Maxplayers to use it here 


    } 

Z góry dziękuję.

Odpowiedz

54

Można zdefiniować klasy otoki, która posiada dwie liczby całkowite:

public class Wrapper 
{ 
    public int onlinePlayers; 
    public int maxPlayers; 
} 

i używać go zamiast Integer:

public class QueryServer extends AsyncTask<String, Void, Wrapper> { 

    protected Wrapper doInBackground(String... serverAddress) { 
     Log.d("QueryServer", ""+serverAddress[0]); 
     MCQuery mcQuery = new MCQuery("" + serverAddress[0] ,25565); 
     QueryResponse response = mcQuery.basicStat(); 

     int onlinePlayers = response.getOnlinePlayers(); //first vaule 
     int maxPlayers = response.getMaxPlayers(); //second vaule 

     Log.d("MCQuery", "" + onlinePlayers + " onlinePlayers"); 
     Wrapper w = new Wrapper(); 
     w.onlinePlayers = onlinePlayers; 
     w.maxPlayers = maxPlayers; 
     return w; 

    } 

    protected void onPostExecute(Wrapper w){ 

     TextView onlinePlayersView = (TextView) findViewById(R.id.online_players); 

     onlinePlayersView.setText(""+w.onlinePlayers+"/"+ w.maxPlayers); //i need to pass Maxplayers to use it here 


    } 
+3

Świetna odpowiedź! dzięki – Harry

+0

Dziękuję, proste i skuteczne –

2

Najprostszym sposobem jest po prostu zadeklarować niewielki obiekt kontenera z pola, które chcesz zwrócić, a następnie zwróć instancję z pola doInBackground:

private class QueryResult { 
    int onlinePlayers; 
    int maxPlayers; 

    public QueryResult(int onlinePlayers, int maxPlayers) { 
     this.onlinePlayers = onlinePlayers; 
     this.maxPlayers = maxPlayers; 
    } 
} 

protected QueryResult doInBackground(String... serverAddress) { 
    // ... 

    return new QueryResult(onlinePlayers, maxPlayers); 
} 
3
public class QueryServer extends AsyncTask <String, Void, Integer>{...} 

Wymień Integer rodzajowe param do ArrayList<Integer> lub Integer[]

ten sposób Państwa doInBackground() będzie wyglądać następująco:

protected Integer[] doInBackground(String... serverAddress) 
{ 
    ...do what you need to do... 
    //Then create an array, fill with data, and return. 
    Integer[] arr = new Integer[10]; 
    for(int i=0; i<10; i++) 
     arr[i] = i; //just an example 
    return arr; 
} 

A w swojej onPostExecute()

protected void onPostExecute(Integer [] Onlineplayers) 
{ 
    //Do whatever you want with your array 
} 
0

Czy próbowałeś podejmowania tych dwóch intów członków QueryServer?

onPostExecute() nie działa do momentu zakończenia doInBackground(), więc nie powinno być żadnych problemów z gwintowaniem.

0

Ogólnie mówiąc, zamień Integer na ArrayList of Objects. z tą tablicą możesz przekazać dowolną kolekcję dowolnego typu. ale oczywiście wadą jest to, że powinieneś obsadzić członków i powinieneś znać wszystkie typy ...

Powiązane problemy