Discussion :

[patatrac13500@free.f]

Bonjour,
je commence a develloper un petit peu et je n'arrive pas a coriger un exercice a trou...
si quelqu'un peu m'aider....en italique les trous que j'ai rempli mais ça fonctionne pas

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
int main(int argc, char *argv[])
{
  int i;
  char nom_produit[20];
  float prix_produit[5];
  float prix_moyen;
  float prix_min;
  
  void moy_min_prix (float prix[5], float *pprix_moyen,float *pprix_min);
  printf ("saisir le nom du produit : ");
  scanf("%s",nom_produit);
  
  for (i=0;i<5;i++)
      {
      printf ("donner le prix du %s dans le magasin %d : ",nom_produit,i+1);
      scanf ("%f",prix_produit);
      }
      
  moy_min_prix ( &prix_produit[5], &prix_moyen, &prix_min);     
  printf("Le prix moyen du %s est : %2f",nom_produit, &prix_moyen);  
printf ("\nLe prix le plus bas du %s est : %2f\n",nom_produit,&prix_min);
  
  system("PAUSE");	
  return 0;
}

void moy_min_prix( float prix[5],float *pprix_moyen,float *pprix_min){
 int j;
 float somme=0;
 
 for (j=0;j<5;j++)
    somme=somme+prix[j];
    *pprix_moyen=somme/5;
    *pprix_min=prix[0];
     
     for (j=1;j<5;j++)
     if (prix[j]<*pprix_moyen);
     *pprix_min=prix[j];
}
;

[juls64]

qu'est ce qui ne marche pas ?
compil pas ?
erreur apres la compilation ?
se serait mieux que tu t'expliques un peu plus pour comprendre rapidement et si tu as des erreurs le mieux c de les citer avec le numero des lignes

[fearyourself]

Si j'ai bien compris voici le code de départ qu'il faut remplir :

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
int main(int argc, char *argv[])
{
  int i;
  char nom_produit[20];
  float prix_produit[5];
  float prix_moyen;
  float prix_min;
  
  void moy_min_prix (____, _____,_____);
  printf ("saisir le nom du produit : ");
  scanf("%s",______);
  
  for (i=0;i<5;i++)
      {
      printf ("donner le prix du %s dans le magasin %d : ",______,i+1);
      scanf ("%f",______);
      }
      
  moy_min_prix ( ______,______, ______);   [/I]  
  printf("Le prix moyen du %s est : %2f",______, ______);[/I]  
  printf ("\nLe prix le plus bas du %s est : %2f\n",______);
  
  system("PAUSE");	
  return 0;
}

void moy_min_prix( ______,______,______){
 int j;
 float somme=0;
 
 for (j=0;j<5;j++)
    somme=somme+prix[j];
    *pprix_moyen=somme/5;
    *pprix_min=prix[0];
     
     for (j=1;j<5;j++)
     if (prix[j]<______);
     ______=prix[j];
}

Et il faut le compléter, c'est ca ?

Jc

PS: Tu pourrais dire à ton prof que le temps de scanf est révolu et que mettre un prototype dans le main....

[Médinoc]

Code :

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

scanf("%s",______);

Ton prof a réussi à regrouper les deux pires utilisations possibles de scanf() en un seul exercice:

• scanf("%s")
• Enseigner scanf() aux débutants.

---> Change de prof.

[Dud225]

Code :

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

scanf("%s",______);

Ton prof a réussi à regrouper les deux pires utilisations possibles de scanf() en un seul exercice:

• scanf("%s")
• Enseigner scanf() aux débutants.

---> Change de prof.

J'ai aussi eu le même genre de prof....mais je peux vous assurer que c'est pas un guignol!
Qu'est ce qu'il y a comme problèmes avec scanf??

[Médinoc]

scanf() est difficile à bien utiliser.
La preuve, scanf("%s") offre à peu près autant de résistance au test du singe que gets().

En clair, tu lances le programme, tu tapes aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa et tu regardes si ça plante ou non.

