Nombre pair_impair

Messages postés
15
Date d'inscription
jeudi 20 octobre 2016
Statut
Membre
Dernière intervention
29 juillet 2018
- - Dernière réponse : Whismeril
Messages postés
14030
Date d'inscription
mardi 11 mars 2003
Statut
Contributeur
Dernière intervention
8 décembre 2019
- 25 avril 2019 à 22:01
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/102376-nombre-pair-impair

Whismeril
Messages postés
14030
Date d'inscription
mardi 11 mars 2003
Statut
Contributeur
Dernière intervention
8 décembre 2019
322 -
Bonsoir

ton dépôt enfreint 2 règles de la charte:
-ton adresse mail est présente dans la description, même si t'as écrit arobase au lieu @
-les fichiers exécutables sont présents dans le source
Merci donc de corriger ces 2 points

Coté réutlisabilité, regrouper les classes "métier", Reel, Complex, Matrice dans un projet dll et appeler ce projet ou la dll dans les autres serait plus judicieux.

Point de vue code, il est facile à lire, même si je n'ai fait que le parcourir rapidement.
Cependant
-test_reel plante car tu n'as pas fourni le fichier
Path.Combine(bureau, "Capteur", "bug_1.txt")

-reg_poly, j'ai tenté de lui mettre Cal.txt en entré, j'ai laissé tourné 3 minutes après avoir cliqué sur Angles C1, puis j'ai stoppé.
Tu dis que c'est un lent à l'exécution, en effet, une petite barre de progression serait la bienvenue. Tu pourrais peut-être optimiser certaines boucle (notamment les for) en faisant du linq qui est sensé être plus rapide (ça dépend des cas, il faut donc avoir benchmarqué les 2 codes)
-même constat avec modelisation (j'ai stoppé à 2 min 30)