Discussion :

[zwan.bourg]

J'ai un programme qui fait un telechargement dans un thread. Et je veux permettre l'annulation du download à partir d'un clique.
Voila donc ce que je fais:

Code :

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

1
2
3
4
5
6
7
8
9
 
 threadDown.Abort();                    
 webResponse.Close();
 strResponse.Close();
 strLocal.Close();
 // Barre de prog à zero
 downlProg.Value = 0;
 lblWhat.Text = "Mise à jour annullée";
 lblNeed.Text = string.Empty;

Mais voila l'exception qui se declenche:

ObjectDisposedException was unhandled. Impossible d'accéder à un objet supprimé.

au niveau de la ligne suivante:

Code :

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

Invoke(new UpdateProgessCallback(this.UpdateProgress), new object[] { strLocal.Length, fileSize });

Cette ligne permet d'afficher la barre de progression.
Apparemment, le thread veut continuer à s'executer meme apres Abort()!
Quelqu'un peut il m'aider?

[Xzander]

La méthode Abort() lance l'exception. Tu ne peux rien y faire, ce n'est pas une "erreur", c'est inévitable. Tu pourrais mettre une structure try catch. Toutefois, ce n'est pas élégant d'utiliser Abort(). Voici ce qu'il convient de faire:

Code :

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

1
2
3
4
5
6
7
8
9
10
 
private void myThreadStartsHere{
 
    while(continueToDownload){
        //LIRE n octets.
        //Si jamais la connexion est lente, faire dormir le thread quelques ms.
        Thread.Sleep( 50 );    
    }
 
}

De cette façon, tu as un booléen qui te permet d'interrompre le téléchargement en lui affectant 'false' lorsque l'utilisateur clique sur arrêter.

[ced600]

Tu pourrais mettre une structure try catch. Toutefois, ce n'est pas élégant d'utiliser Abort().

Par curiosité pourquoi ce n'est pas élégant l'utilisation d'Abort() ? A cause des try catch ?

[TrEiZe85]

Il suffit de mettre ton thread en IsBackGround=true juste avant de faire ton abort. Enfin si j'ais bien compris ton prob

[Xzander]

Par curiosité pourquoi ce n'est pas élégant l'utilisation d'Abort() ? A cause des try catch ?

Car tu te retrouves avec un catch{} vide. Mais si tu peux mettre ton thread en arrière plan, alors vas-y à fond. Toutefois, ce que j'aime en ajoutant un booléen, c'est que tu peux facilement faire du traitement supplémentaire (ex: clairer des ressources, supprimer des fichiers temporaires, etc).

[ced600]

Car tu te retrouves avec un catch{} vide

Il est vide uniquement si tu le laisses vide.
Avec le catch tu peut récupérer l'exception et provoquer par exemple l'écriture dans un fichier de log que le thread a été arrêté normalement. Le catch n'est alors pas vide et c'est propre