Discussion :

[ThanosT]

Bonjour a tous.

Je suis depuis plusieurs heures sur un probleme qui est pourtant tout simple. Je ne vois vraiment pas ou pourait etre le probleme. Bientot je m'arrache les cheveux. Donc si vous pouviez m'eviter la calvitie , j'apprecierai grandement.


je vous presente mon main.

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
32
33
 
int main(int argc, char *argv[])
{
	char option;
	char *Nom = (char*)malloc(sizeof(char*));
	int Num;
	AnnuR a[MAX];
	CreateAnnuR("Test",a);
 
	while(option !='q')
	{
		printf("Commande : ");
		scanf("%c",&option);
		fflush(stdin);
 
		if(option == 'i')
		{
			printf("Nom \t: \t", Nom);
			scanf("%s",Nom);
			fflush(stdin);
			printf("\nNumero \t: \t", Num);
			scanf("%d",&Num);
			fflush(stdin);
			NewUser(a,Nom,Num);
 
		}
		else if(option == 'c')
		{
			ConsultingAnnuR(a);
		}
                }
return 0;
}

Les fonction utilisé CreateAnnuR, NewUser et ConsultingAnnur sont tres simple.

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
32
33
34
35
36
37
38
39
40
 
 
typedef struct AnnuR
{
	char *nomAnnu;
	char *nom;
	int num;
	int indexF;
}AnnuR;
 
void CreateAnnuR(char *NomAnnu, AnnuR *Annu)
{	
	int i;
	Annu->indexF = 0;
	printf("Creation de l'annuaire ... %s\n", NomAnnu);
	for(i=0;i<MAX;i++)
	{
		Annu[i].nom = "";
	}
}
 
void ConsultingAnnuR(AnnuR *a)
{
	int i =0;
	while(i<a->indexF)
	{	
		printf("Nom \t:\t %s\n", a[i].nom);
		printf("Numero \t:\t %d\n", a[i].num);
		printf("======================================\n");
		i++;
 
	}
}
 
void NewUser(AnnuR *Annu, char *unNom, int unNum)
{
	Annu[Annu->indexF].nom = unNom;
	Annu[Annu->indexF].num = unNum;
	Annu->indexF ++ ;
}

Le probleme de ce programme c'est quand je crée 2 NewUser (ou plus). Le nom des precedent User dans le tableau est remplacé par celui du dernier User crée.


Si vous pouviez m'aider.

Merci

[Trap D]

Nom est beaucoup trop petit pour contenir un nom de 20 caractères, ça fonctionne peut-être mais c'est un coup de chance.

D'autre part, si tu utilises NewUser(AnnuR *Annu, char *unNom, int unNum) dans une boucle pour créer des usagrs, tu risques d' affecter toujours la même adresse au éléments de l'annuaire, donc tu auras toujours le même nom afficher (ce qui se produit apparemment).

Réfléchis à ce que je viens de dire et poses des questions si besoin est.

Bon courage.

[ThanosT]

Merci pour l'info.
Tu avais raison pour Nom, il etait trop petit, je lui ai donc allouer plus de memoire.

mais le probleme persiste. J'ai enlever la boucle while (pour faire un test)
et le probleme persiste.

j'envoie un petit aperçus du probleme

[Emmanuel Delahaye]

Code :

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

1
2
		scanf("%c",&option);
		fflush(stdin);

En plus de ce qui a été dit, j'ajoute que telles quelles, les saisies par scanf() sont fragiles, instables et difficile à écrire correctement. D'autre part, fflush() n'est défini que pour les flux sortants.

Je conseille ceci :

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers

[Trap D]

Il faut allouer de la mémoire à chaque nouvel utilisateur :

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
void NewUser(AnnuR *Annu, char *unNom, int unNum)
{
  char *ptr;
 
  ptr = malloc(strlen(unNom) + 1);
  if (ptr != NULL)
  { 
    strcpy(ptr, unNom);
    Annu[Annu->indexF].nom = ptr;
   }
   else
   {  
      fprintf(stderr, "Pb mémoire\n");
      return;
   }
   Annu[Annu->indexF].num = unNum;
   Annu->indexF ++ ;
}

PS il ne faudra pas oublier le libérer la mémoire à la sortie du prog.

[Emmanuel Delahaye]

Tu avais raison pour Nom, il etait trop petit, je lui ai donc allouer plus de memoire.

mais le probleme persiste. J'ai enlever la boucle while (pour faire un test)
et le probleme persiste.

j'envoie un petit aperçus du probleme

[img ]http://www-iutinfo.unice.fr/~dutratfe/Test/annuR.jpg[/img]

Non. Pas d'image. Faire un copié collé.

Dans la structure, il faut stocker une copie de la chaine saisie. Soit en prévoyant de l'espace fixe (tableau de char), soit en attachant un bloc alloué (pointeur, comme tu as fait) avec, par exemple la fonction POSIX.1 'strdup())' (Non standard, mais largement diffusée et facilement reproductible).

Grilled!

[ThanosT]

Merci les gars vous etes au poil (citation de Monstre et companie).
Je vous remercie du fond du coeur.
Vous m'avez sauver ma santé mentale.

Merci encore

[Médinoc]

D'autre part, fflush() n'est défini que pour les flux sortants.

En effet, et cela peut être un problème pour toi si tu essaies de faire fonctionner le programme autre part que sous Windows (Comportement indéfini).
Heureusement, avec des fgets(), on n'a pas trop besoin de vider le buffer (moins qu'avec scanf(), en tout cas).

PS: Emmanuel, tu sais comment on peut vider le buffer en C (ou du moins savoir s'il est vide ou non) ?

Edit: Oui, j'ai lu tes articles, mais ils n'ont pas répondu à ma question...

[Emmanuel Delahaye]

Heureusement, avec des fgets(), on n'a pas trop besoin de vider le buffer (moins qu'avec scanf(), en tout cas).

PS: Emmanuel, tu sais comment on peut vider le buffer en C (ou du moins savoir s'il est vide ou non) ?

Il suffit d'utiliser les bonnes fonctions de saisies et de les utiliser correctement. Je poste ces liens au moins 3 fois par jour...

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers