Discussion :

[koiki78]

Bonjour,

J'ai besoin de récupérer dans une liste chainé les packet UDP (4 packets) avec recvfrom(); cependant lorsque je retourne ma liste le pointeur sur la structure suivante vaut NULL donc je ne peux afficher qu'une seul liste (celle que je retourne), j'ai une fonction afficher(); qui me permet d'afficher toutes les listes chainée qui elle fonctionne bien, le problème se pose donc sur ma fonction de récupération des 4 packets UDP ci-dessous :

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
 
DECOUPE_TIF_STRUC			       *recv_data(int sock)
{
	int						nOctets;
	int						i;
	SOCKADDR_IN				sin = {0};
	int						sinsize = sizeof(sin);
	unsigned char 			        buffer[65508] = {0};
	DECOUPE_TIF_STRUC		       *new_t = NULL;
	DECOUPE_TIF_STRUC		       *temp = NULL;
	for(i = 0; i < 4; i++)
	{
		new_t = (DECOUPE_TIF_STRUC*)malloc(sizeof(*new_t));		
		new_t->next = NULL;
		if (new_t == NULL)
			return (temp);
		if ((nOctets = recvfrom(sock, (char *)new_t->data, sizeof(buffer), 0, (SOCKADDR *)&sin, &sinsize)) == SOCKET_ERROR)
		{
			perror("recvfrom()");
			wprintf(L"bind failed with error %d\n", WSAGetLastError());
			exit(errno);
		}
		buffer[nOctets] = '\0';	
		//printf("NOCTET : %d\n", nOctets);
		if (i == 0)
		{
			 temp = new_t;
		}
	}
	return (temp);
}

Lorsque je change la condition du i == 0 par i == 1 ou 2 ou 3 j'affiche respectivement les packets 0,1,2,3. Je pensais qu'il suffisait que je renvoie la structure de i == 0 la première donc pour pouvoir accéder au suivante, mais non...

Merci pour votre aide !

[diogene]

Tu n'as pas construit du tout une liste chainée : on ne voit nulle part comment chaque enregistrement est lié aux enregistrements précédents.
Chaque nouvelle allocation a son adresse mise dans new_t en écrasant l'adresse de l'allocation précédente.

[koiki78]

Merci diogene de ta réponse, j'y voit un peu plus clair.

Pourrais-tu, s'il te plait, si tu as le temps me faire une ébauche de la bonne manière à suivre selon mon code ?

Par avance merci.

[ternel]

Oui, regarde dans la faq. C'est expliqué.

Je te laisse chercher dedans, tout est bon à lire

[koiki78]

Oui j'avais déjà épluché la FAQ C et les Cours C, ce qui m'a permis d'en arriver là, cependant ce qui est commun à chaque faq est que pour insérer une nouvelle list, ils passent par des fonctions et passe en paramètre la structure précédente ainsi que la donnée à insérer et renvoie un pointeur d'une structure, mais je ne peux faire cela car le temps de copier les données (strcpy) dans la structure je perds les packets(UDP), je dois donc spécifier directement en 2ème paramètre à recvfrom() la nouvelle structure new_t->data et la je bloque totalement.

Est-ce possible de faire comme sa ?

Si oui si tu pouvais reprendre mon code et le modifier je t'en serai grandement reconnaissant, car tu as l'air d'être expert dans ce domaine.

Cordialement.

PS : Je ne suis pas adepte du copier collé à moins de comprendre ce que cela fait.

[diogene]

Pourquoi ne pas allouer dynamiquement un tableau de 4 DECOUPE_TIF_STRUC, les remplir par recvfrom() et renvoyer l'adresse du premier élément de ce tableau ?