mini projet C

**eminemanaas1985** · 07/03/2007, 20h02

Voila mon programme qui conciste a créer une annuaire telephonique
lorsque je comile il me donne 0 erreur et 0 warning mais lorsque je link il me donne 1 erreur 0 warnig

je ne sais pas où est le problème, le programme est le suivant:

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
#include<stdio.h>
struct personne {
	char nom[20];
	char num[20];
}personne[20];
void siasie(struct personne *perso,int nombre)
{
	int i;
	for(i=0;i<nombre;i++)
	{
		printf("entrez le nom = ");
	    scanf("%s",&perso->nom);
        printf("entrez le num = ");
	    scanf("%s",&perso->num);
	}
 
}
 
void affiche (struct personne perso[20],int nombre)
{
	int i;
	for(i=0;i<nombre;i++)
	{
		printf("%d de numero %d", perso->nom,perso->num);
	}
}
 
void main(void)
{
	struct personne perso[20];
	int nombre;
	printf("entrez le nombre");
	scanf("%d",&nombre);
	siasie(perso,nombre);
	affiche(perso,nombre);
}

**scriptoff** · 07/03/2007, 20h07

balise code stp
le code avec les modifs

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
#include <stdio.h>



struct personne {
    char nom[20];
    char num[20];
} personne[20];

void siasie(struct personne *perso, int nombre)
{
    int i;
    for (i = 0; i < nombre; i++) {
	printf("entrez le nom = ");
	scanf("%s", perso[i].nom);
	printf("entrez le num = ");
	scanf("%s", perso[i].num);
    }

}

void affiche(struct personne perso[20], int nombre)
{
    int i;
    for (i = 0; i < nombre; i++) {
	printf("%s de numero %s", perso[i].nom, perso[i].num);
    }
}

int main(void)
{
    struct personne perso[20];
    int nombre;
    printf("entrez le nombre");
    scanf("%d", &nombre);
    siasie(perso, nombre);
    affiche(perso, nombre);
    return 0;
}

**Emmanuel Delahaye** · 07/03/2007, 20h46

Envoyé par eminemanaas1985

Voila mon programme qui conciste a créer une annuaire telephonique
lorsque je comile il me donne 0 erreur et 0 warning

