Effektiver Einsatz von Code-Reviews

Transkript

1 Effektiver Einsatz von Code-Reviews Version: 1.4 Orientation in Objects GmbH Weinheimer Str Mannheim Java Forum Stuttgart 2015

2 Ihr Sprecher Thorsten Maier Trainer, Berater, Entwickler Schwerpunkte Menschen Prozesse Architektur Code-Qualität 2015 Orientation in Objects GmbH Effektiver Einsatz von Code-Reviews 2

3 Agenda Einsteiger Code-Reviews? Noch nie gehört! Skeptiker Wir haben es ausprobiert. Es funktioniert nicht! Profis Wir setzen Code-Reviews bereits erfolgreich ein und wollen noch besser werden Orientation in Objects GmbH Effektiver Einsatz von Code-Reviews 3

4 Agenda Einsteiger Skeptiker Profis Warum Code- Reviews? Do it! Die richtigen Tools 2015 Orientation in Objects GmbH Effektiver Einsatz von Code-Reviews 4

5 Kennen Sie das? Das ist historisch gewachsen. 5

6 Oder das? Das wollten wir schon lange refactoren. 6

7 noch ein Klassiker Thomas ist leider gerade im Urlaub 7

8 Es bleibt nie Zeit für das große Refactoring Code-Reviews als kontinuierliche Lösung 8

9 Bugs finden 9

10 Wissen 10

11 11

12 Zeugnis 1+ Anerkennung 12

13 Entwicklungskosten in % Code-Review Entwicklung 20 0 ohne Code-Review mit Code-Review 13

14 Wissen Bugfixing Code-Review Entwicklung ohne Code-Review mit Code-Review 14

15 Code-Reviews für alle? Projekt oder Produkt 15

16 Agenda Einsteiger Skeptiker Profis Warum Code- Reviews? Do it! Die richtigen Tools 2015 Orientation in Objects GmbH Effektiver Einsatz von Code-Reviews 16

17 Wann? Code- Review Code- Review feature-x feature-z Nach jedem Feature / Bugfix 17

18 Wer? Klickt Einstiegspunkte Hinweise Prüft Hinterfragt Erläuterungen Reviewer Code-Autor 18

19 Jeder mit jedem! 19

20 Wir wollen unsere Anwendung gemeinsam besser machen 20

21 Was war der Grund für dein hier gewähltes Vorgehen? Das ist falsch 21

22 Zeilen Gefundene Fehler 1 2 Stunden Dauer des Reviews 22

23 23

24 24

25 Sinnvolle Namen? Klare Struktur? Korrekte Verwendung von Datentypen? Kommentare? Code smells? Tests vorhanden? Wohldefinierte Schnittstellen? Exception-Handling? I18N? Typische Programmier-Fehler? 25

26 Agenda Einsteiger Skeptiker Profis Warum Code- Reviews? Do it! Die richtigen Tools 2015 Orientation in Objects GmbH Effektiver Einsatz von Code-Reviews 26

27 27

28 28

29 29

30 30

31 31

32 32

33 Controller ServiceImpl 33

34 Controller ServiceImpl DaoImpl 34

35 35

36 (Service) - [DEPENDS_ON] -> (Controller) 36

37 MATCH (s:service)-[:depends_on]->(c:controller) return s,c 37

38 38

39 39

40 Git Issue-Tracker Jira 2. Build erfolgreich CI-Server Bamboo 1. Änderung 3a. Pull-Request 3b. Autor Repo-Verwaltung Stash 4a. Status-Übergang 4b. Pull 40

41 Fazit Einsteiger Wissen Skeptiker Profis Be prepared KISS 2015 Orientation in Objects GmbH Effektiver Einsatz von Code-Reviews 41

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

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