Problème socket serveur (attente du client)

Version imprimable

Bonsoir à tous!!

Voici un problème que je ne comprend pas!
Tout d'abord je suis débutant en c#, le but de mon appli est de créer un serveur /client via internet. Hors j'ai un petit soucis avec le code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
 
richTextBox1.Text = "Démmarage du serveur...\n";
 
            srv = new TcpListener(IPAddress.Any, 3000); //initialise TcpListener détecte automatiquement les connexion, 
            // IPaddress.Any accecpte toute les connexion IP sur le port 3000
            richTextBox1.Text += "Initialisation du serveur...\n";
            srv.Start();                                            //On lance la boucle d'écoute
            richTextBox1.Text += "Le serveur est prêt à écouter\n";
 
 
           while (true)                                            //Boucle à l'infini
            {
                 Socket client = srv.AcceptSocket();                 // Bloquer quand il n'y a pas de client, AcceptSocket renvoi un socket tout prêt
                 if (client.Connected)
                 {
                    richTextBox1.Text += "client connecté\n";
                    Thread thr = new Thread(new ThreadStart(ListenThread)); //création d'un thread client
                    richTextBox1.Text += "thread actif\n";
                    thr.Start();                                            //Démarre thread
                 }
            }

Mon erreur se trouve à la ligne

Code:

Socket client = srv.AcceptSocket();

Enfin ce n'est pas une erreur, c'est ce qui me fait planter mon appli.
J'aurai voulu votre avis sur mon code, je crèe le serveur avent le client.
Car il sera toujours actif je voudrait juste qu'il attende le client sans planter :(

Je ne comprend pas du tout pourquoi..

Et pour ListenThread voici la suite:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
void ListenThread()
        {
            Socket sock = client;                               //récupère le socket prêt
            while (sock.Connected != false)                      //Parler au client tant que ce n'est pas fini
            {
                Byte[] rep = new Byte[32767];                   //Création d'un tableau de byte
                int count = sock.Receive(rep, rep.Length, 0);   //stock donnée dans rep, longeur du champs dans count
                string srep = Encoding.ASCII.GetString(rep);    //Convertir les donnée en string
                string reponse = srep.Substring(0, count);      //On garde les données exacte grâce à la taille
                richTextBox1.Text += reponse + "\n";
                switch (reponse)
                {
                    case "deconnexion":
                        richTextBox1.Text += "déconnexion\n";
                        sock.Shutdown(SocketShutdown.Both);         //si deconnexion on ferme le socket en ecriture/lecture
                        sock.Close();                               // libere le socket
                        break;
                }
            }
            Thread.CurrentThread.Abort();                          //On tue le thread 
            richTextBox1.Text += "thread tué \n";
        }

Merci de m'aider

19/10/2011, 20h07
tomlev

Quelle est l'exception que tu obtiens ? (type, message, stacktrace)
19/10/2011, 20h27
Pol63

si tu as un plantage, synonyme d'erreur, il convient de donner les détails de l'erreur
si tu as un freeze, il contient d'utiliser le vocabulaire adapté lorsqu'on s'exprime en public

