Programmation informatique python

[invitea7a1543f]

Bonjour,
Voici mon énoncé :
utiliser la fonction fact pour écrire une fonction qui renvoie C(n,p):n!/p!(n-p)!

j'ai écris le programme suivant :

def fact(n):
    factorielle=1
    for i in range (2,n+1):
        factorielle=factorielle*i
    return(factorielle)

def combinaison(n,p):
    x=(factorielle(n))/(factorielle(p)*factorielle(n-p))
    x=int(x)
    return(x)
n=int(input("quelle est la valeur de n?"))
n=int(input("que vaut p?"))
print(combinaison(n,p))

le programme tourne, or il me met que p n'est pas définit est-ce que quelqu'un peut m'expliquer pourquoi?
merci d'avance
-----

[albanxiii]

Bonjour,

Je recopie votre code :

n=int(input("quelle est la valeur de n?"))
n=int(input("que vaut p?"))

Vous voyez ?

[invitea7a1543f]

oui j'ai remplacé par :
n=int(input("quelle est la valeur de n?"))
p=int(input("que vaut p?"))

mais maintenant il me met que "factorielle" n'est pas définit

[CM63]

Ben oui, tu as appelé ta fonction fact, et non pas factorielle.

def fact(n):

[A voir en vidéo sur Futura]

[invite44510b00]

Ben oui, tu as appelé ta fonction fact, et non pas factorielle.

def fact(n):

De toute façon, il y a une erreur manifeste dans le code fonction. (et je ne connais rien à Python).

[Loupsio]

De toute façon, il y a une erreur manifeste dans le code fonction. (et je ne connais rien à Python).

Ah? je ne vois pourtant rien d'étrange, a part ce qu'a mentionné CM63, (a moins que ce soit sa formule qui soit fausse)
car une fois corrigé son code donne bien le bon résultat, pour n=8 et p=3 son programme me renvoi 56, le meme résultat que quand j'applique sa formule "x=(factorielle(n))/(factorielle(p)*factorielle(n-p))" à la main

PS, en revanche je te conseille d'utiliser les balise de code quand tu écris du code sur les forums, cela permet de garder l'indentation, et ton code n'en sera que plus lisible, notamment, on devine que ta fonction se termine au "return" mais sans ca on ne sait pas vraiment qu'est ce qui est dans ta fonction et qu'est ce qui est à l'exterieur

[pm42]

Ah? je ne vois pourtant rien d'étrange, a part ce qu'a mentionné CM63, (a moins que ce soit sa formule qui soit fausse)
car une fois corrigé son code donne bien le bon résultat, pour n=8 et p=3 son programme me renvoi 56, le meme résultat que quand j'applique sa formule "x=(factorielle(n))/(factorielle(p)*factorielle(n-p))" à la main

Pareil. Je ne vois pas d'erreur et en testant le code, il semble donner les bons résultats.

[invite44510b00]

Laissez tomber ...

de la m..de dans les yeux ....
avec une excuse : en général cet exercice vise à démontrer l'usage de la récursivité et on s'attend à une fonction factorielle écrite sous forme d'appel récursif .

Du genre :

fact(n)
if n = 0 return 1;
return n * fact(n - 1);