erreur avec les floats

Version imprimable

Re-bonjour,

Je n'en peux plus, je ne comprends pas mon erreur....

Je sais que tout ce qui concerne le nb d'eleve et le j sont inutile pour l'instant.

Code:

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
 
#include<stdlib.h>
#include<stdio.h>
 
int main(){
    int nbnotes,nbeleves, i=0, j=0; 
    float note=0;
    float temp=0;
 
    scanf("%d",&nbeleves);
    fflush(stdin);
 
    int tab_nbnotes[nbeleves];
    float tab_notes[nbeleves][100];
    float tab_moys[nbeleves];
 
    while (note != -1) {
          scanf("%f",&note);
          fflush(stdin);
          if (note < 0){
             tab_notes[j][i] = note;
          }
          i++;
    }
 
 
    i=0;
    tab_nbnotes[j]=0;
    for(i=0;i<=100;i++){
 
                        if(tab_notes[j][i] > 0){
                                           tab_nbnotes[j]++;
                        }
    }
 
    for(i=0;i<=100;i++){
                        temp = temp + tab_notes[j][i];
    }
 
    tab_moys[j] = temp/tab_nbnotes[j];
 
    printf("La moyenne de l'eleve %d est %f",j,tab_moys[j]);
    system("PAUSE");
    return 0;
 
 
 
}

Si je rentre 1(nb d'eleve), 5, 5, 5, 5 (notes), et -1 par exemple, la moyenne affichée foire totalement ce n'est même pas des chiffres...

Quelqu'un vois ou j'ai faux ? merci.

PS : si quelqu'un est motivé et veux m'epliquer sur msn...

Code:

1
2
3
    scanf("%d",&nbeleves); 
 ....
    int tab_nbnotes[nbeleves];

Ceci est accepté par ton compilateur (C99) ?

Code:

1
2
3
4
5
6
7
8
  while (note != -1) { 
          scanf("%f",&note); 
          ....
          if (note < 0){ 
             tab_notes[j][i] = note; 
          } 
          i++; 
    }

Tu es un prof sympa, tu ne mets que des notes négatives! :D

Code:

1
2
3
4
5
for(i=0;i<=100;i++){ 
 
                        if(tab_notes[j][i] > 0){ 
                                           tab_nbnotes[j]++; 
                        }

Des notes >0, du coup,il ne doit pas y avoir foule!

Code:

tab_moys[j] = temp/tab_nbnotes[j];

Ca doit ressembler à une division par zéro!

11/10/2005, 22h37
Pouic

Et je note une utilisation plus qu'hasardeuse de scanf dans une boucle, un innomable fflush(stdin), et une visible aversion d'Halobox pour les allocations dynamiques (cette façon de déclarer les tableaux... beurk)
11/10/2005, 22h48
Halobox

Bah on m'a dit de mettre un fflush(stdin) apres chaque scanf

et pour diogene :

Oui ca fonctionne avec mon compilateur, enfaite dans l'idée, c'est quand je met la note -1 ca s'arrette de compter, par contre piur le 3eme truc :

Citation:

Des notes >0, du coup,il ne doit pas y avoir foule!

bah si pourquoi ? des notes >0 c'est 1,2..... ?

Je ne vois pas ou j'ai faux.

Pouic : Tu les déclarerais comment mes tableaux ?, je ne te compends pas?
11/10/2005, 22h53
diogene
Citation:

bah si pourquoi ? des notes >0 c'est 1,2..... ?

Oui , même 20. Mais tu ne prends les notes QUE si elles sont négatives. Les autres tu les ignores!
Code:

1 2 3 if (note < 0){ tab_notes[j][i] = note; }
11/10/2005, 22h56
Halobox

Ok j'a réctifié, mais ca ne change rien, j'ai toujours un résultat foireux, avec un # et des caractères bizarres.

:(
11/10/2005, 22h58
Pouic
Citation:

Envoyé par Halobox

Bah on m'a dit de mettre un fflush(stdin) apres chaque scanf

Bêtises.... Déjà, fflush(stdin) n'est pas un comportement normalisé. Certains compilo te font croire que ``ca marche'', mais bonjour les surprises...
Quant à vider stdin suite à un scanf, il y a d'autres méthodes (avec les expressions rationnelles, par exemple, ou tout simplement avec un :
Code:

1 2 3 4 5 int c; while ( (c=getchar()) != '\n' && c!= EOF) { }
nettement plus standard...

Citation:

Pouic : Tu les déclarerais comment mes tableaux ?, je ne te compends pas?

J'utiliserais malloc....
11/10/2005, 23h02
Halobox

Waoh tu s'ais hje débute, et je dois me débrouiller avec ce que mon prof m'appris...

Enfin merci qd même...

Re: erreur avec les floats

Citation:

Envoyé par Halobox

Je n'en peux plus, je ne comprends pas mon erreur....
Quelqu'un vois ou j'ai faux ?

Pas mal d'erreurs. Remarques et correction. Attention, le calcul est faux, et je ne vois pas pas pourquoi...

Code:

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
103
104
105
106
107
108
109
110
111
 
#include<stdlib.h>
#include<stdio.h>
 
int main()
{
   int nbnotes,nbeleves, i=0, j=0;
   float note=0;
   float temp=0;
 
   /* -ed- pas facile de taper 'en aveugle'... */
   puts("Nombre d'eleves");
   scanf("%d",&nbeleves);
 
   /* -ed-
   fflush(stdin);
 
   n'existe pas en C.
   fflush() n'est défini que pour les flux sortants...
   
   bidouille infame:
   */
   (void) getchar();
 
    /* -ed- Les VLA sont C99.
            Je ne connais pas d'implementation gratuite
            conforme sur ce point.
     */
    int tab_nbnotes[nbeleves];
    float tab_notes[nbeleves][100];
    float tab_moys[nbeleves];
 
    /* -ed-
   while (note != -1)
 
      Un float ne peut pas etre compare a un entier.
      Il faut faire la difference absolue et comparer a FLT_EPSILON
      (<float.h>) ou, ici, tout simplement >=0
 
   */
   puts("Notes (-1 pour terminer)");
 
   while (note >= 0)
   {
      /* -ed- ajout */
      printf("%2d : ", i + 1);
      fflush (stdout);
 
      scanf("%f",&note);
      (void) getchar();
 
/* -ed-
      if (note < 0)
 
   Tu voulais sans doute dire
*/
      if (note >= 0)
      {
         tab_notes[j][i] = note;
      }
      i++;
   }
 
/* -ed-
   J'ai pas compris a quoi servait ce bazar...
 
   i=0;
   tab_nbnotes[j]=0;
   
   for(i=0;i<=100;i++)
   {
      if(tab_notes[j][i] > 0)
      {
         tab_nbnotes[j]++;
      }
   }
*/
   tab_nbnotes[j] = i;
 
   /* -ed-
   for(i=0;i<=100;i++)
 
      Debordement du tableau (i<100)
      Le tableau n'etant pas initialise,
      le resultat est indertermine.
      Il faut donc se limiter au nombre de notes.
      */
   nbnotes = tab_nbnotes[j];
   for (i = 0; i < nbnotes; i++)
   {
      temp += tab_notes[j][i];
   }
 
   /* -ed-
   tab_moys[j] = temp/tab_nbnotes[j];
   
   Simplification. Le nombre de notes, c'est nbnotes...
   Il faut donc se proteger de nbnotes==0...
   */
   if (nbnotes != 0)
   {
      tab_moys[j] = temp/nbnotes;
 
      printf ("La moyenne de l'eleve %d est %f (%d notes)\n"
             , j
             , tab_moys[j]
             , nbnotes
             );
   }
   return 0;
}

11/10/2005, 23h05
Emmanuel Delahaye

Citation:

Envoyé par Halobox

Bah on m'a dit de mettre un fflush(stdin) apres chaque scanf

Celui qui t'a dit ça mérite qu'on lui ouvre le ventre, qu'on y mette un rat affamé, et qu'on le recouse...
11/10/2005, 23h08
Pouic

Citation:

Envoyé par Emmanuel Delahaye

Citation:

Envoyé par Halobox

Bah on m'a dit de mettre un fflush(stdin) apres chaque scanf

Celui qui t'a dit ça mérite qu'on lui ouvre le ventre, qu'on y mette un rat affamé, et qu'on le recouse...

Recoudre qui ? Le rat ? :mouarf2:
11/10/2005, 23h09
Halobox

Ok merci je vais essayer de dechiffer ton code (aucune critique), sinon pour ta remarque d'aveugle, je sais bien, j'ai simplifier a mort avant de le poster voila pourquoi il n'y a rien.

A+ et merci
11/10/2005, 23h13
Halobox

Enfaite, peux tu m'expliquer cela :

Citation:

Il faut faire la difference absolue et comparer a FLT_EPSILON (<float.h>)

Merci
11/10/2005, 23h19
Emmanuel Delahaye

Citation:

Envoyé par Halobox

Enfaite, peux tu m'expliquer cela :

Citation:

Il faut faire la difference absolue et comparer a FLT_EPSILON (<float.h>)

Merci

Et si tu lisais les réponse qu'on se troue le c*l à te poster ?

http://www.developpez.net/forums/vie...287419#2287419

http://mkmb.net/logocolor/03551SAGMYX5.gif
11/10/2005, 23h23
Halobox

ok escuse dsl, j'ai plus les alertes mails sur l'autre sujet.

A+