(accessoirement, sous un système unixoïde, tu peux aussi redigirer /dev/random sur l'entrée standard.

[Dud225]

scanf() est difficile à bien utiliser.
La preuve, scanf("%s") offre à peu près autant de résistance au test du singe que gets().

En clair, tu lances le programme, tu tapes aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa et tu regardes si ça plante ou non.

(accessoirement, sous un système unixoïde, tu peux aussi redigirer /dev/random sur l'entrée standard.

Oui c'est sûr qu'il faut faire attention au buffer overflow! Mais si tu déclares un tableau très grand et que tu supposes qu'il n'y a pas de gogols qui s'amusent à rentrer des chaînes de caractères de 3km de long un scanf convient parfaitement non?

[odsen.s]

Oui c'est sûr qu'il faut faire attention au buffer overflow! Mais si tu déclares un tableau très grand et que tu supposes qu'il n'y a pas de gogols qui s'amusent à rentrer des chaînes de caractères de 3km de long un scanf convient parfaitement non?

Ah oui c'est super il faut déclarer des tableaux très grands et il y a toujours un risque de plantage... mais c'est génial ! Ca ne prend tout de même pas longtemps de se créer une fonction d'entrées solide, ou d'apprendre à se servir de fgets() !

http://emmanuel-delahaye.developpez.com/inputs.htm

[stephl]

J'ai fait cela vite fait alors je ne suis pas certain d'avoir tout corrigé.

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
#include <stdio.h>
 
 
int main(int argc, char *argv[])
{
  int i;
  char nom_produit[20];
  float prix_produit[5];
  float prix_moyen;
  float prix_min;
 
  void moy_min_prix (float [5], float *,float *);
  printf ("saisir le nom du produit : ");
  scanf("%s",nom_produit);
 
  for (i=0;i<5;i++)
      {
      printf ("donner le prix du %s dans le magasin %d : ",nom_produit,i+1);
      scanf ("%f",prix_produit+i);
      }
 
  moy_min_prix ( prix_produit, &prix_moyen, &prix_min);     
  printf("Le prix moyen du %s est : %f",nom_produit, prix_moyen);  
printf ("\nLe prix le plus bas du %s est : %f\n",nom_produit,prix_min);
 
  system("PAUSE");	
  return 0;
}
 
void moy_min_prix( float prix[5],float *pprix_moyen,float *pprix_min){
 int j;
 float somme=0;
 
 for (j=0;j<5;j++)
    somme=somme+prix[j];
    *pprix_moyen=somme/5;
    *pprix_min=prix[0];
 
     for (j=1;j<5;j++)
     if (prix[j]<*pprix_min)
     *pprix_min=prix[j];
}

[Médinoc]

Code :

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

scanf("%s",nom_produit);

Moi non plus je n'en suis pas certain...

[stephl]

Code :

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

scanf("%s",nom_produit);

Moi non plus je n'en suis pas certain...

Il n'y a pas de problème fondamental sur cette ligne. Dans la suite du programme, seul des float sont attendus donc il n'est pas absolument nécessaire de vider le buffer. En outre, il n'est pas précisé que le nom de produit peut contenir des espaces. Enfin, ce code suppose qu'un nom de produit fait au plus 19 caractères.

[Thierry Chappuis]

Il n'y a pas de problème fondamental sur cette ligne. Dans la suite du programme, seul des float sont attendus donc il n'est pas absolument nécessaire de vider le buffer. En outre, il n'est pas précisé que le nom de produit peut contenir des espaces. Enfin, ce code suppose qu'un nom de produit fait au plus 19 caractères.

Si nom_produit ne contient pas d'espace:

Code :

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

scanf("%19s", nom_produit);

convient.

Je ne comprends pas ta remarque à propos de la non nécessité de vider le tampon avant saisie des flottants. C'est à mon sens une bonne pratique de "nettoyer" le tampon après chaque saisie selon le principe: (1) J'utilise le tampon pour la saisie (2) Je nettoie après mon passage. Ca clarifie les choses, non?

Thierry

[stephl]

Si nom_produit ne contient pas d'espace:

Code :

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

scanf("%19s", nom_produit);

convient.

Tout à fait d'accord. C'est ce que je fais en pratique.

Je ne comprends pas ta remarque à propos de la non nécessité de vider le tampon avant saisie des flottants.

scanf() n'est utilisée par la suite que pour saisir des flottants, donc scanf() ne retournera pas à cause du '\n' qui reste dans le buffer après l'entrée de la chaîne. Il n'est donc pas absolument nécessaire de faire le ménage dans ce cas précis. En revanche, si d'autres entrées de chaînes avaient été programmées, il aurait fallu vider le buffer d'entrée.

[Médinoc]

En outre, il n'est pas précisé que le nom de produit peut contenir des espaces. Enfin, ce code suppose qu'un nom de produit fait au plus 19 caractères.

Il n'est pas non plus précisé qu'il ne peut pas en contenir. Ça fait beaucoup de suppositions, tout cela...

Le scanf("%19s", nom_produit) est un minimum. Descendre en-dessous est suicidaire.

[stephl]

Il n'est pas non plus précisé qu'il ne peut pas en contenir. Ça fait beaucoup de suppositions, tout cela...

Le scanf("%19s", nom_produit) est un minimum. Descendre en-dessous est suicidaire.

Je ne pense pas qu'un éventuel plantage de ce programme entraîne la fin du monde. Je me suis contenté de répondre aux attentes du PO, plutôt que de partir tout de suite en croisade contre les entrées non sécurisées. Vous vous êtes attaché à ces problèmes dès le départ, alors que selon moi, des erreurs telles que:

Code :

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

scanf ("%f",prix_produit);

sont bien plus importantes.

[Thierry Chappuis]

Tout à fait d'accord. C'est ce que je fais en pratique.


scanf() n'est utilisée par la suite que pour saisir des flottants, donc scanf() ne retournera pas à cause du '\n' qui reste dans le buffer après l'entrée de la chaîne. Il n'est donc pas absolument nécessaire de faire le ménage dans ce cas précis. En revanche, si d'autres entrées de chaînes avaient été programmées, il aurait fallu vider le buffer d'entrée.

Qu'est ce qui te fait dire qu'il ne restera que '\n' dans le tampon?

Thierry

[stephl]

Qu'est ce qui te fait dire qu'il ne restera que '\n' dans le tampon?

Thierry

Si l'utilisateur ne tape pas d'espace, le format "%s" aura tout retiré du buffer d'entrée (mis à part le '\n').

[Thierry Chappuis]

Je ne pense pas qu'un éventuel plantage de ce programme entraîne la fin du monde. Je me suis contenté de répondre aux attentes du PO, plutôt que de partir tout de suite en croisade contre les entrées non sécurisées. Vous vous êtes attaché à ces problèmes dès le départ, alors que selon moi, des erreurs telles que:

Code :

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

scanf ("%f",prix_produit);

sont bien plus importantes.

A ceci près qu'un bon compilateur (bien règlé) pourra générer un message d'avertissement dans ce cas, ce qui n'est pas le cas avec un buffer overflow:

Code :

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

1
2
3
4
5
6
7
8
9
10
#include <stdio.h>
#include <stdlib.h>
 
int main(void)
{
    float x;
    scanf("%f", x);
 
    return EXIT_SUCCESS;
}

et le message l'avertissement retourné par gcc avec l'option -Wall:

Code :

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

1
2
/home/thierry/devel/Forums/main.c:7: attention : format ‘%f’ expects type ‘float *’, but argument 2 has type ‘double’
:: === Build finished: 1 errors, 0 warnings ===

C'est suffisamment explicite pour moi, et je préfère me concentrer sur les entrées sécurisées.

Thierry

[Thierry Chappuis]

Si l'utilisateur ne tape pas d'espace, le format "%s" aura tout retiré du buffer d'entrée (mis à part le '\n').

Si l'utilisateur ne tape pas d'espace, en effet.

Thierry

[stephl]

C'est suffisamment explicite pour moi, et je préfère me concentrer sur les entrées sécurisées.

Chacun sa façon de voir. Personnellement, je trouve qu'oublier l'opérateur d'adresse est une faute plus grave: c'est une réelle erreur de programmation. En ce qui concerne les entrées, ce n'est pas une erreur de programmation. Le programme fonctionnera bien, pourvu que l'utilisateur ne rentre pas n'importe quoi.