Code reviews

Code Reviews

Leave your ego at the door

@FrankS

Frank Sons - Code Reviews, Leave your ego at the doorcontinuous lifecycle 10.11.2014

InnoGames GmbH

ca. 15 Jahre Entwickler (PHP, Java, etc.)

PHP Usergroup Hamburg

Und ich lerne immer noch…

2

Kurz zu mir…


Eine kurze Einführung zu Reviews

Collective Code Ownership

Das richtige Werkzeug wählen

Typische Probleme vermeiden

3

Worum geht es hier?

continuous lifecycle 10.11.2014 Frank Sons - Code Reviews, Leave your ego at the door

Eine kurze Einführung zu Reviews

4

Lassen wir den Code mal kurz weg…


Wikipedia: Peer Review

“Peer review methods are employed to maintain standards, improve performance

and provide credibility.”

5

Frank Sons - Code Reviews, Leave your ego at the doorcontinuous lifecycle 10.11.2014 6

Reviews gibt es nicht nur für Code…


IBM: Inspektionen seit 1970ern

7

Zurück zu den Anfängen der Code Reviews

Strikte Checklisten

Alles andere als agile


“Das ist nicht mein Code!”

8

Collective Code Ownership ist eine Einstellung


Jeff Atwood (Stack Overflow ) - @codinghorror

“Everyone has the right and freedom to totally suck in private.”

9


“Attempt to be awesome in public and embrace the suck!”

10



Reviews sind knowledge sharing!


Produktwissen

Technisches Wissen

Coding Standards

Transparenz schaffen!

12

Wissen und Erfahrungen im Team teilen…


Die richtige Review Methode finden

13

Das richtige Werkzeug für den richtigen Zweck!


“Hast du mal kurz Zeit? Ich hab da nen Problem.”

14


Braucht nicht viel Zeit

15

Ad hoc reviews

Geht überall

Löst nur das konkrete Problem


Talk to the duck


“Kannst du dir nachher mal den Code anschauen?”

17


Geht meist asynchron

18

Der “Schreibtischtest”

Gut für große Mengen an Code

Review tools arbeiten so


“Lass uns das zusammen machen!”

19


“Driver” und “Navigator”

20

Pair Programming

Schnelle Iterationen

“Review on the fly”


“Ich zeige euch mal meinen Code.”

21


Autor erklärt den Code

Entwickler stellen Fragen

Ideal zum vorstellen und präsentieren

Risiko etwas zu übersehen

22

Code walkthrough


“Kannst du meinen Code den anderen erklären?”

23


Erklären ohne Code zu kennen

Auch auf das große Ganze achten

Autor beantwortet Fragen

Code gezielt auswählen

24

Code Review - “in a big round”


Typische Probleme vermeiden

25

Es gibt Stolpersteine…


“My ego is not tied to my perfect or imperfect work product.”

26



Das Ego an der Tür lassen

Nicht persönlich werden

Ergebnisse nachbereiten

Nicht missbrauchen!

27

Stolpersteine vermeiden


Nicht zu viele Teilnehmer

Maximal zwei Stunden

Probleme finden, keine Lösungen

Umfang Review und Projekt

28

Ein paar praktische Tipps…


Code wird verständlicher

Wissen wird weiter gegeben

Kommunikation im Team

Leuchtturm Effekt

29

Es gibt viele Vorteile…


Einfach mal machen…

30

Just do it!


Für mehr Hintergrundinfos:


Reviews helfen dem Team

32

Viel Spaß bei euren Reviews!

Die Einstellung ist das wichtigste

Immer wieder anpassen

Vielen Dank!

[email protected]

@FrankS

Sprecht mich an :-)

slideshare.net/frank_sons

mailto:[email protected]

http://slideshare.net/frank_sons

Code reviews

Software

Transcript of Code reviews