UNE CALCULATRICE BIEN FAITE
cs_Julien39
Messages postés
6414
Date d'inscription
mardi 8 mars 2005
Statut
Modérateur
Dernière intervention
29 juillet 2020
-
1 mai 2011 à 19:08
Twinuts Messages postés 5375 Date d'inscription dimanche 4 mai 2003 Statut Modérateur Dernière intervention 14 juin 2023 - 17 mai 2011 à 09:44
Twinuts Messages postés 5375 Date d'inscription dimanche 4 mai 2003 Statut Modérateur Dernière intervention 14 juin 2023 - 17 mai 2011 à 09:44
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/53110-une-calculatrice-bien-faite
https://codes-sources.commentcamarche.net/source/53110-une-calculatrice-bien-faite
17 mai 2011 à 09:44
Je confirme les dires de William44290 vue que j'ai supprimé le premier code de Julien39.
Pour l'histoire de la guéguerre, disons que 3 calculatrices d'affilées dans les sources, ça devient une agression visuel :p
17 mai 2011 à 08:21
Pour mémoire Julien39 a publié un premier code que j'ai lu et que j'ai abondamment critiqué.
Afin de ne pas limiter mon intervention à une simple critique j'ai publié ce code pour présenter un contre exemple.
De là julien39 à retenu quelques arguments que je lui soumettait et à refait une version plus simple.
Ensuite les modérateurs on pris cette discussion comme si c'était une guéguerre et ils nous ont modéré.
16 mai 2011 à 23:42
16 mai 2011 à 23:37
Bref si vous n'avez toujours pas remarqué je ne suis pas l'auteur ici donc si il y a reproche à faire sur mes sources merci de le faire sur ces dernières et non pas sur celle d'un autre membre.
à bonne entendeur.
16 mai 2011 à 23:28
16 mai 2011 à 23:23
lol, je te parle pas de ça... je ne vois pas pourquoi tu parles de moi...
16 mai 2011 à 23:21
16 mai 2011 à 23:08
"on ne peut reproché a Twinuts d'utiliser comme bon lui semble un modèle conceptuel" Gné de quoi tu parles ?
16 mai 2011 à 23:04
Par contre si je peux me permettre une remarque plus subjective : Cessons de coder en franglais ! Et s'il vous plait, lorsque vous publiez un code source, documentez !!!
2 mai 2011 à 17:17
je laisse celle-ci uniquement pour l'aspect MVC, mais svp pas de gueguerre sur qui a la plus grosse calculatrice
1 mai 2011 à 19:08
La source est pas mal, la partie ihm est largement meilleure que la mienne, je vais m'en inspirer pour la version simplifiée de ma calculette.
Au sujet de ton implémentation de MVC, il me semble que normalement, le modèle ne voit pas le contrôleur, mais la vue via le design pattern observer (il voit une interface implémentée par la vue qui contient une méthode update) et la vue, elle voit le contrôleur. Mais bon, je sais qu'il y a plusieurs façons de l'implémenter.
Le principal défaut que je trouve à ton code c'est qu'il n'a pas de conception métier, pas d'objet métier (Operateur, Operande, Operation dans ma source), le modèle ne contient que le résultat du calcul. Là la calculatrice est simple mais dès que tu voudrais ajouter une opération, tu sera oblige de modifier le code de classes existantes alors qu'avec ma source, tu ajoutais une classe et ton pas de régression de code.
Mais bon, c'est vrai, pour ce genre de code simple et pas franchement amené à évoluer, on peut se passer de modèle, mais passer à une calculatrice scientifique demandera de créer un nouveau projet...
Et il y a un problème avec l'ajout des virgules.