Zobacz, jak robi to Facebook z własną aplikacją o nazwie phabricator: http://phabricator.org/
Zasadniczo zatwierdzają dla poszczególnych numerów, a dla każdego numeru wyświetlany jest kod, który ktoś ma sprawdzić. Kod nie trafia do ich głównego repozytorium, dopóki recenzent nie stwierdzi, że jest to w porządku.
Myślę, że to sprawia, że jest to bardziej zabawne.
Być może kod powinien być przypisany dwóm osobom: jednej, która to robi, i drugiej, która go recenzuje.
Chociaż być może twoi koledzy z drużyny nie wierzą w tę recenzję.
Osobiście, z braku recenzentów, użyłem testów jednostkowych dla funkcji niższego poziomu i „testu woźnego” dla całej reszty: test woźnego jest tak nazywany, ponieważ nawet woźny powinien być w stanie zrozumieć Twój kod.
Zwykle usuwałem niektóre drobne części, takie jak nawiasy blokowe / funkcyjne, notki widoczności, czasem nawet typy, i pokazywałem to menedżerom, ekspertom domeny, partnerom, którzy poprosili o kod: „czy tego chcesz?”
Pomaga też osobiście udać się tam i nie wyjechać, dopóki nie dokona się przeglądu.
Lub, jeśli nie masz nic przeciwko zespołowi lub nie są w porządku, wiesz, „jeśli możesz zmienić firmę, zmienić firmę” ...
Masz wiele problemów do rozwiązania - musisz zdobyć serca i umysły oraz upewnić się, że masz czas na recenzje kodu.
Druga część jest prawdopodobnie najłatwiejsza - zgadzasz się (łącznie i to musi obejmować zarządzanie), że pierwszą rzeczą, którą programista robi każdego ranka, są recenzje kodu - jest to proste, zrozumiałe, skuteczne i daje ci fajny kij do pokonania ludzi jeśli nie są zgodne. Co więcej, niczego nie przerywasz, nie prosisz ich, aby przestali pracować nad swoim kodem, nie prosisz ludzi o wyciśnięcie czegoś z listy rzeczy do zrobienia ...
Pierwsza część to prawdziwy problem - uczestnicy procesu sprawdzania muszą widzieć, że ma wartość, w przeciwnym razie nigdy nie zrobią przeglądu kodu (który jest postrzegany jako nie mający wartości), gdy mogą pisać kod lub naprawiać błędy (które jest z pewnością ważniejsze ...?).
Jeśli umiesz połączyć te dwa elementy - po pierwsze upewniając się, że wszyscy wierzą (lub rozumieją), że w recenzjach kodu ma wartość - w najprostszym przypadku powinno to oznaczać mniej błędów, co oznacza więcej nowego kodu, co zwykle sprawia więcej zabawy, a następnie po drugie porządkowanie rzeczy, aby w harmonogramie było wolne miejsce na przegląd kodu, mam nadzieję, że wydarzy się coś dobrego ... stanie się częścią kultury.
Kiedyś będzie to część kultury, może nie być już konieczne mówienie „pierwsza rzecz każdego dnia” - ale powiedziawszy, że uważam, że pasuje do wzorca, w którym prawdopodobnie chce pracować deweloper.
źródło
W większości firm, dla których pracowałem, masz 3 dni na dokończenie recenzji. Niedopuszczalne jest nie robienie recenzji. To część twojej pracy. Jeśli nie zrobisz przyzwoitych recenzji na czas, wpłynie to na twoją ocenę wydajności. I tak, recenzje zawsze wydają się zdarzać w najbardziej nieodpowiednich momentach. Szkoda, naucz się uwzględniać czas oceny w swoich prognozach. W każdym razie, jeśli kierownictwo naprawdę wierzy, że recenzje są ważne (tj. Nakazują przeglądanie całego kodu), wówczas zastosowaliby podobną politykę. Dodatkowo, jeśli ludzie nie ukończą przeglądu w wyznaczonym czasie, oznacza to akceptację materiału.
źródło
Rozważ skorzystanie z narzędzia takiego jak komisja recenzująca . Jest to bardzo pomocne, szczególnie w przypadku długich recenzji.
Możesz przesłać swoje różnice i poczekać, aż recenzent zakończy swoją recenzję. Jeśli masz otwarte recenzje, które uniemożliwiają ci kontynuowanie pracy, możesz to zgłosić podczas codziennych spotkań (Twój zespół chce, aby nowe funkcje zostały sprawdzone, aby można je było jak najszybciej przetestować, prawda?).
źródło
Kilka punktów do dodania, których nie ma w innych odpowiedziach.
Kod do przejrzenia musi zostać zameldowany
Blokowanie zadań ma pierwszeństwo, dlatego recenzje kodu powinny mieć pierwszeństwo przed innymi pracami (ale starając się nie przerywać przepływu). Jako programista powinieneś poprosić innych o sprawdzenie Twojego kodu (ponieważ chcesz go ulepszyć). Mając tę wiedzę, należy niezwłocznie wykonywać recenzje dla innych osób.
Trudniejsze pytania dotyczą tego, kiedy i jak dobrze wykonywać recenzje kodu.
Reguła, która zadziałała dla nas w tym momencie, jest taka, że kod dzielony musi zostać sprawdzony, ponieważ ma szerszy wpływ, podczas gdy w kodzie dla pojedynczej aplikacji (szczególnie biorąc pod uwagę, że używamy programowania opartego na testach) jest mniej ważny.
źródło