Discussion :

[Mickael Baron]

Bonjour,

Developpez.com s'associe avec la chaîne YouTube « Crafties » et de son auteure Nadia Humbert-Labeaumaz pour vous proposer des vidéos destinées à partager des bonnes pratiques de développement logiciel. Inutile d'être un expert, ces vidéos s’adressent à toutes les personnes qui ont envie de les découvrir et les mettre en pratique.

Dans cet épisode, Nadia vous présente le refactoring qui est l'acte d'améliorer le design du code sans changer son comportement. Cette vidéo permet de découvrir quelques techniques de refactoring par la pratique en Java avec l'outil de développement IntelliJ.

N'hésitez pas à laisser vos commentaires à la suite de cette discussion.

Des tutoriels autour des méthodes de développement sont disponibles ici : http://alm.developpez.com/cours

Retrouver les vidéos de Nadia sur Developpez.com

• http://alm.developpez.com/actu/98455...ert-Labeaumaz/
• http://alm.developpez.com/actu/98793...ert-Labeaumaz/

Bon visionnage

[noOneIsInnocent]

Bonjour
Tout d'abord je trouve que la vidéo bien réalisée. Bravo pour le travail.
Ensuite il y a juste un point qui me gêne c'est l’ordonnancement. En effet je pense qu'il aurait été plus logique d'expliquer quelle est la solution retenue et ensuite l'implémentation. Pour les gens expérimentés on voit rapidement le problème et comment le résoudre. En revanche je pense que cela doit être un peu plus difficile pour les débutants.

Dans ce cas précis c'est à la fin de la vidéo qu'on aborde la notion de polymorphisme et de classe abstraite. A mon sens il aurait fallut d'abord expliquer pourquoi le code initial n'est pas bon (c'est ce qui est fait), ensuite il faudrait parler du polymorphisme et pourquoi la solution apportée est plus fiable. Et enfin terminer par l'implémentation ...

Bonne continuation

[Sebajuste]

Je suis du même avis que noOneIsInnocent pour ce qui est de l'amélioration pédagogique.

Et si je ne m'abuse, le test à 4:30 ne devrait pas fonctionner.
En effet, la précédente opération a été de retirer le premier paramètre EmployeeType de la classe abstraite Employee.
Or, toutes les nouvelles classes héritant de Employee (Worker, Salesman et Manager) appellent le constructeur de la classe mère avec EmployeeType en paramètre, ce qui ne devrait plus compiler.