Forum |  HardWare.fr | News | Articles | PC | S'identifier | S'inscrire | Shop Recherche
1149 connectés 

  FORUM HardWare.fr
  Programmation
  C

  [ C ] Archi-debutant segfault sur petit programme :-(

 
 


 Mot :   Pseudo :  
 
Bas de page
Auteur Sujet :

[ C ] Archi-debutant segfault sur petit programme :-(

n°1013303
NiQoZ1
Posté le 15-03-2005 à 13:45:42  profilanswer
 


Mon programme (un petit exo que j'avait fait en perl que je refait en C pour m'apprendre) fonctionne mais il ya une erreur de segmentation des que je le quitte.
(je n'ose pas poster le code car je débute ...sic)
 
Que veut dire ce genre de backtrace ??
 

Code :
  1. 0x00000000 in ?? ()


 


---------------
NiQoZ
mood
Publicité
Posté le 15-03-2005 à 13:45:42  profilanswer
 

n°1013309
Emmanuel D​elahaye
C is a sharp tool
Posté le 15-03-2005 à 13:50:15  profilanswer
 

NiQoZ1 a écrit :

Mon programme (un petit exo que j'avait fait en perl que je refait en C pour m'apprendre) fonctionne mais il ya une erreur de segmentation des que je le quitte.


Donc il ne fonctionne pas.
 

Citation :

Que veut dire ce genre de backtrace ??
 

Code :
  1. 0x00000000 in ?? ()



Rien il faut poster le code.


---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
n°1013322
NiQoZ1
Posté le 15-03-2005 à 14:00:02  profilanswer
 

tu a tout a fait raison.
C'est un programme tout simple de calcul de moyenne le programme demande le prenom et une note en boucle jusqu'a la saisie de 'q' et renvoi les moyennes des prenoms. On peut rentrer plusieur fois le même prenom (c'est la seule difficulté...enfin pour moi).
 

Code :
  1. #include <stdio.h>
  2. #include <string.h>
  3. #include <stdlib.h>
  4. #define BIGMAX 80
  5. #define MAX 20
  6. #define FAIL 1
  9. void RempliList(char *un_prenom, char prenom[]);
  10. void Affiche(char **liste, float tabnote[], short int tab_multi[], unsigned int max);
  15. int
  16. main(void) {
  17. char *listprenoms[MAX];
  18. float tab_notes[MAX];
  19. short int tab_multi[MAX];
  20. float note;
  21. char prenom[BIGMAX];
  22. unsigned int existe, test, i, j = 0;
  23. size_t s;
  27. for (i = 0; i < BIGMAX; i++) {
  28.  listprenoms[i] = NULL;
  29.  tab_multi[i] = 0;
  30. }
  31. i = 0;
  33. while (1) {
  34.  existe = 0;
  35.  printf("Saissisez le prenom ?\n" );
  36.  scanf("%s", prenom);
  37.  /* Sort si 'q' est pressé */
  38.  if ((prenom[0] == 'q') && (prenom[1] == '\0')) break;     s = strlen(prenom);
  39.  if (prenom[s - 1] == '\n') prenom[s - 1] = '\0';
  40.  printf("Saississez la note ?\n" );
  41.  scanf("%f", &note);
  43.   /* Teste dans la liste "listprenoms" si "prenom" existe    et, si oui  modifie sa note. "tab_multi" enregistre le     nombre de notes pour chaque prenom afin d'extraire     ensuite la moyenne...*/
  44.   for(j = 0; j < i; j++) {
  45.    if (listprenoms[j] != NULL) {
  46.     if ((test = strcmp(prenom, listprenoms[j])) == NULL) {
  47.      tab_notes[j] += note;
  48.      tab_multi[j]++;
  49.      existe = 1;
  50.      break;
  51.     }
  52.    }
  53.   }
  54.   /*Allocation dynamique */
  55.   if (!existe) {
  56.   if ((listprenoms[i] = malloc(strlen(prenom) + 1)) == NULL) {
  57.    perror("malloc()" );
  58.    exit(FAIL);
  59.   }
  60.   /* Mise en memoire */
  61.   RempliList(listprenoms[i], prenom);
  62.   tab_notes[i] = note;
  63.   tab_multi[i]++;
  64.   }
  65.   i++; /*incremente i a chaque boucle while */
  66. }
  68.  Affiche(listprenoms, tab_notes, tab_multi, i); /*Affiche les moyennes */
  70. for (j = 0; j < i ; j++) {
  71.  free(listprenoms[j]);
  72.  tab_notes[j] = '0';
  73.  tab_multi[j] = '0';
  74. }
  76. return(0);
  77. }
  79. void RempliList(char *un_prenom, char prenom[])
  80. {
  81. char *ptr;
  82. if ((ptr = strcpy(un_prenom, prenom)) == NULL) {
  83.  perror("strcpy!" );
  84.  exit(1);
  85. }
  87. }
  89. void
  90. Affiche(char **liste, float tabnote[], int short tab_multi[],unsigned int max)
  91. {
  92. int i;
  93. for (i = 0; i < max; i++)
  94.  if (liste[i] != NULL)
  95.   printf("\nPrenom = %s\nNote moyenne =%2.2f\n", liste[i], tabnote[i] / tab_multi[i]);
  97. }


 
Je suis sous linux pour info... Je reprécise que je débute en programmation et c'est mon premier programme en c.


---------------
NiQoZ
n°1013345
Taz
bisounours-codeur
Posté le 15-03-2005 à 14:12:56  profilanswer
 

scanf("%s", prenom);
 
beurk, utilise fgets
 
#  if ((ptr = strcpy(un_prenom, prenom)) == NULL) {
#         perror("strcpy!" );
#         exit(1);
#     }
 
tu peux t'accrocher là ... tu nous appelle si ça arrive un jour
 
tab_notes[j] = '0';
 
'0' != '\0'
 
 
(test = strcmp(prenom, listprenoms[j])) == NULL
 
man strcmp

n°1013356
NiQoZ1
Posté le 15-03-2005 à 14:23:05  profilanswer
 

Merci de donner de ton temp.
 
#  if ((ptr = strcpy(un_prenom, prenom)) == NULL) {
#         perror("strcpy!" );
#         exit(1);
#     }
 
comment effecturait tu la copie alors ?   :??:  
 
même question pour la comparaison strcmp.  
 


---------------
NiQoZ
n°1013359
Taz
bisounours-codeur
Posté le 15-03-2005 à 14:25:48  profilanswer
 

man strcpy
man strcmp
 
tu te vautres complet sur les valeurs de retour

n°1013362
Emmanuel D​elahaye
C is a sharp tool
Posté le 15-03-2005 à 14:31:06  profilanswer
 

NiQoZ1 a écrit :


C'est un programme tout simple de calcul de moyenne le programme demande  


strcmp() retourne un int et non un pointeur. Lire la doc.
 


Message édité par Emmanuel Delahaye le 15-03-2005 à 14:36:28

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
n°1013364
NiQoZ1
Posté le 15-03-2005 à 14:31:19  profilanswer
 

(test = strcmp(prenom, listprenoms[j])) == NULL)
 
serai:  
 
(test = strcmp(prenom, listprenoms[j])) == '0')
 
mais ça ne fonctionne plus à la comparaison ...


---------------
NiQoZ
n°1013366
Taz
bisounours-codeur
Posté le 15-03-2005 à 14:33:31  profilanswer
 

man strcmp !

n°1013369
NiQoZ1
Posté le 15-03-2005 à 14:37:08  profilanswer
 

strcmp prend deux pointeurs en parametre.. est ce là mon erreur car moi c'est des tableau de catactere (chaines)...
Mais je croyai qu'une chaine de caractère etait un pointeur sur le premier caractere ?


---------------
NiQoZ
mood
Publicité
Posté le 15-03-2005 à 14:37:08  profilanswer
 

n°1013382
NiQoZ1
Posté le 15-03-2005 à 14:46:34  profilanswer
 

bon j'ai compris mon erreur sic...sic( un peu boulet le bonhomme) :heink:  
 
donc  
 
(test = strcmp(prenom, listprenoms[j])) == NULL)
 
devien
 
(test = strcmp(prenom, listprenoms[j])) == '\0')
 
