[ Pobierz całość w formacie PDF ]
Zaczynali porządkować swój kod przed przeglądem, dodając zrozumiałe
nazwy zmiennych oraz komentarze, zanim ich o to poproszono. DÅ‚ugie,
skomplikowane procedury stawały się krótkie i zrozumiałe.
Co więcej, nawyki nabyte w trakcie przeglądów kodu utrwaliły się. Po około
trzech miesiącach zmieniliśmy politykę przeprowadzania przeglądów kodu
tak, że każdy pracownik mógł ich dokonywać.
Jeśli jeden lub dwóch Twoich programistów regularnie nie zauważa istot-
nych kwestii w trakcie przeglądów, powinieneś kontrolować ich pracę za po-
mocą powiadomień o zmianie kodu (patrz: podrozdział 14. na stronie 128).
Monitorowanie powiadomień o zmianie kodu pozwala w prosty, nieinwa-
zyjny sposób mieć pod kontrolą każdego pracownika bez siedzenia w jego
biurze i zaglÄ…dania mu przez ramiÄ™.
124 Sprzedaj swój program
Czasami będziesz tak pochłonięty problemem, że przerwa na wykonanie
przeglądu kodu całkowicie wybiłaby Cię z rytmu. Próba powrotu do wła-
snego problemu kosztowałaby Cię mnóstwo czasu.
(Czy pamiętasz dyskusję o przerwaniach? Patrz: podrozdział 11. na stro-
nie 93). Jeśli ktoś przychodzi i prosi o przegląd kodu (zresztą również
o cokolwiek innego), kiedy jesteś pochłonięty problemem, powiedz mu, że
jesteś zajęty teraz, i poproś, żeby przyszedł pózniej. Z drugiej strony, jeśli
to Ty szukasz kogoś do przejrzenia kodu i ktoś mówi, że jest zajęty, daj mu
spokój i zaczekaj na lepszy moment lub idz do kogoś innego.
Większość pracy związanej z programowaniem to praca umysłowa jesteś
pochłonięty problemem, aż będzie on rozwiązany. Nie jest niegrzeczne
poprosić kogoś, aby przyszedł pózniej, jeśli jesteś w sytuacji, która wymaga
koncentracji. W niektórych firmach jest to naturalne, w innych całkiem obce.
Zawsze powinna być zrozumiana prośba, aby ktoś przyszedł za pół godziny
albo po obiedzie.
Wirtualne przeglÄ…dy kodu
Z czasem nauczysz się, na co zwracają szczególną uwagę poszcze-
gólne osoby przeglądające kod. Na przykład kiedyś Jarek napisał
skomplikowany fragment programu, który ku jego satysfakcji
działał. Następnie przeprowadził wirtualny przegląd , próbu-
jąc dostrzec, na co dwóch z jego najbardziej doświadczonych pra-
cowników może zwrócić uwagę. Po wprowadzeniu zmian, co do
których spodziewał się, że zostaną mu zaproponowane, pozwolił im
przeprowadzić rzeczywisty przegląd kodu. Byli zachwyceni! Prze-
glądali razem już tak dużo programów, że był w stanie przeana-
lizować kod z ich punktu widzenia. Jarek nauczył się, co dwaj
programiści (z wieloletnim doświadczeniem) sobie cenili, i był
w stanie wykorzystać to doświadczenie, aby poprawić swoją pracę.
Taki jest więc cel przeglądów kodu nie tylko opracowujesz do-
bre programy, ale również wychowujesz dobrych programistów.
Rozdział 3. " Pragmatyczne techniki projektowe 125
WSKAZÓWKA 17
Można mówić pózniej
Kierownictwo firmy powinno wymagać przeglądu kodu. Jeśli nie ma zale-
cenia kierownictwa, nikt z firmy nie ma oficjalnej motywacji, aby w tym
uczestniczyć. Innymi słowy, jeśli nikt nie dostał polecenia, aby Ci pomóc,
prawdopodobnie nie będzie miał dla Ciebie czasu, zwłaszcza jeśli terminy
są napięte.
Jeśli w firmie nie ma polityki obowiązkowych przeglądów kodu, także wtedy
możesz prosić swoich kolegów o przegląd. Nie cały zespół odniesie korzyść,
ale Twój własny kod będzie lepszy. Osoby dokonujące przeglądu Twojego
kodu z czasem nauczą się, jakie korzyści z tego płyną.
Nie czekaj zbyt długo na konkretną osobę, aż będzie miała czas, by doko-
nać przeglądu Twojego kodu; przejdz się po firmie i znajdz kogoś, kto nie
jest pochłonięty problemem. Idz na drugi koniec budynku, jeśli musisz, ale
znajdz kogoś. Jeśli znajdziesz kogoś, kto nigdy nie wykonywał przeglądu
kodu dla Ciebie, będzie to dla niego świetna okazja, aby dowiedzieć się,
co robisz.
Te szybkie przeglÄ…dy kodu promujÄ… wzajemnÄ… edukacjÄ™ bez dodatkowego
kosztu sformalizowanych programów. Zmieniając programistów, którzy do-
konują przeglądu Twojego kodu, osiągasz korzyść wynikającą z korzystania
z doświadczenia i wiedzy wielu różnych osób. Każdy przeglądający kod
wskaże różne sposoby rozwiązania tego samego problemu. Niektóre lepsze,
niektóre gorsze, ale zawsze różne.
WSKAZÓWKA 18
Zawsze przeglądaj cały kod
Celem jest kreatywne myślenie przy jednoczesnej poprawie jakości pro-
gramu. Naucz się spoglądać na swoje problemy z różnych punktów wi-
dzenia. Krótkie przeglądy kodu staną się wkrótce Twoją drugą naturą
126 Sprzedaj swój program
i podobnie jak z kuchenkami mikrofalowymi będziesz się zastana-
wiał, jak kiedyś bez nich żyłeś. Praktyczna dyskusja nad analizą algorytmu
lub ograniczeniami zasobów jest lekcją, która jest zapamiętywana, ponieważ
wiąże się z praktycznym i natychmiastowym zastosowaniem.
Zamiast uczyć się z książek, będziesz siedział przy biurku różnych fachow-
ców (niektórych świetnych, innych przeciętnych) będziesz uczył się
trochę od każdego z nich, dodawał ich triki do własnych, aż pewnego
dnia sam zostaniesz mistrzem.
Od czego zacząć?
Przeglądy kodu to wspaniałe narzędzie! Kiedy staną się one nawykiem,
[ Pobierz całość w formacie PDF ]