Legacy Code meistern in x einfachen Schritten

Transkript

1 Legacy Code meistern in x einfachen Schritten Version: 1.0 Orientation in Objects GmbH Weinheimer Str Mannheim info@oio.de Über mich Falk Sippach (@sippsack) Trainer, Berater, Entwickler Co-Organisator Schwerpunkte Architektur Agile Softwareentwicklung Codequalität 2016 Orientation in Objects GmbH Legacy Code meistern in x einfachen Schritten 2 1

2 Java, XML und Open Source seit 1998 ) Software Factory ) Schlüsselfertige Realisierung von Java Software Individualsoftware Pilot- und Migrationsprojekte Sanierung von Software Software Wartung ) Object Rangers ) Unterstützung laufender Java Projekte Perfect Match Rent-a-team Coaching on the project Inhouse Outsourcing ) Competence Center) Schulungen, Coaching, Weiterbildungsberatung, Train & Solve-Programme Methoden, Standards und Tools für die Entwicklung von offenen, unternehmensweiten Systemen 2016 Orientation in Objects GmbH Legacy Code meistern in x einfachen Schritten 3 Legacy Code meistern in x einfachen Schritten Nur heute, nicht 1, nicht 2, 2016 Orientation in Objects GmbH Legacy Code meistern in x einfachen Schritten 4 2

3 Legacy Code meistern in x einfachen Schritten In einer idealen Welt würden wir nur "neuen" Code schreiben, der natürlich perfekt und wunderschön ist. Wir müssten nie wieder unseren Code anschauen, geschweige denn 10 Jahre alte Projekte warten. Ende des Tagtraums... Leider ist unsere Welt nicht so ideal, unser Code von gestern ist heute schon Legacy. Diesen im Nachhinein zu verstehen, zu erweitern oder darin Fehler zu beheben, ist immer eine Herausforderung, insbesondere wenn Tests fehlen. Trotzdem gibt es einfache Möglichkeiten, wie man die Qualität von Legacy-Code verbessern kann. Das Wichtigste ist das Einziehen von Fangnetzen, sodass man trotz fehlender Tests guten Gewissens Änderungen durchführen kann. Wer Golden Master, Subclass to Test und Extract Pure Functions an konkreten Beispielen kennenlernen möchte, ist in dieser Session genau richtig Orientation in Objects GmbH Legacy Code meistern in x einfachen Schritten 5 Unser Thema heute: Refactoring Legacy Code Disclaimer: KEIN Projekterfahrungsbericht 6 3

4 Legacy Code Somebody else's code Jeder kennt ihn Keiner mag ihn Was ist mit unserem eigenen Code? 7 Refactoring Legacy Code Warum? Verstehen Erweitern Bugfixing Optimierung Grafik von ClkerFreeVectorImages: (CC0 Public Domain Lizenz) 8 4

5 Refactoring Legacy Code Annahmen Es gibt automatisierte Tests Quellcode ist schon testbar 2016 Orientation in Objects GmbH Legacy Code meistern in x einfachen Schritten 9 Refactoring Legacy Code Code Smells Temporary Field Long Method Feature Envy Code Comments

6 "Gimme Google, Stack Overflow, and this keyboard, and I'll program you anything. " Duplicated Code 11 Refactoring Legacy Code Code without tests 12 6

7 LEGACY Vermächtnis Erbe Altlast Hinterlassenschaft Foto von smpcas, CC0 Public Domain Lizenz, 13 Legacy code is valuable code that we feel afraid to change. J. B. Rainsberger Foto von PublicDomainPictures, CC0 Public Domain Lizenz,

8 Code without tests is bad code. Michael Feathers 15 Es ist egal, wie gut geschrieben der Code ist schön der Code ist objektorientiert der Code ist entkoppelt der Code ist Tests lassen unser Verhalten schnell und verifizierbar ändern Ohne Tests wissen wir nicht, ob der Code besser oder schlechter wird 16 8

9 Die gute Nachricht Keine Wissenschaft Gesunder Menschenverstand Foto von Engel62: (CC0 Public Domain Lizenz) 17 Was macht es dann schwierig? Hello World vs LOC Disziplin (kleine Schritte, ) Aussagekräftige Testabdeckung 18 9

