-
Notifications
You must be signed in to change notification settings - Fork 16
/
Copy pathlegacy.html
218 lines (197 loc) · 9.02 KB
/
legacy.html
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
<!doctype html>
<html>
<head>
<meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=no">
<title>Legacy Code</title>
<meta name="description" content="A framework for easily creating beautiful presentations using HTML">
<meta name="author" content="Gaspard POINTEAU">
<link rel="stylesheet" href="css/reveal.css">
<link rel="stylesheet" href="css/theme/white.css">
<!-- Theme used for syntax highlighting of code -->
<link rel="stylesheet" href="lib/css/zenburn.css">
<!-- Printing and PDF exports -->
<script>
var link = document.createElement('link');
link.rel = 'stylesheet';
link.type = 'text/css';
link.href = window.location.search.match(/print-pdf/gi) ? 'css/print/pdf.css' : 'css/print/paper.css';
document.getElementsByTagName('head')[0].appendChild(link);
</script>
</head>
<body>
<div class="reveal">
<div class="slides">
<section>
<h2>Legacy Code</h2>
<p>
<small>Gaspard POINTEAU<br><a href="http://twitter.com/Gaspard_PO">@Gaspard_PO</a></small>
</p>
</section>
<section>
<section>
<h2>Legacy-Code</h2>
<ul>
<li class="fragment">existant ?</li>
<li class="fragment">de mauvaise qualité ?</li>
<li class="fragment">vieux ?</li>
<li class="fragment">on ne sait pas ce qu'il fait ?</li>
<li class="fragment">plein de dépendances ?</li>
</ul>
</section>
<section>
<h2>Legacy-Code</h2>
<h3 class="fragment">Du code sans tests</h3>
</section>
<section>
<h3>La seule vérité est le code</h3>
<p class="fragment">Les specs ne sont pas à jour et pas fiables</p>
<br>
<h4 class="fragment">Ne rien changer au fonctionnement</h4>
<p class="fragment">Un bug suffisamment vieux devient une feature</p>
<p class="fragment">Toujours demander au PO</p>
</section>
<section>
<h3>Pourquoi refactorer ?</h3>
<p class="fragment">Le code fonctionne très bien, ne pas refactorer pour le principe.<br>
C'est couteux et risqué.</p>
<p class="fragment">Pour comprendre le fonctionnement ?</p>
<p class="fragment">Pour ajouter une feature ?</p>
<p class="fragment">Pour rembourser la dette ?</p>
<blockquote class="fragment">"Make the change easy before making the easy change."<br>
Kent Beck
</blockquote>
</section>
</section>
<section>
<section>
<h3>Gestion de version</h3>
<p class="fragment">Pouvoir toujours revenir à un état stable.</p>
<p class="fragment">S'autoriser à refactorer, renommer, bouger des choses pour rien, juste pour
découvrir le code.</p>
</section>
<section>
<h3>Time-boxer</h3>
</section>
<section>
<h3>Bloc-notes</h3>
</section>
</section>
<section>
<section>
<h3>Golden Master Test</h3>
<p>Le système est une boite noire, on vérifie que pour les mêmes entrées on a toujours les mêmes
sorties</p>
<img src="images/legacy/black-box.jpg" alt="inputs goes to black-box, ouput goes" height="300">
</section>
<section>
<uL>
<li class="fragment">Librairie Approvals</li>
<li class="fragment">Fixer les entrées</li>
<li class="fragment">Sérialiser les objets, ou rediriger les sorties pour avoir du texte</li>
<li class="fragment">Vérifier avec la couverture de code. Attention, pas toujours fiable</li>
<li class="fragment">Vérifier en changeant des bouts de code. (Red du TDD)</li>
<li class="fragment">Utiliser des données représentatives (copie de prod)</li>
<li class="fragment">Vérifier qu'on ne touche pas d'environnement de prod.</li>
</uL>
</section>
</section>
<section>
<section>
<h3>Test de caractérisations</h3>
<p>Créer une batterie de tests unitaires qui permmettent d'avoir des specs.</p>
</section>
<section>
<h3>Rendre le code testable en supprimant les dépendances</h3>
<ul>
<li class="fragment">Extraire les appels statiques dans des méthodes protected</li>
<li class="fragment">Utiliser uniquement les refactos automatisé de l'IDE</li>
<li class="fragment">remplacer les "new Object()" par des paramètres</li>
<li class="fragment">Utiliser l'héritage</li>
</ul>
<aside class="notes">
rendre le code testable -> héritage, avant d'arriver à de l'injection
</aside>
</section>
<section>
<p>Commencer par tester la branche la plus courte</p>
<p class="fragment">La branche avec le moins de complexité (for, if...) et la plus facile à tester.</p>
<p class="fragment">Utiliser la couverture de code pour voir les branches qui ne sont pas encore
testées.</p>
</section>
<section>
<h4 class="fragment">En dernier recours</h4>
<p class="fragment">Reflection</p>
<p class="fragment">Mock d'objet statiques</p>
<aside class="notes">
très risqué !
</aside>
</section>
</section>
<section>
<section>
<h3>Refactorer</h3>
</section>
<section>
<p>Baby-steps</p>
<p>Tester en continu</p>
</section>
<section>
<h4>Nommer les choses</h4>
<ul>
<li class="fragment">Commenter pour donner un nom a un paquet de lignes, à un chiffre</li>
<li class="fragment">Extraire ces lignes dans une méthode, une variable</li>
<li class="fragment">Donner comme nom à la méthode/variable le commentaire</li>
<li class="fragment">Supprimer le commentaire</li>
</ul>
<blockquote class="fragment" du por>Comprendre ce que fait le code et mettre cette compréhension dans le
code.
</blockquote>
</section>
<section>
<h4>Extraire des méthodes</h4>
<p>Commencer par les branches les plus à droite (les plus imbriquées) et par des tous petits blocs.</p>
<p>Extraire des variables, forcer la visibilité du vide pour faire apparaitre de la duplication.</p>
</section>
<section>
<h4>Améliorer le design</h4>
<ul>
<li class="fragment">Casser les appels à l'extérieur (Logs, BDD, fichiers, mails ...)</li>
<li class="fragment">Casser les appels statiques</li>
<li class="fragment">Mettre de l'injection de dépendances</li>
<li class="fragment">Extraire des objets</li>
</ul>
</section>
<section>
<h4>Objets</h4>
<ul>
<li class="fragment">Profiter de la création d'objets pour faire du TDD et mettre des tests unitaires.</li>
<li class="fragment">Procéder par toutes petites itérations : encapsuler un String dans un Objet, puis rajouter d'autres champs, des méthodes ...</li>
<li class="fragment">Bien se demander quel objet est responsable de quoi.</li>
<li class="fragment">append(s) -> s.append()</li>
</ul>
</section>
</section>
</div>
</div>
<script src="lib/js/head.min.js"></script>
<script src="js/reveal.js"></script>
<script>
// More info about config & dependencies:
// - https://github.com/hakimel/reveal.js#configuration
// - https://github.com/hakimel/reveal.js#dependencies
Reveal.initialize({
dependencies: [
{src: 'plugin/markdown/marked.js'},
{src: 'plugin/markdown/markdown.js'},
{src: 'plugin/notes/notes.js', async: true},
{
src: 'plugin/highlight/highlight.js', async: true, callback: function () {
hljs.initHighlightingOnLoad();
}
}
]
});
</script>
</body>
</html>