et là plus d'erreurs de type mais toujour le segfault à la fin


---------------
NiQoZ
n°1013386
Taz
bisounours-codeur
Posté le 15-03-2005 à 14:49:50  profilanswer
 

on va faire comme si j'avais rien lu.
 
strcmp retourne une valeur < 0, == 0 ou > 0
 
strcpy(a, b) retourne a

n°1013394
NiQoZ1
Posté le 15-03-2005 à 14:53:12  profilanswer
 

d'accord sorry Taz !
 
(test = strcmp(prenom, listprenoms[j])) == '0')
mais il ne trouve jamais le prenom ?


---------------
NiQoZ
n°1013402
NiQoZ1
Posté le 15-03-2005 à 15:00:20  profilanswer
 

pourquoi cette comparaison ne fonctionne pas ?  
car man strcmp dit:
-Elle renvoie un entier negatif, nul, ou positif, si s1  est  respectivement inferieure, egale ou superieure a s2
donc si mon entier test est nul c'est que la comparaison est égale.  
Mais ça ne fonctionne pas...
Je pense que ma chaine comparée doit avoir un retour chariot en trop, ou quelque chose comme ça ?


---------------
NiQoZ
n°1013404
Vinx
Posté le 15-03-2005 à 15:00:55  profilanswer
 

