Exécuter asynctask de façon synchrone ou alors faire une temporisation

Version imprimable

Voir 40 message(s) de cette discussion en une page

Bonjour,

Pour mon projet scolaire j'ai besoin de récupérer des données GPS et maintenant depuis les versions récentes on est obligé de passer par une méthode AsyncTask mais, le problème c'est le fait que cela soit asynchronque car lors ce que je lance la classe il continue à faire la suite et en fond de tache il exécute la Classe alors que j'ai besoin des résultats obtenue par elle même directement après son exécution..

Voici le code d'éxécution..
Code:

1 2 3 4 5 6 7 //execution de la requete post pour obtenir l'id et le numéro du repsonsable objPostIdNum.execute(numTel, MDP); Toast.makeText(getApplicationContext(), objPostIdNum.result, Toast.LENGTH_LONG).show(); // LE RESULTAT N'EST PAS ENCORE CONNU //faire tempo, CA NE MARCHE PAS...
Merci

04/05/2015, 14h18
Hephaistos007

Parce que ce n'est pas comme cela qu'une AsyncTask s'utilise. En gros, il faut positionner ton code Toast.makeText(getApplicationContext(), objPostIdNum.result, Toast.LENGTH_LONG).show(); dans une méthode de ton choix dite de "callback", c-a-d qui serait appelée seulement quand l'AsyncTask aura terminé son travail. Explications ici.
05/05/2015, 14h10
mailoy

le problème pour moi c'est justement il faut que la suite s'intérprete seulement si le asynTask est terminé, alors on peux l'exécuter en synchrone ?

Merci
05/05/2015, 14h44
Hizin

Et si, au lieu de tenter de forcer à faire rentrer un objet triangle dans un trou carré, tu changeais pour utiliser l'outil comme il le faut ? :)

Change ton code pour réaliser ce que Hephaistos007 t'a conseillé, c'est la seule bonne manière de faire.
i.e. : comprend la notion de callback et bouge ton code dans le callback de l'AynsTask :)

Citation:

Envoyé par Hizin

Et si, au lieu de tenter de forcer à faire rentrer un objet triangle dans un trou carré, tu changeais pour utiliser l'outil comme il le faut ? :)

Change ton code pour réaliser ce que Hephaistos007 t'a conseillé, c'est la seule bonne manière de faire.
i.e. : comprend la notion de callback et bouge ton code dans le callback de l'AynsTask :)

Tout le code de mon postData il faut que le mette dans le onPostExecute c'est ça ?

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
 @Override
    public String doInBackground(String... params) {
        // TODO Auto-generated method stub
        postData(params[0], params[1], params[2], params[3]);
        return null;
    }
 
    public void onPostExecute() {
 
        //Toast.makeText(getApplicationContext(), objPostIdNum.result, Toast.LENGTH_LONG).show();
    }
 
 
    public String postData(String precisionP, String dateP, String positionPLa, String positionPLo) {
 
 
        try {
            adresseUrlPost = "http://192.168.107.14:146/api/PositionGPS/42202b162";
 
            // 1. Création de  HttpClient
            HttpClient httpclient = new DefaultHttpClient();
 
            // 2. créer la requete POST en lui donnant l'URL
            HttpPost httpPost = new HttpPost(adresseUrlPost);
 
            String json = "";
 
            // 3. Construction d'un jsonObject
            JSONObject jsonObject = new JSONObject();
            jsonObject.accumulate("PrecisionPosition", precisionP);
            jsonObject.accumulate("dateHeurePosition", dateP);
            jsonObject.accumulate("Position.Latitude", positionPLa);
            jsonObject.accumulate("Position.Longitude", positionPLo);
 
 
            // 4. Convertion JSONObject de JSON en String
            json = jsonObject.toString();
 
            // ** Alternative way to convert Person object to JSON string usin Jackson Lib
            // ObjectMapper mapper = new ObjectMapper();
            // json = mapper.writeValueAsString(person);
 
            // 5. set json to StringEntity
            StringEntity se = new StringEntity(json);
 
            // 6. set httpPost Entity
            httpPost.setEntity(se);
 
            // 7. Set some headers to inform server about the type of the content
            httpPost.setHeader("Accept", "application/json");
            httpPost.setHeader("Content-Type", "application/json");
 
 
            // 8. Executer une requete POST en lui donnant l' URL
            HttpResponse httpResponse = httpclient.execute(httpPost);
 
            // 9. receive response as inputStream
            inputStream = httpResponse.getEntity().getContent();
 
 
            // 10. convert inputstream to string
            if (inputStream != null)
                result = convertInputStreamToString(inputStream);
            else
                result = "probleme, ne fonctionne pas";
 
        } catch (Exception e) {
            Log.d("InputStream", e.getLocalizedMessage());
        }
 
        // 11. return result
        return result;
    }

