Discussion :

[momedalhouma]

Bonjour,

J'ai fait un petit exemple qui est très clair et qui fait la même erreur, si je resoud cet exemple je peux résoudre l'ereur dans mon programme.

Voila le 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
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
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
package com.example.ftpthread;
 
import java.io.FileOutputStream;
 
import org.apache.commons.net.ftp.FTPClient;
import org.apache.commons.net.ftp.FTPFile;
import org.apache.commons.net.ftp.FTPReply;
 
import android.app.Activity;
import android.os.AsyncTask;
import android.os.Bundle;
import android.os.Environment;
import android.util.Log;
import android.view.View;
import android.view.View.OnClickListener;
import android.widget.Button;
 
public class MainActivity extends Activity {
 
    Connexion conx=null;
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
 
        Button bt= (Button) findViewById(R.id.button1);
        bt.setOnClickListener(new OnClickListener() {
 
            @Override
            public void onClick(View v) {
                if (conx!=null){
                    Log.i("voila", "voila on est la 1");
                    conx.cancel(true);
                    conx=new Connexion();
                    conx.execute("73383_20130426_Tessenderlo_VBR_3.pdf");
 
                }
 
                conx=new Connexion();
                conx.execute("73383_20130426_Tessenderlo_VBR_3.pdf");
            }
        });
    }
 
    class Connexion extends AsyncTask<String, String, String> {
        FTPClient mFTPClient;
        @Override
        protected String doInBackground(String... params) {
            Log.i("voila", "voila on est la 2");
             String chaine = params[0];
                 try {
                        mFTPClient = new FTPClient();
                        mFTPClient.connect("site", 21);
                        Log.i("voila", "voila on est la 4");
                        if (FTPReply.isPositiveCompletion(mFTPClient.getReplyCode())) {
                            boolean status = mFTPClient.login("user", "pass");
                            mFTPClient.enterLocalPassiveMode();
                            ftpDownload("/fromCIS/" +chaine ,
                                    Environment.getExternalStorageDirectory()
                                            + "/Fromcis/" + chaine);
                              mFTPClient.logout();  
                              mFTPClient.disconnect();
 
                        }
                    } catch (Exception e) {
 
                    }
                 return "zaki";    
        }
 
        @Override
        protected void onPostExecute(String result) {
            // TODO Auto-generated method stub
            super.onPostExecute(result);
            Log.i("voila", "voila on est la onpost");
            conx=null;
 
        }
 
        public boolean ftpDownload(String srcFilePath, String desFilePath) {
            boolean status = false;
            try {
                FileOutputStream desFileStream = new FileOutputStream(
                        desFilePath);
                ;
                status = mFTPClient.retrieveFile(srcFilePath, desFileStream);
                desFileStream.close();
 
                return status;
            } catch (Exception e) {
                Log.d(e.getCause() + "", "download failed");
            }
 
            return status;
        }
    }
}

C'est un exemple de téléchargement de fichier PDF depuis un serveur FTP. Le problème est que quand j'appuie plusieurs fois dans le AsyncTask, ça fonctionne pour les premiers clics mais ensuite ça ne fonctionne plus.

Dans le log ça m'affiche :

Log.i("voila", "voila on est la 1")

donc, la tache est occupée mais normalement d'après le code que j'ai mis, je dois l'interrompre et exécuter une autre tâche. Malheureusement, elle ne fait pas ça et j'ignore ce qui se passe.

D'après ce que j'ai lu c'est que AsyncTask.cancel ne permet pas de terminer la tâche.

Quelqu'un saurait-il m'indiquer d'où peut venir le problème ?

Merci d'avance pour votre aide.

[Simon MARQUIS]

En effet, la méthode cancel() d'un AsyncTask ne l'arrête pas directement.
En fait, cela va ajouter un flag interne à l'AsyncTask exprimant que l'AsyncTask doit s'arrêter. Mais c'est au développeur de le faire. Cela n'est pas automatique.

Pour arrêter correctement un AsyncTask, il faut vérifier fréquemment la valeur de isCancelled() dans la méthode doInBackground. Si cette valeur est égale à false, alors tu peux arrêter le travail, sinon tu peux continuer.

[nicroman]

Ou sinon utiliser cancel(true), et bien être sur de gérer correctement les "InterruptedException" qui se produiront alors dans le doInBackground....

[momedalhouma]

tout d'abord merci pour vos réponses.

le problème d’après plusieurs test c'est que le thread se bloque dans la méthode de téléchargement dans le code : c'est cette méthode:

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
   public boolean ftpDownload(String srcFilePath, String desFilePath) {
            boolean status = false;
            try {
                FileOutputStream desFileStream = new FileOutputStream(
                        desFilePath);
                ;
                status = mFTPClient.retrieveFile(srcFilePath, desFileStream);
                desFileStream.close();
 
                return status;
            } catch (Exception e) {
                Log.d(e.getCause() + "", "download failed");
            }
 
            return status;
        }
    }

j'ai fait un test : j'ai essayé de télécharger le mémé fichier plusieurs fois, les premiers téléchargement le thread fonctionnait très bien, mais une fois il s'est arrête et j'ai vu dans éclipse dans la partie DDMS que le fichier ne s'est pas telechargé complétement, juste une partie, et il reste bloqué dans cette partie.

et quand je lance un nouveau thread , il fait rien .

j'ai essayé ce que vous m'avez dit mais aucune solution.a quoi sert de tester iscanceled dans le doinbackround sachant que je fait le travail une seul fois , mais j''ai pas une boucle.

le problème d’après ce que j'ai vu c'est que le thread reste bloqué dans la méthode de téléchargement , est ce qu'il y a moyen de sortir de cette méthode , ou moyen de tuer le thread . j'ai vu dans un exemple l'utilisation d'une liste de tache et a chaque fois quand veut tuer un thread on le supprime de la liste mais je crois que c'est le même truc si on met la tache dans une variable.

Merci d'avance pour votre aide.

[nicroman]

Non mais cancel(true) il fait rien ? (le "true" est important hein !)

Il devrait envoyer un "interrupt" au thread qui execute la tâche, et donc interrompre immédiatement tout "wait" avec un InterruptedException.

[Scinarf]

Bonjour,

Pour régler le problème :

Déclarer Le AsyncTask dans un nouveau fichier .java
Ajouter un

Code :

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

private boolean ThreadLance = false;

Dans le PostExecute :

Code :

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

threadLance = true;

Définir un getter pour ce boolean

Dans la mainActivity

Code :

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

1
2
3
4
5
6
7
if (!conx.getThreadLance) conx.execute();
else 
{
conx = null;
conx = new connexion()
conx.execute();
}

Cela provient du fait qu'un Thread ne peut être lancé qu'une fois. Donc si il n'est pas recréer il ne se ne lance plus et renvoie en général une erreur.

[nicroman]

Oui enfin... entre stocker l'AsyncTask ou stocker un boolean pour savoir si il est lancé, je vois pas la différence:

Code :

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

1
2
3
4
5
6
 
myTask = ....;
myTask.execute();
 
if (myTask != null) 
    myTask.cancel(true);

Mais bon... là en l'occurence, on sait si il est lancé ou non, on veut jute l'arrêter !
(et le boolean ne permettra jamais de le stopper).