'0' ce n'est pas 0.
'0' ça doit faire 48 si je ne me trompe pas (Caractère '0').


---------------
Slack powa | http://www.racingpneu.com
n°1013405
Taz
bisounours-codeur
Posté le 15-03-2005 à 15:01:50  profilanswer
 

NiQoZ1 a écrit :

d'accord sorry Taz !
 
(test = strcmp(prenom, listprenoms[j])) == '0')
mais il ne trouve jamais le prenom ?


allez dégage ...

n°1013408
NiQoZ1
Posté le 15-03-2005 à 15:03:47  profilanswer
 

Code :
  1. #include <stdio.h>
  2. #include <string.h>
  3. #include <stdlib.h>
  4. #define BIGMAX 80
  5. #define MAX 20
  6. #define FAIL 1
  9. void RempliList(char *un_prenom, char prenom[]);
  10. void Affiche(char **liste, float tabnote[], short int tab_multi[], unsigned int max);
  15. int
  16. main(void) {
  17. char *listprenoms[MAX];
  18. float tab_notes[MAX];
  19. short int tab_multi[MAX];
  20. float note;
  21. char prenom[BIGMAX];
  22. unsigned int existe, test, i, j = 0;
  23. size_t s;
  27. for (i = 0; i < BIGMAX; i++) {
  28.  listprenoms[i] = NULL;
  29.  tab_multi[i] = 0;
  30. }
  31. i = 0;
  33. while (1) {
  34.  existe = 0;
  35.  printf("Saissisez le prenom ?\n" );
  36.  scanf("%s", prenom);
  37.  /* Sort si 'q' est pressé */
  38.  if ((prenom[0] == 'q') && (prenom[1] == '\0')) break;     s = strlen(prenom);
  39.  if (prenom[s - 1] == '\n') prenom[s - 1] = '\0';
  40.  printf("Saississez la note ?\n" );
  41.  scanf("%f", &note);
  43.   /* Teste dans la liste "listprenoms" si "prenom" existe       et, si oui  modifie sa note. "tab_multi" enregistre le       nombre de notes pour chaque prenom afin d'extraire       ensuite la moyenne...*/
  44.   for(j = 0; j < i; j++) {
  45.    if (listprenoms[j] != NULL) {
  47.     if ((test = strcmp(prenom, listprenoms[j])) == '0') {
  48.      tab_notes[j] += note;
  49.      tab_multi[j]++;
  50.      existe = 1;
  51.      break;
  52.     }
  53.    }
  54.   }
  55.   /*Allocation dynamique */
  56.   if (!existe) {
  57.   if ((listprenoms[i] = malloc(strlen(prenom) + 1)) == NULL) {
  58.    perror("malloc()" );
  59.    exit(FAIL);
  60.   }
  61.   /* Mise en memoire */
  62.   strcpy(listprenoms[i], prenom);
  63.   tab_notes[i] = note;
  64.   tab_multi[i]++;
  65.   }
  66.   i++; /*incremente i a chaque boucle while */
  67. }
  69.  Affiche(listprenoms, tab_notes, tab_multi, i); /*Affiche les moyennes */
  71. for (j = 0; j < i ; j++) {
  72.  free(listprenoms[j]);
  73.  tab_notes[j] = '\0';
  74.  tab_multi[j] = '\0';
  75. }
  77. return(0);
  78. }
  81. void
  82. Affiche(char **liste, float tabnote[], int short tab_multi[],unsigned int max)
  83. {
  84. short int i;
  85. for (i = 0; i < max; i++)
  86.  if (liste[i] != NULL)
  87.   printf("\nPrenom = %s\nNote moyenne =%2.2f\n", liste[i], tabnote[i] / tab_multi[i]);
  89. }


 
