PERMUTATION ET PUISSANCE

garslouche
Messages postés
584
Date d'inscription
mardi 26 novembre 2002
Statut
Membre
Dernière intervention
29 mai 2015
- 24 nov. 2003 à 08:13
garslouche
Messages postés
584
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
584
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
584
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.