triplets pythagoriciens help

Version imprimable

Bonjour,
question débutant
je ne comprends pas pourquoi le code ne marche pas
si je remplace le return par un print ça marche

Code:

1
2
3
4
5
6
7
8
 
def pythagoriciens(n):
    for x in range(1,n+1):
        for y in range(x+1,n):
            for z in range(y+1,n):
                if x*x+y*y == z*z:
                    return (x,y,z)
print(pythagoriciens(50))

MERCI de votre aide

12/02/2018, 21h49
wiztricks

Salut,

"return" termine la fonction en retournant le premier x, y, z trouvé.
"print" se contente de les afficher au fur et à mesure.
Dans les 2 cas, le code fonctionne mais il ne fait peut être pas ce que vous vouliez.
Si vous vouliez que la fonction retourne une liste, ben, il faut la créer, la remplir et la retourner...

- W
12/02/2018, 22h16
tyrtamos
Bonjour,

Bizarre: chez moi, ça marche. Et le résultat est très connu (3,4,5) puisque c'est comme ça que les anciens traçaient les angles droits...

Et en remplaçant return par print, on en trouve beaucoup d'autres:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 3 4 5 5 12 13 6 8 10 7 24 25 8 15 17 9 12 15 9 40 41 10 24 26 11 60 61 12 16 20 12 35 37 13 84 85 14 48 50 15 20 25 15 36 39 ...

Citation:

Envoyé par wiztricks

Dans les 2 cas, le code fonctionne mais il ne fait peut être pas ce que vous vouliez.
Si vous vouliez que la fonction retourne une liste, ben, il faut la créer, la remplir et la retourner...
- W

En effet, merci pour l'aide ;-)
le code suivant fonctionne ... sans problème ;-))
Code:

1 2 3 4 5 6 7 8 9 10 def pythagoriciens(n): L=[] for x in range(1,n+1): for y in range(x+1,n): for z in range(y+1,n): if x*x+y*y == z*z : L.append((x,y,z)) return L print(pythagoriciens(150))

Citation:
Envoyé par roger62
le code suivant fonctionne ... sans problème ;-))
Code:

1 2 3 4 5 6 7 8 9 def pythagoriciens(n): L=[] for x in range(1,n+1): for y in range(x+1,n): for z in range(y+1,n): if x*x+y*y == z*z : L.append((x,y,z)) return L print(pythagoriciens(150))
Bonjour

Ton code ne fonctionne pas tout à fait sans problème. Il ne trouve pas (42, 144, 150) ni (90, 120, 150). Probablement parce que tes deux range internes ne vont pas jusqu'à "n".

Sinon, puisque tu sembles motivé, voici deux petits plus
1) le yield.
Ca fonctionne comme un return mais l'environnement de la fonction est en quelque sorte "mémorisé". Et si tu rappelles la fonction, le code repart de là.
Exemple
Code:

1 2 3 4 5 6 7 8 def fct(): yield 1 yield 2 yield 3 yield 4 yield 5 for x in fct(): print(x)
Ca équivaut à ceci:
Code:

1 2 3 4 def fct(): return [1, 2, 3, 4, 5] for x in fct(): print(x)
Toutefois ça évite de créer le tableau en mémoire souvent inutilement (surtout quand le résultat est traité ensuite de façon itérative élément par élément)

Donc pour toi, ton code pourrait devenir cela:
Code:

1 2 3 4 5 6 def pythagoriciens(n): for x in range(1,n+1): for y in range(x+1,n+1): for z in range(y+1,n+1): if x*x+y*y == z*z : yield(x, y, z) print(tuple(pythagoriciens(150)))
2) les listes en intension: c'est une écriture qui permet de créer à la volée une nouvelle liste à partir d'un traitement effectué sur les éléments d'une liste déjà existante. Exemple: le carré des impairs des nombres compris entre 1 et 10: carre_impair=[x**2 for x in range(1, 11) if x%2 != 0].
De plus, la liste en intension peut très bien manger "n" listes à la fois. Les listes internes sont alors automatiquement démultipliées, chaque élément de chaque liste étant alors croisé avec chacun de ceux des autres listes.

A partir de là, ton code pourrait devenir ceci:
Code:

1 2 3 def pythagoriciens(n): return tuple((x, y, z) for x in range(1, n+1) for y in range(x+1, n+1) for z in range(y+1, n+1) if x*x+y*y == z*z) print(pythagoriciens(150))
Et une fonction n'ayant q'une ligne peut alors devenir une lambda:
Code:

1 2 pythagoriciens = lambda n: tuple((x, y, z) for x in range(1, n+1) for y in range(x+1, n+1) for z in range(y+1, n+1) if x*x+y*y == z*z) print(pythagoriciens(150))
Cordialement

PS: tu remarqueras que j'ai transformé ta liste initiale en tuple. Ben oui, un tuple n'étant qu'une liste non modifiable, autant utiliser le container le plus adapté à son utilisation (stop gaspi)...;)