ma comparaison ne fonctionne pas et j'ai un segfault à la fin....


---------------
NiQoZ
n°1013412
NiQoZ1
Posté le 15-03-2005 à 15:07:04  profilanswer
 

c'est pas la peine d'^etre impoli je debute... vieux j'ai 30 balais et forcément c'est plus dur qua 18...
Donc mon 'strcmp' est faux encore, je n'ai donc toujours pas compris. Et bien désolé.


---------------
NiQoZ
n°1013424
Vinx
Posté le 15-03-2005 à 15:14:02  profilanswer
 

NiQoZ1 a écrit :

c'est pas la peine d'^etre impoli je debute... vieux j'ai 30 balais et forcément c'est plus dur qua 18...
Donc mon 'strcmp' est faux encore, je n'ai donc toujours pas compris. Et bien désolé.


Remplace ton '0' par 0 ;)
Ca ira mieux.


---------------
Slack powa | http://www.racingpneu.com
n°1013426
NiQoZ1
Posté le 15-03-2005 à 15:16:29  profilanswer
 

((test = strcmp(prenom, listprenoms[j])) == 0)
 
voila ca marche pour la comparaison les '' etait en trop.
Mais le segfault est toujour là à la fin.
(effectivement avec les cotes sa change tout)


---------------
NiQoZ
n°1013428
NiQoZ1
Posté le 15-03-2005 à 15:17:45  profilanswer
 

merci vinx je venai de m'en rendre compte mais j'ai toujour un segfaut à la fin.


---------------
NiQoZ
n°1013432
Taz
bisounours-codeur
Posté le 15-03-2005 à 15:20:54  profilanswer
 

gcc -g fichier.c
 
gdb ./a.out
run
bt
list

n°1013439
NiQoZ1
Posté le 15-03-2005 à 15:25:24  profilanswer
 

(gdb) run
Starting program: /home/gnoniqoz/tmp/a.out
Saissisez le prenom ?
toto
Saississez la note ?
12
Saissisez le prenom ?
q
 
Prenom = toto
Note moyenne =12.00
 
Program received signal SIGSEGV, Segmentation fault.
0x00000000 in ?? ()
(gdb) bt
#0  0x00000000 in ?? ()
(gdb) list
13
14
15      int
16      main(void) {
17              char *listprenoms[MAX];
18              float tab_notes[MAX];
19              short int tab_multi[MAX];
20              float note;
21              char prenom[BIGMAX];
22              unsigned int existe, test, i, j = 0;
(gdb)  


---------------
NiQoZ
n°1013488
Emmanuel D​elahaye
C is a sharp tool
Posté le 15-03-2005 à 15:53:20  profilanswer
 

NiQoZ1 a écrit :

d'accord sorry Taz !
 
(test = strcmp(prenom, listprenoms[j])) == '0')
mais il ne trouve jamais le prenom ?


On te dit que c'est 0 (la valeur zéro), pas '0' (le caractère 0) ...


