Discussion :

[roger62]

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 :

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

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

[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

[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 :

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
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
...

[roger62]

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 :

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

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))

[Sve@r]

le code suivant fonctionne ... sans problème ;-))

Code :

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

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 python :

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

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 python :

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

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 python :

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

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 python :

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

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 python :

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

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)...