Discussion :

[innosang]

Hello !
Encore une fois j'ai un ptit prob...

J'ai créé une structure Sharer et je veux ajouter à la suite plusieurs Sharer...
on problème est que lorsque j'ajoute, tout est ok, une fois que je fai un affiche après avoir retourner le tout, ce qui a été ajouté c'est n'importe koi ! comme si il le NULL n'a pas été remplacé...

Mes sources:

Code :

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

1
2
3
4
5
6
7
8
9
typedef struct listePartage
{
	struct sockaddr_in * ip ;
	struct listePartage * suivant ;
	int nbConnection ;
 
}Sh;
 
typedef struct listePartage * Sharer ;

Code :

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

1
2
3
4
5
6
7
8
9
10
Sharer sharerNouv()
{
	Sharer  s = (Sharer) malloc(sizeof (struct listePartage)) ;
	s->suivant = NULL ;
	s->nbConnection = 0 ;
 
 
	return s ;
 
}

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
void ajoutSharer(Sharer l, struct sockaddr_in a)
{
	Sharer  s = sharerNouv() ;
 
 
	s->ip=&a ;
	printf("WHHHATTTT %s\n", inet_ntoa(s->ip->sin_addr.s_addr));
 
	Sharer courant = l ;
 
	while(courant->suivant!=NULL)
	{
		courant=courant->suivant ;
 
	}
 
	courant->suivant = s ;
 
	printf("adresse mise %s\n", inet_ntoa(s->ip->sin_addr.s_addr)) ;	
	printf("port mis %d\n", ntohs(s->ip->sin_port)) ;
 
	//return l ;
 
}

Merci d'avance !

[Trap D]

As-tu géré le premier cas ou l'argument l est NULL, ou éventuellement non initialisé ?

[innosang]

oui, tout a été initialisé au préalable...
l pointe toujous la structure initiale que je met en place avant d'ajouter quelque chose...

en fait j'ai l'impression il ne fai pas le lien entre le premier élément et celui qu'on ajoute...

[Trap D]

En fait, mo je ferais comme ça :

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
Sharer ajoutSharer(Sharer l, struct sockaddr_in a)
{
   Sharer  s = sharerNouv() ;
 
   if (s == NULL)
  {
     fprintf(stderr, "Pb mémoire\n");
     return NULL;
  }
   s->ip=&a ;
   printf("WHHHATTTT %s\n", inet_ntoa(s->ip->sin_addr.s_addr));
 
   if (l == NULL)
       return s;
 
   Sharer courant = l ;
 
   while(courant->suivant!=NULL)
   {
      courant=courant->suivant ;
 
   }
 
   courant->suivant = s ;
 
   printf("adresse mise %s\n", inet_ntoa(s->ip->sin_addr.s_addr)) ;   
   printf("port mis %d\n", ntohs(s->ip->sin_port)) ;
 
  return l ;
 
}

Maintenant, si l'ordre n'a pas d'importance, pourquoi insères-tu en fin de liste ??
C'est beaucoup plus simple et plus rapide de faire

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
Sharer ajoutSharer(Sharer l, struct sockaddr_in a)
{
   Sharer  s = sharerNouv() ;
 
   if (s == NULL)
  {
     fprintf(stderr, "Pb mémoire\n");
     return NULL;
  }
   s->ip=&a ;
   printf("WHHHATTTT %s\n", inet_ntoa(s->ip->sin_addr.s_addr));
 
   s->suivant = l;
  return s;
 
}

[innosang]

j'ai utilisé ta deuxieme solution pour ajoutSharer et ça ne change rien. L'adresse qui a été ajoutée dans la liste reste toujours bizarre ... lorsqu'on l'on affiche cette adresse dans la fonction ajoutSharer, elle correspond à ce qu'on attend .. mais lorsque l'on affiche la liste après que la fonction ajoutSharer ait été éxécutée ... l'adresse dans la liste est fausse... pourtant la fonctionne n'est pas en static ni rien je ne vois pas ...

[Trap D]

C'est sans doute un autre problème du a struct sockaddr_in a. Passe plutôt un pointeur vers a

Code :

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

Sharer ajoutSharer(Sharer l, struct sockaddr_in &a)

Car la, tu as l'adresse au moment de l'appel de la fonction qui devient fase dès qu'on sort de cette fonction.

[innosang]

c'est bon ça marche merci ... mais par contre j'ai pas compris cette histoire de fase ... tu peux men dire plus ?

[Trap D]

j'ai pas compris cette histoire de fase

moi non plus
fase : : :
D'autre part, j'ai fait une faute de frappe :

Code :

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

Sharer ajoutSharer(Sharer l, struct sockaddr_in *a)

C'est * qu'il faut mettre, on n'est pas en C++

[innosang]

ouh là lol ...

Car la, tu as l'adresse au moment de l'appel de la fonction qui devient fase dès qu'on sort de cette fonction.

et là phrase exacte serait donc ? lol

[Trap D]

OK je ne me suis pas relu

Car la, tu as l'adresse au moment de l'appel de la fonction qui devient fausse dès qu'on sort de cette fonction.