if (strcmp(prenom, listprenoms[j])) == 0)
{
   /* j'ai trouve'... */
}


Message édité par Emmanuel Delahaye le 15-03-2005 à 15:54:51

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
n°1013506
NiQoZ1
Posté le 15-03-2005 à 16:04:57  profilanswer
 

Code :
  1. #include <stdio.h>
  2. #include <string.h>
  3. #include <stdlib.h>
  4. #define BIGMAX 80
  5. #define MAX 20
  6. #define FAIL 1
  9. void CalculEtAffiche(char **liste, float tabnote[], short int tab_multi[], unsigned int max);
  14. int
  15. main(void) {
  16. char *listprenoms[MAX];
  17. float tab_notes[MAX];
  18. short int tab_multi[MAX];
  19. float note;
  20. char prenom[BIGMAX];
  21. unsigned int existe, test, i, j = 0;
  22. size_t s;
  26. for (i = 0; i < BIGMAX; i++) {
  27.  listprenoms[i] = NULL;
  28.  tab_multi[i] = 0;
  29. }
  30. i = 0;
  32. while (1) {
  33.  existe = 0;
  34.  printf("Saisissez le prenom ?\n" );
  35.  fgets(prenom, BIGMAX, stdin);
  36.  prenom[BIGMAX - 1] = 0;
  37.  /* Sort si 'q' est pressé */
  38.  if ((prenom[0] == 'q') && (prenom[1] == '\n')) break;
  39.  s = strlen(prenom);
  40.  if (prenom[s - 1] == '\n') prenom[s - 1] = '\0';
  41.  printf("Saisissez la note ?\n" );
  42.  scanf("%f", &note);
  43.  getchar();
  45.   /* Teste dans la liste "listprenoms" si "prenom" existe       et, si oui  modifie sa note. "tab_multi" enregistre le       nombre de notes pour chaque prenom afin d'extraire       ensuite la moyenne...*/
  46.   for(j = 0; j < i; j++) {
  47.    if (listprenoms[j] != NULL) {
  49.     if ((test = strcmp(prenom, listprenoms[j])) == 0) {
  50.      tab_notes[j] += note;
  51.      tab_multi[j]++;
  52.      existe = 1;
  53.      break;
  54.     }
  55.    }
  56.   }
  57.   /*Allocation dynamique */
  58.   if (!existe) {
  59.   if ((listprenoms[i] = malloc(strlen(prenom) + 1)) == NULL) {
  60.    perror("malloc()" );
  61.    exit(FAIL);
  62.   }
  63.   /* Mise en memoire */
  64.   strcpy(listprenoms[i], prenom);
  65.   tab_notes[i] = note;
  66.   tab_multi[i]++;
  67.   }
  68.   i++; /*incremente i a chaque boucle while */
  69. }
  71.  CalculEtAffiche(listprenoms, tab_notes, tab_multi, i); /*CalculEtAffiche les moyennes */
  73. for (j = 0; j < i ; j++) {
  74.  free(listprenoms[j]);
  75.  tab_notes[j] = '\0';
  76.  tab_multi[j] = '\0';
  77. }
  79. return(0);
  80. }
  83. void
  84. CalculEtAffiche(char **liste, float tabnote[], int short tab_multi[],unsigned int max)
  85. {
  86. short int i;
  87. for (i = 0; i < max; i++)
  88.  if (liste[i] != NULL)
  89.   printf("\nPrenom = %s\nNote moyenne =%2.2f\n", liste[i], tabnote[i] / tab_multi[i]);
  91. }


 
oui oui, comme ecrit dans les posts précédents les cotes sont enlevées, mais j'ai toujours un segfault.


---------------
NiQoZ
n°1013516
Taz
bisounours-codeur
Posté le 15-03-2005 à 16:11:49  profilanswer
 

ce qui est beau sur mon powerpc, c'est que ce joli programme tourne indéfininment, pile corrompue oblige.
 
char *listprenoms[MAX];
...
 
 
    for (i = 0; i < BIGMAX; i++) {
       listprenoms[i] = NULL;
 
ça fait plein de jolis trucs :)

