Discussion :

[Nigo]

Salut tous le monde, je fais un programme utlisant les sockets et je recherche pas mal d'info la deçu, j'ai trouvé sur ce site un bout de code que j'ai tout de suite testé, mais le compilateur ne compile rien et me retourne pas moins de huit erreurs.(testé en mode console). La premiere erreur etant le caractère en rouge.

Est-ce que quelqu'un d'autre l'a testé et a déjà eu ce problème ?

Je compile sous WindowsXPSP2, DEV-C++.

#include
#pragma comment(lib, "ws2_32.lib")
void main()
{
WSADATA WSAData;
WSAStartup(MAKEWORD(2,0), &WSAData);
SOCKET sock;
SOCKET csock;
SOCKADDR_IN sin;
SOCKADDR_IN csin;
sock = socket(AF_INET, SOCK_STREAM, 0);
sin.sin_addr.s_addr = INADDR_ANY;
sin.sin_family = AF_INET;
sin.sin_port = htons(23);
bind(sock, (SOCKADDR *)&sin, sizeof(sin));
listen(sock, 0);
while(1)
{
int sinsize = sizeof(csin);
if((csock = accept(sock, (SOCKADDR *)&csin, &sinsize)) != INVALID_SOCKET)
{
send(csock, "Hello world!\r\n", 14, 0);
}
}
}

[Nigo]

c'est bon j'ai trouvé.

[PRomu@ld]

Code :

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

1
2
 
#include

Soucis, il manque quelque chose après ...

Code :

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

1
2
 
#pragma comment(lib, "ws2_32.lib")

Les directives pragma ne sont pas portables du tout.

Code :

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

1
2
 
void main()

Il date de quel année ce code ?

Code :

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

1
2
 
sock = socket(AF_INET, SOCK_STREAM, 0);

Toujours tester le retour d'une fonction système

Code :

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

1
2
 
int sinsize = sizeof(csin);

Pour un opérateur sizeof, n'utilise pas un int mais un type size_t.

Code :

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

1
2
3
4
5
6
7
8
9
 
while(1) 
{ 
     int sinsize = sizeof(csin); 
     if((csock = accept(sock, (SOCKADDR *)&csin, &sinsize)) !=INVALID_SOCKET) 
     { 
          send(csock, "Hello world!\r\n", 14, 0); 
     } 
}

Elle s'arrête quand la boucle infinie ?

Tu peux nous dire où tu as trouvé ce code ? (url ?)

[Skyounet]

Code :

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

1
2
 
sock = socket(AF_INET, SOCK_STREAM, 0);

Je dirai meme mieux

Code :

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

1
2
 
sock = socket(PF_INET, SOCK_STREAM, 0);

Code :

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

1
2
3
4
5
6
7
8
9
 
while(1) 
{ 
     int sinsize = sizeof(csin); 
     if((csock = accept(sock, (SOCKADDR *)&csin, &sinsize)) !=INVALID_SOCKET) 
     { 
          send(csock, "Hello world!\r\n", 14, 0); 
     } 
}

Elle s'arrête quand la boucle infinie ?

C'est sensé etre un serveur tournant tout le temps, envoyant Hello World à quiconque se connecte.

Tu peux nous dire où tu as trouvé ce code ? (url ?)

Ici-meme
http://c.developpez.com/WalrusSock/

[Kr00pS]

D'un côté, il date de 2002.

[Skyounet]

D'un côté, il date de 2002.

Et?

2002 c'est pas si bien vieux.

[Kr00pS]

Bah ça fait quand même 4 ans qu'il n'est plus à jour.

[PRomu@ld]

2002 c'est pas si bien vieux.

Tout est relatif, quand je parlais d'année, il s'agissait de la façon dont on créeait le main (pas de void depuis un moment).

C'est sensé etre un serveur tournant tout le temps, envoyant Hello World à quiconque se connecte.

Oui d'accord, vu sous cet angle, mais je n'aime pas la méthode ( j'aurai privilègié un thread pour envoyer le message. ).

De plus, suivant le type de système, il faudrait tout de même gérer les signauxs par exemple pour quitter proprement le serveur (plutôt qu'un arrêt violent...)

[Emmanuel Delahaye]

C'est sensé etre un serveur tournant tout le temps, envoyant Hello World à quiconque se connecte.

Oui d'accord, vu sous cet angle, mais je n'aime pas la méthode ( j'aurai privilègié un thread pour envoyer le message. ).

Dans un code d'initiation ? Et select() ?

De plus, suivant le type de système, il faudrait tout de même gérer les signauxs par exemple pour quitter proprement le serveur (plutôt qu'un arrêt violent...)

On est tous d'accord, un select() aurait été mieux. Mais là, ce n'est pas le code de 'Apache, c'est juste un exemple de base...

Mais je suis d'accord pour que le prototype de main() soit mis en conformité.