si c'est un freeze, c'est tout à fait normal, msdn (l'aide) le précise
Accept est une méthode bloquante, tant que personne se connecte, l'exécution s'arrête là, et ca continue à la connexion d'un client

solutions possibles :
- mettre ca dans un thread (comme ca le thread qui rafraichit l'interface n'est pas bloqué)
- utiliser les méthodes begin* comme beginaccept (un callback est appelé à la connexion d'un client)

et si tu codes les 2 côtés de la connexion, tu peux jeter un oeil sur WCF, c'est 100x plus pratique au final
19/10/2011, 21h48
synolog

Pour tomlev, je n'est pas d"exception, juste mon application gèle.

Pour Pol63, Dsl de mettre mal exprimé, mai oui comme je viens de le dire mon appli gèle.
Je veux bien mettre un thread mais pour quel partie?
Juste pour :

Code:

Socket client = srv.AcceptSocket();

Parce que j'ai essayé sans succès :(

Dsl pour mes erreurs de débutant.

Et les méthode begin* (je ne connais pas), quel est l'intérêt?

Merci! de votre aide.. :ccool:
19/10/2011, 22h00
Pol63
accept est bloquant
beginaccept ne l'est pas
beginaccept (et autres beginreceive, beginsend ...) démarrent un thread qui fait accept (et autres) puis quand ca se débloquent ils appellent un callback que tu leur a fournit (= méthode de retour, presque comme un évènement)

au lieu d'avoir
Code:

1 2 3 4 void { code accept code }
tu as
Code:

1 2 3 4 5 6 7 void { code beginaccept (callback)} void callback { endaccept code }
car si la suite du code doit être effectuée après l'acceptation, celle ci se retrouve dans le callback

les threads sont peut etre plus simple que les callbacks à comprendre ...
dans les 3 cas (j'inclus wcf) il y a des tutos via le lien en haut de page

Voilà j'ai suivi tes conseils!
Mais il me reste un soucis lol, je ne peut pas afficher se que le client m'envoi!
Sinon une erreur apparait à la ligne

Code:

1
2
 
this.Invoke(new testecriture(testecrituretext1), "Message recu : "+content+" \n");

Citation:

Erreur : Keyword 'this' is not valid in a static property, static method, or static field initializer
Erreur : An object reference is required for the non-static field, method, or property 'ServeurAppli.Serveur.testecrituretext(string)'

Je ne vois pas comment elle se crée vu que j'écrit déjà à partir du thread qui contient la ligne.. :cry:

Voici le code :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
         public void Run_Click(object sender, EventArgs e)
        {
            IPHostEntry ipHostInfo = Dns.Resolve(Dns.GetHostName());   // retourne les information réseaux
            IPAddress ipAddress = ipHostInfo.AddressList[0];            //Adresse ip retournée par le DNS
            IPEndPoint localEndPoint = new IPEndPoint(ipAddress, 11000); //combine IP et port
            richTextBox1.Text += "Démarage du serveur...\n";
 
            listener = new Socket(AddressFamily.InterNetwork, SocketType.Stream, ProtocolType.Tcp); //Ipv4 / Envoi - reception / TCP
            richTextBox1.Text += "Socket créer\n";
 
            listener.Bind(localEndPoint);
            listener.Listen(100);
            richTextBox1.Text += "Serveur prêt à écouter...\n";
            test = new Thread(testmeth);
            test.Start();
 
        }

Je lance un thread à la fin pour éviter que mon programme gèle

Code:

1
2
3
4
5
6
7
8
9
10
11
12
 
        public void testmeth()
        {
            while (true)
            {
                allDone.Reset();  // définit l'éenevement à l'état non signalé
                this.Invoke(new testecriture(testecrituretext), "Dans boucle..\n"); //Permet décrire dans richbox sans conflit de thread
                listener.BeginAccept(new AsyncCallback(AcceptReceiveDataCallback), listener);
                allDone.WaitOne();
            }
           this.Invoke(new testecriture(testecrituretext),"Sortie boucle..\n");
        }

Voilà le contenu du thread avec BeginAccept, j'utilise this.Invoke pour écrire dans ma richbox.
Jusqu'ici tout va bien! :P Si vous remarqué des erreurs n'hesiter pas à me les dire ;)

Code:

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
 
        public void AcceptReceiveDataCallback(IAsyncResult ar)
        {
            this.Invoke(new testecriture(testecrituretext), "dans accetp donné..\n");
            listener = (Socket)ar.AsyncState;
            Socket handler = listener.EndAccept(ar); // renvoi le nombre d'octet recu
 
            StateObject state = new StateObject();
            state.workSocket = handler;
            this.Invoke(new testecriture(testecrituretext), "Avant reception");
            handler.BeginReceive(state.buffer, 0, StateObject.BufferSize, 0, new AsyncCallback(ReadCallback), state);
 
        }
 
 
        public static void ReadCallback(IAsyncResult ar)
        {
            String content = String.Empty;
            //this.Invoke(new testecriture(testecrituretext), "Entrer dans beginreceive");
 
            StateObject state = (StateObject)ar.AsyncState;
            Socket handler = state.workSocket;
 
 
            int bytesRead = handler.EndReceive(ar); // lit donné du socket client
 
            if (bytesRead > 0)
            {
                state.sb.Append(Encoding.ASCII.GetString(state.buffer, 0, bytesRead)); // on stock les donné de suite
 
                content = state.sb.ToString();
                this.Invoke(new testecriture(testecrituretext), "Message recu : "+content+" \n");
            }
        }

L'erreur se trouve donc à la dernière ligne, ce qui m'empeche donc de récupérer se que je viens de récupérer :cry:

Merci de votre aide :ccool:

20/10/2011, 13h40
Pol63

enlève les "static" qui font que les méthodes sont partagées entre toutes les instances, et ne s'exécutant pas sur une instance, this n'est pas accessible car il ne désigne rien
20/10/2011, 13h43
synolog

Super ça fonctionne!! Moi et mes static :aie:

Merci!!

Sauf que lorsque je lance la requette avec mon ip public avec le client la connexion echoue :(

Je lance l'application cliente d'un autre poste, donc la connexion n'est pas locale.
Faudrait il que je change des paramètres du client?
Code:

1 2 3 4 IPHostEntry ipHostInfo = Dns.Resolve(Dns.GetHostName()); IPAddress ipAddress = ipHostInfo.AddressList[0]; IPEndPoint remoteEP = new IPEndPoint(IPAddress.Parse("86.***.***.***"), 11000);
Le port client et serveur est le même, hors il n'arrive pas à se connecter!

Merci d'avance

20/10/2011, 14h38
Pol63

le serveur doit écouter sur 0.0.0.0 tant qu'à faire (= toutes les interfaces réseaux de la machine)
le client doit se connecter sur l'ip publique
le port doit être routé s'il y a un routeur
le firewall doit être paramétré s'il y en a un
20/10/2011, 15h24
synolog

Merci de ta réponse!

Alors j'ai désactiver le parefeu Windows,
j'ai mis 0.0.0.0 pour le serveur,
le client se connecte a mon ip publique (Adresse IP WAN : 86.***.***.***) chez Orange

Mais non toujours pas de connexion :(

Faut il que j'inscrive une nouvelle règle dans le NAT?
Si oui quel application/service je doit entrer? FTP SERVER?

Je ne pensais pas que c'était aussi complexe :?
20/10/2011, 16h10
synolog

C'est bon j'ai créer une règle sur le nat! il y a connection! :ccool:
Mais une petite erreur! lol
Je pense que je vais réussir à la réglé

Merci encore!!
20/10/2011, 17h22
synolog

Tout fonctionne parfaitement mais pour un seul client :(
Je n'arrive pas à retourner au début de ma boucle, faut il que je détruise mon socket ?
C'était ma dernière question :D

Merci encore
20/10/2011, 17h47
Pol63

quand le socket server accepte un client, il faut créer un nouveau socket server, le faire écouter sur le même port que le précédent etc...
plusieurs clients peuvent donc être connectés au même moment, donc il ne faut pas se perdre dans les instances de socket

pour le read, si tu as plusieurs read à faire tant que la connexion est ouverte c'est à peu près pareil
il faut dans le callback de receive faire un beginreceive à la fin du void pour pouvoir lire une nouvelle fois (et ainsi de suite)
il faut aussi savoir qu'en réseau, un send peut être découpé en plusieurs receive sur le serveur, en théorie il ne faudrait donc pas se dire je fais un receive et je suis tranquille, ni un receive = un message
d'ailleurs sur un receive il peu y avoir la fin d'un send et le début d'un autre

autre chose, dans le receive quand le nombre d'octet recu = 0 ca veut dire que le client a fermé la connexion

et il y a plein d'autres subtilités dans ce genre
sur ce, bonne chance ^^
20/10/2011, 18h16
synolog

Merci pour la petite explication :)
Je vais m'amuser maintenant lol
:ccool: