Problème a la compilation

Version imprimable

Bonjour je fait un exo qui dit de faire le calculer d'une moyenne et de donner le nombre de note insuffisantes.
Voila ce que j'ai fait:

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
 
#include <stdio.h>
 
float moyenne(float tab_nt[], int nbNt)
{
  int i;
  float moy;
  float som=0;
  for (i=0; i<nbNt; i++)
    som=som+tab_nt[i];
  moy=som/nbNt;
  return moy;
}
 
int insuf(float tab_nt[], float moy, int nbNt)
{
  int i;
  int som=0;
  for (i=0; i<nbNt; i++)
    if (tab_nt[i]<moy)
      som=som+1;
  return som;
 
int main()
{
  float moy;
  int nbNt,i;
  printf("Combien de note seron incerer : ");
  scanf ("%d",&nbNt);
  float tab_nt[nbNt];
  printf("\nQuelle sera la moyenne : ");
  scanf ("%f",&moy);
  float moy_el=0;
  int nt_insuf=0;
  for (i=0; i<nbNt; i++)
    {
      printf("La %d note : ", i+1);
      scanf ("%f",&tab_nt[i]);
    }
  moy_el=moyenne(tab_nt,nbNt);
  nt_insuf=insuf(tab_nt,nbNt,moy);
  printf ("\nLa moyenne et de %.3f avec %d note dessou d'une moyenne de %f\n",moy_el, nt_insuf, moy);
}

Mais j'ai le mesage d'ereur suivent:

Citation:

gcc -std=c99 exo1.c -o exo1
exo1.c: In function <<insuf":
exo1.c:42: erreur: expected declaration or statement at end of input

elle est ou la faute?
merci

13/02/2008, 13h31
Arnaud F.
Salut,

Voici ton prototype :
Code:

1 2 int insuf(float tab_nt[], float moy, int nbNt)
Et la manière dont tu l'appelles :

Code:

nt_insuf=insuf(tab_nt,nbNt,moy);

Ne remarques-tu pas l'erreur par toi-même?

Tu as inversé les paramètres dans ton appel, il faut appeler insuf de la sorte :

Code:

nt_insuf=insuf(tab_nt, moy, nbNt);

@+
13/02/2008, 13h35
krachik

Bonjour
Et aussi qu'il te manque l'accolade de fermeture de ta fonction insuf avant le main ;)
@+
13/02/2008, 13h38
gallos986

Oui c'est vrais. Je l'avais pas remarquer.:lol:

merci
13/02/2008, 14h51
corentin59
Citation:
Envoyé par gallos986

Code:

1 2 3 4 int nbNt; printf("Combien de note seron incerer : "); scanf ("%d",&nbNt); float tab_nt[nbNt];
Bon, à confirmer par les pros mais je ne pense pas que ce soit une bonne idée d'allouer de la mémoire comme cela !!

Sinon, ça n'a rien à voir avec le C, mais essaies d'afficher des phrases écrites en français;)
13/02/2008, 16h05
diden138

Re : bonjour
oui effectivement
Code:

1 2 3 4 5 int nbNt; printf("Combien de note seron incerer : "); scanf ("%d",&nbNt); float tab_nt[nbNt];
le principe même d'un tableau est de réserver un nombre connue d'éléments :aie:
cordialement @+
13/02/2008, 16h17
gallos986

En faite j'ai fait ce pour me rappeler un peut commen on travaille les tableaux et les sous algorithme.

Mois en tout cas c'est comme ca que j'allouer de la mémoire pour un tableau quand je ne conneser pas ca taille dé le depart. Comment je doit faire ci-non?
13/02/2008, 16h20
DaZumba

Citation:

Envoyé par corentin59

Bon, à confirmer par les pros mais je ne pense pas que ce soit une bonne idée d'allouer de la mémoire comme cela !!

C'est C99 (variable-length arrays) mais la qualite du support par les compilateurs actuels est assez variable (en general, support incomplet). Il vaut mieux ne pas les utiliser et allouer la memoire via malloc().
13/02/2008, 17h06
Thierry Chappuis

Citation:

Envoyé par DaZumba

C'est C99 (variable-length arrays) mais la qualite du support par les compilateurs actuels est assez variable (en general, support incomplet). Il vaut mieux ne pas les utiliser et allouer la memoire via malloc().

Concrètement, au moins gcc et le compilateur C de Microsoft ne le supporte pas à l'heure actuelle.

Thierry
13/02/2008, 17h08
gallos986

J'ai pas trop compri 8O :roll: ce que tu veut dire DaZumba
13/02/2008, 17h48
DaZumba

Citation:

Envoyé par gallos986

J'ai pas trop compris ce que tu veux dire DaZumba

Dans ton code, la dimension nbNt n'est pas connue a la compilation, puisqu'elle est entree par l'utilisateur. Tu t'en sers tout de meme pour dimensionner le tableau tab_nt. Ce n'est possible que dans une version assez "recente" du C standard, appelee C99. On appelle cela les tableaux a taille variable (variable-length arrays, ou VLAs). Dans les versions precedentes du C standard (C90 notamment), il faut que la taille des tableaux soit connue a la compilation.

Evidemment, les VLAs sont simples a utiliser, et on pourrait croire que depuis 1999 ou 2000, les compilateurs ont eu le temps d'implementer l'ensemble des changements introduits par C99, dont les VLAs. Ce n'est pas le cas, soit par decision de ne pas implementer C99 (parce qu'on prefere travailler sur l'implementation du C++, par exemple), soit parce que c'est difficile a faire. Du coup, il faut faire attention lorsqu'on utilise des nouveautes C99.

Dans ton cas, je pense qu'un compilateur "a peu pres" C99 (comme gcc) va pouvoir se debrouiller: ton programme est tres simple. Mais je te conseille de faire aussi une version sans VLA, qui utilise malloc() pour allouer un tableau dynamiquement. Voir ton livre de C ou les tutorials sur le Net pour apprendre a utiliser malloc() et free().
13/02/2008, 23h55
Emmanuel Delahaye

Citation:

Envoyé par corentin59

Bon, à confirmer par les pros mais je ne pense pas que ce soit une bonne idée d'allouer de la mémoire comme cela !!

Sinon, ça n'a rien à voir avec le C, mais essaies d'afficher des phrases écrites en français;)

C'est techniquement possible en C99, mais ce n'est officiellement pas au point avec gcc...