Embed Size (px)
DESCRIPTION
Code Reviews werden häufig eingesetzt, um Bugs im Code zu finden. Aber welche Probleme findet man wirklich? Kosten Code Review Meetings in der heutigen, agilen Umgebung nicht auch einfach zu viel Zeit? Pair Programming und vielleicht noch ein passendes tool sind doch schon mehr als genug um die Fehler zu finden, oder nicht? Der Vortrag geht auf die unterschiedlichen Möglichkeiten und Ziele von Code Reviews ein und zeigt auf, welchen Nutzen es auch in einem agilen Umfeld haben kann.
Citation preview
Was bringen Code Reviews wirklich?Wir sind doch schon agil!
Meetings, tools und agile Praktiken
@FrankS
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Worum geht’s?
Code Review tools vs. meetings
Code Review != Code Review
Code Reviews und die agile Welt
Also: Your mileage may vary!
2
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Und das ist der Plan:
Ein Blick auf die agilen Anfänge
Code Reviews: Ziele und Realität
Code Reviews und die agilen Werte
Erfahrungen und Tipps
3
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Ein Blick auf die agilen AnfängeEs war einmal XP…
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
So sieht man das in XP…
Wir machen pair programming!
Wir machen Unit tests!
Wir haben Collective Code Ownership!
Wir brauchen keine extra Code Reviews!
5
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Ausserdem macht das eh nur Probleme…
“Bin grad mitten im code, jetzt nicht”
“Wir diskutieren da eh nur rum”
“Dein Code ist auch nicht viel besser!”
“Können wir nicht ausfallen lassen?”
6
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Was kommt oft dabei raus?
7
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Code Review Ziele und agile RealitätBrauchen wir Code Reviews?
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Typische Ziele für Code Reviews
Standards einhalten
Wissen weitergeben
Auf korrekte Funktion prüfen
Bugs finden
9
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Wie war das noch mal in der agilen Welt?
Standards = ?
Wissen = Pair Programming
Korrekte Funktion = TDD
Bugs = kurze Iterationen
10
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Was finden Code Review (meetings) den wirklich?
Zeigen Diskussionsbedarf auf
Unklare Standards
Schlechte Architektur
Ab und zu auch einige gute WTFs
11
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
WTF / Minute
12
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Code Reviews und die agilen WerteMal ins Agile Manifest schauen…
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Die core values…
14
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Principle #06 - http://agilemanifesto.org/principles.html
“The most efficient and effective method of conveying information to and within a development team
is face-to-face conversation.”
15
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Principle #06 - http://agilemanifesto.org/principles.html
“The most efficient and effective method of conveying information to and within a development team is face-to-face conversation.”
16
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Principle #09 - http://agilemanifesto.org/principles.html
“Continuous attention to technicalexcellence and good design enhances agility.”
17
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Principle #09 - http://agilemanifesto.org/principles.html
“Continuous attention to technical excellence and good design enhances agility.”
18
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Principle #11 - http://agilemanifesto.org/principles.html
“The best architectures, requirements, and designs emerge from self-organizing teams.”
19
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Principle #11 - http://agilemanifesto.org/principles.html
“The best architectures, requirements, and designs emerge from self-organizing teams.”
20
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Also was bringen Code Reviews nun?
Gemeinsame Werte
Wissensaustausch
Etablieren von Standards
Interaktionen!
21
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Erfahrungen und TippsRein subjektiv natürlich…
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Richtig einsetzen ist wichtig
Nachträgliche Reviews != agil
23
Continuous Reviews == agil
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Wie läuft das bei uns?
Alle 2 Wochen “in a big round”
Code Review ständig anpassen
30min jeden Tag mit tool
Review in der Definition of Done
24
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Unsere Learnings
Gemeinsames Verständnis
Bessere Architektur
Alle wollen Reviews
Meetings vs. Tools?
25
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Unsere Learnings
Gemeinsames Verständnis
Bessere Architektur
Alle wollen Reviews
Meetings und Tools!
26
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Dann es geht Hand in Hand
27
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
Das kann helfen:
Mitte der Iteration - 2h
Täglich fixer Termin
Ständig anpassen
Pair Programming hilft
28
Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014
- Jeff Sutherland (co-founder von Scrum)
“Code Review is necessary for scrum”
29
Vielen Dank!
@FrankS
@PHPUGHH
http://phpughh.de
