Discussion :

[Hgo-Crltt]

Bonsoir,

Je me permet de vous poser une question concernant mon code dont l'objectif est de saisir pour un certain nombre d'étudiant, leurs nom et prénom ainsi que 3 notes. Le programme a la fin affiche les coordonnées de chaque étudiant avec leurs nom, prénom, leurs notes et la moyenne.
Or je viens débuter il y a peu en C et je ne comprend pas du tout où se trouve le problème dans la saisie de mes coordonnées ....le programme ne me laisse pas saisir le nom du premier étudiant. De plus lors de l'affichage, le nom et prénom de l'étudiant s'affiche mal sans que je n'arrive à comprendre mon erreur.

Merci d'avance pour l'aide que vous pourrez m'apporter.

Bonne soirée à vous.

voici mon programme :

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
 
#include <stdio.h>
#define nmax 30
 
 
//-----------------------
// Fonction saisir le nombre d'étudiant
//-----------------------
int saisir_nombre_etudiant()
{
    int a ;
  do{
    printf(" donner un nombre entre 1 et %d:",nmax );
    scanf("%d",&a);
 }while(a<=0 || a>=nmax );
    return(a);
}
//-----------------------
// procédure saisir coordonnées
//-----------------------
void saisir_coordonnes(int a, char nom[a][30], char prenom[a][30], char id[a][61])
{
    int i;
    for (i = 0; i < a; i++) {
        printf("\nNom : ");
        gets(nom[i]);
 
        printf("\nPrenom : ");
        gets(prenom[i]);
        strcpy(id[i] , prenom[i] ) ;
        strcat(id[i] , " " );
        strcat(id[i], nom[i]);
}}
//-----------------------
// procédure saisir notes
//-----------------------
void saisir_notes ( int note1[nmax], int note2[nmax], int note3[nmax],int a)
   {
 
    int i;
    for (i=0; i<a; i++)
    {
     printf("\n saisir les notes de l'etudiant %d :", i+1);
     printf("\nnote 1 :");
     scanf("%d",&note1[i]);
 
     printf("\nnote 2 :");
     scanf("%d",&note2[i]);
 
     printf("\nnote 3 :");
     scanf("%d",&note3[i]);
    }
 
   }
//-----------------------
// procédure moyenne
//-----------------------
 
void moyenne_tableau (int note1[nmax],int note2[nmax],int note3[nmax], int a ,float moy[nmax])
{
    int i;
    for ( i=0; i<a ; i++)
        moy[i]=((float)note1[i]+(float)note2[i]+(float)note3[i])/3;
 
}
 
//-----------------------
// Procedure afficher tableau
//-----------------------
void afficher_tableau (int note1[nmax],int note2[nmax], int note3[nmax],int a,float moy[nmax], char id[a][30] )
{
    int i ;
    printf("\nLes relevé de notes des éleves :\n : ");
    for ( i=0; i<a; i++)
    {
        printf("\n----------------------");
        printf(" \netudiant n°%d : %s ",i+1,id[i]);
        printf("\n----------------------");
 
        printf("\nnote 1 : %d",note1[i]);
        printf("\nnote 2 : %d",note2[i]);
        printf("\nnote 3 : %d",note3[i]);
        printf("\n----------------------");
        printf(" \nmoyenne :%f ", moy[i]);
 
    }
 
}
main(){
int note1[nmax],note2[nmax],note3[nmax],a;
float moy[nmax];
char nom[a][30] ,prenom[a][30],id[a][61];
a=saisir_nombre_etudiant();
saisir_coordonnes(a, nom, prenom,id);
 
saisir_notes(note1, note2, note3, a);
 
moyenne_tableau(note1, note2, note3, a, moy);
 
afficher_tableau(note1, note2, note3, a, moy, id);
 
}

[edgarjacobs]

Hello,

Ligne 92, tu déclares des tableaux de type vla avec la variable 'a'. Oui, pourquoi pas, mais cette variable 'a' n'est initialisée qu'à la ligne suivante....
Je n'ai pas été plus loin.

[Hgo-Crltt]

Merci beaucoup pour ta réponse aussi rapide, effectivement je n'avais pas fait attention à ce détail mais il ne semblait pas avoir d'impact sur le code après modification ( et je ne sais pas pourquoi le programme fonctionnait quand même alors qu'il n'avais pas la valeur de a ).

[Sve@r]

Bonjour

( et je ne sais pas pourquoi le programme fonctionnait quand même alors qu'il n'avais pas la valeur de a ).

Comportement indéterminé (UB en anglais). C'est le pire cas possible en C car indétectable.
Le C est un langage fait pour aller vite donc il ne vérifie rien de ce que tu écris. Soit tu respectes ses obligations et tout va bien, soit tu ne les respectes pas et il exécute tes instructions quelles qu'elles soient et là ça peut fonctionner ou ne pas fonctionner. Ou fonctionner une semaine puis tu rajoutes un truc banal et ça ne fonctionne plus (et là va chercher la cause du souci qui n'est pas dans le rajout mais dans l'erreur précédente !!!)
Dans l'absolu ça peut même t'envoyer un écran bleu ou éteindre ta machine. C'est la définition parfaite de "indéterminé" => qui ne peut pas être prévu.
Dans le cas présent, tu ne peux pas dire que "a" n'a pas de valeur car une variable a toujours une valeur. Donc ça crée bien un tableau de "a" éléments.

Sinon pour le souci principal (à savoir la "non-saisie" du nom) cela vient de la saisie précédente (le scanf dans la fonction "saisir_nombre_etudiant"). Quand tu entres ton nombre, tu valide par <return>. Le scanf() récupère donc le nombre (normal) et laisse le <return> qui n'est pas un nombre dans le clavier (normal aussi). Ensuite à la saisie suivante le gets (saisie chaine) fait son travail et récupère la chaine toujours présente dans le clavier (le <return>). Tout est cohérent.

Je ne sais pas qui t'a appris le C mais il est clair que lui-même n'y connait rien en C. Dans l'immédiat, si tu veux voir ton code tourner, rajoute getchar() juste sous le scanf() de la ligne 16. C'est juste un pansement sommaire qui permet de récupérer dans le vide (donc de virer) le <return>. Plus tard tu apprendras à faire de la saisie propre donc sans passer par scanf() qui n'est pas faite pour ça (mais fatalement on la montre immédiatement car elle permet de commencer des exercices).

Ensuite pour avoir un code un peu plus propre, alors

• remplace gets(nom[i]) par fgets(nom[i], 30+1, stdin) (et pareil au prénom)
• soit cohérent avec id[] définit à 61 dans le main() et à 30 dans saisir_coordonnees()
• rajoute son type int à la fonction main()
• n'utilise pas de VLA (les tableaux avec une variable en taille) car c'est non protégé (si ça plante tu ne le sais pas et UB). Puisque tu as nmax élèves alors tu crées un tableau à nmax. Plus tard tu apprendras à demander à l'OS de la mémoire si tu veux créer des tableaux vraiment variables et là tu pourras gérer si l'OS ne peut pas répondre à ta demande.
• mets tes macros (nmax) en majuscules
• indente correctement. Ca montrera que tu es heureux et fier d'écrire ce programme qui te permet d'imposer des calculs et traitements de ton choix à cette machine stupide et que le C n'est pas pour toi qu'une corvée à expédier avant de retourner jouer à COD. Ou, à défaut de respecter ton code, ça montrera au-moins que tu respectes ceux à qui tu demandes de le lire

