[C] Programme simple...

Programme simple... [C] - C - Programmation

Marsh Posté le 31-07-2004 à 18:03:13    

Mon programme foire completement,
je ne comprends pas bien l'utilisation des fonctions et des structures
voila pourquoisans doute tant de problemes...
 
merci d'aider un pauvre debutant comme moi...
 

Code :
  1. #include <stdio.h>
  2. #include <string.h>
  3. #define MAX 50
  4. #define SIZE 20
  6. struct etudiant
  7. {
  8. char nom[SIZE] ;
  9. char code ;
  10. int cote1 ;
  11. int cote2 ;
  12. int cote3 ;
  13. };
  15. int menu(void) ;
  16. void saisie(struct etudiant tab[], int *dim) ;
  17. void affichage(struct etudiant tab[], int dim) ;
  19. main()
  20. {
  21. int choix ;
  22. int end = 0 ;
  23. int dim ;
  24. struct etudiant tableau[MAX] ;
  27. do
  28. {
  29.  choix = menu() ;
  30.  switch(choix)
  31.  {
  32.  case 1 :
  33.   saisie(tableau, &dim) ;
  34.   break ;
  35.  case 2 :
  36.   affichage(tableau, dim) ;
  37.   break ;
  38.  case 3 :
  39.   end = 1 ;
  40.   break ;
  41.  }
  42. }
  43. while(end == 0) ;
  44. }
  46. int menu(void)
  47. {
  48. int x ;
  50. printf("\n\n\n-----  Menu  -----\n\n\n" ) ;
  51. printf("1 : Saisie des etudiants\n" ) ;
  52. printf("2 : Affichage des etudiants avec la moyenne des cotes\n" ) ;
  53. printf("6 : Quitter\n\n\n" ) ;
  54. printf("Votre choix : " ) ;
  55. scanf("%d",&x) ;
  57. return x ;
  58. }
  61. void saisie(struct etudiant tab[], int *dim)
  62. {
  63. int x ;
  64. int i=0, j ;
  65. int result, position ;
  66. struct etudiant temp ;
  68. do
  69. {
  70.  printf("\n\nCombien d'etudiant voulez-vous saisir (max 50) " ) ;
  71.  scanf("%d", &x) ;
  72. }
  73. while(x > MAX || x < 1) ;
  74. *dim = x ;
  76. printf("\n\n\n-----  Etudiant numero %d  -----\n\n", x) ;
  77. printf("Nom  : " ) ;
  78. fflush(stdin) ;
  79. gets(temp.nom) ;
  80. printf("Code (1 caractere) : " ) ;
  81. scanf("%c", &temp.code) ;
  82. printf("Cote 1 : " ) ;
  83. scanf("%d", &temp.cote1) ;
  84. printf("Cote 2 : " ) ;
  85. scanf("%d", &temp.cote2) ;
  86. printf("Cote 3 : " ) ;
  87. scanf("%d", &temp.cote3) ;
  88. }
  89. void affichage(struct etudiant tab[], int dim)
  90. {
  91. int i, moyenne, moyenne_totale=0 ;
  93. for(i=1; i<=dim; i++)
  94. {
  95.  moyenne = (tab[i].cote1 + tab[i].cote2 + tab[i].cote3)/3 ;
  96.  moyenne_totale = moyenne_totale + moyenne ;
  98.  printf("\n\n\n-----  Etudiant %d  -----\n\n", i) ;
  99.  printf("Nom  : %s\n", tab[i].nom) ;
  100.  printf("Code : %c\n\n", tab[i].code) ;
  101.  printf("Cote 1 : %d\n", tab[i].cote1) ;
  102.  printf("Cote 2 : %d\n", tab[i].cote2) ;
  103.  printf("Cote 3 : %d\n\n", tab[i].cote3) ;
  104.  printf("Moyenne : %d", moyenne) ;
  106.  if((i%2)==0)
  107.   fflush(stdin) ;
  108.   getchar() ;
  109. }
  110. moyenne_totale = moyenne_totale/dim ;
  111. printf("\n\n\nMoyenne totale : %d", moyenne_totale) ;
  112. }

Reply

Marsh Posté le 31-07-2004 à 18:03:13   

Reply

Marsh Posté le 31-07-2004 à 18:10:18    

1) vire les gets
2) fais des allocations dynamiques
3) fflush(stdin) ne sert à rien du tout
4) un scanf peut lire plusieurs choses à la fois, pas la peine d'en faire 36
4) 1 printf suffit, pas la peine d'en faire 36
5) while(x > MAX || x < 1) ; superbe
6) ta fonction saisie ne fait rien


