Discussion :

[iCitron]

Bonjour à tous,
Pour un TP scolaire, je dois écrire un algorithme capable de donner tout les entiers impairs, multiples de 5 et/ou contenant un 2; compris dans un interval [n,m] (Avec n,m en donné par l'utilisateur)
J'e débouche sur cet algorithme, qui dans ma tête est censé fonctionner, et j'en arrive à une erreur incompréhensible, comme si dans mon "while m<n:", toutes les conditions et autres "while" sont ignorés, sauf une (le "if a==5:")
Voici un screenshot, dl plus bas.


Si vous avez besoin d'autres informations que j'aurai pu oublier, dites le.
Merci d'avoir pris le temps de lire mon message.

[fred1599]

Bonjour, le sujet n' est pas très explicite. Imaginons que je prenne tous les nombres entre 0 et 100, le seul résultat est 25, car impair, multiple de 5 et le chiffre 2 présent, non?

[VinsS]

Salut,

La question est bien si le nombre est impair and (multiple de 5 or contient un "2") ?

Si j'ai bien compris voici ma proposition:

Code :

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

1
2
3
4
5
6
7
8
9
 
Python 2.7.3 (default, Apr 10 2013, 05:46:21) 
[GCC 4.6.3] on linux2
Type "help", "copyright", "credits" or "license" for more information.
>>> a = 10
>>> b = 50
>>> l = [x for x in range(a, b) if x % 2 == 1 and (x % 5 == 0 or '2' in str(x))]
>>> l
[15, 21, 23, 25, 27, 29, 35, 45]

Si je n'ai rien compris, dites le hein !

[iCitron]

Je me suis mal exprimé désole, je dois obtenir tout les nombres:
-impairs
-multiples de 5
-contenant un 2

[fred1599]

Ok c'est plus clair, donc juste

Code :

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

1
2
3
4
5
6
7
8
9
10
11
>>> for i in range(10):
...     if i%2 or not i%5 or "2" in str(i):
...         print(i)
... 
0
1
2
3
5
7
9

devrait suffire non?

[iCitron]

Merci beaucoup
C'est la partie de code qui m’intéressait, je débute donc je m'y perd !
Encore merci a toute la communauté !

[VinsS]

OK on change alors

Code :

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

1
2
 
l = [x for x in range(a, b) if x % 2 == 1 or x % 5 == 0 or '2' in str(x)]

Mais entretemps j'ai vérifié ton code.

Tout d'abord:

Code :

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

1
2
3
4
5
6
 
    i=1
    while n>i|n==0:
        a=(n-(i*(n//i)))//(i/10)
        i=i*10
        if a==2:

Si n == 0 tu es dans une boucle infernale.
Si a == 2 ne dit pas si n contient un "2"

Ensuite

Code :

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

1
2
 
    a=n-(10*(n//10))

Je pense qu'ici c'est le modulo de x par y que tu veux, ce qui s'écrit comme ceci:

Code :

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

1
2
 
    m = x % y

Ensuite:

Code :

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

1
2
 
    if a==1|a==3|a==7|a==9:

Ce n'est rien d'autre que not x modulo 2

Code :

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

1
2
3
 
    if not n % 2:
        print("%s est impair" % n)

Côté esthétique, il est apprécié de mettre des espaces de part et d'autre des opérateurs, de préférer or à | et and à &.

Et surtout de copier simplement son code entre des balises du même nom plutôt que de coller une image, ton code est incopiable.

[iCitron]

Ah oui, j'avais pas pensé au cas du n=0
C'est beaucoup plus clair de cette manière, mais je ne connaissais pas le str(x)
Je crois avoir compris, et c'est vraiment très simple en fait je me suis compliqué la vie pour rien !
Et j'ai pas pensé au n%2!=0, c'est quand même plus simple que d’écrire toutes les possibilités !
Merci mais j'ai vraiment toujours pas compris ce qu'il se passait dans mon ancien code, pourquoi il se comportait si bizarrement ..

[fred1599]

Merci mais j'ai vraiment toujours pas compris ce qu'il se passait dans mon ancien code, pourquoi il se comportait si bizarrement ..

Tu ne devrais pas réfléchir sur un code qui à la base a été mal pensé.

La bonne méthode c'est de réfléchir sur papier et de représenter avec une syntaxe le plus approprié possible (possibilité du langage) ta réflexion.

Si tu avais fais ceci, tu n'aurais sans doute pas pondu ce code.

Bonne continuation...