Pour l'élégance tu peux remplacer tes strcat() à répétition par sprintf(id[i], "%s %s", prenom[i], nom[i]) et moy[i]=((float)note1[i]+(float)note2[i]+(float)note3[i])/3 par moy[i]=(note1[i] + note2[i] + note3[i]) / 3.0 (un seul float suffit pour avoir la division en float et aérer son code ne fait pas de mal). Accessoirement moi j'aurais fait des fonctions permettant de gérer "un" étudiant et je les aurais appelées pour chaque étudiant (c'est plus souple donc plus réutilisable).

Et fais attention aux tableaux 2D car en C, dès qu'on dépasse 1D il faut être super rigoureux. Hélas une chaine étant déjà un tableau, vouloir rien qu'un tableau de strings c'est déjà de la 2D.

[Hgo-Crltt]

Bonjour Sve@r

Tout d'abord, merci énormément pour votre réponse, cela m'a bien aidé. Votre idée du getchar() a bien fonctionné mais un problème que je ne saurais expliquer ( votre expérience le saura peut être) est toujours présent : pourquoi lors de l'exécution du programme et plus particulièrement lors de l'affichage du tableau, seul le nom d'un étudiant sur 2 apparait ( je ne vois pas d'erreur dans la boucle for ou dans la saisie de ID). Dans mon exemple j'ai pris 4 étudiants ( Claude Francois, Jean Schultheis, Gilbert Montagné et Johnny Hallyday), vous trouverez ci-joint l'affichage du tableau en sortie.

Merci aussi pour les conseils qui vont permettre une meilleure visibilité du code

Par ailleurs auriez vous des conseils afin d'éviter à l'avenir les comportements indéterminés ? ( des fautes types, une règle de base a connaitre….)

Merci d'avance

Bonne journée et bon Dimanche

[Sve@r]

Votre idée du getchar() a bien fonctionné

C'est pas "mon" idée, le truc est assez connu. Et attention, c'est juste une rustine. Tape "5xxxxxxx" à la saisie du nb et ça ne fonctionne plus (le getchar() ne supprime qu'un seul caractère).

pourquoi lors de l'exécution du programme et plus particulièrement lors de l'affichage du tableau, seul le nom d'un étudiant sur 2 apparait ( je ne vois pas d'erreur dans la boucle for ou dans la saisie de ID). Dans mon exemple j'ai pris 4 étudiants ( Claude Francois, Jean Schultheis, Gilbert Montagné et Johnny Hallyday), vous trouverez ci-joint l'affichage du tableau en sortie.

Alors ton code rectifié chez-moi (le getchar(), remplacer gets() par fgets()) fonctionne correctement. As-tu pensé au fgets() ?
Dans ta copie d'écran je vois un "°" devant "Jean". J'ai l'impression que lors de la saisie, il enregistre des trucs en plus ce qui reproduit le même phénomène (la saisie suivante récupère le truc en plus de la saisie précédente) et si ça marche chez-moi c'est que je suis sous Linux (OS écrit en C donc dans lequel le C lui est implicitement lié). Toi tu sembles bosser sous Windows pour qui le C n'est qu'une verrue qu'il tolère à peine et où ses compilateurs ont été écrits avec les pieds. Mais si tu as bien corrigé alors reposte le nouveau code.

Par ailleurs auriez vous des conseils afin d'éviter à l'avenir les comportements indéterminés ? ( des fautes types, une règle de base a connaitre….)

Désolé, si on avait un truc pour l'éviter ce ne serait pas la terreur des devs C. Ton cerveau est la meilleurs arme (utiliser "a" avant de le remplir...)

[Hgo-Crltt]

Bonsoir Sve@r,

Encore merci pour votre promptitude, j'ai pris en compte toutes vos remarques pour la modification de mon code, le voici donc actualisé et je dispose toujours de la même erreur d'affichage en sortie :

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
 
#include <stdio.h>
#define NMAX 30
 