n°1013524
NiQoZ1
Posté le 15-03-2005 à 16:17:43  profilanswer
 

il se moque... Bon alors donc c'est tout faux il faut que je le reprenne totalement ou quelques parties sont "juste" ?
t'en pense quoi monsieur Taz...


---------------
NiQoZ
n°1013530
NiQoZ1
Posté le 15-03-2005 à 16:20:40  profilanswer
 

en fait je ne sait pas comment effectuer une recherche de chaine dans un tableau de pointeur sur chaine de caractère...


---------------
NiQoZ
n°1013535
Taz
bisounours-codeur
Posté le 15-03-2005 à 16:21:58  profilanswer
 

si tu commençais par pendre en compte les remarques, ça serait déjà un grand pas. alors tu corrige ce problème de borne, et tu reprends le topic depuis le début, et tu corriges, surtout les farces genre scanf("%s" :)

n°1013538
NiQoZ1
Posté le 15-03-2005 à 16:24:31  profilanswer
 

ya plus de cotes et scanf(%s est enleve au benefice de fgets.
regarde le dernier code posté à 16h04 c'est corrigé...
MAis toujours mon probleme de segfault


---------------
NiQoZ
n°1013540
NiQoZ1
Posté le 15-03-2005 à 16:26:03  profilanswer
 

et le fait que ca tourne indefiniment sur le Ppc j'espere que ton systeme n'est pas corrompu à cause de moi (ppc c'est stable ...)


---------------
NiQoZ
n°1013588
NiQoZ1
Posté le 15-03-2005 à 16:51:16  profilanswer
 

Code :
  1. #include <stdio.h>
  2. #include <string.h>
  3. #include <stdlib.h>
  4. #define BIGMAX 80
  5. #define MAX 20
  6. #define FAIL 1
  9. void CalculEtAffiche(char **liste, float tabnote[], short int tab_multi[], unsigned int max);
  14. int
  15. main(void) {
  16. char *listprenoms[MAX];
  17. float tab_notes[MAX];
  18. short int tab_multi[MAX];
  19. float note;
  20. char prenom[BIGMAX];
  21. unsigned int existe, test, i, j = 0;
  22. size_t s;
  26. for (i = 0; i < BIGMAX; i++) {
  27. /* listprenoms[i] = NULL;*/
  28.  tab_multi[i] = 0;
  29. }
  30. i = 0;
  32. while (1) {
  33.  existe = 0;
  34.  printf("Saisissez le prenom ?\n" );
  35.  fgets(prenom, BIGMAX, stdin);
  36.  prenom[BIGMAX - 1] = 0;
  37.  /* Sort si 'q' est pressé */
  38.  if ((prenom[0] == 'q') && (prenom[1] == '\n')) break;
  39.  s = strlen(prenom);
  40.  if (prenom[s - 1] == '\n') prenom[s - 1] = '\0';
  41.  printf("Saisissez la note ?\n" );
  42.  scanf("%f", &note);
  43.  getchar();
  45.   /* Teste dans la liste "listprenoms" si "prenom" existe       et, si oui  modifie sa note. "tab_multi" enregistre le       nombre de notes pour chaque prenom afin d'extraire       ensuite la moyenne...*/
  46.   for(j = 0; j < i; j++) {
  47.    if (listprenoms[j]) {
  48.     if ((test = strcmp(prenom, listprenoms[j])) == 0) {
  49.      tab_notes[j] += note;
  50.      tab_multi[j]++;
  51.      existe = 1;
  52.      break;
  53.     }
  54.    }
  55.   }
  56.   /*Allocation dynamique */
  57.   if (!existe) {
  58.   if ((listprenoms[i] = malloc(strlen(prenom) + 1)) == NULL) {
  59.    perror("malloc()" );
  60.    exit(FAIL);
  61.   }
  62.   /* Mise en memoire */
  63.   strcpy(listprenoms[i], prenom);
  64.   tab_notes[i] = note;
  65.   tab_multi[i]++;
  66.   }
  67.   i++; /*incremente i a chaque boucle while */
  68. }
  70.  CalculEtAffiche(listprenoms, tab_notes, tab_multi, i); /*CalculEtAffiche les moyennes */
  72. for (j = 0; j < i ; j++) {
  73.  free(listprenoms[j]);
  74.  tab_notes[j] = '\0';
  75.  tab_multi[j] = '\0';
  76. }
  78. return(0);
  79. }
  82. void
  83. CalculEtAffiche(char **liste, float tabnote[], int short tab_multi[],unsigned int max)
  84. {
  85. short int i;
  86. for (i = 0; i < max; i++)
  87.  if (liste[i] != NULL)
  88.   printf("\nPrenom = %s\nNote moyenne =%2.2f\n", liste[i], tabnote[i] / tab_multi[i]);
  90. }


 
chez moi ça fonctionne parfaitement...ouf
pour saisir des nombre entiers ou flottants faut t'il utiliser scanf ? si non quelle fonction ?


---------------
NiQoZ
n°1013606
NiQoZ1
Posté le 15-03-2005 à 16:55:50  profilanswer
 

Merci à tous pour vos réponses, je peut comprendre que j'agace les puristes. Mais je ne le fait pas expres. D'ailleur je me suis posé la question si je postait ce code ou pas. Mais parfois il faut un peu de courage temp pis pour le phrases assasines et le ridicule.


---------------
NiQoZ
n°1013713
Emmanuel D​elahaye
C is a sharp tool
Posté le 15-03-2005 à 18:02:26  profilanswer
 

NiQoZ1 a écrit :

chez moi ça fonctionne parfaitement...ouf
pour saisir des nombre entiers ou flottants faut t'il utiliser scanf ? si non quelle fonction ?


Voici une version plus industrielle de ton code. On peut faire mieux avec un tableau de structures plutôt que 3 tableaux de même taille...
 

  • Réduction de la portée des variables
  • simplification du code et des types
  • fonctions de saisie solide.



#include <stdio.h>
#include <string.h>
#include <stdlib.h>
 
void CalculEtAffiche (char **liste
                      ,double tabnote[]
                      ,int tab_multi[]
                      ,size_t max)
{
   size_t i;
 
   for (i = 0; i < max; i++)
   {
      if (liste[i] != NULL)
      {
         printf ("\nPrenom = %s\n"
                 "Note moyenne =%2.2f\n"
                 ,liste[i]
                 ,tabnote[i] / tab_multi[i]);
      }
   }
}
 
void get_s (char *s, size_t size)
{
 
   fgets (s, size, stdin);
   {
      /* search ... */
      char *p = strchr (s, '\n');
 
      if (p != NULL)
      {
         /* ... and kill... */
         *p = 0;
      }
      else
      {
         /* ... or clean. */
         int c;
 
         while ((c = getchar ()) != '\n' && c != EOF)
         {
         }
      }
   }
}
 
double get_d (void)
{
   double val;
   char s[32];
 
   get_s (s, sizeof s);
 
   val = strtod (s, NULL);
 
   return val;
}
 
 
int main (void)
{
#define MAX 20
 
   int tab_multi[MAX] =
   {0};
   char *listprenoms[MAX] =
   {0};
   double tab_notes[MAX] =
   {0};
 
   size_t i = 0;
 
   while (1)
   {
      int existe = 0;
      char prenom[80];
 
      printf ("Saisissez le prenom ?\n" );
 
      get_s (prenom, sizeof prenom);
 
      /* Sort si 'q' est presse */
      if (strcmp (prenom, "q" ) == 0)
      {
         break;
 
      }
 
      printf ("Saisissez la note ?\n" );
 
      {
         double note = get_d ();
 
         /* Teste dans la liste "listprenoms" si "prenom" existe        
          * et, si oui  modifie sa note. "tab_multi" enregistre le        
          * nombre de notes pour chaque prenom afin d'extraire        
          * ensuite la moyenne...  
          */
         {
            size_t j;
 
            for (j = 0; j < i; j++)
            {
               if (listprenoms[j])
               {
                  if (strcmp (prenom, listprenoms[j]) == 0)
                  {
                     tab_notes[j] += note;
                     tab_multi[j]++;
                     existe = 1;
                     break;
                  }
               }
            }
         }
 
         /* Allocation dynamique */
         if (!existe)
         {
            if ((listprenoms[i] = malloc (strlen (prenom) + 1)) == NULL)
            {
               perror ("malloc()" );
               exit (EXIT_FAILURE);
            }
            /* Mise en memoire */
            strcpy (listprenoms[i], prenom);
            tab_notes[i] = note;
            tab_multi[i]++;
         }
         i++;                   /* incremente i a chaque boucle while */
      }
   }
 
   /* CalculEtAffiche les moyennes */
   CalculEtAffiche (listprenoms, tab_notes, tab_multi, i);
 
   {
      size_t j;
 
      for (j = 0; j < i; j++)
      {
         free (listprenoms[j]);
         tab_notes[j] = 0;
         tab_multi[j] = 0;
      }
   }
   system ("pause" );
 
   return 0;
}


Message édité par Emmanuel Delahaye le 15-03-2005 à 21:54:10

---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
n°1013715
Emmanuel D​elahaye
C is a sharp tool
Posté le 15-03-2005 à 18:04:10  profilanswer
 

NiQoZ1 a écrit :

Merci à tous pour vos réponses, je peut comprendre que j'agace les puristes. Mais je ne le fait pas expres. D'ailleur je me suis posé la question si je postait ce code ou pas. Mais parfois il faut un peu de courage temp pis pour le phrases assasines et le ridicule.


Le problème n'est pas ton code intial qui est ce qu'il est, mais plutôt quand tu lis mal les réponses, et ça, ça énerve...


---------------
Des infos sur la programmation et le langage C: http://www.bien-programmer.fr Pas de Wi-Fi à la maison : http://www.cpl-france.org/
n°1013761
NiQoZ1
Posté le 15-03-2005 à 18:30:49  profilanswer
 

merci beaucoup pour ton exemple je vais de ce pas l'etudier en detail .


---------------
NiQoZ
n°1013800
Taz
bisounours-codeur
Posté le 15-03-2005 à 19:19:39  profilanswer
 

Emmanuel > Franchement arrête de faire les corrigés. Tu vois bien que certaines personnes ne font pas d'effort. Je lui fait une remarque sur la borne sup d'un tableau ... il commente l'instruction au petit bonheur la chance, ET à la ligne en dessous, c'est exactement le même problème : dépassement de capacité ...
C'est vraiment pas bien de faire des corrigés, surtout quand la personne ne comprend rien et n'a pas envie de.

n°1014161
NiQoZ1
Posté le 16-03-2005 à 08:53:21  profilanswer
 

il a manger quoi le bisounours pour être aussi con. Car si la ligne d'en dessou ne lui plai pas c'est peu être que l'apprenti codeur n'a pas compri la 'remarque'...
Alors heureusement qu'il y a des gars comme emmanuel sinon les débutants ne feraient que de ce heurter à des remarques de ce genre et ne progresserai pas d'un yota.  
A bon entendeur (mais à mon avis il est sourd) salut...


---------------
NiQoZ
mood
Publicité
Posté le   profilanswer
 


Aller à :
Ajouter une réponse
  FORUM HardWare.fr
  Programmation
  C

  [ C ] Archi-debutant segfault sur petit programme :-(

 

Sujets relatifs
Programmation d'un programme de détection d'attaque réseau[Débutant] wxStyledTextCtrl
modification d'un programme[RECH] Jeu de foot pour programme de tracking
Programme de pic[C++]Mon programme compile mais plante.
Modifier les bordures d'un tableau PHP (débutant inside)debutant pb structure
Programme C++ utilisant une base de donnéemon programme fonctionne t il chez vous?
Plus de sujets relatifs à : [ C ] Archi-debutant segfault sur petit programme :-(

Forum MesDiscussions.Net, Version 2010.2
(c) 2000-2011 Doctissimo
Page générée en 0.230 secondes

Copyright © 1997-2022 Hardware.fr SARL (Signaler un contenu illicite / Données personnelles) / Groupe LDLC / Shop HFR