Upload
proidea
View
121
Download
0
Embed Size (px)
DESCRIPTION
Krótkie przedstawienie tego, czym jest code review oraz tego, jakie są sposoby na realizowanie tej aktywności. Następnie poruszenie problematycznych kwestii, których często doświadczają zespoły deweloperskie (pochłaniania czasu, poziom szczegółowości, kiedy kod jest akceptowalny, itp.). Poza tym, wskazówki jak unikać tych problemów i jak usprawnić ten proces w swoim projekcie.
Citation preview
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