Problème programme : Le rendu de pièces de monnaie

Bonjour,

Je rencontre quelques difficultés concernant ce programme. La consigne est :

- On veut réaliser une somme d'argent (limité à 10€) avec des pièces de 2€, 1€, 50c, 20c, 10c, 5c, 2c, 1c, en utilisant le moins de pièces possible. Par exemple, 5,88€ se paye avec deux pièces de 2€, une pièce d'1€, une pièce de 50c, une pièce de 20c, une pièce de 10c, une pièce de 5c, une pièce de 2c et une pièce de 1c. Ecrire un algorithme demandant à l'utilisateur de saisir une somme d'argent, puis détermine le détail des pièces à utiliser pour constituer la somme saisie en utilisant le moins de pièces possible.

J'ai un problème au niveau de la pièce de 1 centime. Comme vous pouvez le voir, j'ai saisi un montant de 5,88€ et logiquement, il y a une pièce de 1 centime. Or, le programme refuse de me l'afficher.

Pouvez vous m'aider ? Merci d'avance !

Pièce jointe 473076
Pièce jointe 473077

Bonjour,
En passant ...

Afficher la valeur de q8 pour voir ce qu'il y a dedans.
S'il y a 1
=>
Et si
à la place de

Citation:

---

if m//2

---

on mettait

Code:

---

if q1 !=0

---

à la place de

Citation:

---

if r1//1

---

on mettait

Code:

---

if q2 !=0

---

etc.

Pour moi 2 intérêts :
1) m//2, r1//1 etc. ce sont des calculs qu'on a déjà effectués et dont le résultat a été mis dans q1, q2,etc. donc pas besoin de refaire tous ces calculs...
2) Ensuite faire If "un calcul" ça me parait bizarre je préfère If "mon quotient est différent de 0"

Bonjour

Déjà sur la forme: rarement vu un code aussi pourri. Déjà tu nous mets une copie d'écran ce qui nous oblige à tout retaper si on veut tester. Ca t'aurait trop fatigué les doigts de copier ton code dans le post ?
Ensuite tu calcules (par exemple) q1=m//2 puis tu refais ce calcul dans if m//2. Donc autant t'as la flemme de nous poster ton code, autant pour ton programme t'as pas ce genre de scrupules car tu n'hésites pas à le faire bosser deux fois plus que nécessaire.

Peut pas utiliser un tableau de valeurs et programmer une boucle ? Ok ça impose de réfléchir un peu au début mais ensuite c'est tellement simple de relire le code et le faire évoluer si besoin (le jour où par exemple on te demande d'inclure les billets de 5, 10 et 20€ !!!).

Sinon pour tes erreurs c'est dû à l'imprécision du calcul en flottants (exécute l'instruction 1.88 % 1 en direct dans ton idle et tu verras ce que je veux dire). Donc soit tu convertis ta somme initiale en centimes et tu travailles avec des entiers, soit tu utilises le module decimal pour faire tes calculs en précision exacte.

Citation:

---

Envoyé par vttman

2) Ensuite faire If "un calcul" ça me parait bizarre je préfère If "mon quotient est différent de 0"

---

Tu peux alors utiliser le booléen implicite et réduire à if mon_quotient...

J’essaierai la fonction différent de car il est vrai que si r7 est différent de 0, il reste bien 1 centime.

Je te remercie ! :)

Parfait ...

Citation:

---

Envoyé par anastxp

J’essaierai la fonction différent de car il est vrai que si r7 est différent de 0, il reste bien 1 centime.

---

Ca ne changera pas le fait que q8 vaut 0 alors qu'il devrait valoir 1. Alors ok tu afficheras bien un message correct à l'écran mais ton code continuera à ne pas calculer la bonne valeur. Si ça te suffit... :roll:
Pièce jointe 473235

Ceci dit, si vraiment ça te suffit je te suggère alors de tester ton algo sur la somme "1€40" et regarder combien de pièces de 20 centimes il te sort (variable q4).
Pièce jointe 473239
Tu pourras toujours alors tenter "r3 différent de 0" :ptdr:

Citation:

---

Envoyé par anastxp

En effet, je suis étudiante en première année de BTS informatique. Il est clair que je suis donc en apprentissage.

---

Je n'ai pas dit qu'en BTS tu devrais être forte en prog, j'ai dit qu'en BTS tu devrais avoir plus de conscience professionnelle dans la conception d'un algorithme. On voit de suite la différence entre un code maladroit mais écrit par quelqu'un qui en veut et le code d'un "je m'en foutiste" doublé d'un "j'en fais le minimum". Tu ne feras croire à personne qu'on ne t'a pas appris les tableaux !!!

Citation:

---

Envoyé par anastxp

Je te remercie ! :)

---

De rien. Ah oui, c'est vrai, ce n'est pas moi que tu remercies. Pourtant c'est moi qui t'ai donné la bonne solution (celle qui calcule la bonne valeur pour q8 !!!)
Pièce jointe 473217

Bonjour d'un autre vieux qui a très bien vécu
Personnellement j'aurais utilisé une liste et une boucle.
Premièrement, je sépare les euros des cents et vérifie la division entière des euros par 2 et son reste, ce qui me donne les pièces de 2 et 1 euros
Deuxièmement, j'utilise une boucle pour tester toutes les pièces en cents
En petit exemple sans prétention :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

m= 8.97 print(m) #on sépare les euros des cents euros=int(m) #nombre de cents cents=int(m*100)%100 print("valeur =",euros,"€ et ",cents,"cent(s)") # les pièces en euros: un certain nombre de pièces de 2 et éventuellement 1 pièce de 1 print("Soit ",euros//2,"pièce(s) de 2€", "et",euros%2,"pièce de 1€") print("----------------------------------- ") #Liste des pièces en cents pcents=[50, 20, 10, 5, 2, 1] # on parcourt cette liste print("Pièces en cents ") for n in pcents: p=cents//n cents=cents%n if p>0 : print (p," piece(s) de", n,"c")

---

Enfin une réponse claire et polie ! Je te remercie Michel ! :)

Citation:

---

Envoyé par Michel

Premièrement, je sépare les euros des cents

---

Pourquoi séparer ? Ça complexifie inutilement le traitement (rappelle-toi le pep 20 de Python: "simply is better than complex"). Quand on rend la monnaie, le calcul des pièces de 2€ est exactement le même (ou devrait être exactement le même) que le calcul des pièces de 50cts. Ainsi un traitement unique et commun sur une liste de toutes les valeurs de monnaie disponibles permettra de le faire évoluer très facilement le jour où on veut l'adapter aux billets ou le jour où de nouvelles pièces font leur apparition...