Discussion :

[Dr walid]

bonsoir,

j'ai trouvé

Ecrire un programme qui demande à l'utilisateur de taper des entiers strictement positifs et qui affiche leur moyenne. Lorsqu'on tape une valeur négative, le programme affiche ERREUR et demande de retaper une valeur. Lorsqu'on tape 0, cela signifie que le dernier entier a été tapé. On affiche alors la moyenne. Si le nombre d'entiers tapés est égal à 0, on affiche PAS DE MOYENNE.

j'ai repondu ansi :

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
 
 
# include <stdio.h>
 
int main()
{
	 int N,i=0,S=0;
	 float moy;
     do
	 {
     printf(" Taper un entier : ");
	 scanf("%d",&N);
	 while(N<0)
	 {
		 printf("erreur Tapper un entier : ");
			 scanf("%d",&N);
	 }
	 S=S+N;
	 if(N > 0)
	 i++;
	 }
	 while(N!=0);
	 if(S==0)
		 printf(" PAS DE MOYENNE.\n");
	 else
	 {
      moy=(float)S/i;
	printf(" La moyenne est %f \n",moy);
	 }
	return 0;
}

pouvez-vous me dire si il y a une faute et merci !

[adrien1]

Salut,

a premiere vue ca m'a l'air correct cependant attention à :
- au depassement de tampon,
- si tu tapes autre chose qu'un chiffre ( stupid end-user)
- et si l'utilisateur voulait rentrer un 0 ???

[ram-0000]

D'abord des problèmes d'indentation qui rendent ton code difficile à lire. Revoici le code avec ma présentation (qui est aussi discutable mais mieux indentée)

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
43
# include <stdio.h>
 
int main()
{
   int N;
   int i=0;
   int S=0;
   float moy;
 
   do
   {
      /* saisie d'un nombre entier tant qu'il n'est pas >= 0 */
      printf("Taper un entier : ");
      scanf("%d", &N);
      while(N < 0)
      {
         printf("erreur Tapper un entier : ");
         scanf("%d",&N);
      }
 
      if(N > 0)
      {
         /* modifie la somme des nombres saisis */
         S = S + N;
 
         /* ainsi que le nombre de nombres de la série */
         i++;
      }
   }
   while(N != 0);
 
   if(S == 0)
   {
      printf("La moyenne est NULLE.\n");
   }
   else
   {
      moy = (float)S / i;
      printf("La moyenne est %f\n", moy);
   }
 
   return 0;
}

Tu pourrais faire un effort sur les noms des variables afin de rendre ton code encore plus clair

• i peut devenir nb_nombre;
• S peut devenir Somme;
• N peut devenir NombreSaisi
• moy peut devenir Moyenne

Je n'ai pas testé mais que fait le programme si au lieu de saisir un nombre, tu saisis une chaine de caractères (abc par exemple).

Au niveau sémantique, dire que que si la somme est nulle, il n'y a pas de moyenne, c'est un peu faux (même si je comprend ce que tu veux dire). Tu peux tout simplement dire que la moyenne est NULLE.

Sinon, le programme semble faire ce que tu dis (je ne l'ai pas testé, je l'ai juste lu).