Slide 1

Slide 1 text

Praha, 2. března 2013 David Majda @dmajda Josef Reidinger (twitterless) Code Reviews FTW!

Slide 2

Slide 2 text

Kontext 1

Slide 3

Slide 3 text

Proč? 2

Slide 4

Slide 4 text

lepší kód lepší design bus factor učení odhad kvality tlak okolí

Slide 5

Slide 5 text

Jak? 2

Slide 6

Slide 6 text

Úprava kódu Pull request Review Merge Přijetí Odmítnutí

Slide 7

Slide 7 text

Checklist 3

Slide 8

Slide 8 text

1. Je kód dobře čitelný? 2. Rozumím tomu, co kód dělá? 3. Skautské pravidlo 4. Jde to udělat lépe?

Slide 9

Slide 9 text

Kvalita kódu 4

Slide 10

Slide 10 text

Viděl jsem to už někde?

Slide 11

Slide 11 text

Rozšiřitelnost

Slide 12

Slide 12 text

Pokrytí testy

Slide 13

Slide 13 text

Ošetření chyb

Slide 14

Slide 14 text

Štábní kultura

Slide 15

Slide 15 text

Správná úroveň abstrakce?

Slide 16

Slide 16 text

Izolace změn

Slide 17

Slide 17 text

Chyby se rády druží!

Slide 18

Slide 18 text

Overengineering!

Slide 19

Slide 19 text

Sociální aspekty 5

Slide 20

Slide 20 text

No content

Slide 21

Slide 21 text

No content

Slide 22

Slide 22 text

No content

Slide 23

Slide 23 text

No content

Slide 24

Slide 24 text

Problémy 6

Slide 25

Slide 25 text

Velké větve

Slide 26

Slide 26 text

Nekonečný příběh

Slide 27

Slide 27 text

Jednejte, jako kdybyste daný kód od zítřka měli na starost!

Slide 28

Slide 28 text

http://www.doporucim.cz/mobile/DEVELREVIEWS