Ca me fera toujours rire...

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums2\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `siasie':
main.c:14: warning: char format, different type arg (arg 2)
main.c:16: warning: char format, different type arg (arg 2)
main.c: In function `affiche':
main.c:24: warning: int format, pointer arg (arg 2)
main.c:24: warning: int format, pointer arg (arg 3)
main.c: At top level:
main.c:28: warning: return type of 'main' is not `int'
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 1 seconds)
0 errors, 5 warnings

http://emmanuel-delahaye.developpez....tm#cfg_compilo

**Thierry Chappuis** · 07/03/2007, 21h51

Voici quelques modififcations suite à la contribution de scriptoff. Saisir des chaines de caractères avec scanf("%s", tampon), c'est fragile et ça devrait être interdit. Deux solutions: on apprend à utiliser scanf() avec cet excellent tutoriel http://xrenault.developpez.com/tutoriels/c/scanf/ ou on utilise fgets():

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
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
#include <stdio.h>
#include <string.h>
 
#define NMAX 20
 
struct personne {
    char nom[20];
    char num[20];
};
 
/**
 * -TC- Vide le tampon du flux d'entrée standard.
 */
static void vider_stdin(void)
{
    scanf("%*[^\n]");
    getchar();
}
 
/**
 * -TC- Saisit un chaîne de caractère entrée par l'utilisateur de manière
 * sécurisée.
 *
 * Si la chaîne entrée par l'utilisateur est trop longue pour être contenue
 * le tampon prévu à cet effet, un message d'erreur est affiché, et
 * l'utilisateur est invité à répéter sa saisie.
 *
 * @param s_tampon Tampon destiné à recevoir la chaîne de l'utilisateur.
 * @param taille Taille du tampon.
 * @param s_message Invite à afficher à l'utilisateur.
 */
static void saisie_chaine_stdin(char *s_tampon,
                                size_t taille,
                                char const *s_message)
{
    int saisie_ok = 0;
    char *pc = NULL;
 
    while (saisie_ok != 1)
    {
        /* -TC- On affiche le message à l'utilisateur */
        printf("%s ", s_message);
        fflush(stdout);
 
        /*-TC- Saisie de la chaîne */
        fgets(s_tampon, taille, stdin);
        if ((pc = strchr(s_tampon, '\n')) != NULL) /* -TC- saisie OK */
        {
            saisie_ok = 1;
            *pc = '\0';
        }
        else /* -TC-La chaîne de l'utilisateur a été tronquée */
        {
            /*-TC- Si la chaine est trop longue, on affiche un message
              d'erreur, on vide le tampon et on recommence */
            fprintf(stderr, "Votre entrée est trop longue! Réessayez..\n");
            vider_stdin();
        }
    }
}
 
void saisie(struct personne perso[], int nombre)
{
    int i;
    for (i = 0; i < nombre; i++) {
        saisie_chaine_stdin(perso[i].nom, sizeof perso[i].nom,
                            "entrez le nom = ");
        saisie_chaine_stdin(perso[i].num, sizeof perso[i].num,
                            "entrez le num = ");
    }
 
}
 
void affiche(struct personne perso[], int nombre)
{
    int i;
    for (i = 0; i < nombre; i++) {
        printf("%s de numero %s\n", perso[i].nom, perso[i].num);
    }
}
 
int main(void)
{
    struct personne perso[NMAX];
    int nombre;
    int rc;
 
    printf("entrez le nombre (entre 0 et %d): ", NMAX);
    /* On force l'écriture sur stdout */
    fflush(stdout);
 
    /* -TC- sécurisons un peu la saisie avec scanf() */
    while ((rc = scanf("%d", &nombre)) != 1 || nombre < 0 || nombre > NMAX)
    {
        /*-TC- La saisie a échoué: on vide la tampon et on recommence */
        vider_stdin();
        printf("entrez le nombre (entre 0 et %d): ", NMAX);
        fflush(stdout);
    }
    /*-TC- Ici, il reste au minimum '\n' sur le tampon de stdin. On purge cela*/
    vider_stdin();
 
    saisie(perso, nombre);
    affiche(perso, nombre);
    return 0;
}

Thierry

**Emmanuel Delahaye** · 07/03/2007, 22h37

Envoyé par mujigka

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
int main(void)
{
    /* -TC- sécurisons un peu la saisie avec scanf() */
    while ((rc = scanf("%d", &nombre)) != 1 || nombre < 0 || nombre > NMAX)
    {
        /*-TC- La saisie a échoué: on vide la tampon et on recommence */
        vider_stdin();
        printf("entrez le nombre (entre 0 et %d): ", NMAX);
        fflush(stdout);
    }
    /*-TC- Ici, il reste au minimum '\n' sur le tampon de stdin. On purge cela*/
    vider_stdin();
 
    saisie(perso, nombre);
    affiche(perso, nombre);
    return 0;
}

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
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
 
#include "ed/inc/io.h"
 
#include <stdio.h>
#include <string.h>
#include <limits.h>
 
#define NMAX 20
 
struct personne
{
   char nom[20];
   char num[20];
};
 
static void saisie_chaine (char *s_tampon, size_t taille,
                           char const *s_message)
{
   int err = 0;
   do
   {
      printf ("%s ", s_message);
      fflush (stdout);                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                               /*-TC- Saisie de la chaîne */
 
      err = get_s (s_tampon, taille);
      if (err)
      {                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                            /*-TC- Si la chaine est trop longue, on affiche un message d'erreur, on vide le tampon et on recommence */
         fprintf (stderr, "Votre entree est trop longue! Ressayez..\n");
      }
   }
   while (err);
}
 
static void saisie (struct personne perso[], int nombre)
{
   int i;
   for (i = 0; i < nombre; i++)
   {
      saisie_chaine (perso[i].nom, sizeof perso[i].nom, "entrez le nom = ");
      saisie_chaine (perso[i].num, sizeof perso[i].num, "entrez le num = ");
   }
}
 
static void affiche (struct personne perso[], int nombre)
{
   int i;
   for (i = 0; i < nombre; i++)
   {
      printf ("%s de numero %s\n", perso[i].nom, perso[i].num);
   }
}
 
int main (void)
{
   struct personne perso[NMAX];
   int nombre = -1;
   int err;
 
   do
   {
      long n;
      printf ("entrez le nombre (entre 0 et %d): ", NMAX);
      fflush (stdout);
      err = get_l (&n) || n < INT_MIN || n > INT_MAX;
      if (!err)
      {
         nombre = (int) n;
      }
      else
      {
         fprintf (stderr, "Nombre errone ou hors limites! Reessayez..\n");
      }
   }
   while (err || nombre < 0 || nombre > NMAX);
 
   saisie (perso, nombre);
   affiche (perso, nombre);
   return 0;
}

http://emmanuel-delahaye.developpez.com/clib.htm
Module IO

**Thierry Chappuis** · 07/03/2007, 22h48

Using à gaz? Pourquoi? Avec scanf(), je ne vois pas comment faire autrement, à part encapsuler tout ça dans une fonction saisir_entier()... Ai-je mal fait?

EDIT: J'admets qu'il y a plus simple...

Thierry

**Thierry Chappuis** · 07/03/2007, 23h38

Envoyé par Emmanuel

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
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
 
#include "ed/inc/io.h"
 
#include <stdio.h>
#include <string.h>
#include <limits.h>
 
#define NMAX 20
 
struct personne
{
   char nom[20];
   char num[20];
};
 
static void saisie_chaine (char *s_tampon, size_t taille,
                           char const *s_message)
{
   int err = 0;
   do
   {
      printf ("%s ", s_message);
      fflush (stdout);                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                               /*-TC- Saisie de la chaîne */
 
      err = get_s (s_tampon, taille);
      if (err)
      {                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                            /*-TC- Si la chaine est trop longue, on affiche un message d'erreur, on vide le tampon et on recommence */
         fprintf (stderr, "Votre entree est trop longue! Ressayez..\n");
      }
   }
   while (err);
}
 
static void saisie (struct personne perso[], int nombre)
{
   int i;
   for (i = 0; i < nombre; i++)
   {
      saisie_chaine (perso[i].nom, sizeof perso[i].nom, "entrez le nom = ");
      saisie_chaine (perso[i].num, sizeof perso[i].num, "entrez le num = ");
   }
}
 
static void affiche (struct personne perso[], int nombre)
{
   int i;
   for (i = 0; i < nombre; i++)
   {
      printf ("%s de numero %s\n", perso[i].nom, perso[i].num);
   }
}
 
int main (void)
{
   struct personne perso[NMAX];
   int nombre = -1;
   int err;
 
   do
   {
      long n;
      printf ("entrez le nombre (entre 0 et %d): ", NMAX);
      fflush (stdout);
      err = get_l (&n) || n < INT_MIN || n > INT_MAX;
      if (!err)
      {
         nombre = (int) n;
      }
      else
      {
         fprintf (stderr, "Nombre errone ou hors limites! Reessayez..\n");
      }
   }
   while (err || nombre < 0 || nombre > NMAX);
 
   saisie (perso, nombre);
   affiche (perso, nombre);
   return 0;
}

http://emmanuel-delahaye.developpez.com/clib.htm
Module IO

OK! Mais si ton usine à gaz faisait référence à scanf(), ce n'est pas moi mais le PO de ce fil qui ai fait le choix d'utiliser cette fonction. Je voulais le rendre attentif au fait qu'il était possible que la saisie avec scanf() échoue, de même que la nécessité que le nombre saisi soit borné. J'admets que le code que j'ai proposé n'était pas des plus élégants, mais pas spécialement difficile à assimiler.

Thierry

**Emmanuel Delahaye** · 07/03/2007, 23h50

Envoyé par mujigka

OK! Mais si ton usine à gaz faisait référence à scanf(),

Surtout à tout le code parasite que ça génère. Pourquoi il fait vider stdin() ici, ou là dans le code applicatif...

Non seulement, les syntaxes de scanf() sont imbitables, mais en plus on passe son temps à nettoyer le tuyau...

Je dis qu'il faut passer 1 ou 2 jours à résoudre le problèmes de saisies une bonne fois pour toutes (c'est d'ailleurs la raison d'être d'une partie des exercices du K&R...), et ensuite, affaire réglée définitivement. On passe à autre chose.

**Thierry Chappuis** · 08/03/2007, 00h12

Envoyé par Emmanuel Delahaye

Surtout à tout le code parasite que ça génère. Pourquoi il fait vider stdin() ici, ou là dans le code applicatif...

Non seulement, les syntaxes de scanf() sont imbitables, mais en plus on passe son temps à nettoyer le tuyau...

Je dis qu'il faut passer 1 ou 2 jours à résoudre le problèmes de saisies une bonne fois pour toutes (c'est d'ailleurs la raison d'être d'une partie des exercices du K&R...), et ensuite, affaire réglée définitivement. On passe à autre chose.

OK! Mais le fait est que de nombreux débutants débarquent sur ce forum avec du code et des scanf() partout. Cela ne fait pas beaucoup avancer les choses si je balance dans le tas les fonctions de ma bibliothèque toute faite. Je suis d'accord avec ta façon d'aborder les choses, puisque ton site m'a guidé et me guident toujours dans mon apprentissage (et en particulier sur la gestion des I/O, mais aussi tous les autres modules de la clib).

J'aime beaucoup l'approche de K&R, et je regrette que ce ne soit pas de cette façon que le C soit enseigné à l'université, dans les écoles d'ingénieur, ou ailleurs. scanf() est une usine à gaz, nous somme bien d'accord. Mais lorsque quelqu'un l'utilise à la légère, je préfère mettre le doigt sur les faiblesses que présentent son code, et éventuellement l'aiguiller ensuite sur la construction de ses propres primitives de saisie. Je ne pense que ce soit bénéfique qu'un débutant utilise directement les fonctions de la clib. Il est plus profitable qu'il en reprenne l'interface, et qu'il ré-implante les fonctions à coups de fgetc(), strtol, strtod, etc.

Thierry

**Emmanuel Delahaye** · 08/03/2007, 00h19

Envoyé par mujigka

Je ne pense que ce soit bénéfique qu'un débutant utilise directement les fonctions de la clib. Il est plus profitable qu'il en reprenne l'interface, et qu'il ré-implante les fonctions à coups de fgetc(), strtol, strtod,

Evidemment. J'ai balancé intentionnellement un exemple un peu provoquant pour montrer que "si on veut, on peut"...

... alors qu'avec scanf(), cette fonction du Diable, on a beau faire des efforts louables, on arrive à une magnifique usine à gaz difficile à utiliser, à maintenir, à faire évoluer... Ca parait trop fragile ces purges dans tous les coins... Ca ne donne pas confiance, et je pense que ça fait peur aux débutants...

En tout cas, moi, ça me fout la trouille !

**Thierry Chappuis** · 08/03/2007, 00h36

Envoyé par Emmanuel Delahaye

Evidemment. J'ai balancé intentionnellement un exemple un peu provoquant pour montrer que "si on veut, on peut"...

... alors qu'avec scanf'(), cette fonction du Diable, on a beau faire des efforts louables, on arrive à une magnifique usine à gaz difficile à utiliser, à maintenir, à faire évoluer... Ca parait trop fragile ces purges dans tous les coins... Ca ne donne pas confiance, et je pense que ça fait peur aux débutants...

En tout cas, moi, ça me fout la trouille !

Nous sommes donc d'accord au sujet de scanf(). Comment proposes-tu d'orienter un programmeur débutant en langage C qui utilise scanf() pour tout et en plus souvent de manière peu robuste et absolument pas sécurisée (la mode, ça semble être scanf("%s", tampon): je ne sais pas où on apprend ça). Je vais essayer de réfléchir à la meilleure manière d'aider les débutants dans ce domaine. Ton site est très bien pour cela. Pour une correction de code à cours terme... Je pense qu'avant de se lancer dans la conception de fonctions de saisie personnalisées, un débutant a besoin de pouvoir mesurer quelles sont les faiblesses des méthodes qu'il a apprises en cours. Il comprendra alors qu'il ne peut lui être que profitable de construire sa propre bibliothèque.

Thierry