Code Review jak zyskać więcej niż tracić? Sebastian Malaca

Co to jest Code Review?

Code Review jest techniką polegającą na systematycznym przeglądzie kodu źródłowego. Ma na celu znalezienie i poprawienie błędów przeoczonych na etapie jego pisania, poprawiając tym samym jakość oprogramowania oraz umiejętności programisty.

[Wikipedia.org]

Code Review wpływa na:

•  Jakość kodu i oprogramowania

Code Review wpływa na:

•  Jakość kodu i oprogramowania •  Wymiana wiedzy nt. projektu

Code Review wpływa na:

•  Jakość kodu i oprogramowania •  Wymiana wiedzy nt. projektu •  Umiejętności developerów

Code Review wpływa na:

•  Jakość kodu i oprogramowania •  Wymiana wiedzy nt. projektu •  Umiejętności developerów •  Czytelność kodu

Code Review to:

•  „Konstruktywne” komentarze

Code Review to:

•  „Konstruktywne” komentarze •  Detale

Code Review to:

•  „Konstruktywne” komentarze •  Detale •  Niekończące się dyskusje

Code Review to:

•  „Konstruktywne” komentarze •  Detale •  Niekończące się dyskusje •  Błędy „projektowe”

Code Review to:

•  „Konstruktywne” komentarze •  Detale •  Niekończące się dyskusje •  Błędy „projektowe” •  Wydłużony proces developmentu

Pozbyć się Code Review?!

Wyrzuć to, co nie potrzebne!

•  Standardy kodowania

Wyrzuć to, co nie potrzebne!

•  Standardy kodowania •  Jałowe komentarze

Wyrzuć to, co nie potrzebne!

•  Standardy kodowania •  Jałowe komentarze •  Etap projektowania

Co jest ważne, a co nie?

•  Każdy pisze inaczej

Co jest ważne, a co nie?

•  Każdy pisze inaczej •  Sugestie, a błędy

Co jest ważne, a co nie?

•  Każdy pisze inaczej •  Sugestie, a błędy •  Pomyśl, nim coś napiszesz

Co jest ważne, a co nie?

•  Każdy pisze inaczej •  Sugestie, a błędy •  Pomyśl, nim coś napiszesz •  Nie pisz komentarzy!

Co jest ważne, a co nie?

•  Każdy pisze inaczej •  Sugestie, a błędy •  Pomyśl, nim coś napiszesz •  Nie pisz komentarzy! •  #whatever

Kod, który warto oglądać

•  Nie każda zmiana jest tak samo ważna

Kod, który warto oglądać

•  Nie każda zmiana jest tak samo ważna •  Nie każdy musi oglądać Twój kod

Kod, który warto oglądać

•  Nie każda zmiana jest tak samo ważna •  Nie każdy musi oglądać Twój kod •  Każdy powinien oglądać kod

Odwieczne problemy

•  Post- czy pre- commit?

Odwieczne problemy

•  Post- czy pre- commit? •  Z toolem czy bez?

Odwieczne problemy

•  Post- czy pre- commit? •  Z toolem czy bez? •  Kiedy poprawiać?

Pułapki i problemy

•  Komentarze, a niezrozumienie kodu

Pułapki i problemy

•  Komentarze, a niezrozumienie kodu •  Cross-team review i pair programming

Pułapki i problemy

•  Komentarze, a niezrozumienie kodu •  Cross-team review i pair programming •  Kwestia sporne i arbiter

Pytania ?!

Dziękuję

http://sebastian-malaca.blogspot.com/ http://twitter.com/SebastianMalaca