Upgrade to Pro — share decks privately, control downloads, hide ads and more …

Confident refactors

C74bdcd3fa7c3d3f23290f46430b1463?s=47 Mehdi Lahmam B.
June 19, 2017
Technology
1
35

Confident refactors

C74bdcd3fa7c3d3f23290f46430b1463?s=128

Mehdi Lahmam B.

June 19, 2017
Tweet

More Decks by Mehdi Lahmam B.

See All by Mehdi Lahmam B.
C74bdcd3fa7c3d3f23290f46430b1463?s=48 mehlah
0
370
C74bdcd3fa7c3d3f23290f46430b1463?s=48 mehlah
1
130
C74bdcd3fa7c3d3f23290f46430b1463?s=48 mehlah
0
9
C74bdcd3fa7c3d3f23290f46430b1463?s=48 mehlah
0
110
C74bdcd3fa7c3d3f23290f46430b1463?s=48 mehlah
2
730
C74bdcd3fa7c3d3f23290f46430b1463?s=48 mehlah
0
110
C74bdcd3fa7c3d3f23290f46430b1463?s=48 mehlah
1
260
C74bdcd3fa7c3d3f23290f46430b1463?s=48 mehlah
1
350
C74bdcd3fa7c3d3f23290f46430b1463?s=48 mehlah
0
49

Other Decks in Technology

See All in Technology
9e7a653ed313b3da4b263a772bfc5026?s=48 clustervr
0
170
5b9550b2ea725a86cde115633e7cd162?s=48 muras
0
120
332f89cc697355902a817506b6995f2b?s=48 ytaka23
5
1.7k
85da685d91fda190e2e3162d0de248a4?s=48 recruitengineers
0
140
3c635ac67557f954b04d37a24254f436?s=48 becolomochi
0
170
7d065deebd80754be06ba6d71049f1be?s=48 apcom
1
450
F2ccc400e285972d80feab0e4e57b5f7?s=48 fu3ak1
0
980
9cdb1d21375062bc2cd0d1358c2fa1f0?s=48 noir_neo
0
130
D9e65f4b0af059ae9ba243c8c2265e4f?s=48 pohjus
1
2.1k
35f1e35d8c02c612e281c6c7a9d22341?s=48 hirosys
0
150
3009eedce0d0f7ae45987a7bd8f57b85?s=48 nkjzm
1
870
88964b936e864ca7d326272eaa70fa9a?s=48 hgsgtk
4
1.2k

Featured

See All Featured
56c4053438af8e8b90d6f53cbb7573be?s=48 jakevdp
775
200k
7c8469ee8c9e594c65c59b919626c08d?s=48 scottboms
251
11k
A0517b08532aec8ab2aae3f65d40ad86?s=48 hannesfritz
27
930
D47656e20ff5e42f125fc5ea0fd636c6?s=48 tmm1
61
8.5k
0c6fd6a08d0f898159cda7cafcacf07f?s=48 afnizarnur
176
14k
E6c6e133e74c3b83f04d2861deaa1c20?s=48 searls
204
35k
B3d6434763caa0ef5dc4b792662c49f7?s=48 smashingmag
229
18k
B3ace07412a5178ea778e7eec161fc0a?s=48 jcasabona
7
520
4262b9cfacfb6b2c77f23e1d0310cd3e?s=48 myddelton
109
11k
0bce06bd06ee7a2c4f5500f25dcf7f18?s=48 paulrobertlloyd
71
3.6k
719435d98d452de7ac367c828266cf01?s=48 erikaheidi
13
4.2k
F57e2136eb9895eb95ff5dc8a1c02677?s=48 zakiwarfel
88
3.3k

Transcript

  1. Confident Refactors

  2. Mehdi Lahmam @mehlah

  3. 25kV Rails app, started in Feb 2009 Never rewritten 23k+

    commits from 52 contributors

  4. Ruby is a massively successful language!

  5. Early success Making it easy to make new things

  6. Later success Making it easy to maintain old things

  7. Today, let’s refactor some legacy code

  8. Today, let’s refactor some legacy code

  9. Refactor - verb To change the design of code without

    changing its observable behavior.

  10. Refactor - verb To change in advance of a new

    feature or bug fix, making the job easier.

  11. Today, let’s refactor some legacy code

  12. Legacy - noun Old code.

  13. Legacy - noun Code without tests.

  14. Legacy - noun Code that we don’t like.

  15. Legacy - noun Code we don’t understand well enough
 to

    change confidently.

  16. Today, let’s refactor some legacy code

  17. Refactoring is hard

  18. Refactoring legacy code is very hard

  19. Easy to accidentally break functionality

  20. Legacy refactors feels unsafe

  21. Legacy refactors are hard to sell

  22. Business priority Cost / risks

  23. Business priority Cost / risks New features

  24. Business priority Cost / risks New features Bugs fixes

  25. Business priority Cost / risks New features Bugs fixes Testing

  26. Business priority Cost / risks New features Bug fixes Testing

    Refactoring

  27. Business priority Cost / risks New features Bug fixes Testing

    Refactoring no selling needed easy to sell can often sell very hard to sell

  28. Selling refactoring is hard ⏲ ⛔

  29. Business priority Cost / risks Refactoring

  30. Too much pressure ⏳ ⚒

  31. Refactors are scary

  32. What techniques we have? PDD - Pray Driven Development Fowler’s

    refactoring book Characterization testing A/B testing, experiments

  33. Refactors as surgeries

  34. Refactors as surgeries Requires careful planning Follow a clear process

    Multiple observations Flexible tools Things can get bloody

  35. 1. Plan 2. Cut 3. Record 4. Validate 5. Refactor

    6. Verify 7. Compare 8. Fallback 9. Delete Clear process
  36. None
  37. None
  38. None

  39. Cut

  40. Record

  41. Validate ✅

  42. Refactor <Insert magic here />

  43. Verify

  44. Compare ⚖

  45. Fallback

  46. Delete ✂

  47. We dit it

  48. We dit it Time to celebrate hard