Discussion :

[gl]

Code :

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

1
2
3
4
5
6
struct etu 
{
       int    mat;
       char    nom[20];
       float    cote[maxcotes];
}       tab[maxetu]   ;

C'est ce que tu appelles ne plus avoir de global ?

Je ne vois pas de globales, c'est juste une structure. Tu voudrais le mettre ou ce bout de code toi ?

[marc_dd]

Je ne vois pas de globales, c'est juste une structure. Tu voudrais le mettre ou ce bout de code toi ?

merci il me semblait aussi

[diogene]

tab sauf erreur de ma part est une variable déclarée en dehors de toute fonction et est du type tableau de struct etu

[diogene]

1- D'abord et surtout soigne l'indentation du programme
2- tu définis maxetu et maxcotes : tu devrais donc les utiliser et on ne devrait pas voir de 25 ou de 3 dans ton programme. Sinon, ce n'était pas la peine
3- supprime la variable globale tab : les variables globales ne sont pas une bonne habitude à prendre. Elles sont dangereuses dès que le programme devient un tant soit peu complexe

Code :

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

1
2
3
4
5
6
7
8
9
struct etu
{
....
};
void main ()
{
 struct etu tab[maxetu];
....
}

et passe cette variable en argument aux fonctions.
4- n'écris pas des choses inutiles qui peuvent troubler le lecteur. exemple

Code :

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

int  lire_float (char nbf[5])

renvoie toujours 0. Autant qu'il ne renvoie rien! Dans cette déclaration le 5 ne sert à rien. A supprimer. Idem pour lire_int

Code :

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

void  lire_float (char *nbf)

5- Lire_mat renvoie 1 ou 2. C'est bizarre et on se demande pourquoi. Le code ne l'utilise que pour comparer à 1. D'après son code, elle cherche si le matricule existe dans le tableau, ce qui ne correspond pas à son nom. Les identificateurs doivent être choisi pour être au lecteur une source d'informations, pas pour les induire en erreur. De plus, son code est faux, puisque la valeur de trouve ne dépend que du DERNIER élément du tableau. La variable matt[5] ne sert à rien et le code est tordu : pourquoi mettre trouve à 1 pour renvoyer 2 et à 0 pour renvoyer 1 ? Il suffit de qq chose du genre

Code :

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

1
2
3
4
5
6
7
int Existe_mat ( int  m)
{
 int i;
 for (i=0;i<maxetu;i++)
   if (m==tab[i].mat) return 1;
 return 0;
}

qui permet des tests du genre while(Existe_mat(-----
6- lire_cotes : à quoi sert le else; ?
7- init n'utilise pas maxetu et maxcotes . Son strcpy est une grosse erreur.
8- etc ...

[Emmanuel Delahaye]

Code :

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

1
2
3
4
5
6
struct etu 
{
       int    mat;
       char    nom[20];
       float    cote[maxcotes];
}       tab[maxetu]   ;

C'est ce que tu appelles ne plus avoir de global ?

Je ne vois pas de globales, c'est juste une structure.

Huh! Nettoyage des yeux, please! 'tab' est un tableau global de structures.

Tu voudrais le mettre ou ce bout de code toi ?

A début du source :

Code :

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

1
2
3
4
5
6
7
 
struct etu 
{
   int    mat;
   char    nom[20];
   float    cote[maxcotes];
};

Dans main()

Code :

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

struct etu tab[maxetu];

ou

Code :

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

static struct etu tab[maxetu];

si le tableau est trop grand.
Et ces paramètres pour chaques fonctions qui en ont besoin :

Code :

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

   f (struct etu *tab, size_t n)

[diogene]

A Emmanuel : Entièrement d'accord

[gl]

tab sauf erreur de ma part est une variable déclarée en dehors de toute fonction et est du type tableau de struct etu

Oups, je l'avais vu a l'interieur des accolades. Effectivement c'est bien une globale.

[gl]

A Emmanuel : Entièrement d'accord

Moi aussi ! J'avais juste lu ca trop rapidement.