Petite précision sur un programme

Bonjour,

Ce programme calcule les somme produit et moyenne d'une série de chiffres tapée au clavier,série qui prend fin quand on tape 0 .

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

#include<stdio.h>   main() { int X;/*le chiffre courant*/ int N=0;/*le compteur des données*/ int SOM=0;/*la somme actuelle*/ long PROD=1;/*le produit actuel-type long à cause de la grandeur du résultat*/   do {/*saisie des données (pour perfectionnnistes)*/ printf("Entrez le %d%s chiffre : ",(N+1),(N)?"e":"er"); scanf("%d",&X);   if(X<0||X>9)printf("\a"); else if(X) { N++; SOM+=X; PROD*=X; } else if(!X&&N>0) { /*seulement si au moins un chiffre a été accepté*/ printf("La somme des chiffres est %d\n",SOM); printf("Le produit des chiffres est %ld\n",PROD); printf("La moyenne des chiffres est %f\n",(float)SOM/N); } }while(X);   return 0; }

---

Ma question se porte sur la ligne 22 et l'instruction

Citation:

---

else if(!X&&N>0)

---

Je voudrais juste savoir si c'est important de mettre '&&N>0' ou si c'est inutile ?

Citation:

---

Envoyé par chicabonux

Je voudrais juste savoir si c'est important de mettre '&&N>0' ou si c'est inutile ?

---

c'est surtout que c'est l'autre élément qui est inutile :aie:

• Premier if : tu testes si X est strictement négatif ou strictement supérieur à 9
• premier else : tu testes si X n'est pas nul (donc compris entre 1 et 9)
• deuxième else : le cas qui reste est X = 0

le N>0 est nécessaire, car cela participe au nombre, alors qu'un zéro en tête ne change rien..

Note :
Mais je viens de voir que ton while est while(X)

Donc ton dernier else ne sert à rien..

Il faut juste mettre ton print à la sortie du do..while..

Citation:

---

Ce programme calcule les somme produit et moyenne d'une série de chiffres tapée au clavier,série qui prend fin quand on tape 0

---

Ben justement cette précision est importante pour connaître la fin du programme ? Où alors je comprends encore mal le déroulement du programme ?

regarde ma note..

ok merci

Je trouve pas clair cette expression :

Code:

---

else if(!X&&N>0)

---

On est toujours obligé de réfléchir à la priorité des opérateurs, alors que de belles parenthèses, c'est pratique.

D'ailleurs, là je pensais que c'était équivalent à

Code:

---

else if((!X&&N)>0)

---

mais il semblerait que je me sois trompé :aie:

Moi, j'aime les parenthèses :lol:

(et il faut que révise les priorités des opérateurs :aie:)

Citation:

---

Envoyé par gwenBZH

...
D'ailleurs, là je pensais que c'était équivalent à

Code:

---

else if((!X&&N)>0)

---

mais il semblerait que je me sois trompé

---

Oui, tu t'es trompé.
Personnellement, j'écrirai explicitement

Code:

---

1 2 3 4 5 6 7 8 9 10

do { .... if(X<0||X>9)printf("\a"); else if(X!=0) .... else if(X==0 && N>0) // et comme le dit souviron34 le X==0 est inutile ... }while (x !=0)

---

Voilà maintenant un programme qui calcule à l'écran un nombre lu à rebours qui contient une suite de chiffres non nuls ,terminée par zéro et en controlant qu'il s'agisse vraiment de chiffres .

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

#include<stdio.h>   main() { int X,N=0; long VALD=1,NOMB=0;   do { printf("Entrez le %d%s chiffre : ",(N+1),(N)?"e":"er"); scanf("%d",&X);   if(X<0||X>9)printf("\a"); else if(X) { NOMB+=VALD*X; N++; VALD*=10; } else printf("La valeur du nombre renverse est %ld\n",NOMB);   }while(X);   return 0; }

---

Ce code ne vérifie pas si ce qui est tapé au clavier est un chiffre ? Comment réaliser ce contrôle ?
(Le code est un corrigé d'un exercice à effectuer )

Citation:

---

Ce code ne vérifie pas si ce qui est tapé au clavier est un chiffre ? Comment réaliser ce contrôle ?

---

La première chose est de vérifier la valeur de retour de scanf(). Mais ceci ne garantit pas que la chaine entrée soit entièrement numérique. Pour affiner le test, il faut tester caractère par caractère pour vérifier que chacun est un chiffre.

Citation:

---

Envoyé par diogene

La première chose est de vérifier la valeur de retour de scanf(). Mais ceci ne garantit pas que la chaine entrée soit entièrement numérique. Pour affiner le test, il faut tester caractère par caractère pour vérifier que chacun est un chiffre.

---

disons qu'avec son format c'est difficile ;)

Code:

---

scanf("%d",&X);

---

@chicabonux :

si tu veux faire ça, il faut accepter une chaîne (ce que dit diogene) en faisant par exemple un fgets, puis traverser toute la chaîne pour vérifier que tu n'as que des chiffres consécutifs (pour les entiers), ou à la limite des + ou - en tête, et éventuellement un . en tête, à la fin, ou au milieu (pour des réels). On peut encore raffiner pour les réels en acceptant éventuellement la notation scientifique (e+3)..

Je pourrais utiliser la fonction isdigit ? Mais je ne sais pas de quelle façon le faire ?

Citation:

---

Envoyé par diogene

La première chose est de vérifier la valeur de retour de scanf(). Mais ceci ne garantit pas que la chaine entrée soit entièrement numérique. Pour affiner le test, il faut tester caractère par caractère pour vérifier que chacun est un chiffre.

---

On peut également saisir la ligne complète sous forme de chaine de caractères à l'aide de fgets(), réaliser la conversion en entier à l'aide de strtol() et ensuite se servir du deuxième argument de cette dernière fonction pour vérifier si l'adresse retournée pointe sur le caractère nul.

Thierry