2012-08-06 20 views
22

Ho un asynctask che recupera due vault int e voglio passarli a onPostExecute per mostrarli nella vista.
Ecco il mio codice:Asynctask: passa due o più valori da 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 


    } 

Grazie in anticipo.

risposta

54

È possibile definire una classe wrapper che contiene due interi:

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

e usarlo al posto di 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

ottima risposta! grazie – Harry

+0

Grazie, semplice ed efficace –

2

Il modo più semplice è semplicemente dichiarare un oggetto contenitore piccolo con la campi che si desidera restituire, quindi restituire un'istanza di quello da 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>{...} 

Sostituire il parametro generica Integer-ArrayList<Integer> o Integer[]

questo modo il vostro doInBackground() sarà simile a questa:

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

E la nella vostra onPostExecute()

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

Hai provato fare quei due membri inte di QueryServer?

onPostExecute() non viene eseguito fino a dopo doInBackground() è finito, quindi non ci dovrebbero essere problemi di threading.

0

In generale, sostituire Integer con ArrayList of Objects. con questo array, puoi passare qualsiasi raccolta di qualsiasi tipo. ma ovviamente, il rovescio della medaglia è che si dovrebbe cast membri e si dovrebbe conoscere tutti i tipi ...

Problemi correlati