SlideShare ist ein Scribd-Unternehmen logo
1 von 30
Downloaden Sie, um offline zu lesen
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änge
Es 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ät
Brauchen 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 Werte
Mal 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 technical

excellence 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 Tipps
Rein 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!
frank.sons@starside.de
@FrankS
@PHPUGHH
http://phpughh.de

Weitere ähnliche Inhalte

Andere mochten auch

Presentatie profeten 2v12 2 afwijzing - vergeving vyu 05022012
Presentatie profeten 2v12 2 afwijzing  - vergeving vyu 05022012Presentatie profeten 2v12 2 afwijzing  - vergeving vyu 05022012
Presentatie profeten 2v12 2 afwijzing - vergeving vyu 05022012vineyardutrecht
 
Building Disciples in the Practice: Getting Started
Building Disciples in the Practice: Getting StartedBuilding Disciples in the Practice: Getting Started
Building Disciples in the Practice: Getting StartedAllan Carrington
 
Open Source for the greater good
Open Source for the greater goodOpen Source for the greater good
Open Source for the greater goodDavid Coallier
 
Cuestionariojornadadereflexion
CuestionariojornadadereflexionCuestionariojornadadereflexion
CuestionariojornadadereflexionJuan Castillo
 
やはり日本の技術基準適合証明はまちがっている?
やはり日本の技術基準適合証明はまちがっている?やはり日本の技術基準適合証明はまちがっている?
やはり日本の技術基準適合証明はまちがっている?Yoshitake Takata
 
PaaSで簡単Railsアプリを公開しよう!
PaaSで簡単Railsアプリを公開しよう!PaaSで簡単Railsアプリを公開しよう!
PaaSで簡単Railsアプリを公開しよう!Yoshitake Takata
 
Ds dunia seni visual thn 2 (2)
Ds dunia seni visual thn 2 (2)Ds dunia seni visual thn 2 (2)
Ds dunia seni visual thn 2 (2)Nor Amalina
 
Минкомсвязь об использовании ЕСИА кредитными организациями
Минкомсвязь об использовании ЕСИА кредитными организациямиМинкомсвязь об использовании ЕСИА кредитными организациями
Минкомсвязь об использовании ЕСИА кредитными организациямиVadim Andreev
 
Маркетинговое исследование "Качество питания", БЦ Трехпрудный, 2005
Маркетинговое исследование "Качество питания", БЦ Трехпрудный, 2005Маркетинговое исследование "Качество питания", БЦ Трехпрудный, 2005
Маркетинговое исследование "Качество питания", БЦ Трехпрудный, 2005Vadim Andreev
 
Comscore 2012 US Digital Futur in Focus
Comscore 2012 US Digital Futur in FocusComscore 2012 US Digital Futur in Focus
Comscore 2012 US Digital Futur in FocusMediamaispasque
 
RIAアーキテクチャー研究会 第3回 セッション4 Mvpvm pattern
RIAアーキテクチャー研究会 第3回 セッション4 Mvpvm patternRIAアーキテクチャー研究会 第3回 セッション4 Mvpvm pattern
RIAアーキテクチャー研究会 第3回 セッション4 Mvpvm patternMami Shiino
 
De Panter Presentatie Door Mick De Krom
De Panter Presentatie Door Mick De KromDe Panter Presentatie Door Mick De Krom
De Panter Presentatie Door Mick De KromMick
 
Regionaal Netwerk Februari 2010
Regionaal Netwerk Februari 2010Regionaal Netwerk Februari 2010
Regionaal Netwerk Februari 2010Johan Lapidaire
 
Jahresrückblick 2012 PHP Usergroup Hamburg
Jahresrückblick 2012 PHP Usergroup HamburgJahresrückblick 2012 PHP Usergroup Hamburg
Jahresrückblick 2012 PHP Usergroup HamburgFrank Sons
 
RESTful APIs and FRAPI
RESTful APIs and FRAPIRESTful APIs and FRAPI
RESTful APIs and FRAPIDavid Coallier
 
Esri Dev Summit 2009 Rest and Mvc Final
Esri Dev Summit 2009 Rest and Mvc FinalEsri Dev Summit 2009 Rest and Mvc Final
Esri Dev Summit 2009 Rest and Mvc Finalguestcd4688
 

Andere mochten auch (20)