10 Legacy zu Clean Code? Clean Code ist NICHT das Ziel Hauptfokus: testbarer Code 19 Dann schreiben wir halt Tests viel zu teuer Code meist kaum/nicht testbar starke Kopplung, geringe Kohäsion 20 10

11 refactoren bräuchte man Tests, Tests würden helfen, Code zu verstehen, um Code zu verstehen könnte man Code refactoren, um Code zu refactoren bräuchte man Tests, Tests würden helfen, Code zu verstehen, um Code zu verstehen könnte man Code refactoren, um Code zu refactoren bräuchte man Tests, Tests würden helfen, Code zu verstehen, um Code zu verstehen könnte man Code refactoren, um Code zu refactoren bräuchte man Tests, Tests würden helfen, Code zu verstehen, um Code zu Henne-Eiverstehen könnte man Code refactoren, um Code zu refactoren bräuchte Problem man Tests, Tests würden helfen, Code zu verstehen, um Code zu verstehen könnte man Code refactoren, um Code zu refactoren bräuchte man Tests, Tests würden helfen, Code zu verstehen, um Code zu verstehen könnte man Code refactoren, um Code zu refactoren bräuchte man Tests, Tests würden helfen, Code zu verstehen, um Code zu verstehen könnte man Code refactoren, um Code zu refactoren bräuchte man Tests, Tests würden helfen, Code zu verstehen, um Code zu verstehen könnte man Code refactoren, um Code zu refactoren bräuchte man Tests, Tests würden helfen, Code zu 21 um Tests zu schreiben muß der Code testbar sein, um testbaren Code zu erhalten müßte man refactoren, um zu refactoren könnte man Tests schreiben, um Tests zu schreiben muß der Code testbar sein, um testbaren Code zu erhalten müßte man refactoren, um zu refactoren könnte man Tests schreiben, um Tests zu schreiben muß der Code testbar sein, um testbaren Code zu erhalten müßte man refactoren, um zu refactoren könnte man Tests schreiben, um Tests zu schreiben muß der Code testbar sein, um testbaren Code zu erhalten müßte man refactoren, um zu refactoren könnte man Tests schreiben, um Tests zu schreiben muß der Code testbar sein, um testbaren Code zu erhalten Die Katze müßte beißt man refactoren, sich um zu refactoren könnte man Tests schreiben, in um den Tests Schwanz! zu schreiben muß der Code testbar sein, um testbaren Code zu erhalten müßte man refactoren, um zu refactoren könnte man Tests schreiben, um Tests zu schreiben muß der Code testbar sein, um testbaren Code zu erhalten müßte man refactoren, um zu refactoren könnte man Tests schreiben, um Tests zu schreiben muß der Code testbar sein, um testbaren Code zu erhalten müßte man refact 22 11

12 Vorgehen 1. Identify what to change 2. Identify what to test 3. Break dependencies 4. Write the tests 5. Modify and refactoring 2016 Orientation in Objects GmbH Legacy Code meistern in x einfachen Schritten 23 No Silver Bullet Jedes Projekt individuell Vorsicht beim Beheben von offensichtlichen Fehlern Foto von stevepb: (CC0 Public Domain Lizenz) 24 12

13 Dann mal her mit den x einfachen Schritten! 25 A Sicherheitsnetz + Tests Foto von bella67: (CC0 Public Domain Lizenz) 26 13

14 B Sanierung Foto von KlausHausmann: (CC0 Public Domain Lizenz) 27 1 Golden Master Foto von istara: (CC0 Public Domain Lizenz) 28 14

15 29 Live-Coding Foto von StockSnap: (CC0 Public Domain Lizenz) 30 15

16 1 Golden Master Vorsicht bei Zufallsgeneratoren Festlegen von Seeds (Pseudo-Random) 31 2 Subclass To Test Foto von chrisli8020: (CC0 Public Domain Lizenz) 32 16

