Classe per fare una richiesta POST

Premetto che ho cercato un bel po' su internet prima di scrivere qui.
Ciò di cui ho bisogno è una classe che prende come parametri l'URL e i parametri che devono essere inviati tramite POST, ed espone un metodo pubblico che esegue la richiesta POST e ritorna una String con il contenuto restutuito dal server.

<br /> PostReq pr= new PostReq(url,parametri);<br /> String risposta = pr.getRisposta();<br />

Attraverso gli esempi trovati su internet sono riuscito ad eseguire richieste POST e leggerne il contenuto, ma questi metodi si trovavano all'internto dell'activity. Io voglio creare una classe che una volta istanziata esegua questa operazione in asincrono, senza che ogni volta sia necessario scrivere tutto il codice.

In c# per WP ho già fatto un paio di app in questo modo. Per android non ci sono ancora riuscito. Facendo un paragone al c# lì ho il codice che istanzia la classe e la utilizza è questo (notare l'utilizzo di async e await :cool: ):

<br /> async void button_click() {<br /> PostReq pr= new PostReq(url,parametri);<br /> String risposta = await pr.getRisposta();<br /> }<br />

Suggerimenti?

Ciao. Io ho fatto il mese scorso una classe con le funzionalità che ti servono e con un minimo di impegno é sicuramente possibile estenderla alle chiamate get e forse anche alle put e delete.

Contattami in pvt che cosi mi ricordo di spiegarti la cosa :D


Inviato dal mio Galaxy Nexus usando Androidiani App

Qualcun'altro che mi possa aiutare?

Sono arrivato a questo punto:

codice:

---

   
 PostReq postreq= new PostReq();
      postreq.execute(dict);

      String ciao="";
      while(ciao==""){ ciao= postreq.contenuto;}
     
System.out.println("rispostaServer "+ ciao);

---

PostReq è la classe che esegue la richiesta Post al server in modo asincrono. dict è il List<NameValuePair> , cioè è una lista che contiene l'url e i parametri post da inviare al server.
Il problema è che se non metto quel while, la funzione di print stampa la variabile ciao come vuota perchè non ha atteso che il thread postreq.execute(dict) abbia completato l'operazione.
Ho forzato quindi l'attesa con un while ma quel while blocca ovviamente il thread principale. Il problema adesso è come far attendere che il metodo sia stato eseguito e quindi assegni il valore alla variabile ciao in modo asincrono.

AsyncTask oppure future


Inviato dal mio Nexus 7 usando Androidiani App

ok, ma .. un esempio pratico di come posso usarlo nel mio caso?

guarda che google non morde e con la query di ricerca "android asynctask" questo è il primo risultato --> AsyncTask | Android Developers
Oltre che essere la reference ufficiale c'è anche un esempio piuttosto completo per l'utilizzo degli asynctask, l'esempio non è proprio il tuo caso ma ci si avvicina parecchio

Già vista e rivista quella pagina.. ma non risolto ancora :/

ma scusa, li c'è il migliore esempio che puoi trovare per un asynctask....
nell'onPreExecute crei la richiesta, nel doInBackground fai la richiesta (questo metodo è asincrono per cui non devi sbatterti te a gestire il ciclo vitale del thread o del runnable o di una qualsiasi altra forma di programmazione concorrente presente in java/android) se vuoi durante il doInBackground chiami onProgressUpdate per aggiornare una tua UI che notifica all'utente che la tua app non è morta e poi appena finito il doInBackground viene chiamato il onPostExecute che riceve come parametro il risultato di doInBackground.

E per citare Snoop Dog in "Starsky & Hutch": Io te l'ho cantata adesso stà a te suornarla

forse non ci siamo capiti.. io ho già creato una classe che esegue la richiesta asincrona con asynctask.. ma è una classe ESTERNA.. ho solo bisogno che il thread chiamante (il metodo che si trova nel mainacitivy, per capirci) non si bloccchi mentre attende la risposta, mi serve quel che fa' await in c#, lo stesso in java..

posta l'intero codice della classe asynctask ed il metodo che chiama l'asynctask

PostReq.java -->

codice:

---

