Discussion :

[ptitjoz]

Bonjour
Je cherche une fonction en Python pour calculer la factorielle d'un nombre mais je n'ai pas trouvé.. ça existe ?
Sinon j'ai fait ce petit script

Code :

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

1
2
3
4
5
def fact(x):
  f=1
  for i in range(x):
    f=f+f*i
  return f

>>> from fact import *
>>> fact(8)
40320

est-ce correct comme code ?
merci

[nekcorp]

Salut,

Alors si ton code te fournit le résultat attendu alors c'est qu'il est correct.

Après sans vouloir faire de la pub à une personne de ce forum (Tyrtamos), je te pousserai à faire un tour par ici : factorielle si tu veux voir plusieurs solutions.

Sinon tu as dans le module math une fonction déjà toute faite :-)

En espérant avoir pu t'aider.

[VinsS]

Salut,

Regarde ici

[ptitjoz]

Salut,

Alors si ton code te fournit le résultat attendu alors c'est qu'il est correct.

Après sans vouloir faire de la pub à une personne de ce forum (Tyrtamos), je te pousserai à faire un tour par ici : factorielle si tu veux voir plusieurs solutions.

Sinon tu as dans le module math une fonction déjà toute faite :-)

En espérant avoir pu t'aider.

Salut,

Regarde ici

Merci à vous deux !
Cordialement

[Sve@r]

est-ce correct comme code ?

Bonjour

C'est une situation peu banale: ton code merdoie... mais il fonctionne.

Pourquoi merdoie-t-il ? Parce qu'il ne représente pas la définition de la factorielle. La factorielle de (par exemple) 5, c'est 1x2x3x4x5. On s'attendrait donc à trouver un algo représentatif de cette formule. Par exemple for i in range(1, n+1): f=f*i. Or là, il y a ce f=f+f*i qui décontenance tout lecteur habitué à la factorielle (depuis quand une factorielle inclut-elle une addition ???).

Pourquoi fonctionne-t-il ? Parce que cette dernière addition permet de rétablir la vraie multiplication qui ne se fait pas entièrement du fait que le range s'arrête trop tôt.
Si par exemple je veux multiplier un truc par 500, je peux écrire "truc=truc * 500" ou bien écrire "truc=truc * 499 + truc". La seconde écriture, bien qu'assez inusitée et donc assez surprenante (et aussi bien plus lourde) fonctionne. Toutefois, la première reste quand-même bien plus simple et à écrire, et à relire.

Toi tu as utilisé la seconde.

Code python :

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

1
2
3
4
5
def fact(x):
  f=1
  for i in range(2, x+1):
    f=f*i
  return f

[ptitjoz]

Bonsoir @Sve@r

Merci pour ta réponse explicative. En effet mon code ne me paraissait pas très lisible et quelque peu compliqué à comprendre du premier abord. C'est un peu pour cela que je demandais un avis.

Mon idée de départ était de rajouter à la variable le nouveau produit. Même si ça fonctionne, je me range à ton avis que ce n'est pas vraiment bien pensé...

Ta solution est bien, facile à comprendre et répond bien à la définition de la factorielle.

Arriver à un résultat, même correct, demande tout de même une logique qui était pas la meilleure.

Merci d'avoir pris le temps de répondre.

Bien cordialement