FatimaZahraAkherraz
Messages postés15Date d'inscriptionjeudi 20 octobre 2016StatutMembreDernière intervention29 juillet 2018
-
3 déc. 2017 à 14:31
Whismeril
Messages postés19024Date d'inscriptionmardi 11 mars 2003StatutContributeurDernière intervention18 avril 2024
-
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.
Whismeril
Messages postés19024Date d'inscriptionmardi 11 mars 2003StatutContributeurDernière intervention18 avril 2024656 Modifié le 25 avril 2019 à 22:02
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)
Modifié le 25 avril 2019 à 22:02
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
-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)