Message édité par Taz le 31-07-2004 à 18:10:33
Reply

Marsh Posté le 01-08-2004 à 00:10:27    

lol


---------------
job
Reply

Marsh Posté le 01-08-2004 à 01:52:35    

Taz a écrit :

1) vire les gets
2) fais des allocations dynamiques
3) fflush(stdin) ne sert à rien du tout
4) un scanf peut lire plusieurs choses à la fois, pas la peine d'en faire 36
4) 1 printf suffit, pas la peine d'en faire 36
5) while(x > MAX || x < 1) ; superbe
6) ta fonction saisie ne fait rien


 
A part que c'est laid et tordu, je ne vois pas trop le probleme, cela semble cadrer avec ce qu'il desire.
 
=>nicolas
Principal probleme dans saisie: tu ne remplis pas ton tableau d'etudiant (enfin tu en fais 1)
Sinon tu ferais bien de suivre les conseils deja donnes, en particulier pour les printf, scanf et fflush. Pour les allocations dynamiques, attends un peu de bien tout digerer :o

Reply

Marsh Posté le 01-08-2004 à 02:11:35    

thermocline a écrit :

A part que c'est laid et tordu, je ne vois pas trop le probleme, cela semble cadrer avec ce qu'il desire.

y a des moyens plus simple de faire boucler indéfiniment un programme.

Reply

Marsh Posté le 01-08-2004 à 02:36:53    

Taz a écrit :

y a des moyens plus simple de faire boucler indéfiniment un programme.


 
 :??:  
 
Il veut que l'utilisateur entre un nombre compris entre 1 et MAX.  
 
Il a bien mis ||, les regles de parentheses sont respectees, alors je ne vois pas ou est le probleme (en meme temps j'avais la tete en bas toute la journee, je dois avoir le cerveau abime).
 
C'est même la partie la plus belle de son code, s'il doit y en avoir une :o

Reply

Marsh Posté le 01-08-2004 à 03:19:37    

et que ce passe-t-il si x remplis justement une de ces conditions ?

Reply

Marsh Posté le 01-08-2004 à 03:30:04    

Taz a écrit :

et que ce passe-t-il si x remplis justement une de ces conditions ?


 
he ben l'utilisateur qui a rentrée une telle valeur sera puni comme il le merite, l'enfoiré :O

Reply

Marsh Posté le 01-08-2004 à 05:28:45    

Pas mieux

Reply

Marsh Posté le 01-08-2004 à 17:50:50    

1) vire les scanf, prefere les fgets.
2) Getchar n'est pas standard
3) int main (void) a la rigueur, mais pas main tout court, main doit retourner 0 si tout c'est bien passé.

Reply

Marsh Posté le 01-08-2004 à 17:50:50   

Reply

Marsh Posté le 01-08-2004 à 17:51:37    

Taz a écrit :

y a des moyens plus simple de faire boucler indéfiniment un programme.


 
while(1)  :whistle:

Reply

Marsh Posté le 01-08-2004 à 18:36:22    

Tamahome a écrit :

2) Getchar n'est pas standard

ah bon ... ca te va pas bien de jouer au Taz

Reply

Marsh Posté le 02-08-2004 à 18:25:43    

Taz a écrit :

et que ce passe-t-il si x remplis justement une de ces conditions ?


 
Ben ça rebouclera une fois pour refaire la saisie de x, qui a une valeur invalide...
Ce while n'a strictement aucun problème, je vois pas pourquoi il rebouclerait indéfiniment... (par contre, après, ça suit pas...)

Reply

Marsh Posté le 02-08-2004 à 18:35:10    

et il se passe quoi si je tape 'a' ? et le flux n'est pas flushé d'ailleurs, alors désolé, c'est une boucle infinie à la moindre erreur, sur certains systèmes, la saisie sera réalisée avec le prompt

Reply

Marsh Posté le 02-08-2004 à 18:37:27    

Honte à moi, j'avais que des chiffres dans la tête :jap:

Reply

Marsh Posté le 03-08-2004 à 18:10:29    

Taz a écrit :

ah bon ... ca te va pas bien de jouer au Taz


 
ca va les chevilles ? :sleep:
 
Edit : j'ai confondu avec getch ()
 
Quand à scanf, et le pourquoi il ne vaut mieux pas l'utiliser, je renvoi a l'excellente FAQ de fclc : http://www.isty-info.uvsq.fr/~rume [...] 8.html#q_8


Message édité par Tamahome le 03-08-2004 à 18:13:15

---------------
Hobby eien ／人◕ ‿‿ ◕人＼
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed