2012-08-06 697 views
22

Tengo una AsyncTask que recupera dos vaules int y quiero pasarlos a onPostExecute para mostrarlos en la vista.
Aquí está mi código:AsyncTask: pase de dos o más valores de doInBackground a 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 


    } 

gracias de antemano.

Respuesta

54

Se puede definir una clase de contenedor que contiene dos enteros:

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

y utilizar en su lugar de 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

gran respuesta! gracias – Harry

+0

Gracias, simple y eficaz –

2

La forma más sencilla es simplemente declarar un objeto pequeño recipiente con la campos que desea devolver, luego devuelva una instancia de 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>{...} 

Vuelva a colocar la Integer parámetro genérico para ArrayList<Integer> o para Integer[]

De esta manera su doInBackground() se verá así:

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; 
} 

Y el en su onPostExecute()

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

Ha intentado hacer esos dos integrantes miembros de QueryServer?

onPostExecute() no se ejecuta hasta después doInBackground() ha terminado, por lo que no debería haber ningún problema de roscado.

0

En términos generales, en lugar de Integer con ArrayList of Objects. con esta matriz, puede pasar cualquier colección de cualquier tipo. pero, obviamente, el inconveniente es que debe emitir los miembros y usted debe saber todos los tipos ...

Cuestiones relacionadas