Discussion :

[the_penguin]

Bonjour,
Je viens de créer mon premier code utilisant les sockets (presque entièrement pompé sur le tuto de The Walrus) et celui ci ne fonctionne pas :-(
Voici le code 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
#include <iostream>
#include <winsock2.h>
 
int main()
{
    WSADATA WSAData;
    WSAStartup(MAKEWORD(2,0), &WSAData);
    SOCKET sock;
    SOCKADDR_IN sin;
    char *buffer = new char[255];
    sock = socket(AF_INET, SOCK_STREAM, 0);
    sin.sin_addr.s_addr			= inet_addr("127.0.0.1");
    sin.sin_family				= AF_INET;
    sin.sin_port				= htons(4148);
    connect(sock, (SOCKADDR *)&sin, sizeof(sin));
    recv(sock, buffer, sizeof(buffer), 0);
    closesocket(sock);
    WSACleanup();
    std::cout << buffer[0] << std::endl;
    system("PAUSE");
}

Et le 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
 
#include <winsock2.h>
 
int 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(4147);
    bind(sock, (SOCKADDR *)&sin, sizeof(sin));
    listen(sock, 0);
    for(;;)
    {
        int sinsize = sizeof(csin);
        if((csock = accept(sock, (SOCKADDR *)&csin, &sinsize)) != INVALID_SOCKET)
        {
            send(csock, "Hello world!\r\n", 14, 0);
        }
    }
}

Je lance les 2 programmes sur ma machine (en commençant par le seveur) et la chaine que le client doit afficher est remplacée par d'autres caractères (toujours les mêmes).
Je ne trouve pas mon erreur.
Merci

[Skyounet]

Code :

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

1
2
 
    std::cout << buffer[0] << std::endl;

C ou C++ à toi de choisir...

TU n'utilises pas les meme ports sur le serveur et le client...

[granquet]

ce qui serait bien, c'est que ton client se connecte sur le port que le server utilise.
et teste le retour de tes fonctions ... connect etc...
et puis y'as des trucs que je comprend pas ... ca ressemble a du C++.
et pis conserve le type int pour tes sockets, windows en feras pas une maladie, et ce seras plus facilement portable.

edit:: haben grillé ...

et:

sock = socket(PF_INET, SOCK_STREAM, 0);

[Emmanuel Delahaye]

Voici le code client :

Code :

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

#include <iostream>

Pas du C. C++, c'est à coté

MODO : A déplacer sur C++. Merci

[zul]

sizeof(buffer) == sizeof(char*) != 255 ...
Ca devrait résoudre le souci. Cette manie d'utiliser une allocation dynamique pour un buffer de taille fixe ...

[the_penguin]

Suite à vos conseils, j'ai modifié mon code : J'ai mis le même port pour le client et le serveur et j'ai mis un buffer static pour simplifier.
Voici mon code 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
 
#include <stdio.h>
#include <winsock2.h>
 
int main()
{
    WSADATA WSAData;
    WSAStartup(MAKEWORD(2,0), &WSAData);
    SOCKET sock;
    SOCKADDR_IN sin;
    char buffer[255];
    sock = socket(AF_INET, SOCK_STREAM, 0);
    sin.sin_addr.s_addr			= inet_addr("127.0.0.1");
    sin.sin_family				= AF_INET;
    sin.sin_port				= htons(4147);
    connect(sock, (SOCKADDR *)&sin, sizeof(sin));
    recv(sock, buffer, sizeof(buffer), 0);
    closesocket(sock);
    WSACleanup();
    std::printf("%s", buffer);
    system("PAUSE");
}

Et le 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
 
#include <winsock2.h>
 
int 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(4147);
    bind(sock, (SOCKADDR *)&sin, sizeof(sin));
    listen(sock, 0);
    for(;;)
    {
        int sinsize = sizeof(csin);
        if((csock = accept(sock, (SOCKADDR *)&csin, &sinsize)) != INVALID_SOCKET)
        {
            send(csock, "Hello world!\r\n", 14, 0);
        }
    }
}

Mon "Hello World!" s'affiche bien chez le client mais il est suivit de plusieurs caractères indésirables.
De plus, je n'ai pas compris la différence entre AF_INET et PF_INET.

[Emmanuel Delahaye]

Code :

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

1
2
3
4
 
    sock = socket(AF_INET, SOCK_STREAM, 0);
    sin.sin_addr.s_addr			= inet_addr("127.0.0.1");
    sin.sin_family				= AF_INET;

De plus, je n'ai pas compris la différence entre AF_INET et PF_INET.

PF_xxx est une constante désignant la Protocol Family. C'est le premier paramètre de socket(). (ici, le type 'Internet', soit Protocole Internet, ou IP)

AF_xxx est une constante désignant la Address Family. C'est la valeur qu'attend le champ sin_family qui concerne l'adressage (ici, de type Internet, c'est à dire IP).

[Emmanuel Delahaye]

Voici mon code client :

Code :

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

1
2
 
    std::printf("%s", buffer);

Pas du C. le C++, c'est à coté...

[Emmanuel Delahaye]

Voici mon code client :

Code :

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

1
2
3
4
 
    char buffer[255];
 
    recv(sock, buffer, sizeof(buffer), 0);

Erreur récurrente. Pas de 0 final, pas de place pour le 0 final...

Code :

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

1
2
3
4
5
6
7
 
   char buffer[255];
 
   int n = recv(sock, buffer, sizeof buffer - 1, 0);
   if (n >= 0)
   {
      char buffer[n] = 0;

Apprenez le C avant de jouer avec les sockets...