Discussion :

[suppadrakn]

Bon alors comme il me l'a été demandé ou suggéré ( je pense que le choix me revenais au final ) je crée un nouveau sujet pour soumettre mon travail face à la horde de professionnels que vous êtes.

Comme cité dans mon précédent topic ( qui devrait continuer à être alimenté ?), je me suis remis au travail et me voila à l'exercice 34 ( comment ça je chôme ? Mais pas du tout ! ) dont voici le sujet :

Écrire un programme qui, étant données deux bornes entières a et b, additionne les nombres multiples de 3 et de 5 compris entre ces bornes.

avec tout de même un tout petit peu d'aisance ( il ne faut sans doute pas trop exagérer ) je suis parvenu à ceci :

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
# -*- coding: utf-8 -*-
#Ce programme permet d'identifier tous les multiples de 3 et de 5 dans une
# borne donnée par l'utilisateur.
# il additionne ensuite ses valeurs et affiche le resultat a l'ecran
 
minimum = int(raw_input("veuillez entrer une valeur minimal:"))
maximum = int(raw_input("veuillez entrer une valeur maximal:"))
a = minimum
b = maximum
compteur = 0
liste = []
index = 0
m1 = 3
m2 = 5
resultat = 0
while compteur < b:
    a = a + 1
    compteur = compteur + 1
    if a % m1 == 0 and a % m2 == 0:
        liste.append(a)
        index = index + 1
        resultat = resultat + a
        print("les valeurs situees entre", minimum, "et", maximum, "qui")
        print("sont a la fois multiples de 3 et de 5 sont les suivants:")
        print(liste, "la somme de ces elements est egale a :")
        print(resultat)

Encore une fois, notons a quel point on se sent fière est satisfait lorsqu'on parvient à écrire un programme d'approximativement 30 lignes et qui fonctionne en plus !

Un gros merci à vous de m'épaulez au travers de mon périple..Au passage, et vous l'aurez sans doute compris, je souhaiterais que vous commentiez mon code, faite ressortir les points qui pourraient être améliorés pour plus de clarté peut être ( et c'est la que je repense à cette phrase précieuse au début du bouquin, concernant l'importance des commentaires ) ou tout simplement autour de la démarche logique du problème.

[PauseKawa]

Bonjour,

Pas mal de déclaration de variables dans tout cela.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
current = minimum = int(raw_input("veuillez entrer une valeur minimal: "))
maximum = int(raw_input("veuillez entrer une valeur maximal: "))
liste = []
while current <= maximum:
    if current % 3 == 0 and current % 5 == 0:
        liste.append(current)
    current += 1
print "les valeurs situees entre", minimum, "et", maximum, "qui"
print "sont a la fois multiples de 3 et de 5 sont les suivants:"
print liste
print "la somme de ces elements est egale a :", sum(liste)

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
current = minimum = int(raw_input("veuillez entrer une valeur minimal: "))
maximum = int(raw_input("veuillez entrer une valeur maximal: "))
somme = 0
liste = []
while current <= maximum:
    if current % 3 == 0 and current % 5 == 0:
        somme += current  # somme = somme + current
        liste.append(current)
    current += 1
print "les valeurs situees entre", minimum, "et", maximum, "qui"
print "sont a la fois multiples de 3 et de 5 sont les suivants:"
print liste
print "la somme de ces elements est egale a :", somme

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
current = minimum = int(raw_input("veuillez entrer une valeur minimal: "))
maximum = int(raw_input("veuillez entrer une valeur maximal: "))
somme = 0
liste = []
while current <= maximum:
    if not current % 3  and not current % 5:
        somme += current  # somme = somme + current
        liste.append(current)
    current += 1
print "les valeurs situees entre", minimum, "et", maximum, "qui"
print "sont a la fois multiples de 3 et de 5 sont les suivants:"
print liste
print "la somme de ces elements est egale a :", somme

@+

[PauseKawa]

Question: Puisque vous utilisez Python 2.x pourquoi n'utilisez vous pas input ?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
current = minimum = input("veuillez entrer une valeur minimal: ")
maximum = input("veuillez entrer une valeur maximal: ")
somme = 0
liste = []
while current <= maximum:
    if not current % 3  and not current % 5:
        somme += current  # somme = somme + current
        liste.append(current)
    current += 1
print "les valeurs situees entre", minimum, "et", maximum, "qui"
print "sont a la fois multiples de 3 et de 5 sont les suivants:"
print liste
print "la somme de ces elements est egale a :", somme

[PauseKawa]

Comme je doute que not, a = b = c et += ne soit abordés dans votre tuto à ce niveau une version plus légère

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
minimum = input("veuillez entrer une valeur minimal: ")
maximum = input("veuillez entrer une valeur maximal: ")
somme = 0
liste = []
current = minimun
while current <= maximum:
    if current % 3 == 0 and current % 5 == 0:
        somme = somme + current
        liste.append(current)
    current = current + 1
print "les valeurs situees entre", minimum, "et", maximum, "qui"
print "sont a la fois multiples de 3 et de 5 sont les suivants:"
print liste
print "la somme de ces elements est egale a :", somme

Comme vous le voyez pas besoin de multiplier les variables .

Ceci dit votre code est 'juste'

[fred1599]

Salut PauseKawa,

Je suis pas totalement d'accord avec ta phrase

Question: Puisque vous utilisez Python 2.x pourquoi n'utilisez vous pas input ?

Dans la doc il est bien spécifié

Consider using the raw_input() function for general input from users

Mais aussi

This function does not catch user errors

Peut-être voulais-tu dire cela pour la version 3 python, et non la 2?

[PauseKawa]

Bonjour fred,

Je suis bien d'accord avec This function does not catch user errors mais si l'on considère int(raw_input("veuillez entrer une valeur minimal: ")) c'est un moindre mal.
Non, je ne parle pas de Python 3 puisque raw_input() n'existe plus.
En fait puisque suppadrakn écrit int(raw_input()) c'est qu'il considère que l'utilisateur vas entrer un nombre. Dans le cas contraire nous savons tous ce qui vas se passer, y compris suppadrakn.
Autant utiliser input plutôt qu'un int(raw_input()) assez 'moche'.
Mais bon, j'aurais attendu une réflexion du PO

En fait, vu que % semble acquis, la pédagogie de l'exercice (désolé si je me trompe mais aucune envie de regarder à quoi cela correspond) me semble porter sur and/or > les conditions multiples. Dans ce sens suppadrakn s'en ai bien tiré.

@+