code retreat

CODE RETREAT8 JUIN 2015

Charles Bouttaz @CharlesBouttaz Emilien Pecoul @Ouarzy

Haïkel Guémar @hguemar

MERCI SPONSORS!

PRINCIPEItération 1 heureNouvelles contraintes par itérationSortir de sa zone de confortVous n’êtes pas au boulotC’est un lieu d’apprentissage avant toutAmusez vous!

ITÉRATION 1• Pas de Pair Programming• Pas de TU

Mais…

• Un livre surprise à gagner pour le premier!

ITÉRATION 1.0Initialisation

• Projet console• Un fichier en entrée• Un fichier en sortie• Avoir la même chose dans les 2 fichiers

ITÉRATION 1.1DNA Kata, la biotech du futur!

• Supprimer toutes les lettres qui ne sont pas des paires de bases valides

• Ne pas utiliser de librairies tierces ou natives de manipulation de chaînes pour des raisons "évidentes" de sécurité et de performance (dixit le CTO)

ITÉRATION 1.1Input.txt >> ATUGACRE

Output.txt >> ATGAC

ITÉRATION 1.2• Supprimer toutes les lettres qui ne sont pas des paires de

bases valide• Inversion de la séquence ADN si la première ligne du

fichier est «reverse »

ITÉRATION 1.2Input.txt >> reverseATGC

Output.txt >> CGTA


bases valide• Inversion de la séquence ADN si la première ligne du fichier

est «reverse »• Compter le nombre d’apparition d’un fragment voulu si la

1ere ligne est « count fragmentVoulu »

ITÉRATION 1.3Input.txt >> Count GAGATTACA

Output.txt >> 1

ITÉRATION 1.4• Supprimer toutes les lettres qui ne sont pas des paires de bases

valide• Inversion de la séquence ADN si la première ligne du fichier est

«reverse »• Compter le nombre d’apparition d’un fragment voulu si la 1ere ligne

est « count fragmentVoulu »• Insérer un fragment voulu si la 1ere ligne est « insert

fragmentVoulu positionInsertion »

ITÉRATION 1.4Input.txt >> insert GA 4GATTACA

Output.txt >> GATTGAACA



est «reverse »• Compter le nombre d’apparition d’un fragment voulu si la 1ere

ligne est « count fragmentVoulu »• Ne pas utiliser de librairies tierces ou natives de manipulation

de chaînes pour des raisons "évidentes" de sécurité et de performance (dixit le CTO)

• Insérer un fragment voulu si la 1ere ligne est « insert fragmentVoulu positionInsertion »

• Compléter la séquence si la 1ere ligne est « complete »

ITÉRATION 1.5Input.txt >> completeGATTACA

Output.txt >> GATTACACTAATGT



est «reverse »• Compter le nombre d’apparition d’un fragment voulu si la 1ere

ligne est « count fragmentVoulu »• Ne pas utiliser de librairies tierces ou natives de manipulation

de chaînes pour des raisons "évidentes" de sécurité et de performance (dixit le CTO)

• Insérer un fragment voulu si la 1ere ligne est « insert fragmentVoulu positionInsertion »

• Compléter la séquence si la 1ere ligne est « complete »• Insérer un tag si la 1ere ligne est « tag tagContent

fragmentToTag »

ITÉRATION 1.6Input.txt >> tag junk ACAGATTACA

Output.txt >> GATTACA |junk

ITÉRATION 1 RÉTRO TIME• Quel est le sujet de ce code retreat ?• Et le gagnant est ?• On ne jette pas le code !

ITÉRATION 1 OBJECTIF• Générer du code legacy :

« Du code legacy est du code qui n’est pas couvert par des tests unitaires » - Michael Feathers

• Contexte entrainant du mauvais code:• Enjeu court-termiste• Pression plus ou moins positive• Isolation

ITÉRATION 2 INTRO• On a bien joué a être au travail• Maintenant prenons le temps de progresser pour de vrai

ITÉRATION 2• Aucune contrainte

ITÉRATION 2• Plusieurs commandes peuvent se cumuler (A traiter par

ordre d’apparition dans le fichier)• Le fichier input peut contenir une séquence simple ou

complète• Ajouter 2 lettres X et Y (http://www.maxisciences.com/adn/adn-

des-chercheurs-creent-deux-nouvelles-lettres-dans-l-039-alphabet-du-vivant_art32558.html)

• Ajouter la commande « simple » qui converti une séquence double en séquence simple

• Ajouter la commande « noSynthetic » qui supprime les paire de base X et Y

ITÉRATION 2 RETRO TIME• Quel choix lorsqu’on a plus de contraintes?

ITÉRATION 2 OBJECTIF• Choisir librement une solution pour travailler sur du legacy?

ITÉRATION 3 INTRORappel 1: SOLID

simple ResponsibilityOpen CloseLiskov Substitution PrincipleInterface SegregationDependency Inversion

ITÉRATION 3 INTRORappel 2: 4 rules of simple design (Kent Beck)

Passes its testsMinimizes duplicationMaximizes clarityHas fewer elements

ITÉRATION 3• Travail en pair obligatoire• Changement de pilote toutes les 5 min• Supprimer toutes dépendances implicites





• Ajouter la commande « simple » qui convertit une séquence double en séquence simple


ITÉRATION 3 RÉTRO TIME• Quelle différence quand on travail en paire ?

ITÉRATION 3 OBJECTIF• Sensibiliser à l’importance d’une paire quand on modifie du

code non testé• Identifier le fait que du code non testé est souvent fortement

couplé (dépandances implicites)• On test avant ou après avoir supprimer ces dépandances?

ITÉRATION 4• Modification de code seulement si couvert par un TU• Changement de pilotes en ping pong Test• Ne pas travailler sur votre code• Travail en paire obligatoire• Supprimer toutes dépendances implicites





• Ajouter la commande « simple » qui converti une séquence double en séquence simple


ITÉRATION 4 RETRO TIME• Est-ce que le test fait gagner :

• En temps ?• En qualité ?• En assurance ?

ITÉRATION 4 OBJECTIF• Tester après, c’est plus douloureux

ITÉRATION 5• Pas de if ni de switch en dehors de Factory• Pas de mutation d'objet• Modification de code seulement si couvert par un TU• Changement de pilotes en ping pong Test• Travail en paire obligatoire• Supprimer toutes dépendances implicites

ITÉRATION 5 RETRO TIME• C’est quoi la POO ?• C’est quoi la programmation fonctionnelle ?

ITÉRATION 5 OBJECTIF• Quelle concept mettre en œuvre pour améliorer son code ?

Même si on arrive à le tester ?

ITÉRATION 6 (BONUS)• Refaire l'exercice de la première itération en TDD

RÉTRO TIME

CONCLUSION• Le legacy ce n’est pas juste le code des autres• Vision court termiste vs maintenance à long terme• Comment gérer du code legacy ?• Commencer dès demain par ne plus en produire !

code retreat

Engineering