Effektiver Einsatz von Code-Reviews

Größe: px

Ab Seite anzeigen:

Download "Effektiver Einsatz von Code-Reviews"

Heiko Adler
vor 5 Jahren
Abrufe

1 - Schneller, Billiger, Besser - Effektiver Einsatz von Code-Reviews Dev Day in Dresden 27. Mai 2015 Version: 1.3 Orientation in Objects GmbH Weinheimer Str Mannheim info@oio.de Ihr Sprecher Thorsten Maier Trainer, Berater, Entwickler Schwerpunkte Menschen Prozesse Architektur Code-Qualität 2 1

2 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. 3 Agenda Einsteiger Skeptiker Profis Warum Code- Reviews? Do it! Die richtigen Tools 4 2

3 Kennen Sie das? Das ist historisch gewachsen. 5 Oder das? Das wollten wir schon lange refactoren. 6 3

4 noch ein Klassiker Thomas ist leider gerade im Urlaub 7 Es bleibt nie Zeit für das große Refactoring Code-Reviews als kontinuierliche Lösung 8 4

5 Bugs finden 9 Wissen 10 5

6 11 Zeugnis 1+ Anerkennung 12 6

7 Qualität kostet Geld Entwicklungskosten in % Code-Review Entwicklung 0 ohne Code-Review mit Code-Review 13 Langfristige Kostenentwicklung Wissen Bugfixing Code-Review Entwicklung ohne Code-Review mit Code-Review 14 7

8 Qualität kostet Anstrengung Warum testen? Vertrauen Sie Ihrem eigenen Code nicht? 15 Code-Reviews für alle? Projekt oder Produkt 16 8

9 Agenda Einsteiger Skeptiker Profis Warum Code- Reviews? Do it! Die richtigen Tools 17 Formale Inspektion nach Michael Fagan (1976) Planung Überblick Vorbereitung Inspektion Nacharbeit Nachverfolgung Manager Moderator Autor Prüfer Schriftführer 18 9

10 Formale Inspektion nach Michael Fagan (1976) "Currently 1% of our code is inspected Planung Überblick Vorbereitung Inspektion Nacharbeit Nachverfolgung "We believe by the end of the year we can get it up to 7%." Manager Moderator Autor Prüfer Schriftführer

11 Wann? Code- Review Code- Review feature-x feature-z Nach jedem Feature / Bugfix 21 Wer? Klickt Einstiegspunkte Hinweise Prüft Hinterfragt Erläuterungen Reviewer Code-Autor 22 11

12 Ein gemeinsames Ziel Wir wollen unsere Anwendung gemeinsam besser machen 23 Stelle Fragen Was war der Grund für dein hier gewähltes Vorgehen? Das ist falsch 24 12

13 Nicht zu viel und nicht zu lange Zeilen Gefundene Fehler 1 2 Stunden Dauer des Reviews 25 Do it!!! 26 13

14 Qualität kostet Geld und deshalb messen! 27 Checkliste Sinnvolle Namen? Klare Struktur? Korrekte Verwendung von Datentypen? Kommentare? Code smells? Tests vorhanden? Wohldefinierte Schnittstellen? Exception-Handling? I18N? Typische Programmier-Fehler? 28 14

15 Guidelines Sun Java Code Conventions Clean Code Tool-Standard-Konfiguration Eclipse: Errors, Warnings und Formatter Findbugs PMD Checkstyle 29 Agenda Einsteiger Skeptiker Profis Warum Code- Reviews? Do it! Die richtigen Tools 30 15

16 31 Versionsverwaltung & Diff-Tool 32 16

17 Gute Vorbereitung ist die halbe Miete 33 Alles in Ordnung? 34 17

18 FindBugs 35 Alles in Ordnung? 36 18

19 Schichtenarchitektur? Controller ServiceImpl 37 Automatische Architektur-Tests Controller ServiceImpl DaoImpl

20 Wünsche an ein Code-Review-Tool 39 Atlassian Stash 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 20

21 Atlassian Stash 41 Fazit Einsteiger Wissen Skeptiker Profis Be prepared KISS 42 21

22 Mehr von OIO zum Thema Schulung: Refactoring Workshop Schulung: Git kompakt Schulung: Versionsverwaltung mit Git Schulung: Atlassian Stash Enterprise Git Schulung: JIRA Fachliche Administration 43?? Fragen??? Orientation Objects GmbH Weinheimer Str Mannheim

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

Ähnliche Dokumente

Effektiver Einsatz von Code-Reviews

Effektiver Einsatz von Code-Reviews Version: 1.4 Orientation in Objects GmbH Weinheimer Str. 68 68309 Mannheim www.oio.de info@oio.de 18. Java Forum Stuttgart 2015 Ihr Sprecher Thorsten Maier Trainer,