Application client serveur

**_SamSoft_** · 11/10/2007, 14h24

Bonjour, je viens de terminer mes applications (une app client et une app serveur)

Ca compile et fonctionne parfaitement (sauf preuve du contraire

)

J'aimerai avoir des commentaires sur le code, des choses à modifier dans le but de le rendre plus stable.

J'aimerai aussi avoir des "idées" sous forme de pseudo code style :

Code :

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

1
2
3
4
5
 
si truc = 0
boucle
affiche("s");
tant que x=1

Dans le but de réaliser un petit logiciel de conversation par ip

(client/serveur) et là je vois flou, c'est le brouillard.

Je dois partir en cours, je vais réfléchir sur le chemin à la façon de mettre en oeuvre un tel programme (gestion des boucles...)

code serveur

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
99
100
101
102
103
104
105
106
107
108
109
/*
  Name: MySock
 Email: <a href="mailto:samyhocine@hotmail.fr">samyhocine_at_hotmail.fr</a>
 Website: <a href="http://www.samsoft.ift.fr" target="_blank">samsoft.ift.fr</a>
  Copyright: Gpl v2
  Author: Samy Hocine
  Date: 07/10/07 18:14
  Description: Partie serveur
*/
#include <stdio.h>
#include <stdlib.h>
#ifdef __cplusplus
#error Be sure you are using a C compiler
#endif
#if defined (WIN32)
#include <winsock2.h>
#elif defined (UNIX)
/* Rien pour le moment */
#else
#error not defined for this platform
#endif
#define MY_PORT 234
#define SUCCESS 0
#define FAILURE -1
int main(void)
{
 #if defined (WIN32)
        WSADATA WSAData;
     int error = WSAStartup(MAKEWORD(2,0), &WSAData);
 #else
    int error = 0;
  #endif
 if(!error)
 {
  /* 
  Prototype : int socket(int domain, int type, int protocol);
  SOCK_STREAM c'est uniquement pour TCP/IP. Pour UDP/IP c'est SOCK_DGRAM
  On construit une socket
  */
  SOCKET mySocket = socket(AF_INET, SOCK_STREAM, 0);
  /* Si ma socket est valide */
  if(mySocket != INVALID_SOCKET)
  {
   SOCKADDR_IN sin;
   int sock_err;
   char txt[256] = "Bienvenue sur notre serveur\0";
   fprintf(stdout, "La socket %d est ouverte en mode TCP/IP\n", mySocket);
 
   /* Adresse ip automatique */
     sin.sin_addr.s_addr = htonl(INADDR_ANY);
     sin.sin_family = AF_INET;
     /* Ecoute d'un port */
     sin.sin_port = htons(MY_PORT);
   sock_err = bind(mySocket, (SOCKADDR *) &sin, sizeof sin); 
   if(sock_err != SOCKET_ERROR)
   {
    /* On commence le listage */
    sock_err = listen(mySocket, 5); 
    fprintf(stdout, "Listage du port %d ...\n", MY_PORT);
    if(sock_err != SOCKET_ERROR)
    {
     /*On attend que quelqu'un se connecte */
     fprintf(stdout, "Attente de connection sur le port %d ...\n", MY_PORT);
     SOCKADDR_IN newSin;
          int recsize = (int) sizeof newSin;
          SOCKET newSocket = accept (mySocket, (SOCKADDR *) &newSin, &recsize);
          if (newSocket != INVALID_SOCKET)
          {
      int rsltOfShutdown;
      fprintf (stdout, "Connection d'un client avec la socket %d de %s:%d\n", newSocket, inet_ntoa (newSin.sin_addr), htons (newSin.sin_port));
 
      sock_err = send(newSocket, txt, sizeof(txt), 0);
 
      if(sock_err != SOCKET_ERROR)
      {
       fprintf(stdout, "Le message [%s] est envoye'\n", txt);
      }
      else
      {
       fprintf(stdout, "Erreur lors de l'envoie du message\n");
      }
      rsltOfShutdown = shutdown(newSocket, 2);
 
      if(rsltOfShutdown != SUCCESS)
      { 
       fprintf(stdout, "Erreur lors de la fin de connection\n");
      }
      else
      {
             fprintf (stdout, "Fermeture du client avec la socket %d\n", newSocket);
             closesocket(newSocket);
       fprintf(stdout, "Fermeture du serveur");
      }
     }
     }
    }
  }
  else
  {
   fprintf(stderr, "Socket %d invalide", mySocket);
  }
 }
 
 #if defined (WIN32)
        WSACleanup();
  #endif
 getchar();  
  return EXIT_SUCCESS;
}

client

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
 