//-----------------------
// Fonction saisir le nombre d'étudiant
//-----------------------
int saisir_nombre_etudiant()
{
    int a ;
    do{
        printf(" donner un nombre entre 1 et %d:",NMAX );
        scanf("%d",&a);
        getchar();
    }while(a<=0 || a>=NMAX );
    return(a);
}
//-----------------------
// procédure saisir coordonnées
//-----------------------
void saisir_coordonnes(int a, char nom[a][30], char prenom[a][30], char id[a][61])
{
    int i;
    for (i = 0; i < a; i++) {
        printf("\nNom : ");
        fgets(nom[i],30+1,stdin);
 
        printf("\nPrenom : ");
        fgets(prenom[i],30+1,stdin);
        strcpy(id[i] , prenom[i] ) ;
        sprintf(id[i],"%s %s", prenom[i],nom[i]);
}}
//-----------------------
// procédure saisir notes
//-----------------------
void saisir_notes ( int note1[NMAX], int note2[NMAX], int note3[NMAX],int a)
   {
 
    int i;
    for (i=0; i<a; i++)
    {
        printf("\n saisir les notes de l'etudiant %d :", i+1);
        printf("\nnote 1 :");
        scanf("%d",&note1[i]);
 
        printf("\nnote 2 :");
        scanf("%d",&note2[i]);
 
        printf("\nnote 3 :");
        scanf("%d",&note3[i]);
    }
 
   }
//-----------------------
// procédure moyenne
//-----------------------
 
void moyenne_tableau (int note1[NMAX],int note2[NMAX],int note3[NMAX], int a ,float moy[NMAX])
{
    int i;
    for ( i=0; i<a ; i++)
        moy[i]=(note1[i]+note2[i]+note3[i])/3.0;
 
}
 
//-----------------------
// Procedure afficher tableau
//-----------------------
void afficher_tableau (int note1[NMAX],int note2[NMAX], int note3[NMAX],int a,float moy[NMAX], char id[a][30] )
{
    int i ;
    printf("\nLes relevé de notes des éleves :\n : ");
    for ( i=0; i<a; i++)
    {
        printf("\n----------------------");
        printf(" \netudiant n°%d : %s ",i+1,id[i]);
        printf("\n----------------------");
 
        printf("\nnote 1 : %d",note1[i]);
        printf("\nnote 2 : %d",note2[i]);
        printf("\nnote 3 : %d",note3[i]);
        printf("\n----------------------");
        printf(" \nmoyenne :%f ", moy[i]);
 
    }
 
}
int main(){
    int note1[NMAX],note2[NMAX],note3[NMAX],a;
    float moy[NMAX];
    a=saisir_nombre_etudiant();
    char nom[a][30] ,prenom[a][30],id[a][61];
 
    saisir_coordonnes(a, nom, prenom,id);
 
    saisir_notes(note1, note2, note3, a);
 
    moyenne_tableau(note1, note2, note3, a, moy);
 
    afficher_tableau(note1, note2, note3, a, moy, id);
 
}

Petite question : Dans le cadre scolaire, je suis amené a travailler avec CodeBlocks, peut-il y avoir des différences lors de la compilation du code sur une application malgré la différence de système d'exploitation ?

Merci d'avance pour votre précieuse aide,

Bonne soirée.

[edgarjacobs]

Hello,

Tu as laissé la dimension 30 (plutot que 61) pour id dans la fonction afficher_tableau(). C'est pour cela que l'affichage n'est pas correct.

[Sve@r]

Encore merci pour votre promptitude, j'ai pris en compte toutes vos remarques pour la modification de mon code, le voici donc actualisé et je dispose toujours de la même erreur d'affichage en sortie :

Rajoute #include <string.h> en début, c'est nécessaire pour strcpy(). Et gaffe la fonction affiche_tableau() taille toujours id[a] à [30]. Mais je ne pense pas que ce soit la cause (d'autant plus que chez-moi il fonctionne nickel). Les indentations c'est pas encore ça (deux accolades en ligne 35 !!!)

