fonction : calcul du nombre d'enrgistrements d'une table

Version imprimable

Bonsoir tout le monde,

Voila j'ai fait une petite fonction qui calcule le nombre d'enregistrements d'une table quelconque. Je l'ai compilé sans faute mais le problème est qu' l'execution il me donne toujours 1 alors que dans la table il ya plus d'1 enr.
Voici le code:

Code:

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
 
#include <stdio.h>
#include <conio.h>
#include <string.h>
#define MAXENR 30 // taille maxmumun de la table
 
struct t_enr // une structure qui permet d'encoder un numero matricule et un nom
    { int matr; char nom[30];};
 
    void remplirTable(struct t_enr* t) // Une procédure qui permet d'encoder 
                                       // et la condition d'arrêt est matr = 0
    {
      int i=0;
      printf(" Matricule: ");
      scanf("%d", &t[i].matr);
      while( t[i].matr != 0)
      {
	printf(" Nom: ");
	gets(t[i].nom);
	printf(" Matricule: ");
	scanf("%d", &t[i].matr);
 
      }
    }
 
    int Haut(struct t_enr* t) /* Voila la fonction qui me permet de calculer la taille de la table. 
                                 Elle returne le numero du dernier enregistrement qui correspond 
                                 à la taille de la table*/
    {
     int h=1;
     while (t[h].matr) h++; return h-1;}
 
 
 
void main() // ma fonction main()
{
 
   struct t_enr tb[MAXENR];
 
   remplirTable(tb); // je remplis la table
 
   int h = Haut(tb); // j'appelle la fonction qui pemet de calculer la taille de la table
   printf("Le nombre des enregistrements est %d", h);
 
}

Le problème du programme c'est qu'il me returne toujours 1 à l'exécution. J'arrive pas à comprendre.

Vos contributions sont les bienvenues. Merci.

11/06/2006, 01h41
thewho

Bonjour,

1) dans la fonction remplirTable, la variable i qui pointe dans ton tableau n'est jamais incrémentée ==> tu stockes toujours dans l'enregisttrement 0

2) dans la fonction Haut, pourquoi commencer avec h = 1, pour renvoyer ensuite h-1 ? ==> tu sautes l'enregistrement 0

3) ton tableau n'est pas initialisé, rien ne te garantit que les tab[i].matr dans lesquels tu n'écris rien sont à 0.

4) ta boucle de remplissage de ton tableau, dans la fonction remplirTable est très mal construite :
- entre autres, pas de vérification de fin de tableau
- avec une meilleure construction de la boucle, tu n'aurais pas répéter la ligne de lecture

Code:

scanf("%d", &t[i].matr);

Citation:

dans la fonction remplirTable, la variable i qui pointe dans ton tableau n'est jamais incrémentée ==> tu stockes toujours dans l'enregisttrement 0

J'ai incrémenté comme suit:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 void remplirTable(struct t_enr* t) // Une procédure qui permet d'encoder // et la condition d'arrêt est matr = 0 { int i=0; printf(" Matricule: "); scanf("%d", &t[i].matr); while( t[i].matr != 0) { printf(" Nom: "); gets(t[i].nom); i++;// c'est ici que j'ai mis l'incrémentation j'espère que c'est correct printf(" Matricule: "); scanf("%d", &t[i].matr);
Citation:

ton tableau n'est pas initialisé, rien ne te garantit que les tab[i].matr dans lesquels tu n'écris rien sont à 0.

Pour cela j'ai créé une procédure pour initialiser la table:
Code:

1 2 3 void videTable(struct t_enr* t) {t[1].matr = 0;}
Je l'appelle dans main comme suit:
Code:

1 2 videTable(tb);
Citation:

ta boucle de remplissage de ton tableau, dans la fonction remplirTable est très mal construite :
- entre autres, pas de vérification de fin de tableau
- avec une meilleure construction de la boucle, tu n'aurais pas répéter la ligne de lecture

Je ne vois pas comment je peux boucler autrement. Aucune d'idée:roll:

Citation:
Envoyé par zui
J'ai incrémenté comme suit:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 void remplirTable(struct t_enr* t) // Une procédure qui permet d'encoder // et la condition d'arrêt est matr = 0 { int i=0; printf(" Matricule: "); scanf("%d", &t[i].matr); while( t[i].matr != 0) { printf(" Nom: "); gets(t[i].nom); i++;// c'est ici que j'ai mis l'incrémentation j'espère que c'est correct printf(" Matricule: "); scanf("%d", &t[i].matr);
Si tu ne changes pas ta boucle, ça ira, mais il faut quand même ajouter le contrôle du nombre d'enregistrements.
Citation:
Envoyé par zui
Pour cela j'ai créé une procédure pour initialiser la table:
Code:

1 2 3 void videTable(struct t_enr* t) {t[1].matr = 0;}
ta procédure videtable se contente de mettre t[1].matr à 0. Il vaudrait mieux initialiser toutes les valeurs du tableau.
Et rappel : en C, les indices des tableaux commencent à 0 (zéro)
tu peux faire quelque chose comme ça:
Code:

1 2 3 4 5 6 7 8 void videTable(struct t_enr* t) { for (int i=0;i<MAXENR;++i) { t[i].matr = 0; t[i].nom[0] = 0; } }
Citation:

Envoyé par zui

Je ne vois pas comment je peux boucler autrement. Aucune d'idée:roll:

C'est une boucle où tu dois passer au moins une fois, donc première piste:
au lieu de faire une boucle
Code:

1 2 3 4 while (condition) { ... }
fais une boucle
Code:

1 2 3 4 do { ... } while (condition);

11/06/2006, 12h05
Pragmateek
Code:

1 2 3 4 int main(){ ... return EXIT_SUCCESS; }
est standard.