/*
  Name: ClientMySocket
 Email: <a href="mailto:samyhocine@hotmail.fr">samyhocine_at_hotmail.fr</a>
 Website: <a href="http://www.samsoft.ift.fr" target="_blank">samsoft.ift.fr</a>
  Copyright: Gpl v2
  Author: Samy Hocine
  Date: 10/10/07 14:14
  Description: Partie client de MySock
*/
#include <stdio.h>
#include <stdlib.h>
#ifdef __cplusplus
#error Be sure you are using a C compiler
#endif
#if defined (WIN32)
#include <winsock2.h>
#elif
/* Rien pour le moment */
#else
#error not defined for this platform
#endif
#define MYPORT 234
int main(void)
{
  #if defined (WIN32)
        WSADATA WSAData;
     int error = WSAStartup(MAKEWORD(2,0), &WSAData);
 #else
    int error = 0;
  #endif
 if(!error)
 {
  /* On construit la socket */
  SOCKET mySocket = socket(AF_INET, SOCK_STREAM, 0);
 
  if(mySocket != INVALID_SOCKET)
  {
   SOCKADDR_IN sin;
   sin.sin_addr.s_addr = inet_addr("127.0.0.1");
   sin.sin_family = AF_INET;
   sin.sin_port = htons(MYPORT);
   /* Si la connection est re'ussie */
   if(connect(mySocket, (SOCKADDR *) &sin, sizeof(sin)) != SOCKET_ERROR)
   {
    char txt[256] = "";
    fprintf(stdout, "Connection a %s sur le port %d\n", inet_ntoa(sin.sin_addr), htons(sin.sin_port));
 
    if(recv(mySocket, txt, sizeof(txt), 0) != SOCKET_ERROR)
    {
     fprintf(stdout, "Message du serveur : %s\n", txt);
    }
    else
    {
     fprintf(stdout, "Erreur lors de la re'ce'ption du message");
    }
   }
   else
   {
    fprintf(stdout, "Erreur lors de la connection\n");
   }
   /* Fermeture de la socket */
   closesocket(mySocket);
  }
 }
 #if defined (WIN32)
        WSACleanup();
  #endif
  getchar();
  return EXIT_SUCCESS;
}

Merci d'avance

PS: Je me suis basé principalement sur le cours réseau : http://emmanuel-delahaye.developpez.com/reseaux.htm (pour le fond)

**Skyounet** · 11/10/2007, 14h53

Indentation horrible.

Le port < 1024, il faudra lancer ton appli en root sur Unix, c'est pas normal.

En français on dit connexion et non connection

Code :

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

char txt[256] = "Bienvenue sur notre serveur\0";

Euh pourquoi tu mets un \0 à la main ? Ca le fait tout seul hein.

J'aurais plutôt mis un truc dans ce genre

Code :

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

char const * const txt = "Bienvenue sur notre serveur";

Et pour envoyer n'utilise pas sizeof mais strlen comme c'est une chaine de caractères.

Code :

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

char txt[256] = "";

Pas besoin du ""

Et pour le recv il faut mettre sizeof(txt) - 1 histoire que si on t'en envoi plus ben tu n'ai pas de soucis.

Normalement un truc comme ça

Code :

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

1
2
3
4
5
6
int n = 0;
if((n = recv(mySocket, txt, sizeof(txt) - 1, 0)) != SOCKET_ERROR)
{
    txt[n] = 0;
    ...
}

Y'a d'autres petites erreurs de ce genre.

**|PaRa-BoL** · 11/10/2007, 15h12

Attention a ne pas oublier le \0 final de ce que tu as recu.
Aussi il est préférable de faire une boucle autour de recv tant que tu recois des données (qui te dis que tu ne recevra pas plus de 256 octets) et d'allouer dynamiquement la mémoire à ton buffer de sortie.

**Emmanuel Delahaye** · 11/10/2007, 15h34

Envoyé par _SamSoft_

J'aimerai avoir des commentaires sur le code, des choses à modifier dans le but de le rendre plus stable.

J'aimerai aussi avoir des "idées" sous forme de pseudo code style :

Code :

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

1
2
3
4
5
 
si truc = 0
boucle
affiche("s");
tant que x=1

Dans le but de réaliser un petit logiciel de conversation par ip

(client/serveur) et là je vois flou, c'est le brouillard.

Je dois partir en cours, je vais réfléchir sur le chemin à la façon de mettre en oeuvre un tel programme (gestion des boucles...)

code serveur

Code :

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

 sock_err = send (newSocket, txt, sizeof (txt), 0);