Entre maintenant la difficile phase d'investigation. Déjà met en commentaire la fonction saisir_notes() en ligne 98 et retente car m'est avis qu'il doit y avoir une influence (de toute façon dès qu'il y a scanf() il faut se méfier). Si ça vient de là, alors voici une fonction de saisie d'entier propre

Code c :

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

1
2
3
4
5
6
7
8
9
10
// Fonction saisie entier
int saisieInt() {
	char data[256];
	int v;
	while (1) {
		fgets(data, 256, stdin);
		if (sscanf(data, "%d", &v) == 1) return v;
		fputs("Erreur de saisie - Recommencez\n", stdout);
	}
}

Ensuite la saisie du nb d'étudiants devient...

Code c :

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

1
2
3
4
5
6
7
8
int saisir_nombre_etudiant() {
	int a ;
	do {
		printf(" donner un nombre entre 1 et %d:",nmax );
		a=saisieInt();
	} while (a <= 0 || a >= nmax);
	return a;
}

Et pour les notes c'est note1[i]=saisieInt() (idem note2 et note3). A voir si ça règle le souci...

Et sur un forum on se tutoie.

Petite question : Dans le cadre scolaire, je suis amené a travailler avec CodeBlocks, peut-il y avoir des différences lors de la compilation du code sur une application malgré la différence de système d'exploitation ?

Non, si un compilateur respecte la norme alors c'est ok et CodeBlocks a très bonne réputation. Je ne peux que pencher pour l'OS...

PS: mes essais: 4 élèves nommés "aaa" "aaa", "bbb" "bbb", "ccc" "ccc" et "ddd ddd". Et pour mes notes c'est 1 2 3 puis 2 3 4 puis 3 4 5 puis 4 5 6 (faut tout tenter)

[Hgo-Crltt]

Bonsoir edgarjacobs,

Ah effectivement merci j'avais regardé la mauvaise fonction tout a l'heure. Mon erreur était que Id était trop petit et ne pouvais contenir que deux noms/prénom ? Mais dans ce cas pourquoi aurait-il ajouté des caracteres en plus ( le "°" tout a l'heure) et pourquoi avoir mis le deuxième prénom au niveau du troisième ?

Navré de vous embeter avec mes questions/problèmes un dimanche soir

Merci d'avance,

Bonne soirée.

[edgarjacobs]

Le tableau id, en mémoire, n'est en réalité qu'une suite d'octets. C'est nous qui voyons une représentation 2D de la chose.

Si id est déclaré [x][61] et qu'on suppose que id[0][0] se trouve à l'adresse 1000, id[1][0] se trouve 61 octets plus loin que id[0][0], soit à l'adresse 1061, et id[2][0] à l'adresse 1000+2*61, etc...
Comme dans la fonction tu as déclaré id[x][30], vu que id[0][0] est à l'adresse 1000, id[1][0] se trouve à l'adresse 1000+1*30, id[2][0] à l'adresse 1000+2*30, etc.

Tu vois la différence, je suppose. Et comme tu n'as pas initialisé id à zéro dans le main(), il peut contenir n'importe quoi, d'où le '°' qui s'affiche.

Et oui, si le C est simple (peu de types, peu d'instructions), il peut être est une catastrophe à la moindre erreur de programmation.

[Sve@r]

Tu as laissé la dimension 30 (plutot que 61) pour id dans la fonction afficher_tableau(). C'est pour cela que l'affichage n'est pas correct.

Sans déconner !!! En plus le compilo me le disait et moi je corrigeais sans penser que c'était la cause première. Comme quoi le moindre détail compte !!!
Bien vu

[Hgo-Crltt]

En tout cas merci énormément à vous deux pour votre aide et votre patience malgré mes erreurs assez bêtes ! Honnêtement je n'y serais jamais arrivé seul ( je n'avais pas vu durant mes 4h de cours le getchar(), le fgets, ni le sprintf) mais merci énormément !!!!

Passez une très belle soirée