Discussion :

[jbarreau-mainson]

Bonjour à tous,

J'ai un petit soucis lors du cast d'une structure, je ne vois pas comment faire

En fait je tente de faire un serveur multi-clients, le client.c s'occupe de "declarer" un nouveau client au serveur...
Ces codes ne concernent que le serveur.

Code :

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

1
2
3
4
5
6
7
8
9
 
/*main.c*/
       for(;;)
        {
            argc = Serveur(SERVEUR_LISTEN, 0);
            Serveur(SERVEUR_ACCEPT, &argc);
        }
//argc recupere (int)Clients[Id];
//voir client.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
 
 
 
/*serveur.c*/
 
int Serveur(int sc, int pa)
{
    switch(sc)
    {
    case SERVEUR_ACCEPT:
        //La fonction accepte la connection d'un socket clients sur le socket serveur
        if((((t_Clients)&pa).ClientSock = accept(serveur.ServeurSock, (SOCKADDR *)&((t_Clients)pa).ClientsSin, &((t_Clients)pa).SinSize)) == INVALID_SOCKET)
        {
            Fin("Echec de la connection du clients.\n");
        }
        else
        {
            printf("\nNouvelle connexion : %s se connecte.\n", inet_ntoa(((t_Clients)pa).ClientsSin.sin_addr));
            strcpy(*Buff, "-- Bienvenue sur le serveur");
            send(((t_Clients)pa).ClientsSock, *Buff, (int)strlen(*Buff), 0);
        }
        break;
}
}

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
 
t_Clients *Clients;
 
/*client.c*/
int Client(int sc, int pa)
{
    switch(sc)
    {
    case CLIENT_NEW:
        puts("Creation d'un nouveau client.");
        Id++;
        Clients[Id].SinSize = sizeof(Clients[Id].ClientSin);
        return (int)Clients[Id];
        break;
}
}

Mon problème se pose au niveau du serveur.c, notamment ici :

Code :

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

 if((((t_Clients)&pa).ClientSock

[Médinoc]

J'ai l'impression que le type de pa n'est pas ce qu'il devrait être...

[jbarreau-mainson]

En fait je cast Clients[Id] en int pour ensuite le recaster en (t_clients).
Je ne vois pas en quoi d'autre il devrait etre !

[crocodilex]

Code :

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

Serveur(SERVEUR_ACCEPT, &argc);

Code :

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

1
2
int Serveur(int sc, int pa)
{

Ce n'est pas très cohérent tout ça....

[Médinoc]

En fait je cast Clients[Id] en int pour ensuite le recaster en (t_clients).
Je ne vois pas en quoi d'autre il devrait etre !

Pourquoi pas en t_clients ou t_Clients*?

Ou bien, à défaut, en intptr_t minimum. On vit à l'ère du 64 bits maintenant, et rien ne garantit qu'un int soit assez grand pour contenir un pointeur.

[jbarreau-mainson]

Pourquoi pas en t_clients ou t_Clients*?

c'est déjà en t_clients.
Si je met en t_clients * j'ai l'erreur suivante :

Code :

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

error: request for member `ClientSock' in something not a structure or union|

On vit à l'ère du 64 bits maintenant, et rien ne garantit qu'un int soit assez grand pour contenir un pointeur.

La plateforme est 32bits.

Ce n'est pas très cohérent tout ça....

J'ai enlevé le "&".

[Médinoc]

Et si tu faisais tout simplement ceci?

Code C :

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
int Serveur(int sc, clients_t *pa)
{
	switch(sc)
	{
	case SERVEUR_ACCEPT:
		/*La fonction accepte la connection d'un socket clients sur le socket serveur*/
		pa->ClientSock = accept(serveur.ServeurSock, (SOCKADDR *)&(pa->ClientsSin), &(pa->SinSize));
		if(pa->ClientSock == INVALID_SOCKET)
		{
			Fin("Echec de la connection du clients.\n");
		}
		else
		{
			printf("\nNouvelle connexion : %s se connecte.\n", inet_ntoa(pa->ClientsSin.sin_addr));
			strcpy(*Buff, "-- Bienvenue sur le serveur");
			send(pa->ClientsSock, *Buff, (int)strlen(*Buff), 0);
		}
		break;
	}
}

PS: Buff est un pointeur de pointeur?