Discussion :

[chlamy]

Bonjour à tous,

Voilà j'ai actuellement ce code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
 
dialogChargement = ProgressDialog.show(ListeReponses.this, "", getString(R.string.chargement), true, true, new DialogInterface.OnCancelListener() {
            @Override
            public void onCancel(DialogInterface dialog) {
                threadConnexion.interrupt();
            }
        });
        threadConnexion = new Thread(new Runnable() {
            public void run() {
 
                unJSONArray[0] = uneConnexionHTTP.chercherREP(uneIdentification.getPseudo());
                runOnUiThread(new Runnable() {
                    @Override
                    public void run() {
                        try {
                            for (int i = 0; i < unJSONArray[0].length(); i++) {
                                JSONObject jsonLigne = unJSONArray[0].getJSONObject(i);
                                String pseudo = jsonLigne.getString("PSEUDOEXP");
                                String MSG = jsonLigne.getString("DATE");
 
                                HashMap<String, String> map = new HashMap<String, String>();
                                map.put("id", String.valueOf(i));
                                map.put("pseudo", pseudo);
                                map.put("MSG", MSG);
                                listItem.add(map);
                            }
                        } catch (JSONException e) {
                            e.printStackTrace();
                        }
                    }
                });
                dialogChargement.dismiss();
                b[0] = false;
            }
        });
        threadConnexion.start();
 
        while(b[0])
        {
 
        }

Le problème c'est que je veux que la boucle while s'arrête quand le runOnUiThread fini en mettant b[0]=false; après le try/catch, et quand je fais ça l'application ne répond plus et je ne sais pas pourquoi.

Merci

[joel.drigo]

Salut,

Ta boucle while ne s'arrête pas probablement parce que b[0] reste à true. Que fait au juste ta méthode runOnUIThread() ? Et dismiss() de ProgressDialog ? Vérifie que le b[0]=false; est exécuté en ajoutant par exemple un System.out.println("b[0]="+b[0]);. juste après. D'ailleurs, es-tu bien sûr qu'il s'agisse de la même instance de tableau dans les deux méthodes ?

Il est possible également que le changement de valeur de b[0] dans le thread ne soit pas vu par l'autre thread qui exécute la boucle while : sans rentrer dans les détails, il y a un système d'optimisation de Java qui procède parfois à une copie de valeur (voir notion de Memory Consistency Errors). On peut régler ça avec le mot clef volatile, mais on ne peut pas faire un tableau de volatiles. On peut utiliser un AtomicXXXArray (ou XXX est le type) pour avoir un tableau d'élements volatiles, mais il n'existe pas de AtomicBooleanArray. Toutefois, dans ton cas tu peux utiliser un AtomicBoolean à la place, puisque tu n'a qu'une seule valeur. Essayes de remplacer ton tableau par un AtomicBoolean pour voir.

Par ailleurs, la méthode interrupt() n'arrête en aucun cas un thread. Elle change un état qui provoque la levée d'un InterruptedException sur des méthodes d'attente telle que Thread.sleep(), ou Object.wait().

[Hizin]

Change ton implémentation et utilise un pattern observer, soit via un Handler, soit via un système d'interface.
C'est plus propre, plus compréhensible, ce n'est pas de l'attente active et ça ne crame pas la batterie du terminal (sans compter la chaleur qui sera dégagée).
Avec ton attention active, tu exécute en continu ta boucle, quoi qu'il se passe, d'où ton "Application not responding".

@joel.drigo : la méthode runOnUiThread() est une built-in d'Android, qui porte bien son nom : elle permet d'exécuter une portion de code sur le thread graphique. Sur Android, l'interface graphique ne peut être modifiée QUE dans le thread graphique, et dans aucun autre.
dismiss() est une autre built-in concernant les popup, qui permet de les faire disparaître.

[joel.drigo]

@joel.drigo : la méthode runOnUiThread() est une built-in d'Android, qui porte bien son nom : elle permet d'exécuter une portion de code sur le thread graphique. Sur Android, l'interface graphique ne peut être modifiée QUE dans le thread graphique, et dans aucun autre.
dismiss() est une autre built-in concernant les popup, qui permet de les faire disparaître.

J'avais pas fait attention qu'il s'agissait d'une question Android. Je me doutais bien de ce qu'elle faisait, mais n'étant pas standard en Java, je ne pouvais être sûr de comment.

[Hizin]

Pas de souci
La question a été posée, et elle a simplement été répondue