17 2 Subclass To Test Seam (Nahtstelle) Ein Seam ist eine Stelle, an der man das Verhalten editieren kann, ohne direkt an dieser Stelle zu ändern. Aufbrechen stark gekoppelter Abhängigkeiten aka Extract and Override 33 Live-Coding Foto von StockSnap: (CC0 Public Domain Lizenz) 34 17

18 3 Extract Pure Functions Foto von PublicDomainPictures: (CC0 Public Domain Lizenz) 35 3 Extract Pure Functions seiteneffektfrei keine Statusänderung "It's a classic, we call it a Klassiker" 36 18

19 3 Extract Pure Functions "pure function".substring(5); UrlEncoder.encode("pure function"); Math.max(x, y); System.out.println("unpure"); list.add(3); Collections.sort(list); 37 3 Extract Pure Functions Codestellen isolieren Ziele Separat testbar Duplikation reduzieren 38 19

20 Live-Coding Foto von StockSnap: (CC0 Public Domain Lizenz) 39 4 Remove Duplication Don't Repeat Yourself Beachte: Rule of Three Foto von AdinaVoicu: (CC0 Public Domain Lizenz) 40 20

21 Live-Coding Foto von StockSnap: (CC0 Public Domain Lizenz) 41 5 Extract Class Large class SRP verletzt Ziele Unabhängiges Testen einzelner Teile Sauberes OO-Design Foto von blickpixel: (CC0 Public Domain Lizenz) 42 21

22 Live-Coding Foto von StockSnap: (CC0 Public Domain Lizenz) 43 6 Dependency Inversion 7 Test Dependency non-public Inversion member 8 Dependency Mocking Framework Inversion 44 22

23 Zusammenfassung Golden Master Subclass To Test Extract Pure Functions Remove Duplication Extract Class Dependency Inversion Test non-public member Mocking Framework 46 23

24 1 Golden Master Subclass To Test Extract Pure Functions Extract Class Dependency Inversion Test non-public member Mocking Framework 48 24

25 3 4 5 Extract Pure Functions Remove Duplication Extract Class 49 Tools Code Coverage Approval Tests Infinitest Eclipse Metrics 2016 Orientation in Objects GmbH Legacy Code meistern in x einfachen Schritten 50 25

26 Legacy Code Retreat Foto von Jmabel: (CC BY-SA 3.0 Lizenz) 51 Links Code-Beispiel der Live-Demo anderes Code-Beispiel für Legacy Code Blog: Techniken zu Legacy Code-Retreat Orientation in Objects GmbH Legacy Code meistern in x einfachen Schritten 52 26

27 Literaturhinweise Refactoring Sprache: Englisch Gebunden Seiten - Addison Wesely Erscheinungsdatum: 1. Juni 1999 ISBN: Working Effectively with Legacy Code Sprache: Englisch Gebunden 2016 Orientation in Objects GmbH Legacy Code meistern in x einfachen Schritten 53?? Fragen??? Orientation Objects GmbH Weinheimer Str Mannheim info@oio.de?? 27

28 Vielen Dank für ihre Aufmerksamkeit! Orientation in Objects GmbH Weinheimer Str Mannheim Zusatzmaterial 6 Dependency Inversion 7 Test Dependency non-public Inversion member 8 Dependency Mocking Framework Inversion 2016 Orientation in Objects GmbH Legacy Code meistern in x einfachen Schritten 56 28

29 6 Dependency Inversion Entkoppeln durch explizites Setzen der Abhängigkeiten Foto von tatlin: (CC0 Public Domain Lizenz) 57 Live-Coding Foto von StockSnap: (CC0 Public Domain Lizenz) 58 29

30 7 Test non-public member Wirklich? Gute Alternative bei Legacy Code! Foto von Guian Bolisay: (CC BY-SA 2.0) 59 Live-Coding Foto von StockSnap: (CC0 Public Domain Lizenz) 60 30

31 8 Mocking Subclass To Test on Steorids! Interaktion mit Umgebung testen Erwartete Parameter und Aufrufreihenfolge Foto von tatlin: (CC0 Public Domain Lizenz) 61 Live-Coding Foto von StockSnap: (CC0 Public Domain Lizenz) 62 31