Discussion :

[vazer7070]

Bonjour,

Voilà, quand je lance ma forme, j'ai un lonng morceau de code qui se lance et qui me permet de récupérer des informations de mon site pour les mettre sur la dite form.
Néanmoins, quand la form s'ouvre, l'application freeze, alors je sais que c'est ce bout de code qui gêne car j'ai mis en commentaire l'ordre de lancement (puisque c'est une fonction à part exprès justement) et là l'application va très bien.
J'ai lu le cours sur les threads en c# (lien ici), mais quand je place mon code dans le threadsloop, mes label et autres apparaissent en erreurs, comment je dois le formuler svp.?

Cordialement
Vazer7070

[phil_tema]

Salut,

Peux-tu poster un bout de ton code pour nous aider ?

@+,
Philippe

[vazer7070]

Je vous place le gros morceau qui fait freeze mon application:
en lui-même, pour l'avoir tester, il fonctionne comme je veux.

(code plus bas)

[Biomega]

Quelques chose du style , utilise async/await et les methodes associées .

Si tu préfère utiliser les .Dispose met les dans Try{}catch(exceptions)Finally{res.Dispose();}

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
public async Task check()
        {
                        string url = "ftp://[yourIp]/public_html/Friend/" + Properties.Settings.Default.search1 + "_amis/";
                        string responseFromServer = null;
 
                        FtpWebRequest request = (FtpWebRequest)WebRequest.Create(url);
                        request.Credentials = new NetworkCredential("user", "pass");
                        request.Method = WebRequestMethods.Ftp.ListDirectoryDetails;
                        using (FtpWebResponse response = (FtpWebResponse )await request.GetResponseAsync())
                        using (Stream stream = response.GetResponseStream())
                        using (StreamReader readuser = new StreamReader(stream))
                        {
                           int nb = -2;
                           while ((readuser.Peek() > -1))
                           {
                              await readuser.ReadLineAsync();
                              nb += 1;
                            }
                           monoFlat_LinkLabel1.Text = nb.ToString();
                        }
}

Je te conseils aussi de séparé , //amis,//rang , etc , dans des méthodes séparées et dans des Thread séparé.

Code :

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

1
2
3
4
5
6
7
public async Task Check()
        {
            Task amis = Task.Run(async () => await Amis());
            Task rang = Task.Run(async () =>  await Rang());
 
            await Task.WhenAll(amis,rang);
        }

Ps:évite de donner ton ip sur des forums

[vazer7070]

(c'est l'ip du serveur, donc après si ils veulent hack, c'est comme ils veulent, mais les québécois sont hardus x))

Je te remercie de ton aide, je vais tester et revenir vers toi^^

[François DORIN]

Bonjour,

Du coup, on n'a plus le code original ! Mais bon, la proposition de Biomega est tout à fait correct. L'utilisation de async/await est une manière d'éviter les gels de l'interface graphique.

Une autre manière est d'utiliser un thread qui récupère et prépare toutes les informations nécessaires et ensuite notifie le thread graphique pour effectuer les mises à jour.