Presentatie profeten 2v12 2 afwijzing - vergeving vyu 05022012
Presentatie profeten 2v12 2 afwijzing  - vergeving vyu 05022012Presentatie profeten 2v12 2 afwijzing  - vergeving vyu 05022012
Presentatie profeten 2v12 2 afwijzing - vergeving vyu 05022012
 
Building Disciples in the Practice: Getting Started
Building Disciples in the Practice: Getting StartedBuilding Disciples in the Practice: Getting Started
Building Disciples in the Practice: Getting Started
 
Open Source for the greater good
Open Source for the greater goodOpen Source for the greater good
Open Source for the greater good
 
Cuestionariojornadadereflexion
CuestionariojornadadereflexionCuestionariojornadadereflexion
Cuestionariojornadadereflexion
 
Shine Brighter!
Shine Brighter!Shine Brighter!
Shine Brighter!
 
やはり日本の技術基準適合証明はまちがっている?
やはり日本の技術基準適合証明はまちがっている?やはり日本の技術基準適合証明はまちがっている?
やはり日本の技術基準適合証明はまちがっている?
 
PaaSで簡単Railsアプリを公開しよう!
PaaSで簡単Railsアプリを公開しよう!PaaSで簡単Railsアプリを公開しよう!
PaaSで簡単Railsアプリを公開しよう!
 
Ds dunia seni visual thn 2 (2)
Ds dunia seni visual thn 2 (2)Ds dunia seni visual thn 2 (2)
Ds dunia seni visual thn 2 (2)
 
Минкомсвязь об использовании ЕСИА кредитными организациями
Минкомсвязь об использовании ЕСИА кредитными организациямиМинкомсвязь об использовании ЕСИА кредитными организациями
Минкомсвязь об использовании ЕСИА кредитными организациями
 
Маркетинговое исследование "Качество питания", БЦ Трехпрудный, 2005
Маркетинговое исследование "Качество питания", БЦ Трехпрудный, 2005Маркетинговое исследование "Качество питания", БЦ Трехпрудный, 2005
Маркетинговое исследование "Качество питания", БЦ Трехпрудный, 2005
 
Comscore 2012 US Digital Futur in Focus
Comscore 2012 US Digital Futur in FocusComscore 2012 US Digital Futur in Focus
Comscore 2012 US Digital Futur in Focus
 
RIAアーキテクチャー研究会 第3回 セッション4 Mvpvm pattern
RIAアーキテクチャー研究会 第3回 セッション4 Mvpvm patternRIAアーキテクチャー研究会 第3回 セッション4 Mvpvm pattern
RIAアーキテクチャー研究会 第3回 セッション4 Mvpvm pattern
 
Intro To Social Media
Intro To Social MediaIntro To Social Media
Intro To Social Media
 
De Panter Presentatie Door Mick De Krom
De Panter Presentatie Door Mick De KromDe Panter Presentatie Door Mick De Krom
De Panter Presentatie Door Mick De Krom
 
Regionaal Netwerk Februari 2010
Regionaal Netwerk Februari 2010Regionaal Netwerk Februari 2010
Regionaal Netwerk Februari 2010
 
Bpv 05022010
Bpv 05022010Bpv 05022010
Bpv 05022010
 
Jahresrückblick 2012 PHP Usergroup Hamburg
Jahresrückblick 2012 PHP Usergroup HamburgJahresrückblick 2012 PHP Usergroup Hamburg
Jahresrückblick 2012 PHP Usergroup Hamburg
 
RESTful APIs and FRAPI
RESTful APIs and FRAPIRESTful APIs and FRAPI
RESTful APIs and FRAPI
 
Code reviews
Code reviewsCode reviews
Code reviews
 
Esri Dev Summit 2009 Rest and Mvc Final
Esri Dev Summit 2009 Rest and Mvc FinalEsri Dev Summit 2009 Rest and Mvc Final
Esri Dev Summit 2009 Rest and Mvc Final
 

Ähnlich wie Was bringen Code Reviews wirklich? Wir sind doch schon agil.

Code Qualität in agilen Teams - code.talks Hamburg 2015
Code Qualität in agilen Teams - code.talks Hamburg 2015Code Qualität in agilen Teams - code.talks Hamburg 2015
Code Qualität in agilen Teams - code.talks Hamburg 2015Frank Sons
 
"Effektive Code Reviews" - Abendvortrag bei oose.de
"Effektive Code Reviews" - Abendvortrag bei oose.de"Effektive Code Reviews" - Abendvortrag bei oose.de
"Effektive Code Reviews" - Abendvortrag bei oose.deFrank Sons
 
