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: 5372
Date d'inscription: dimanche 4 mai 2003
Statut: Modérateur
Dernière intervention: 24 mai 2022

- 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

Twinuts

Messages postés: 5372
Date d'inscription: dimanche 4 mai 2003
Statut: Modérateur
Dernière intervention: 24 mai 2022

111
17 mai 2011 à 09:44

Salut,

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

William44290

Messages postés: 15
Date d'inscription: vendredi 20 février 2009
Statut: Membre
Dernière intervention: 30 avril 2011

17 mai 2011 à 08:21

FUTTY, je comprends rien non plus de vos interventions.

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é.

cs_futty

Messages postés: 11
Date d'inscription: lundi 14 février 2011
Statut: Membre
Dernière intervention: 18 mai 2011

16 mai 2011 à 23:42

Oh ! Toutes mes excuses .. Je me suis effectivement trompé. Je voulais donc parler de William44290 et non pas de 'vous' Twinuts. Mea culpa !

Twinuts

Messages postés: 5372
Date d'inscription: dimanche 4 mai 2003
Statut: Modérateur
Dernière intervention: 24 mai 2022

111
16 mai 2011 à 23:37

Haaaa mais nan il y a méprise ici, je ne comprend pas 'vos' propos, 'vous' parlez de moi sans *aucune* raison, mon intervention sur cette source n'est qu'une intervention de 'police', du coups parler de moi et de mon non MVC (qui n'est pas totalement faux en soit sachant que les sources que j'ai déposé sur le site n'ont pas pour vocation de parler de MVC)...
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.

cs_futty

Messages postés: 11
Date d'inscription: lundi 14 février 2011
Statut: Membre
Dernière intervention: 18 mai 2011

16 mai 2011 à 23:28

Premièrement, nous n'avons pas gardé les cochons ensemble ! Ensuite: je vous défendais !

cs_Julien39

Messages postés: 6414
Date d'inscription: mardi 8 mars 2005
Statut: Modérateur
Dernière intervention: 29 juillet 2020

366
1 mai 2011 à 19:08

Bonjour William44290,

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.

Afficher les 11 commentaires