Discussion :

[sniperone]

Bonjour,

voici mon programme qui permet d'enregistrer des étudiants et de les afficher.
il ne fonctionne pas correctement. étant nouveaux dans la programmation en C, pouvez-vous vérifier mon code pour moi.

Merci

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
#include <stdio.h>
#include <stdlib.h>
struct etudiant
       { 
         int matricule;
         char nom[15];
         int age;
       };
int i;
 
void saisir();
void afficher();
void saisir()
{
     struct etudiant etu[i]; 
     for(i=0;i<=1;i++) {
 
     printf("entrer le matricule de l'etudiant %d \n",i+1);
     scanf("%d",&etu[i].matricule);
      printf("entrer le nom de l'etudiant %d \n",i+1);
     scanf("%s",&etu[i].nom);   
      printf("entrer l age de l'etudiant %d \n",i+1);
     scanf("%d",&etu[i].age);
     } 
};
void afficher()
{
     struct etudiant etud[i]; 
      for(i=0;i<=1;i++) {
 
      printf("------------ Etudiant %d -------------- \n",i+1);
      printf("MAtricule : %d \n",etud[i].matricule);         
      printf("Age: %d \n",etud[i].age); 
      printf("Nom: %s \n",etud[i].nom); 
 
      printf("\n");
     }
     printf("--------------------------------");
};
 
main()
{    
      saisir();
      afficher();
      system("pause");
}

[Invité]

Bonjour,

• Ton indentation dans les boucles for est à revoir.
  
• int i; à la ligne ligne 9 : on évite au maximum l'utilisation des variables globales.
  
• struct etudiant etu[i]; Ceci étant un cas d'école, je doute que tu veuilles utilise un VLA du C99. Il te faux utiliser une constante #define, ou faire de l'allocation dynamique malloc()/free().
  
• De plus, entre tes deux fonctions, les deux structures déclarées n'ont rien à voir entre-elles. Ce sont deux variables bien distinctes.
  Je te suggère de déclarer ta structure dans le main() et la faire passer en paramètre à tes deux fonctions.
  
• Le prototype du main() est incorrect, on utilise
  

  Code :

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

  1
  2
  3
  int main(void)
  /* ou */
  int main(int argc, char *argv[]) /* si l'on souhaite utiliser les arguments */

  Ce qui amène à retourner une valeur. (return 0; si tout s'est bien déroulé.)
  
• system("pause"); de Windows n'est pas portable, je te conseille d'utiliser getchar(); à la place.
  
• Concernant cette ligne : scanf("%s",&etu[i].nom);, je te conseille la lecture de ce tutoriel.

[Sve@r]

Bonjour,

voici mon programme qui permet d'enregistrer des étudiants et de les afficher.
il ne fonctionne pas correctement. étant nouveaux dans la programmation en C, pouvez-vous vérifier mon code pour moi.

Salut

Ce serait trop long de te dire tout ce qui ne va pas. Je préfère retaper un code correct

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
#include <stdio.h>
#include <stdlib.h>
 
#define NB_ETUDIANT        (10)
 
struct s_etudiant
{ 
    int matricule;
    char nom[15];
    unsigned short age;
};
 
void saisir(struct s_etudiant*, size_t);
void afficher(struct s_etudiant*, size_t);
 
void saisir(struct s_etudiant *etu, size_t nb)
{
     size_t i;
 
     for (i=0; i < nb; i++) {
        printf("entrer le matricule de l'etudiant %lu\n", i+1);
        scanf("%d", &etu[i].matricule);
 
        printf("entrer le nom de l'etudiant %lu\n", i+1);
        scanf("%s", etu[i].nom);     // Pas de "&" car nom est déjà un pointeur
        // Ou alors scanf("%s", &etu[i].nom[0]);
 
        printf("entrer l age de l'etudiant %lu\n", i+1);
        scanf("%hu", &etu[i].age);
     } 
}
 
void afficher(struct s_etudiant *etu, size_t nb)
{
    size_t i;
    for (i=0; i<=nb; i++) {
        printf("------------ Etudiant %lu -------------- \n",i+1);
        printf("Matricule : %d\n", etu[i].matricule);         
        printf("Age: %hu\n", etu[i].age); 
        printf("Nom: %s\n", etu[i].nom); 
        printf("\n");
    }
    printf("--------------------------------\n");
}
 
int main()
{
      struct s_etudiant etudiant[NB_ETUDIANT];    
      saisir(etudiant, NB_ETUDIANT);
      afficher(etudiant, NB_ETUDIANT);
      system("pause");
}

Voilà. Essaye d'utiliser les remarques de Winjerome associées à ce code pour comprendre comment fonctionne le C. Et surtout essaye aussi de lire qq bouquins/tutos...

PS: Si le type size_t n'est pas connu (je crois qu'il faut inclure <sys/types.h> alors remplace-le par un "unsigned long")

PS2: Essaye d'aérer ton code (espaces après les virgules et les points-virgules et après le for). Cela aidera à la relecture (comme en français quoi). En revanche les espaces avant le "\n" sont inutiles.

PS3: s'il s'agit d'un devoir, surtout ne rend pas ce que je viens d'écrire. Le prof verrait de suite qu'il n'est pas de toi...

[karara20]

Tu dois changer struct etudiant etu[i]; par struct etudiant etu[2];.
<i> nombre de max des élevés ::: et tu dois supprimer & de la chaine
résultat :

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
#include <stdio.h>
#include <stdlib.h>
struct etudiant
       {
         int matricule;
         char nom[15];
         int age;
       };
int i;
 
void saisir();
void afficher();
void saisir()
{
     struct etudiant etu[2];
     for(i=0;i<=1;i++) {
 
     printf("entrer le matricule de l'etudiant %d \n",i+1);
     scanf("%d",&etu[i].matricule);
      printf("entrer le nom de l'etudiant %d \n",i+1);
     scanf("%s",etu[i].nom);
      printf("entrer l age de l'etudiant %d \n",i+1);
     scanf("%d",&etu[i].age);
     }
};
void afficher()
{
     struct etudiant etud[2];
      for(i=0;i<=1;i++) {
 
      printf("------------ Etudiant %d -------------- \n",i+1);
      printf("MAtricule : %d \n",etud[i].matricule);
      printf("Age: %d \n",etud[i].age);
      printf("Nom: %s \n",etud[i].nom);
 
      printf("\n");
     }
     printf("--------------------------------\n");
};
 
main()
{
      saisir();
      afficher();
      system("pause");
}