Jede Veränderung braucht Ver-Rückte
Jede Veränderung braucht Ver-RückteJede Veränderung braucht Ver-Rückte
Jede Veränderung braucht Ver-RückteMartin Heider
 
Unit testing - Überblick für (Projekt) Manager
Unit testing - Überblick für (Projekt) ManagerUnit testing - Überblick für (Projekt) Manager
Unit testing - Überblick für (Projekt) ManagerFrank Sons
 
eparo – IA und agile Softwareentwicklung verbinden (Vortrag IA-Konferenz 2009...
eparo – IA und agile Softwareentwicklung verbinden (Vortrag IA-Konferenz 2009...eparo – IA und agile Softwareentwicklung verbinden (Vortrag IA-Konferenz 2009...
eparo – IA und agile Softwareentwicklung verbinden (Vortrag IA-Konferenz 2009...eparo GmbH
 
SEOkomm 2015 - SEO Grundlagen: Suchabsichten verstehen
SEOkomm 2015 - SEO Grundlagen: Suchabsichten verstehenSEOkomm 2015 - SEO Grundlagen: Suchabsichten verstehen
SEOkomm 2015 - SEO Grundlagen: Suchabsichten verstehenBianca Zang
 
Architektur Kata - Workshop Solutions
Architektur Kata - Workshop SolutionsArchitektur Kata - Workshop Solutions
Architektur Kata - Workshop SolutionsFrank Sons
 
Responsive Multichannel-E-Commerce: Vorgehen und Learnings aus der Praxis
Responsive Multichannel-E-Commerce: Vorgehen und Learnings aus der PraxisResponsive Multichannel-E-Commerce: Vorgehen und Learnings aus der Praxis
Responsive Multichannel-E-Commerce: Vorgehen und Learnings aus der PraxisRoberto Rizzi
 
Mobile Produktentwicklung – wie geht das? Konzeption, Design, Entwicklung, Ve...
Mobile Produktentwicklung – wie geht das? Konzeption, Design, Entwicklung, Ve...Mobile Produktentwicklung – wie geht das? Konzeption, Design, Entwicklung, Ve...
Mobile Produktentwicklung – wie geht das? Konzeption, Design, Entwicklung, Ve...Bokowsky + Laymann GmbH
 
Der Kunde ist vor Ihrer Haustüre
Der Kunde ist vor Ihrer HaustüreDer Kunde ist vor Ihrer Haustüre
Der Kunde ist vor Ihrer HaustüreFrank Fuchs
 
Agiles Lernen und Software Entwicklung das OSGi Code Camp 2010
Agiles Lernen und Software Entwicklung das OSGi Code Camp 2010Agiles Lernen und Software Entwicklung das OSGi Code Camp 2010
Agiles Lernen und Software Entwicklung das OSGi Code Camp 2010Christian Baranowski
 
SEODAY2016 - 10 SEO Coder Hooks
SEODAY2016 - 10 SEO Coder HooksSEODAY2016 - 10 SEO Coder Hooks
SEODAY2016 - 10 SEO Coder HooksConstantin
 
Kevin Indig - SEO and Growth Hacking
Kevin Indig - SEO and Growth HackingKevin Indig - SEO and Growth Hacking
Kevin Indig - SEO and Growth HackingKevin Indig
 
Steuerung von Internetagenturen internet-briefing
Steuerung von Internetagenturen internet-briefingSteuerung von Internetagenturen internet-briefing
Steuerung von Internetagenturen internet-briefingWalter Schärer
 
23 Dinge, die Sie über Software-Entwicklung in Teams wissen sollten.
23 Dinge, die Sie über Software-Entwicklung in Teams wissen sollten.23 Dinge, die Sie über Software-Entwicklung in Teams wissen sollten.
23 Dinge, die Sie über Software-Entwicklung in Teams wissen sollten.Stephan Schmidt
 
Legacy php - Sanieren oder Ablösen?
Legacy php  - Sanieren oder Ablösen?Legacy php  - Sanieren oder Ablösen?
Legacy php - Sanieren oder Ablösen?Johann-Peter Hartmann
 

Ähnlich wie Was bringen Code Reviews wirklich? Wir sind doch schon agil. (20)

Code Qualität in agilen Teams - code.talks Hamburg 2015
Code Qualität in agilen Teams - code.talks Hamburg 2015Code Qualität in agilen Teams - code.talks Hamburg 2015
Code Qualität in agilen Teams - code.talks Hamburg 2015
 
"Effektive Code Reviews" - Abendvortrag bei oose.de
"Effektive Code Reviews" - Abendvortrag bei oose.de"Effektive Code Reviews" - Abendvortrag bei oose.de
"Effektive Code Reviews" - Abendvortrag bei oose.de
 
Jede Veränderung braucht Ver-Rückte
Jede Veränderung braucht Ver-RückteJede Veränderung braucht Ver-Rückte
Jede Veränderung braucht Ver-Rückte
 
Unit testing - Überblick für (Projekt) Manager
Unit testing - Überblick für (Projekt) ManagerUnit testing - Überblick für (Projekt) Manager
Unit testing - Überblick für (Projekt) Manager
 
Advanced Continuous Integration
Advanced Continuous IntegrationAdvanced Continuous Integration
Advanced Continuous Integration
 
eparo – IA und agile Softwareentwicklung verbinden (Vortrag IA-Konferenz 2009...
eparo – IA und agile Softwareentwicklung verbinden (Vortrag IA-Konferenz 2009...eparo – IA und agile Softwareentwicklung verbinden (Vortrag IA-Konferenz 2009...
eparo – IA und agile Softwareentwicklung verbinden (Vortrag IA-Konferenz 2009...
 
SEOkomm 2015 - SEO Grundlagen: Suchabsichten verstehen
SEOkomm 2015 - SEO Grundlagen: Suchabsichten verstehenSEOkomm 2015 - SEO Grundlagen: Suchabsichten verstehen
SEOkomm 2015 - SEO Grundlagen: Suchabsichten verstehen
 
Architektur Kata - Workshop Solutions
Architektur Kata - Workshop SolutionsArchitektur Kata - Workshop Solutions
Architektur Kata - Workshop Solutions
 
Responsive Multichannel-E-Commerce: Vorgehen und Learnings aus der Praxis
Responsive Multichannel-E-Commerce: Vorgehen und Learnings aus der PraxisResponsive Multichannel-E-Commerce: Vorgehen und Learnings aus der Praxis
Responsive Multichannel-E-Commerce: Vorgehen und Learnings aus der Praxis
 
Das Pfadfinderprinzip in DevOps
Das Pfadfinderprinzip in DevOpsDas Pfadfinderprinzip in DevOps
Das Pfadfinderprinzip in DevOps
 
Mobile Produktentwicklung – wie geht das? Konzeption, Design, Entwicklung, Ve...
Mobile Produktentwicklung – wie geht das? Konzeption, Design, Entwicklung, Ve...Mobile Produktentwicklung – wie geht das? Konzeption, Design, Entwicklung, Ve...
Mobile Produktentwicklung – wie geht das? Konzeption, Design, Entwicklung, Ve...
 
DevOps jenseits der Tools
DevOps jenseits der ToolsDevOps jenseits der Tools
DevOps jenseits der Tools
 
Der Kunde ist vor Ihrer Haustüre
Der Kunde ist vor Ihrer HaustüreDer Kunde ist vor Ihrer Haustüre
Der Kunde ist vor Ihrer Haustüre
 
Agiles Lernen und Software Entwicklung das OSGi Code Camp 2010
Agiles Lernen und Software Entwicklung das OSGi Code Camp 2010Agiles Lernen und Software Entwicklung das OSGi Code Camp 2010
Agiles Lernen und Software Entwicklung das OSGi Code Camp 2010
 
Thementag 2023 06 Dieses Mal machen wir alles richtig - 9 Hacks für wandelbar...
Thementag 2023 06 Dieses Mal machen wir alles richtig - 9 Hacks für wandelbar...Thementag 2023 06 Dieses Mal machen wir alles richtig - 9 Hacks für wandelbar...
Thementag 2023 06 Dieses Mal machen wir alles richtig - 9 Hacks für wandelbar...
 
SEODAY2016 - 10 SEO Coder Hooks
SEODAY2016 - 10 SEO Coder HooksSEODAY2016 - 10 SEO Coder Hooks
SEODAY2016 - 10 SEO Coder Hooks
 
Kevin Indig - SEO and Growth Hacking
Kevin Indig - SEO and Growth HackingKevin Indig - SEO and Growth Hacking
Kevin Indig - SEO and Growth Hacking
 
Steuerung von Internetagenturen internet-briefing
Steuerung von Internetagenturen internet-briefingSteuerung von Internetagenturen internet-briefing
Steuerung von Internetagenturen internet-briefing
 
23 Dinge, die Sie über Software-Entwicklung in Teams wissen sollten.
23 Dinge, die Sie über Software-Entwicklung in Teams wissen sollten.23 Dinge, die Sie über Software-Entwicklung in Teams wissen sollten.
23 Dinge, die Sie über Software-Entwicklung in Teams wissen sollten.
 
Legacy php - Sanieren oder Ablösen?
Legacy php  - Sanieren oder Ablösen?Legacy php  - Sanieren oder Ablösen?
Legacy php - Sanieren oder Ablösen?
 

Mehr von Frank Sons

An Introduction to pair programming methods
An Introduction to pair programming methodsAn Introduction to pair programming methods
An Introduction to pair programming methodsFrank Sons
 
Introduction to architecture katas
Introduction to architecture katasIntroduction to architecture katas
Introduction to architecture katasFrank Sons
 
Pair programming
Pair programmingPair programming
Pair programmingFrank Sons
 
Effective Code Reviews (IPC 2018)
Effective Code Reviews (IPC 2018)Effective Code Reviews (IPC 2018)
Effective Code Reviews (IPC 2018)Frank Sons
 
Architecture Katas - How can we learn to create better architectures?
Architecture Katas - How can we learn to create better architectures?Architecture Katas - How can we learn to create better architectures?
Architecture Katas - How can we learn to create better architectures?Frank Sons
 
Creating a good architecture
Creating a good architectureCreating a good architecture
Creating a good architectureFrank Sons
 
Effective Code Reviews
Effective Code ReviewsEffective Code Reviews
Effective Code ReviewsFrank Sons
 
Imposter Syndrom - Lightning talk PHP Unconference 2015
Imposter Syndrom - Lightning talk PHP Unconference 2015Imposter Syndrom - Lightning talk PHP Unconference 2015
Imposter Syndrom - Lightning talk PHP Unconference 2015Frank Sons
 
Code Qualität in agilen Teams - Webmontag Hamburg
Code Qualität in agilen Teams - Webmontag HamburgCode Qualität in agilen Teams - Webmontag Hamburg
Code Qualität in agilen Teams - Webmontag HamburgFrank Sons
 
Das Hochstapler Syndrom
Das Hochstapler SyndromDas Hochstapler Syndrom
Das Hochstapler SyndromFrank Sons
 
Presenting to win - how I changed some old slides
Presenting to win - how I changed some old slidesPresenting to win - how I changed some old slides
Presenting to win - how I changed some old slidesFrank Sons
 
Selbstmotivation - vom Intro zum Extro
Selbstmotivation - vom Intro zum ExtroSelbstmotivation - vom Intro zum Extro
Selbstmotivation - vom Intro zum ExtroFrank Sons
 
Code Reviews - PHP Unconference Europe 2013
Code Reviews - PHP Unconference Europe 2013Code Reviews - PHP Unconference Europe 2013
Code Reviews - PHP Unconference Europe 2013Frank Sons
 
PHP Usergroup 2013/03/12
PHP Usergroup 2013/03/12PHP Usergroup 2013/03/12
PHP Usergroup 2013/03/12Frank Sons
 
Code Reviews - Vortrag für Innogames
Code Reviews - Vortrag für InnogamesCode Reviews - Vortrag für Innogames
Code Reviews - Vortrag für InnogamesFrank Sons
 
The Singleton Dilemma
The Singleton DilemmaThe Singleton Dilemma
The Singleton DilemmaFrank Sons
 
Unit Testing: Workshop Erfahrungen
Unit Testing: Workshop ErfahrungenUnit Testing: Workshop Erfahrungen
Unit Testing: Workshop ErfahrungenFrank Sons
 
"Es rechnet sich!" - Argumente für den Einsatz von PHPUnit
"Es rechnet sich!" - Argumente für den Einsatz von PHPUnit"Es rechnet sich!" - Argumente für den Einsatz von PHPUnit
"Es rechnet sich!" - Argumente für den Einsatz von PHPUnitFrank Sons
 
Code reviews - Leave your ego at the door
Code reviews - Leave your ego at the doorCode reviews - Leave your ego at the door
Code reviews - Leave your ego at the doorFrank Sons
 

Mehr von Frank Sons (20)

An Introduction to pair programming methods
An Introduction to pair programming methodsAn Introduction to pair programming methods
An Introduction to pair programming methods
 
Introduction to architecture katas
Introduction to architecture katasIntroduction to architecture katas
Introduction to architecture katas
 
Pair programming
Pair programmingPair programming
Pair programming
 
Effective Code Reviews (IPC 2018)
Effective Code Reviews (IPC 2018)Effective Code Reviews (IPC 2018)
Effective Code Reviews (IPC 2018)
 
Architecture Katas - How can we learn to create better architectures?
Architecture Katas - How can we learn to create better architectures?Architecture Katas - How can we learn to create better architectures?
Architecture Katas - How can we learn to create better architectures?
 
Creating a good architecture
Creating a good architectureCreating a good architecture
Creating a good architecture
 
Effective Code Reviews
Effective Code ReviewsEffective Code Reviews
Effective Code Reviews
 
Imposter Syndrom - Lightning talk PHP Unconference 2015
Imposter Syndrom - Lightning talk PHP Unconference 2015Imposter Syndrom - Lightning talk PHP Unconference 2015
Imposter Syndrom - Lightning talk PHP Unconference 2015
 
Code Qualität in agilen Teams - Webmontag Hamburg
Code Qualität in agilen Teams - Webmontag HamburgCode Qualität in agilen Teams - Webmontag Hamburg
Code Qualität in agilen Teams - Webmontag Hamburg
 
Das Hochstapler Syndrom
Das Hochstapler SyndromDas Hochstapler Syndrom
Das Hochstapler Syndrom
 
Code Review
Code ReviewCode Review
Code Review
 
Presenting to win - how I changed some old slides
Presenting to win - how I changed some old slidesPresenting to win - how I changed some old slides
Presenting to win - how I changed some old slides
 
Selbstmotivation - vom Intro zum Extro
Selbstmotivation - vom Intro zum ExtroSelbstmotivation - vom Intro zum Extro
Selbstmotivation - vom Intro zum Extro
 
Code Reviews - PHP Unconference Europe 2013
Code Reviews - PHP Unconference Europe 2013Code Reviews - PHP Unconference Europe 2013
Code Reviews - PHP Unconference Europe 2013
 
PHP Usergroup 2013/03/12
PHP Usergroup 2013/03/12PHP Usergroup 2013/03/12
PHP Usergroup 2013/03/12
 
Code Reviews - Vortrag für Innogames
Code Reviews - Vortrag für InnogamesCode Reviews - Vortrag für Innogames
Code Reviews - Vortrag für Innogames
 
The Singleton Dilemma
The Singleton DilemmaThe Singleton Dilemma
The Singleton Dilemma
 
Unit Testing: Workshop Erfahrungen
Unit Testing: Workshop ErfahrungenUnit Testing: Workshop Erfahrungen
Unit Testing: Workshop Erfahrungen
 
"Es rechnet sich!" - Argumente für den Einsatz von PHPUnit
"Es rechnet sich!" - Argumente für den Einsatz von PHPUnit"Es rechnet sich!" - Argumente für den Einsatz von PHPUnit
"Es rechnet sich!" - Argumente für den Einsatz von PHPUnit
 
Code reviews - Leave your ego at the door
Code reviews - Leave your ego at the doorCode reviews - Leave your ego at the door
Code reviews - Leave your ego at the door
 

Was bringen Code Reviews wirklich? Wir sind doch schon agil.

  • 1. Was bringen Code Reviews wirklich?
 Wir sind doch schon agil! Meetings, tools und agile Praktiken @FrankS
  • 2. 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
  • 3. 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
  • 4. Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014 Ein Blick auf die agilen Anfänge Es war einmal XP…
  • 5. 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
  • 6. 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
  • 7. Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014 Was kommt oft dabei raus? 7
  • 8. Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014 Code Review Ziele und agile Realität Brauchen wir Code Reviews?
  • 9. 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
  • 10. 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
  • 11. 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
  • 12. Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014 WTF / Minute 12
  • 13. Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014 Code Reviews und die agilen Werte Mal ins Agile Manifest schauen…
  • 14. Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014 Die core values… 14
  • 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.” 15
  • 16. 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
  • 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.” 17
  • 18. 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
  • 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.” 19
  • 20. 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
  • 21. 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
  • 22. Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014 Erfahrungen und Tipps Rein subjektiv natürlich…
  • 23. Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014 Richtig einsetzen ist wichtig Nachträgliche Reviews != agil 23 Continuous Reviews == agil
  • 24. 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
  • 25. 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
  • 26. 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
  • 27. Frank Sons - Was bringen Code Reviews wirklich?code.talks 09.10.2014 Dann es geht Hand in Hand 27
  • 28. 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
  • 29. 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