Discussion :

[Ludo75]

salut jai un souci avec une fonction en C. Je voudrais faire en sorte que dans mon fichier il n'y est pas de doublon. Donc pas 2 fois la meme personne avec le meme nom et le meme prenom. Jai donc fait une fonction.
Pour l'instant je ne fais pas de paramètres, jai juste testé. Je demande donc le nom et le prenom, et je verifie si cette personne est deja inscrite dans le fichier. Mais ça passe pas. Jai l'impression qu'il me bloque au niveau de la 1ere personne inscrite.
Voila mon code

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
void doublon()
{
	char n[LG];
	char nom[LG];
	char prenom[LG];
	FILE *f;
	int trouve=0;
	seleve eleve;
 
	strcpy(n,nomdufichier());
 
 
f=fopen(n,"r");	
 
 
		do{
 
			printf("\n\nEntrez un nom:");
			flushall();
			gets(nom);
			flushall();
			printf("\nEntrez un prenom:");
			flushall();
			gets(prenom);
			flushall();
			fread(&eleve,sizeof(eleve),1,f);
			if( (strcmp(nom,eleve.nom)==0) && (strcmp(prenom,eleve.prenom)==0) )
			{
 
				trouve=1;
				flushall();
				printf("Y a un doublon");
			}
		}while ( (!trouve) && (!feof(f)) );
 
flushall();
printf("Y a pas de doublon");
fclose(f);
getch();
}

Comment faire pour que ça marche?
Merci d'avance

[Anomaly]

Qu'est-ce que tu cherches à faire ?

Si tu veux vérifier qu'une telle personne n'existe pas dans ton fichier, alors tu as mal placé ta boucle. Mets tes fonctions de saisies en-dehors de la boucle.

Si tu veux vérifier qu'il n'y a pas de doublons dans un fichier, alors c'est plus complexe que ça. Tu peux procéder de deux manières :
a) Soit utiliser deux boucles imbriquées pour comparer chaque élément du fichier avec tout ceux qui le suivent
b) Soit utiliser un fichier trié, et faire une boucle qui compare chaque élément avec son suivant

Ensuite, ton code est rempli "d'horreurs" :

Code :

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

void doublon()

Pas d'arguments ? Il faut mettre "void" : void doublon(void)

Code :

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

flushall();

Vu que tu ne cherches qu'à afficher tes printf(), un fflush(stdout) serait plus efficace.

Code :

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

gets(nom);

Surtout ne jamais utiliser gets() !! http://c.developpez.com/faq/c/?page=...EEN_gets_fgets

Code :

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

getch();

Code non standard. Utilise getchar() en boucle ou fgets() pour faire une pause.

[Ludo75]

merci de ta reponse, je vais travaillé ça. Et désolé si mon code est rempli d'horreurs, je suis pas un pro du C

[dirty_boy]

Salut,

Je vois aussi à rajouter aux erreurs preécédentes celle dans le cas ou tu a un doublon:

dans ce cas tu vas afficher qu'il y a un doublon puis sortir de ta boucle while et dire qu'il n'y a pas de doublon.

Perso je mettre l'envoie de doublon/pas doublon dans un if else or de la boucle.

cordialement.