Non. On émet uniquement ce qui nous intéresse soit strlen(txt). Il n'y a aucune raison de transmettre des 0 quand on manipule du texte. Par contre,
on transmet des lignes, c'est à dire du texte terminé par un marque de fin de ligne (CR, LF, CRLF etc. selon le protocole utilisé ou ses propres choix).

Code :

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

1
2
3
4
5
6
7
 
/* EOL = CRLF */
#define EOL "\r\n"
<...>
         char txt[256] = "Bienvenue sur notre serveur"EOL;
<...>
         sock_err = send (newSocket, txt, strlen (txt), 0);

Merci de ne pas mentionner mon site si c'est pour écrire des horreurs pareilles :

Code :

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

1
2
3
4
 
    char txt[256] = "";
<...> 
    if(recv(mySocket, txt, sizeof(txt), 0) != SOCKET_ERROR)

Ou alors, il faut que tu lises un peu mieux mes articles...

http://emmanuel-delahaye.developpez....eaux.htm#texte

Ensuite, pour un serveur mono-client, je conseille un interpréteur de commandes :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
   socket := demarrer_serveur(PORT)
   client := attendre_client(socket)
   envoyer (client, "Hello")
   REPETER
      recevoir (client, commande)
      fin := interpreter (commande)
      IF NOT fin
         saisir (commande)
         emettre (client, commande)
      ENDIF
   JUSQU'A fin
   deconnecter (client)
   arreter_serveur (socket)

Coté client, c'est tout simple :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
   socket := demarrer_client(IP, PORT)
   REPETER
      recevoir (socket, reponse)
      fin := interpreter (reponse)
      IF NOT fin
         saisir (commande)
         emettre (socket, commande)
      ENDIF
   JUSQU'A fin
   arreter_client (socket)

Pour éviter tout blocage, on peut aussi séparer la commande de la réponse dans 2 threads différents :
Emission

Code :

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

1
2
3
4
5
 
   REPETER
         saisir (commande)
         emettre (socket, commande)
   JUSQU'A fin

Réception

Code :

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

1
2
3
4
5
 
   REPETER
      recevoir (socket, reponse)
      fin := interpreter (reponse)
   JUSQU'A fin

'fin' étant partagé proprement entre les 2 taches.

Nota les 2 fonctions 'interpreter' (celle du client et celle du serveur) constituent la partie 'applicative'. A chaque commande ou réponse reçue, un traitement particulier va être effectué. En gros, si un mot clé est reconnu, on fait une action particulière, sinon, on se contente d'afficher le message reçu.

Cet exemple simpliste peut être réalisé pour vérifier la mise en en oeuvre du dialogue, la gestion des commandes etc.

Dans la pratique, on ne fait pas comme ça. En effet, il n'y a aucun intérêt à mobiliser les capacité d'un serveur pour un seul client. On a donc une architecture construite comme ceci :

Dialogue point à point

client <---> serveur <---> client

Les applications client sont identiques. Le serveur sert dans un premier temps à établir les connexions client/serveur (authentification, mise en attente de relation), puis à mettre en relation 2 clients à leur demande. (on est très proche d'un système téléphonique, sauf qu'il n'y a pas "d'appel". on est simplement 'en attente'.).

Comme toujours, les détails de comportement et de protocole doivent être parfaitement spécifiés avant de se lancer dans l'analyse puis le codage.

Nota : rien de tout ça n'a à voir avec le langage C.

**Emmanuel Delahaye** · 11/10/2007, 16h06

Envoyé par |PaRa-BoL

Attention a ne pas oublier le \0 final de ce que tu as recu.

Non, on ne reçoit pas de 0. Par contre, comme l'a fait remarqué Skyrunner, il faut prévoir une place en réception et le positionner correctement pour en faire une chaine C.

**|PaRa-BoL** · 11/10/2007, 17h46

Non, on ne reçoit pas de 0.

Oui c'est ce que j'ai dis "de ne pas oublier d'ajouter le \0 à ce qu'il a recu"

**_SamSoft_** · 11/10/2007, 18h19

Merci pour ces commentaires, j'ai donc adapté mon code.

Serveur

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
99
100
101
102
103
104
105
106
107
108
109
110
 
/*
  Name: MySock
 Email: samyhocine_at_hotmail.fr
 Website: samsoft.ift.fr
  Copyright: Gpl v2
  Author: Samy Hocine
  Date: 07/10/07 18:14
  Description: Partie serveur
*/
#include <stdio.h>
#include <stdlib.h>
#ifdef __cplusplus
#error Be sure you are using a C compiler
#endif
#if defined (WIN32)
#include <winsock2.h>
#elif defined (LINUX)
/* Rien pour le moment */
#else
#error not defined for this platform
#endif
#define MY_PORT 1025
#define SUCCESS 0
#define FAILURE -1
int main(void)
{
 #if defined (WIN32)
        WSADATA WSAData;
     int error = WSAStartup(MAKEWORD(2,0), &WSAData);
 #else
    int error = 0;
  #endif
 if(!error)
 {
  /* 
  Prototype : int socket(int domain, int type, int protocol);
  SOCK_STREAM c'est uniquement pour TCP/IP. Pour UDP/IP c'est SOCK_DGRAM
  On construit une socket
  */
  SOCKET mySocket = socket(AF_INET, SOCK_STREAM, 0);
  /* Si ma socket est valide */
  if(mySocket != INVALID_SOCKET)
  {
   SOCKADDR_IN sin;
   int sock_err;
   char txt[256] = "Bienvenue sur notre serveur";
   fprintf(stdout, "La socket %d est ouverte en mode TCP/IP\n", mySocket);
 
   /* Adresse ip automatique */
     sin.sin_addr.s_addr = htonl(INADDR_ANY);
     sin.sin_family = AF_INET;
     /* Ecoute d'un port */
     sin.sin_port = htons(MY_PORT);
   sock_err = bind(mySocket, (SOCKADDR *) &sin, sizeof sin); 
   if(sock_err != SOCKET_ERROR)
   {
    /* On commence le listage */
    sock_err = listen(mySocket, 5); 
    fprintf(stdout, "Listage du port %d ...\n", MY_PORT);
    if(sock_err != SOCKET_ERROR)
    {
     /* On attend que quelqu'un se connecte */
     fprintf(stdout, "Attente de connexion sur le port %d ...\n", MY_PORT);
     SOCKADDR_IN newSin;
          int recsize = (int) sizeof newSin;
          SOCKET newSocket = accept (mySocket, (SOCKADDR *) &newSin, &recsize);
          if (newSocket != INVALID_SOCKET)
          {
      int rsltOfShutdown;
      fprintf (stdout, "connexion d'un client avec la socket %d de %s:%d\n", newSocket, inet_ntoa (newSin.sin_addr), htons (newSin.sin_port));
 
      sock_err = send(newSocket, txt, strlen(txt), 0);
 
      if(sock_err != SOCKET_ERROR)
      {
       fprintf(stdout, "Le message est envoye'\n");
      }
      else
      {
       fprintf(stdout, "Erreur lors de l'envoie du message\n");
      }
      rsltOfShutdown = shutdown(newSocket, 2);
 
      if(rsltOfShutdown != SUCCESS)
      { 
       fprintf(stdout, "Erreur lors de la fin de connexion\n");
      }
      else
      {
             fprintf (stdout, "Fermeture du client avec la socket %d\n", newSocket);
             closesocket(newSocket);
       fprintf(stdout, "Fermeture du serveur");
      }
     }
     }
    }
  }
  else
  {
   fprintf(stderr, "Socket %d invalide", mySocket);
  }
 }
 
 #if defined (WIN32)
        WSACleanup();
  #endif
 getchar();  
  return EXIT_SUCCESS;
}

Client

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
 
/*
  Name: ClientMySocket
 Email: samyhocine_at_hotmail.fr
 Website: samsoft.ift.fr
  Copyright: Gpl v2
  Author: Samy Hocine
  Date: 10/10/07 14:14
  Description: Partie client de MySock
*/
#include <stdio.h>
#include <stdlib.h>
#ifdef __cplusplus
#error Be sure you are using a C compiler
#endif
#if defined (WIN32)
#include <winsock2.h>
#elif defiend (LINUX)
/* Rien pour le moment */
#else
#error not defined for this platform
#endif
#define MYPORT 1025
int main(void)
{
  #if defined (WIN32)
        WSADATA WSAData;
     int error = WSAStartup(MAKEWORD(2,0), &WSAData);
 #else
    int error = 0;
  #endif
 if(!error)
 {
  /* On construit la socket */
  SOCKET mySocket = socket(AF_INET, SOCK_STREAM, 0);
 
  if(mySocket != INVALID_SOCKET)
  {
   SOCKADDR_IN sin;
   sin.sin_addr.s_addr = inet_addr("127.0.0.1");
   sin.sin_family = AF_INET;
   sin.sin_port = htons(MYPORT);
   /* Si la connection est re'ussie */
   if(connect(mySocket, (SOCKADDR *) &sin, sizeof(sin)) != SOCKET_ERROR)
   {
    char txt[256];
    fprintf(stdout, "connexion a %s sur le port %d\n", inet_ntoa(sin.sin_addr), htons(sin.sin_port));
 
    if(recv(mySocket, txt, sizeof(txt), 0) != SOCKET_ERROR)
    {
     fprintf(stdout, "Message du serveur : %s\n", txt);
    }
    else
    {
     fprintf(stdout, "Erreur lors de la re'ce'ption du message");
    }
   }
   else
   {
    fprintf(stdout, "Erreur lors de la connexion\n");
   }
   /* Fermeture de la socket */
   closesocket(mySocket);
  }
 }
 #if defined (WIN32)
        WSACleanup();
  #endif
  getchar();
  return EXIT_SUCCESS;
}

Sinon EOL ne sert pas à grand chose vu que :

Code :

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

1
2
 
char txt[256] = "mon texte\n";

Fonctionne très bien

(Enfin je sais pas si c'est correct après)

J'ai aussi penser à un système d'interpreteur de commandes :

com:>Fin
/* Quitte et affice à l'autre qu'il est parti */
com:>salut
//Affiche salut car c'est pas répertorié comme une commande
/* En gros */
com:>truc
/* Si truc est une commande alors effectuer une certaine action sinon afficher truc */

Emmanuel (votre site ne fonctionne pas pour le moment

)

Pouvez vous m'expliquer un peu plus comment fonctionne le système :

client <-> serveur <-> client2

Avec un petit pseudo code (sans rentrer dans les détails) ou me passer un lien en français

Merci d'avance

PS: Le site refonctionne je regarde ca. Quel est le problème avec le port 234 (je l'ai pris au hasard sachant qu'il faut être en root sous linux pour utiliser le 23).

**Skyounet** · 11/10/2007, 18h31

Sous Linux pour ouvrir un port < 1024 il faut être en root, donc 243 < 1024...

Revois ton indentation s'il te plait.

**Emmanuel Delahaye** · 11/10/2007, 18h37

Envoyé par |PaRa-BoL

Oui c'est ce que j'ai dis "de ne pas oublier d'ajouter le \0 à ce qu'il a recu"

Tu ne l'avais pas formulé comme ça, ce qui a rendu ton message incompréhensible...

**Emmanuel Delahaye** · 11/10/2007, 18h53

Envoyé par _SamSoft_

Merci pour ces commentaires, j'ai donc adapté mon code.

Serveur

Code :

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

1
2
 
   char txt[256] = "Bienvenue sur notre serveur";

Et la marque de fin de ligne ?

Code :

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

1
2
3
4
 
    /* On commence le listage */
    sock_err = listen(mySocket, 5); 
    fprintf(stdout, "Listage du port %d ...\n", MY_PORT);

Quel listage ? listen signifie 'écouter' en anglais. Il faut connaitre un minimum d'anglais pour pouvoir faire de la programmation.

Code :

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

      sock_err = send(newSocket, txt, strlen(txt), 0);

OK

Client

Code :

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

1
2
3
4
 
    char txt[256];
 
    if(recv(mySocket, txt, sizeof(txt), 0) != SOCKET_ERROR)

Tu es vraiment une tête de mule. On te dis qu'il faut placer le 0 final. C'est fait où ?

Sinon EOL ne sert pas à grand chose vu que :

Code :

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

1
2
 
char txt[256] = "mon texte\n";

Fonctionne très bien

Et alors ? J'ai jamais dit le contraire. Simplement ce type de terminaison est un choix possible parmi plusieurs. L'avantage d'utiliser l'abstraction EOL est qu'on peut changer de terminaison de ligne facilement.

(En http, c'est pas '\n', si tu veux tout savoir...)

J'ai aussi penser à un système d'interpréteur de commandes :

OK.

Pouvez vous m'expliquer un peu plus comment fonctionne le système :

client <-> serveur <-> client2

Avec un petit pseudo (sans rentrer dans les détails) ou me passer un lien en français

Je crois que c'est le moment d'imaginer le scénario. Si tu ne réflechis jamais à un problème, tu ne vas pas avancer. Mets toi à la place de l'utilisateur et essaye d'imaginer ce qui se passe 'en coulisse'

1 - quand tu ne connectes au serveur
2 - quand l'autre client se connecte au serveur

**_SamSoft_** · 11/10/2007, 19h04

Désolé j'avais supprimer la fin de ligne

Serveur

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
99
100
101
102
103
104
105
106
107
108
109
/*
  Name: MySock
 Email: samyhocine_at_hotmail.fr
 Website: samsoft.ift.fr
  Copyright: Gpl v2
  Author: Samy Hocine
  Date: 07/10/07 18:14
  Description: Partie serveur
*/
#include <stdio.h>
#include <stdlib.h>
#ifdef __cplusplus
#error Be sure you are using a C compiler
#endif
#if defined (WIN32)
#include <winsock2.h>
#elif defined (LINUX)
/* Rien pour le moment */
#else
#error not defined for this platform
#endif
#define MY_PORT 1025
#define SUCCESS 0
#define FAILURE -1
int main(void)
{
 #if defined (WIN32)
        WSADATA WSAData;
     int error = WSAStartup(MAKEWORD(2,0), &WSAData);
 #else
    int error = 0;
  #endif
 if(!error)
 {
  /* 
  Prototype : int socket(int domain, int type, int protocol);
  SOCK_STREAM c'est uniquement pour TCP/IP. Pour UDP/IP c'est SOCK_DGRAM
  On construit une socket
  */
  SOCKET mySocket = socket(AF_INET, SOCK_STREAM, 0);
  /* Si ma socket est valide */
  if(mySocket != INVALID_SOCKET)
  {
   SOCKADDR_IN sin;
   int sock_err;
   char txt[256] = "Bienvenue sur notre serveur\n";
   fprintf(stdout, "La socket %d est ouverte en mode TCP/IP\n", mySocket);
 
   /* Adresse ip automatique */
      sin.sin_addr.s_addr = htonl(INADDR_ANY);
      sin.sin_family = AF_INET;
      /* Ecoute d'un port */
      sin.sin_port = htons(MY_PORT);
   sock_err = bind(mySocket, (SOCKADDR *) &sin, sizeof sin); 
   if(sock_err != SOCKET_ERROR)
   {
    /* On commence l'e'coute */
    sock_err = listen(mySocket, 5); 
    fprintf(stdout, "Ecoute du port %d ...\n", MY_PORT);
    if(sock_err != SOCKET_ERROR)
    {
     /* On attend que quelqu'un se connecte */
     fprintf(stdout, "Attente de connexion sur le port %d ...\n", MY_PORT);
     SOCKADDR_IN newSin;
             int recsize = (int) sizeof newSin;
             SOCKET newSocket = accept (mySocket, (SOCKADDR *) &newSin, &recsize);
          if (newSocket != INVALID_SOCKET)
          {
      int rsltOfShutdown;
      fprintf (stdout, "connexion d'un client avec la socket %d de %s:%d\n", newSocket, inet_ntoa (newSin.sin_addr), htons (newSin.sin_port));
 
      sock_err = send(newSocket, txt, strlen(txt), 0);
 
      if(sock_err != SOCKET_ERROR)
      {
       fprintf(stdout, "Le message est envoye'\n");
      }
      else
      {
       fprintf(stdout, "Erreur lors de l'envoie du message\n");
      }
      rsltOfShutdown = shutdown(newSocket, 2);
 
      if(rsltOfShutdown != SUCCESS)
      { 
       fprintf(stdout, "Erreur lors de la fin de connexion\n");
      }
      else
      {
                fprintf (stdout, "Fermeture du client avec la socket %d\n", newSocket);
                closesocket(newSocket);
       fprintf(stdout, "Fermeture du serveur");
      }
     }
     }
    }
  }
  else
  {
   fprintf(stderr, "Socket %d invalide", mySocket);
  }
 }
 
 #if defined (WIN32)
        WSACleanup();
  #endif
 getchar();  
  return EXIT_SUCCESS;
}

Client

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
/*
  Name: ClientMySocket
 Email: <a href="mailto:samyhocine@hotmail.fr">samyhocine@hotmail.fr</a>
 Website: <a href="http://www.samsoft.ift.fr" target="_blank">http://www.samsoft.ift.fr</a>
  Copyright: Gpl v2
  Author: Samy Hocine
  Date: 10/10/07 14:14
  Description: Partie client de MySock
*/
#include <stdio.h>
#include <stdlib.h>
#ifdef __cplusplus
#error Be sure you are using a C compiler
#endif
#if defined (WIN32)
#include <winsock2.h>
#elif defiend (LINUX)
/* Rien pour le moment */
#else
#error not defined for this platform
#endif
#define MYPORT 1025
int main(void)
{
  #if defined (WIN32)
        WSADATA WSAData;
     int error = WSAStartup(MAKEWORD(2,0), &WSAData);
 #else
    int error = 0;
  #endif
 if(!error)
 {
  /* On construit la socket */
  SOCKET mySocket = socket(AF_INET, SOCK_STREAM, 0);
 
  if(mySocket != INVALID_SOCKET)
  {
   SOCKADDR_IN sin;
   sin.sin_addr.s_addr = inet_addr("127.0.0.1");
   sin.sin_family = AF_INET;
   sin.sin_port = htons(MYPORT);
   /* Si la connection est re'ussie */
   if(connect(mySocket, (SOCKADDR *) &sin, sizeof(sin)) != SOCKET_ERROR)
   {
    char txt[256];
    fprintf(stdout, "connexion a %s sur le port %d\n", inet_ntoa(sin.sin_addr), htons(sin.sin_port));
 
    if(recv(mySocket, txt, sizeof(txt), 0) != SOCKET_ERROR)
    {
     fprintf(stdout, "Message du serveur : %s\n", txt);
    }
    else
    {
     fprintf(stdout, "Erreur lors de la re'ce'ption du message");
    }
   }
   else
   {
    fprintf(stdout, "Erreur lors de la connexion\n");
   }
   /* Fermeture de la socket */
   closesocket(mySocket);
  }
 }
 #if defined (WIN32)
        WSACleanup();
  #endif
  getchar();
  return EXIT_SUCCESS;
}

Sur votre site :

Code :

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

1
2
char text[128]; /* taille arbitraire */
   int n = recv (sock, text, sizeof text - 1, 0);

Sur mon code :

Code :

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

1
2
3
4
 
char txt[256];  
if(recv(mySocket, txt, sizeof(txt), 0) != SOCKET_ERROR)
...

Si je met -1 : "erreur de connexion"

Je réfléchis à la mise en place d'un système d'interpreteur pour le dialogue client/serveur

(pour le moment uniquement client/serveur)

PS: Désolé pour l'indentation, c'est au moment du copier collé que ca foire, chez moi c'est bon

**Skyounet** · 11/10/2007, 19h06

Envoyé par _SamSoft_

Si je met -1 : "erreur de connexion"

Pardon ?
N'importe quoi. Ca n'a strictement rien à voir, pourquoi tu aurais erreur de connexion sur une tentative de lecture ?

**Emmanuel Delahaye** · 11/10/2007, 19h09

Envoyé par _SamSoft_

Sur votre site :

Code :

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

1
2
char text[128]; /* taille arbitraire */
   int n = recv (sock, text, sizeof text - 1, 0);

Sur mon code :

Code :

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

1
2
3
4
 
char txt[256];  
if(recv(mySocket, txt, sizeof(txt), 0) != SOCKET_ERROR)
...

Si je met -1 : "erreur de connexion"

Montre le code qui ne va pas, parce que "Si je met -1", je ne sais pas ce que ça veut dire. Je ne suis pas dans ta tête.

PS: Désolé pour l'indentation, c'est au moment du copier collé que ca foire, chez moi c'est bon

On l'a déjà expliqué 3000 fois... Il faut que l'éditeur soit configuré de manière à remplacé les tabulations par un nombre fixe d'espaces (2 à 4).

**_SamSoft_** · 11/10/2007, 19h44

Pour l'indentation (j'avais mis deux mais ca plaisait pas à skyrunner

j'ai mis 4)

Emmanuel: Ce que je ne comprend pas c'est dans quelle mesure :

Code :

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

1
2
 
char txt[256];

Est incorrect

Je suis au début d'une ébauche qui ne fonctionne pas et qui ne fonctionnera pas pour le moment car c'est un pseudo code C

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
 
/* Test pour un interpreteur de commandes */
int end = 1;
do
{
 char commande[256];
 int err;
 err = recv(truc, commande);
 
 if(!err)
 {
  if(strcmp(commande, "STAT_OFF")==0)
  {
   send("%d est absent", ip);
  }
  else if(strcmp(commande, "STAT_ON");
  {
   send("%d est pre'sent", ip);
  }
  else
  {
   fprintf(stdout, "%s", commande);
  }
 }
}
while(end!=0);

C'est juste pour voir si en gros l'idée est là ou si je suis à l'ouest.

**Skyounet** · 11/10/2007, 19h48

Ben moi je vois 1 espace pour une tabulation, c'est trop peu.

**Emmanuel Delahaye** · 11/10/2007, 19h58

Envoyé par _SamSoft_

Emmanuel: Ce que je ne comprend pas c'est dans quelle mesure :

Code :

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

1
2
 
char txt[256];

Est incorrect

J'ai pas dit que cette partie du code était incorrecte. C'est l'appel de recv() est faux et il manque quelque chose après. Tout est expliqué dans l'exemple que j'ai publié sur mon site. Tu as lu la doc de recv() ? Je suis prêt à parier que non et que tu te contentes de recopier du code sans chercher à comprendre comment il fonctionne.

Je suis au début d'une ébauche qui ne fonctionne pas et qui ne fonctionnera pas pour le moment car c'est un pseudo code C

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
 
/* Test pour un interpreteur de commandes */
int end = 1;
do
{
 char commande[256];
 int err;
 err = recv(truc, commande);
 
 if(!err)
 {
  if(strcmp(commande, "STAT_OFF")==0)
  {
   send("%d est absent", ip);
  }
  else if(strcmp(commande, "STAT_ON");
  {
   send("%d est pre'sent", ip);
  }
  else
  {
   fprintf(stdout, "%s", commande);
  }
 }
}
while(end!=0);

C'est juste pour voir si en gros l'idée est là ou si je suis complement à l'ouest.

Il y a de ça, mais pour définir un protocole, on n'utilise pas un algorithme, mais plutôt une liste de commandes avec une description précise de leurs paramètres et de leurs effets.

(Au fait, le code posté à une indentation de 1).

**_SamSoft_** · 11/10/2007, 20h02

Pour le code, j'ai absolument compris comment il fonctionne sinon j'aurais abadonné depuis longtemps, je sais qu'il manque le -1 après sizeof(txt)
mais si je le met, rien ne s'affiche.

Pour l'indentation, chez moi elle est de 4, quand je colle sur le site elle revient à 1

Comment ca on utilise une déscription détaillée de chaque fonction ?

**Emmanuel Delahaye** · 11/10/2007, 20h15

Envoyé par _SamSoft_

Pour le code, j'ai absolument compris comment il fonctionne sinon j'aurais abadonné depuis longtemps, je sais qu'il manque le -1 après sizeof(txt)
mais si je le met, rien ne s'affiche.

Tu as compris à quoi servait ce -1 ?
Tu as compris le sens de la valeur retournée par recv()
Tu as compris comment on plaçait le 0 dans la chaine reçu ?

Alors montre le code qui ne va pas.

Pour l'indentation, chez moi elle est de 4, quand je colle sur le site elle revient à 1

Avec ton éditeur, tu es bien en mode 'espace' et non 'tabs' ? Tu utilises Code::Blocks ?

Comment ca on utilise une déscription détaillée de chaque fonction ?

Ne déforme pas mes propos. Je n'ai pas parlé de fonction.

Pour le moment, tu en es à la description du protocole. Alors :

Envoyé par -ed-

<...> pour définir un protocole, on n'utilise pas un algorithme, mais plutôt une liste de commandes avec une description précise de leurs paramètres et de leurs effets.

C'est bien un interpréteur de commande que tu veux faire ? Alors on attend la liste des commandes...

**_SamSoft_** · 11/10/2007, 20h23

Tu as compris à quoi servait ce -1 ?

Oui, il sert à supprimer le \0 de la chaine (non ?)

Tu as compris le sens de la valeur retournée par recv()

-1 s'elle échoue sinon le nombre d'octes recus.

Pour la troisième question, je ne vois pas

Pour le moment et jusqu'à la fin de ce projet je suis sous devcpp, je passe après à code::blocs

C'est bien un interpréteur de commande que tu veux faire ? Alors on attend la liste des commandes...

Je ne vous comprend plus ? Vous voulez une sorte de documation des commandes ?

**millie** · 11/10/2007, 20h34

recv retourne le nombre d'octet effectivement lu.
Retourne -1 en cas d'erreur, il n'y a pas de rapport avec les sockets.

Si ça retourne -1, il faut voir errno sous Unix ou éventuellement WSAGetLastError sous Windows.
(désolé, c'est du code C++, mais vous comprendrez le principe) :

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
       // Receive a chunk of bytes
        received = recv(_isocket, data, static_cast<int>(dataSize), 0);
 
        // Check the number of bytes received
        if (received < 0)
        {
            logError("[Socket] Received<0");
#ifdef WIN32
            /*partie reprie de SocketInputStream de java de sun*/
            switch (WSAGetLastError())
            {
                case WSAEINTR:
                    logError("Socket closed");
                    throw SocketException("socket closed");
                case WSAECONNRESET:
                case WSAESHUTDOWN:
                    /*
                     * Connection has been reset - Windows sometimes reports
                     * the reset as a shutdown error.
                     */
                     logError("Connection Reset");
                    throw ConnectionResetException("Connection reset");
                case WSAETIMEDOUT :
                    logError("Read timed out");
                    throw SocketTimeoutException("Read timed out");
                default:
                    logError("Read failed");
                    throw SocketException("Read failed");
            }
#else
            switch(errno)
            {
                case ECONNRESET:
                case EPIPE:
                    logError("Connection reset");
                    throw ConnectionResetException("Connection reset");
                case EBADF:
                    logError("Socket closed");
                    throw SocketException("Socket closed");
                case EINTR:
                    logError("Operation interrupted");
                    throw InterruptedIOException("Operation interrupted");
                default:
                    logError("Read failed");
                    throw SocketException("Read failed");
            }
#endif
 
        }

Sinon, jette un oeil au man

Application client serveur

Réseau C

Discussions similaires

Partager

Partager