PERMUTATION ET PUISSANCE

garslouche Messages postés 583 Date d'inscription mardi 26 novembre 2002 Statut Membre Dernière intervention 29 mai 2015 - 24 nov. 2003 à 08:13
garslouche Messages postés 583 Date d'inscription mardi 26 novembre 2002 Statut Membre Dernière intervention 29 mai 2015 - 24 nov. 2003 à 17:58
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/18170-permutation-et-puissance

garslouche Messages postés 583 Date d'inscription mardi 26 novembre 2002 Statut Membre Dernière intervention 29 mai 2015 1
24 nov. 2003 à 17:58
Là je ne suis pas tout-à-fait d'accord.
cin et cout sont des fonctions qui sont apparues avec le C++.

Cette source est en C pur ... autant l'y laisser
cs_PiX3L Messages postés 102 Date d'inscription dimanche 27 juillet 2003 Statut Membre Dernière intervention 13 août 2005 1
24 nov. 2003 à 17:46
Sans oublier les printf et scanf qui sont obselètes. Utilises cout << et cin >> !!
garslouche Messages postés 583 Date d'inscription mardi 26 novembre 2002 Statut Membre Dernière intervention 29 mai 2015 1
24 nov. 2003 à 08:13
Non, non et non !

Dejà que cette source n'est pas d'une grande utilité, mais en plus elle est mal programmée! Là c'est pas une source POUR les débutants mais une source PAR un débutant !

1) Le nom des variables doit être explicite, 'e' ne veut pas dire grand chose
2) Si l'utilisateur entre 0, on lui demande quand même de rentrer un nombre (puisque tu utilises do...while)
3) Pire encore : si l'utilisateur entre un nombre négatif, tu rentres dans une boucle infinie!

Remplace 'e' par qc comme 'nombre', et surtout remplace do...while par for !

Et perso j'aurais même fait une fonction auxiliaire qui calcule la somme indépendament du main. Mais là c'est une question de point de vue.
Rejoignez-nous