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

2 Über mich Falk Sippach Trainer, Berater, Entwickler Co-Organisator Schwerpunkte Architektur Agile Softwareentwicklung Codequalität 2016 Orientation in Objects GmbH Legacy Code meistern in x einfachen Schritten 2

3 Java, XML und Open Source seit 1998 ) Software Factory ) ) Object Rangers ) ) Competence Center) Schlüsselfertige Realisierung von Java Software Individualsoftware Pilot- und Migrationsprojekte Sanierung von Software Software Wartung Unterstützung laufender Java Projekte Perfect Match Rent-a-team Coaching on the project Inhouse Outsourcing 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

4 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

5 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

6 Unser Thema heute: Refactoring Legacy Code Disclaimer: KEIN Projekterfahrungsbericht 6

7 Legacy Code Somebody else's code Jeder kennt ihn Was ist mit unserem eigenen Code? Keiner mag ihn 7

8 Refactoring Legacy Code Warum? Verstehen Erweitern Bugfixing Optimierung Grafik von ClkerFreeVectorImages: (CC0 Public Domain Lizenz) 8

9 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

10 Refactoring Legacy Code Code Smells Temporary Field Long Method Feature Envy Code Comments 10

11 "Gimme Google, Stack Overflow, and this keyboard, and I'll program you anything. " Duplicated Code 11

12 Refactoring Legacy Code Code without tests 12

13 LEGACY Vermächtnis Erbe Altlast Hinterlassenschaft Foto von smpcas, CC0 Public Domain Lizenz, 13

14 J. B. Rainsberger Legacy code is valuable code that we feel afraid to change. Foto von PublicDomainPictures, CC0 Public Domain Lizenz, 14

15 Code without tests is bad code. Michael Feathers 15

16 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

17 Die gute Nachricht Keine Wissenschaft Gesunder Menschenverstand Foto von Engel62: (CC0 Public Domain Lizenz) 17

18 Was macht es dann schwierig? Hello World vs LOC Disziplin (kleine Schritte, ) Aussagekräftige Testabdeckung 18

19 Legacy zu Clean Code? Clean Code ist NICHT das Ziel Hauptfokus: testbarer Code 19

20 Dann schreiben wir halt Tests viel zu teuer Code meist kaum/nicht testbar starke Kopplung, geringe Kohäsion 20

21 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

22 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üßtebeiß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

23 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

24 No Silver Bullet Jedes Projekt individuell Vorsicht beim Beheben von offensichtlichen Fehlern Foto von stevepb: (CC0 Public Domain Lizenz) 24

25 Dann mal her mit den x einfachen Schritten! 25

26 A Sicherheitsnetz + Tests Foto von bella67: (CC0 Public Domain Lizenz) 26

27 B Sanierung Foto von KlausHausmann: (CC0 Public Domain Lizenz) 27

28 1 Golden Master Foto von istara: (CC0 Public Domain Lizenz) 28

29 29

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

31 1 Golden Master Vorsicht bei Zufallsgeneratoren Festlegen von Seeds (Pseudo-Random) 31

32 2 Subclass To Test Foto von chrisli8020: (CC0 Public Domain Lizenz) 32

33 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

35 3 Extract Pure Functions Foto von PublicDomainPictures: (CC0 Public Domain Lizenz) 35

36 3 Extract Pure Functions seiteneffektfrei keine Statusänderung "It's a classic, we call it a Klassiker" 36

37 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

38 3 Extract Pure Functions Codestellen isolieren Ziele Separat testbar Duplikation reduzieren 38

40 4 Remove Duplication Don't Repeat Yourself Beachte: Rule of Three Foto von AdinaVoicu: (CC0 Public Domain Lizenz) 40

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

44 6 Dependency Inversion 7 Dependency Inversion Test non-public member Dependency Inversion 8 Mocking Framework 44

45 Zusammenfassung 45

46 1 Golden Master 47

47 Subclass To Test Extract Pure Functions Extract Class Dependency Inversion Test non-public member Mocking Framework 48

48 3 4 5 Extract Pure Functions Remove Duplication Extract Class 49

49 Tools Code Coverage Approval Tests Infinitest Eclipse Metrics 2016 Orientation in Objects GmbH Legacy Code meistern in x einfachen Schritten 50

50 Legacy Code Retreat Foto von Jmabel: (CC BY-SA 3.0 Lizenz) 51

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

52 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

53 ??? Fragen?? Orientation in Objects GmbH? Weinheimer Str Mannheim

54 Vielen Dank für ihre Aufmerksamkeit! Orientation in Objects GmbH Weinheimer Str Mannheim

55 Zusatzmaterial 6 Dependency Inversion 7 Dependency Inversion Test non-public member Dependency Inversion 8 Mocking Framework 2016 Orientation in Objects GmbH Legacy Code meistern in x einfachen Schritten 56

56 6 Dependency Inversion Entkoppeln durch explizites Setzen der Abhängigkeiten Foto von tatlin: (CC0 Public Domain Lizenz) 57

58 7 Test non-public member Wirklich? Gute Alternative bei Legacy Code! Foto von Guian Bolisay: (CC BY-SA 2.0) 59

60 8 Mocking Subclass To Test on Steorids! Interaktion mit Umgebung testen Erwartete Parameter und Aufrufreihenfolge Foto von tatlin: (CC0 Public Domain Lizenz) 61