Discussion :

[Altor]

Bonjour,

j'ai programmé un petit client IRC avec un TcpClient, mais quand je me connecte, la form se met en "Ne répond pas" et je ne comprend pas pourquoi.

Peut-être parce que le Input read trop rapidement ? Le timer1 est set à 100 en interval. - même mis à 1000 ça fige -

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
TcpClient client;
        StreamReader input;
        StreamWriter output;
        int Port;
        string Nick, Server, Buffer;
 
        public Form1()
        {
            InitializeComponent();
        }
 
        private void Form1_Load(object sender, EventArgs e)
        {
 
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
            richTextBox1.Text += "Connect in progress... \n";
 
            if (textBox2.Text == "") { richTextBox1.Text += "Nick is empty. \n";  }
            else if (textBox3.Text == "") { richTextBox1.Text += "Server is empty.\n"; }
            else if (textBox4.Text == "") { richTextBox1.Text += "Port is empty.\n";   }
            else
            {
                client = new TcpClient();
                timer1.Start();
                Nick = textBox2.Text;
                Server = textBox3.Text;
                Port = int.Parse(textBox4.Text);
                Name = "Irc client";
                client.Connect(Server, Port);
                input = new StreamReader(client.GetStream());
                output = new StreamWriter(client.GetStream());
 
                richTextBox1.Text += "Connected. Identifying... \n";
 
                output.Write(
                    "USER " + Nick + " 0 * :" + Name + "\r\n" +
                    "NICK " + Nick + "\r\n"
                    );
                richTextBox1.Text += 
                    "USER " + Nick + " 0 * :" + Name + "\r\n" +
                    "NICK " + Nick + "\r\n"
                   ;
                output.Flush();
            }
 
        }
 
        private void timer1_Tick(object sender, EventArgs e)
        {
            Buffer = input.ReadLine();
            if (Buffer != null)
            {
                richTextBox1.Text += Buffer + "\n";
 
                if (Buffer.StartsWith("PING ")) { 
                    output.Write(Buffer.Replace("PING", "PONG") + "\r\n");
                    richTextBox1.Text += Buffer.Replace("PING", "PONG") + "\r\n";
                    output.Flush(); 
                }
                Buffer = null;
            }
 
 
        }
 
        private void richTextBox1_TextChanged(object sender, EventArgs e)
        {
            richTextBox1.ScrollToCaret();
        }

Où serait mon erreur ? *-*

Merci d'avance pour votre aide

[Altor]

J'ai remarqué que même si la form se place en mode "Ne répond pas", l'utilisateur reste tout de même connecté au serveur :

-
Odd is Odd@localhost-C490E8E0 * Irc client
Odd is using modes +iwx
Odd is connecting from *@localhost 127.0.0.1
Odd using irc.local.host LocalHost
Odd has been idle 16secs, signed on Mon Feb 15 15:34:12
Odd End of /WHOIS list.
-

[charouel]

Si ta Form ça fige utilise un backgroundWorker

[Altor]

Qu'est-ce que c'est ?

[charouel]

Tu peux voir ici c'est quoi un backgroundWorker

https://msdn.microsoft.com/fr-fr/lib...(v=vs.95).aspx

[Altor]

J'ai regardé l'article, j'ai même trouvé un tutoriel ici.

Mais je n'arrive toujours pas à comprendre comment implémenter le BW dans mon code et comment l'utiliser.
Mon formateur de C# n'utilisant pas ce système, il n'a pas pu m'aider,

quelqu'un pourrait il pardonner mon manque de connaissance et aider un petit débutant comme moi à corriger tous ces bugs s'il vous plaît ?

Merci d'avance pour votre aide

[J@ckHerror]

Bonjour,

Je ne pense pas que l'on puisse qualifier cela de bug ou alors il est au niveau de l'interface clavier-chaise !

Ton problème est en faite un problème d'asynchronisme, car tout les x milliseconde tu occupes le thread de ton IU à faire quelque chose qui te prend y millisecondes, et donc pendant ce temps ton interface ne peut plus réagir (d’où le ne répond pas) et donc si y devient plus grand que x alors tu ne pourra plus du tout utiliser l'UI car elle sera bloquée en permanence.

L'idée de charouel est donc de déporter le traitement que tu veux faire dans un backgroundworker qui va te permettre de faire du multi thread assez simplement.

sinon tu pourrais aussi aller voir du coté de l'asynchronisme et utiliser async/await si tu es en framework >= 4.5.

J@ck

[raOOn]

Ça devrait ressembler à quelque chose près à ça

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
 
private void button1_Click(object sender, EventArgs e)
{
        BackgroundWorker bw = new BackgroundWorker(); //initialisation
        bw.DoWork += new DoWorkEventHandler(bw_DoWork); //la méthode que tu veux exécuter en parallèle
        bw.RunWorkerAsync();
}
 
//ici tout le traitement que tu veux paralléliser.. (ceux sont des exemples)
void bw_DoWork(object sender, DoWorkEventArgs e) {
 
    ...
    client.Connect(Server, Port););
    ...
    input = new StreamReader(client.GetStream());
    output = new StreamWriter(client.GetStream());
    ...
    etc...
}

[Altor]

Merci pour votre aide précieuse; ça marche !

En fouillant le net (oui j'ai que ça à foutre de ma vie ) je suis tombé sur un topic similaire au mien sur un autre forum. J'ai éclairci mon code et j'ai trouvé une solution qui s'apparente à ce morceau de 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
public async Task Reading()
        {
            try
            {
                string data;
 
                while ((data = await input.ReadLineAsync()) != null)
                {
                    ShowData(data);
                }
            }
            catch (ObjectDisposedException)
            {
               // void
            }
        }

ShowData(); est une fonction personnelle qui affiche les données dans la richtextbox, c'est dans cette fonction que je traiterais les résultats pour les formater à l'écran ([xxxx] <nick> message) et ça marche plutôt pas mal