06/05/2015, 09h40
Hizin

C'est une possibilité.

Tu peux aussi passer par l'architecture "Observer" pour simplement dire à l'observateur que le traitement est fini :)

Documentation : http://design-patterns.fr/observateur
06/05/2015, 10h07
mailoy

Citation:

Envoyé par Hizin

C'est une possibilité.

Tu peux aussi passer par l'architecture "Observer" pour simplement dire à l'observateur que le traitement est fini :)

Documentation : http://design-patterns.fr/observateur

Oui mais alors lors de l'éxécution comment je fais pour faire en sorte que cela ne continue pas à interpréter la suite tant que justement mon asyncTask n'est pas terminé ?

Merci
06/05/2015, 11h52
Hephaistos007

Citation:

Envoyé par mailoy

Tout le code de mon postData il faut que le mette dans le onPostExecute c'est ça ?

Non. Regardes ce que dis le tuto, crées une méthode populate() dans ton Activity qui fera le toast. Cette méthode sera appelée depuis le onPostExecute() de l'AsyncTask.

Citation:

Envoyé par Hephaistos007

Non. Regardes ce que dis le tuto, crées une méthode populate() dans ton Activity qui fera le toast. Cette méthode sera appelée depuis le onPostExecute() de l'AsyncTask.

je crois que tu n'as pas compris..
Code:

1 2 3 4 5 6 7 8 //execution de la requete post pour obtenir l'id et le numéro du repsonsable objPostIdNum.execute(numTel, MDP); // faire ça en bloquant la suite // vue que objPostIdNum.result="" alors // bloquer, tant que le resultat de AsyncTaskPostId==null // ou alors faire tempo faire tempo // Ici j'ai besoin de la valeur de objPostIdNum.result=""

07/05/2015, 13h56
Hephaistos007

Si si, j'ai parfaitement compris. Soit tu t'entêtes à utiliser n'importe comment un AsyncTask, soit tu fais ce que je t'ai conseillé.
11/05/2015, 11h08
mailoy

Citation:

Envoyé par Hephaistos007

Si si, j'ai parfaitement compris. Soit tu t'entêtes à utiliser n'importe comment un AsyncTask, soit tu fais ce que je t'ai conseillé.

Je ne vois pas comment justement, j'ai lu le cours-tuto que tu m'as envoyé et sincérement cela ne montre pas comme exécuter un asyncTask en sync... justement c'est plutot l'inverse à se que j'ai compris..
11/05/2015, 11h59
Hizin

Citation:

Envoyé par mailoy

[...] cela ne montre pas comme exécuter un asyncTask en sync

Donc tu t'entêtes :D
C'est normal que le tuto ne le montre pas, puisque :
- c'est inepte
- c'est une mauvaise pratique
- c'est, comme je l'ai écrit, tenter de faire rentrer un carré dans un rond

Pour la énième fois : intéresse-toi au mécanisme de callback.
Tu ne dois en aucun cas transformer ton AysncTask en tâche synchrone, mais tu dois faire en sorte que le traitement suivant soit déclenché par l'AsyncTask une fois que celle-ci s'est terminée. Il faut que tu appelles la suite de ton traitement dans le onPostExecute de ton AsyncTask, et pour ça, c'est typiquement un mécanisme de Callback avec un patron de conception (design pattern) Observateur (Observer) :)
15/05/2015, 11h21
mailoy
merci pour vos tuto je crois que j'ai compris :), mais pas sur que ce soit la meilleure solution ce que je fait..

Je retourne la valeur du resultat dans mon onPostExecute()
Code:

1 2 3 public void onPostExecute() { getretour(); }
et de l'autre coté je test si j'ai reçu la réponse ou pas .. ça va ou pas et tant que j'ai pas la réponse bah j'attend.. ?

J'interviens rapidement.... sur le terme: "ben j'attends"

