Discussion :

[clampin]

Bonjour,

J'ai trouvé du temps pour continuer mon apprentissage du C...

Je me suis mis dans l'idée d'écrire dans un fichier texte une série de quelques renseignement d'un carnet d'adresses.

voici le code complet

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
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
 
#include<stdio.h>
#include<stdlib.h>
 
int main (int argc, char const* argv[])
{
	FILE *fp;
	char nom[20], ch;
	char prenom[20];
	char filename[20];
	int age;
 
	puts ("Introduisez un nom de fichier");
	gets(filename);
 
	if ((fp = fopen(filename,"a")) == NULL)
	{
		fprintf(stderr,"Erreur à l'ouverture du fichier %s", filename);
		exit(1);
	}
 
 
	while(1)
	{
		puts("Entrez votre nom : ");
		scanf("%s", &nom);
 
		puts("Entrez votre prenom : ");
		scanf("%s", &prenom);
 
		puts("Entrez votre age : ");
		scanf("%d", &age);
 
		printf("Vous avez entre\n\nNom : %s \nPrenon : %s \nAge : %d\n", nom, prenom, age);
 
		if (age < 18)
		{
			printf("Attention :Le sujet est mineur\n");
		}
		else
		{
			printf("OK le sujet est majeur\n");
		}
 
		fprintf(fp,"Nom = %s - Prenom = %s - Age = %d\n", nom, prenom, age);
 
		puts("\nAutre contact [o]ui ou [n]on ?");
 
		do
		{
			ch = getchar();
		} while (ch != 'n' && ch != 'o');
		if (ch =='n')
			/*fclose(fp);*/
			break;
	}
		printf("\n");
 
		fclose(fp);
 
	return 0;
}

Est-ce que je fais dans le bon, ou bien est-ce que je fais totalement fausse route ? (je programme sous macos x), mais je ne pense pas que j'utilise des procédure propre à osx.

Merci de me corriger...

[Médinoc]

Au niveau algorithme, c'est plus ou moins potable, mais pour le reste, tu fais des erreurs qu'il vaut mieux corriger immédiatement :

1. Utilisation de gets(). Cette fonction est un bug et ne devrait pas exister : Utiliser fgets() à la place (voir détails plus bas pour élimination du \n en plus).
2. Utilisation de scanf("%s") : C'est l'une des deux pires utilisations possibles de scanf() (l'autre étant d'enseigner scanf() à des débutants). Utiliser fgets() à la place.
3. Utilisation de & pour une chaîne passée à scanf(). Un tableau de caractère est automatiquement converti en pointeur sur caractère quand on le passe en paramètre d'une fonction. Comme c'est déjà un pointeur, le & est donc inutile (et même nuisible si tu remplaces le tableau par un char*).

[Médinoc]

Pour le fgets(), la fonction a un "inconvénient", qui est de placer dans la chaîne le \n entré. Il faut donc le retirer soi-même.
On utilise donc généralement des fonctions ainsi faites.

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
#include <stdio.h>
#include <string.h>
 
/* Vide une ligne dans le buffer d'entrée. */
void PurgeStdin()
{
	int c;
	while((c=getchar()) != '\n' && c != EOF)
	{}
}
 
/* Retire le \n de fgets() et vire les caractères en trop du buffer d'entrée */
void PurgeString(char *s)
{
	/* Commence par rechercher le \n (LF=Line Feed) */
	char *pLF = strchr(s, '\n');
	if(pLF != NULL)
	{
		/* Si LF trouvé, le supprime du buffer */
		*pLF = '\0';
	}
	else
	{
		/* LF non-trouvé: il est encore dans le flux */
		PurgeStdin();
	}
}

Ainsi, on peut utiliser fgets() avec un buffer:

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
#include <stdio.h>
#include "fonctionsPurge.h"
 
/* Macro pour donner la taille d'un tableau en éléments */
#ifndef ARRAYSIZE
#define ARRAYSIZE(x) (sizeof(x)/sizeof(x[0]))
#endif
 
int main(void)
{
	char buf[16] = "";
	int age = 0;
	int res;
	puts("Entrer votre nom :");
	fgets(buf, ARRAYSIZE(buf), stdin);
	PurgeString(buf);
 
	puts("Entrer votre age :");
	res = scanf("%d", &age);
	PurgeStdin();
 
	if(res != 1)
		puts("Erreur d'age");
	else
		printf("Vous vous appelez : \"%s\" et votre age est %d.\n", buf, age);
	return 0;
}

Je recommande également d'utiliser la fonction PurgeStdin() après un scanf("%d"), comme indiqué.

PS: Je n'ai pas testé, mais ça compile.

[clampin]

Merci pour les remarques, j'examine vos codes pour voir. Et je continue a lire les différents tutoriaux... comme j'ai pas vraiment beaucoup de temps (3 a 4 heures par semaine), je fais encore trop de fautes.... Il faudrait que je prenne plus de temps. Promis j'essayerai d'avoir plus de temps pour apprendre le C.

Je vais aussi me mettre à jours en math et en algo.

Encore merci pour vos remarques et bonne continuation au forum.