class PostReq extends AsyncTask<List<NameValuePair>, Integer, String>{
        String contenuto="";
@Override
public String doInBackground(List<NameValuePair>... params) {
//TODO Auto-generated method stub
postData(params[0]);
return contenuto;
}

protected void onPostExecute(String result){
//Toast.makeText(null, "command sent", Toast.LENGTH_LONG).show();
       
}


public void postData(List<NameValuePair> dict) {
        String url= dict.get(0).getValue();
        dict.remove(0);
HttpClient httpclient = new DefaultHttpClient();
HttpPost httppost = new HttpPost(url);

try {

httppost.setEntity(new UrlEncodedFormEntity(dict));

HttpResponse response = httpclient.execute(httppost);
contenuto= inputStreamToString(response.getEntity().getContent());

System.out.println("risposta "+ contenuto);

} catch (ClientProtocolException e) {
        System.out.println("risposta errorecl");

//TODO Auto-generated catch block
} catch (IOException e) {
        System.out.println("risposta erroreio");
//TODO Auto-generated catch block
}
}

private String inputStreamToString(InputStream is) {
    String s = "";
    String line = "";
    BufferedReader rd = new BufferedReader(new InputStreamReader(is));
    try {
                while ((line = rd.readLine()) != null) { s += line; }
        } catch (IOException e) {
                e.printStackTrace();
        }
    return s;
}
}

---

E questo è il metodo che si trova nell'activityMain, eseguito quando l'utente clicca sul button -->

codice:

---

public void btnWeb_Click(View view){
        String url="http://192.168.1.14/test/index.php";
 
      List<NameValuePair> dict = new ArrayList<NameValuePair>();
      dict.add(new BasicNameValuePair("url",url));
      dict.add(new BasicNameValuePair("id", "contDelPost"));
     
      PostReq postreq= new PostReq();
      postreq.execute(dict);

      String result="";
      while(result==""){ result= postreq.contenuto;}
     
      System.out.println(result);

}

---

Ho tolto tutti gli import dalla classe PostReq, basta premere SHIFT + CTRL + O im eclipse per importarli automaticamente

Caro mio mi sa che devi ancora capire cosa c'è scritto nella documentazione

Questa implementazione è fatta a botto e quindi possono esserci degli errori di battitura e deve essere ottimizzata ma in linea di massima è così

codice:

---

public class PostReq extends AsyncTask<NameValuePair, Void, String> {

        private MainActivity ma;
        private String url;
        private String contenuto = null;
        private HttpClient httpClient;
        private HttpPost httpPost;

        public PostReq(MainActivity ma, String url) {
                this.ma = ma;
                this.url = url;
        }

        @Override
        public String doInBackground(NameValuePair... params) {
                try {
                        // imposto i parametri che NON SONO FILE
                        httpPost.setEntity(new UrlEncodedFormEntity(Arrays.asList(params)));
                        // effettuo la richiesta ed ottengo la risposta
                        HttpResponse response = httpClient.execute(httpPost);
                        // TODO Cambiati tu poi le cose in base agli status code che ottieni
                        // se ho una risposta ok
                        if (response.getStatusLine().getStatusCode() == 200) {
                                //
                                contenuto = inputStreamToString(response.getEntity().getContent());
                        } else {
                                contenuto = "Capitato un errore " + response.getStatusLine().getStatusCode();
                        }
                } catch (ClientProtocolException e) {
                        System.err.println("Errore ClientProtocolException");
                        e.printStackTrace();
                } catch (IOException e) {
                        System.err.println("Errore IOException");
                        e.printStackTrace();
                }
                // restituisco il risultato
                return contenuto;
        }

        protected void onPostExecute(String result) {
                Toast.makeText(ma, "Risposta: " + result, Toast.LENGTH_LONG).show();
                // TODO Qua ti fai i compitini a inserire la tua risposta nell'activity
        }

        protected void onPreExecute() {
                httpClient = new DefaultHttpClient();
                httpPost = new HttpPost(url);
        }

        private String inputStreamToString(InputStream is) {
                StringBuilder s = new StringBuilder();
                BufferedReader rd = new BufferedReader(new InputStreamReader(is));
                String line;
                try {
                        while ((line = rd.readLine()) != null) {
                                s.append(line).append("\n");
                        }
                } catch (IOException e) {
                        e.printStackTrace();
                }
                return s.toString();
        }
}

---

codice:

---

        public void btnWeb_Click(View view) {
                String url = "http://192.168.1.14/test/index.php";

                PostReq postreq = new PostReq(this, url);
                postreq.execute(new BasicNameValuePair("id", "contDelPost"));
        }

---

ehmm.. cioè? la mia classe PostReq funzionava! Non è mica quella che va modificata.. o_O
il problema è nel meotodo

codice:

---

public void btnWeb_Click(View view){
        String url="http://192.168.1.14/test/index.php";
 
      List<NameValuePair> dict = new ArrayList<NameValuePair>();
      dict.add(new BasicNameValuePair("url",url));
      dict.add(new BasicNameValuePair("id", "contDelPost"));
     
      PostReq postreq= new PostReq();
      postreq.execute(dict);

      String result="";
      while(result==""){ result= postreq.contenuto;}
     
      System.out.println(result);

}

---

che non attende asincronicamente la risposta del server e per farlo devo usare un while che blocca tutto finche il server non ha risposto.. ho bisogno che l'oggetto postreq una volta eseguito restituisca il contenuto ottenuto dal server come stringa nel mainActivity senza bloccare tutto........

Quote:

---

Originariamente inviato da jejpeeg

ehmm.. cioè? la mia classe PostReq funzionava! Non è mica quella che va modificata.. o_O
il problema è nel meotodo

codice:

---

public void btnWeb_Click(View view){
        String url="http://192.168.1.14/test/index.php";
 
      List<NameValuePair> dict = new ArrayList<NameValuePair>();
      dict.add(new BasicNameValuePair("url",url));
      dict.add(new BasicNameValuePair("id", "contDelPost"));
     
      PostReq postreq= new PostReq();
      postreq.execute(dict);

      String result="";
      while(result==""){ result= postreq.contenuto;}
     
      System.out.println(result);

}

---

che non attende asincronicamente la risposta del server e per farlo devo usare un while che blocca tutto finche il server non ha risposto.. ho bisogno che l'oggetto postreq una volta eseguito restituisca il contenuto ottenuto dal server come stringa nel mainActivity senza bloccare tutto........

---

Non entro nel merito del codice che hai scritto, dando per scontato che sia tutto corretto...
Come metodo da seguire basta seguire queste regole:
1) la prima parte (quella che crea e lancia l'AsyncTask) mi sembra corretta
2) i valori che devi ricavare dal server li ricavi nel metodo doInBackground o da altri metodi richiamati dallo stesso (cosa che mi sembra hai fatto). Tutto ciò è eseguito in un thread distinto;
3) da doInBackground passi il tuo risultato a onPostExecute con return
4) onPostExecute viene eseguito nel thread di esecuzione della GUI. Quindi da lì potresti anche modificare direttamente la tua interfaccia visualizzando il risultato in qualche componente o aggiornando il valore delle tue variabili.

NB. Se la classe che estende AsyncTask (PostReq) è una classe innestata nella classe principale, da essa dovrebbe essere possibile accedere direttamente a tutte le variabili di istanza della classe esterna. Quindi puoi impostare direttamente il valore delle tue variabili.
Se invece PostReq è una top level class (come sembrerebbe nel tuo caso), allora per modificare il valore delle tue variabili nella classe di partenza o le dichiari pubbliche (metodo sconsigliato), oppure passi i valori tramite un interfaccia. Io userei una classe innestata, che è il metodo più semplice.

In ogni caso non serve alcun loop nel metodo di partenza.

Nel tuo caso secondo me l'errore è che nel metodo di partenza cerchi di ricavare il risultato prima ancora che questo sia stato recuperato dal tuo AsyncTask.
La logica è che è AsyncTask che comunica il risultato alla classe di partenza quando lo ha ricavato, come detto sopra; non è la classe di partenza che continua ad interrogare l'AsyncTask fino a quando non è stato ottenuto il risultato.

Spero di essere stato chiaro.

Ok, grazie :)
Anche se la classe innestata è proprio quello che volevo evitare.. perchè vorrebbe dire che se ho 3 o 4 Activity che generano richieste post, dovrei riscrivere la classe anche in quelle activity. E' per questo che cercavo qualcosa come AWAIT che ho in c#