Non... le programme n'attends pas... un programme ne dois jamais attendre. Un programme réagit à des événements:
* La fenetre est affiché ? => le programme démarre une tâche de récupération de données.
* Les données sont arrivées => le programme affiche les données et permet à l'utilisateur d'interagir avec.
* L'utilisateur a interagi avec les données => le programme fait ce qu'il a à faire avec ces nouvelles données.

A aucun moment le programme "n'attends" explicitement.

Par exemple: j'ai besoin pour envoyer un message d'une position GPS:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 onCreate: * On affiche l'interface de message * On démarre "l'écoute" de localisation ("onLocationUpdated") * On setup le lien bouton "envoyer" avec "onButtonPressed" * On récupère la dernière position connue (et valide). Si pas disponible, on grise le bouton "envoyer" (avec éventuellement une progress bar pour dire à l'utilisateur qu'on attend un truc). onButtonPressed * On envoie le message avec la dernière localisation connue (elle est forcément valide à ce moment). onLocationUpdated * On met à jour la dernière localisation connue. Si nouvelle, on rend disponible le bouton "envoyer" (et éventullement on enleve la progress bar).
Exemple avec un appel à un webservice entre l'edit d'un champ, et l'envoi du message:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 onCreate: * On affiche l'interface de message. * on setup le lien edit-text "destinataire" avec "onRecipientChange" * on setup le lien bouton "envoyer" avec "onSend" * on interdit l'utilisation du bouton "envoyer" (tant qu'on n'a pas le destinataire) onRecipientChange * on desactive le bouton envoyer * Si on a déjà une tâche de récupération du tel, on l'annule (cancel(true)) * On démarre une nouvelle tâche de récupération du tel. Quand celle-ci se termine (onPostExecute) elle appelera onRecipientNumRetrieved onRecipientNumReceived(num) * on met à jour le tel stocké dans l'activité * on réactive le bouton envoyer onSend * ici on a forcément le tel stocké dans l'activité correspondant au recipient.

Citation:
Envoyé par nicroman
J'interviens rapidement.... sur le terme: "ben j'attends"

Non... le programme n'attends pas... un programme ne dois jamais attendre. Un programme réagit à des événements:
* La fenetre est affiché ? => le programme démarre une tâche de récupération de données.
* Les données sont arrivées => le programme affiche les données et permet à l'utilisateur d'interagir avec.
* L'utilisateur a interagi avec les données => le programme fait ce qu'il a à faire avec ces nouvelles données.

A aucun moment le programme "n'attends" explicitement.

Par exemple: j'ai besoin pour envoyer un message d'une position GPS:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 onCreate: * On affiche l'interface de message * On démarre "l'écoute" de localisation ("onLocationUpdated") * On setup le lien bouton "envoyer" avec "onButtonPressed" * On récupère la dernière position connue (et valide). Si pas disponible, on grise le bouton "envoyer" (avec éventuellement une progress bar pour dire à l'utilisateur qu'on attend un truc). onButtonPressed * On envoie le message avec la dernière localisation connue (elle est forcément valide à ce moment). onLocationUpdated * On met à jour la dernière localisation connue. Si nouvelle, on rend disponible le bouton "envoyer" (et éventullement on enleve la progress bar).
Exemple avec un appel à un webservice entre l'edit d'un champ, et l'envoi du message:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 onCreate: * On affiche l'interface de message. * on setup le lien edit-text "destinataire" avec "onRecipientChange" * on setup le lien bouton "envoyer" avec "onSend" * on interdit l'utilisation du bouton "envoyer" (tant qu'on n'a pas le destinataire) onRecipientChange * on desactive le bouton envoyer * Si on a déjà une tâche de récupération du tel, on l'annule (cancel(true)) * On démarre une nouvelle tâche de récupération du tel. Quand celle-ci se termine (onPostExecute) elle appelera onRecipientNumRetrieved onRecipientNumReceived(num) * on met à jour le tel stocké dans l'activité * on réactive le bouton envoyer onSend * ici on a forcément le tel stocké dans l'activité correspondant au recipient.
Désolé mais clairement je ne comprend pas grand chose...

Après des tonnes de recherches il est vrai que j'ai trouvé des solution bien plus propre en créant une interface etc...
Mais j'ai trouvé quelque de rapide mais qui me parait pas top j'aimerais votre avis .
Code:

