jdd2014: code review - jak zyskać więcej niż tracić? - sebastian malaca
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.TRANSCRIPT
![Page 1: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/1.jpg)
Code Review jak zyskać więcej niż tracić? Sebastian Malaca
![Page 2: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/2.jpg)
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]
![Page 3: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/3.jpg)
Code Review wpływa na:
• Jakość kodu i oprogramowania
![Page 4: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/4.jpg)
Code Review wpływa na:
• Jakość kodu i oprogramowania • Wymiana wiedzy nt. projektu
![Page 5: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/5.jpg)
Code Review wpływa na:
• Jakość kodu i oprogramowania • Wymiana wiedzy nt. projektu • Umiejętności developerów
![Page 6: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/6.jpg)
Code Review wpływa na:
• Jakość kodu i oprogramowania • Wymiana wiedzy nt. projektu • Umiejętności developerów • Czytelność kodu
![Page 7: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/7.jpg)
Code Review to:
• „Konstruktywne” komentarze
![Page 8: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/8.jpg)
Code Review to:
• „Konstruktywne” komentarze • Detale
![Page 9: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/9.jpg)
Code Review to:
• „Konstruktywne” komentarze • Detale • Niekończące się dyskusje
![Page 10: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/10.jpg)
Code Review to:
• „Konstruktywne” komentarze • Detale • Niekończące się dyskusje • Błędy „projektowe”
![Page 11: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/11.jpg)
Code Review to:
• „Konstruktywne” komentarze • Detale • Niekończące się dyskusje • Błędy „projektowe” • Wydłużony proces developmentu
![Page 12: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/12.jpg)
![Page 13: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/13.jpg)
Pozbyć się Code Review?!
![Page 14: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/14.jpg)
Wyrzuć to, co nie potrzebne!
• Standardy kodowania
![Page 15: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/15.jpg)
Wyrzuć to, co nie potrzebne!
• Standardy kodowania • Jałowe komentarze
![Page 16: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/16.jpg)
Wyrzuć to, co nie potrzebne!
• Standardy kodowania • Jałowe komentarze • Etap projektowania
![Page 17: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/17.jpg)
Co jest ważne, a co nie?
• Każdy pisze inaczej
![Page 18: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/18.jpg)
Co jest ważne, a co nie?
• Każdy pisze inaczej • Sugestie, a błędy
![Page 19: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/19.jpg)
Co jest ważne, a co nie?
• Każdy pisze inaczej • Sugestie, a błędy • Pomyśl, nim coś napiszesz
![Page 20: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/20.jpg)
Co jest ważne, a co nie?
• Każdy pisze inaczej • Sugestie, a błędy • Pomyśl, nim coś napiszesz • Nie pisz komentarzy!
![Page 21: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/21.jpg)
Co jest ważne, a co nie?
• Każdy pisze inaczej • Sugestie, a błędy • Pomyśl, nim coś napiszesz • Nie pisz komentarzy! • #whatever
![Page 22: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/22.jpg)
Kod, który warto oglądać
• Nie każda zmiana jest tak samo ważna
![Page 23: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/23.jpg)
Kod, który warto oglądać
• Nie każda zmiana jest tak samo ważna • Nie każdy musi oglądać Twój kod
![Page 24: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/24.jpg)
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
![Page 25: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/25.jpg)
Odwieczne problemy
• Post- czy pre- commit?
![Page 26: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/26.jpg)
Odwieczne problemy
• Post- czy pre- commit? • Z toolem czy bez?
![Page 27: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/27.jpg)
Odwieczne problemy
• Post- czy pre- commit? • Z toolem czy bez? • Kiedy poprawiać?
![Page 28: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/28.jpg)
Pułapki i problemy
• Komentarze, a niezrozumienie kodu
![Page 29: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/29.jpg)
Pułapki i problemy
• Komentarze, a niezrozumienie kodu • Cross-team review i pair programming
![Page 30: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/30.jpg)
Pułapki i problemy
• Komentarze, a niezrozumienie kodu • Cross-team review i pair programming • Kwestia sporne i arbiter
![Page 31: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/31.jpg)
Pytania ?!
![Page 32: JDD2014: Code review - jak zyskać więcej niż tracić? - Sebastian Malaca](https://reader033.vdocuments.net/reader033/viewer/2022052907/55933bb01a28abed2c8b471a/html5/thumbnails/32.jpg)
Dziękuję
http://sebastian-malaca.blogspot.com/ http://twitter.com/SebastianMalaca