Post on 01-Jul-2015
description
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