1 2 3 4 MyAsyncTaskPostIdNum objPostIdNum = new MyAsyncTaskPostIdNum(); //execution de la requete post pour obtenir l'id et le numéro du repsonsable try { objPostIdNum.execute(numTel, "cdfIris2015").get();
Le get va bloquer le thread du main temps qu'il n'a pas la réponse.. du coup ça met en arret le main mais au moins il ne fait pas la suite le temps qu'il n'a pas la réponse..

Je demande votre avis parce que j'ai trouvé plus propre ici :
https://learnandroidtoday.wordpress....of-async-task/
ou
http://www.codeproject.com/Questions...k-in-main-thre

Merci

25/05/2015, 19h14
Hephaistos007

C'est fort, c'est très fort :mouarf:

Le tuto que tu cites https://learnandroidtoday.wordpress....of-async-task/ expliques justement que le get() bloques l'interface (ce qui est une erreur) et donc propose en réponse ... exactement le même pattern (processFinish() est totalement équivalent à populate()) avec en plus une interface (au sens Java du terme) pour faire plus propre.

Je vais être très clair : ta solution, celle à laquelle tu tiens tant (on ne sait pas vraiement pourquoi) n'existe pas. Ce n'est pas une possibilité, ni même une simplification, c'est juste du code qui ne marche pas et ne marchera jamais. Nous sommes plusieurs à te l'avoir dit. Si tu veux rester bloquer 5 ans sur la même erreur, c'est ton choix mais nous clôturons ce fil de discussion.
25/05/2015, 20h40
mailoy

Citation:

Envoyé par Hephaistos007

C'est fort, c'est très fort :mouarf:

Le tuto que tu cites https://learnandroidtoday.wordpress....of-async-task/ expliques justement que le get() bloques l'interface (ce qui est une erreur) et donc propose en réponse ... exactement le même pattern (processFinish() est totalement équivalent à populate()) avec en plus une interface (au sens Java du terme) pour faire plus propre.

Je vais être très clair : ta solution, celle à laquelle tu tiens tant (on ne sait pas vraiement pourquoi) n'existe pas. Ce n'est pas une possibilité, ni même une simplification, c'est juste du code qui ne marche pas et ne marchera jamais. Nous sommes plusieurs à te l'avoir dit. Si tu veux rester bloquer 5 ans sur la même erreur, c'est ton choix mais nous clôturons ce fil de discussion.

Je vais alors faire la solution du tuto et si je suis bloqué à un endroit je viendrais poser des questions

Merci :)
25/05/2015, 21h26
mailoy

Citation:

Envoyé par Hephaistos007

C'est fort, c'est très fort :mouarf:

Le tuto que tu cites https://learnandroidtoday.wordpress....of-async-task/ expliques justement que le get() bloques l'interface (ce qui est une erreur) et donc propose en réponse ... exactement le même pattern (processFinish() est totalement équivalent à populate()) avec en plus une interface (au sens Java du terme) pour faire plus propre.

Je vais être très clair : ta solution, celle à laquelle tu tiens tant (on ne sait pas vraiement pourquoi) n'existe pas. Ce n'est pas une possibilité, ni même une simplification, c'est juste du code qui ne marche pas et ne marchera jamais. Nous sommes plusieurs à te l'avoir dit. Si tu veux rester bloquer 5 ans sur la même erreur, c'est ton choix mais nous clôturons ce fil de discussion.

J'ai fais comme le tuto même si j’avoue ne pas avoir tout compris.. pas d’erreur mais malheureusement je ne vois pas de différence avec la récupération de base.. au final il fait la suite sans forcement avoir la réponse du AsyncTask... donc la je suis bien embêté..

pouvez vous m'expliquer se que fait finalement la technique utilisée dans ce tuto ?

Merci
25/05/2015, 21h28
Hephaistos007

Si tu suis le tuto, tu as la garantie que tout le code mis dans populate() sera fait après la fin de l'AsyncTask. Tu peux y mettre ton Toast et tout le code que tu veux...
25/05/2015, 21h33
mailoy

Citation:

Envoyé par Hephaistos007

Si tu suis le tuto, tu as la garantie que tout le code mis dans populate() sera fait après la fin de l'AsyncTask. Tu peux y mettre ton Toast et tout le code que tu veux...

justement .. au final cela ne change rien à la méthode de reception basic.... car la suite du code est effectué dans une autre méthode qui elle est appelée lors ce que la asyntask est terminée..

Mais moi j'ai besoin d’effectuer le code qui est à la suite du lancement de l'asyncTask qui au final ce code est dans la même méthode..

Voir 40 message(s) de cette discussion en une page