Nombre pair_impair

Signaler
Messages postés
15
Date d'inscription
jeudi 20 octobre 2016
Statut
Membre
Dernière intervention
29 juillet 2018
-
Messages postés
14491
Date d'inscription
mardi 11 mars 2003
Statut
Contributeur
Dernière intervention
10 avril 2020
-
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

Messages postés
14491
Date d'inscription
mardi 11 mars 2003
Statut
Contributeur
Dernière intervention
10 avril 2020
377
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)