Jak radzić sobie z kimś, kto nie lubi pomysłu na recenzje kodu?

Oczywiście, jeśli kierownictwo kupuje spędzać czas na przeglądaniu kodu, wszyscy muszą to zrobić.

Ale zawsze są tacy faceci (lub dziewczęta), którzy opierają się każdej uncji swojego istnienia.

W jaki sposób skutecznie radzisz sobie z tym scenariuszem, gdy masz do czynienia z recenzentem?

Opiera się ze strachu . Ta regulacja może być wynikiem wcześniej złe doświadczenia (ów) około poddawane przeglądowi, jako dziecko, w szkole, w pracy czy nawet w swojej obecnej drużyny. W naszych współczesnych społeczeństwach bardzo często mylimy czyjąś pracę z jego wartością jako istoty ludzkiej. Dlatego recenzje w pracy nie są dobrze postrzegane. Dlatego też przemawia publicznie w jednej z najbardziej rozpowszechnionych fobii (strach przed osądem).

Aby uniknąć takiego zachowania, potrzebujesz psychologii. Musisz udowodnić swojemu jaszczurkowemu mózgowi, że tak się nie stanie (nie zostanie osądzony, upokorzony, zabity, cokolwiek ...) poprzez znieczulenie go na recenzje kodów.

Jedną z najbardziej skutecznych metod, jaką znalazłem, aby odblokować kogoś, jest poproszenie go o sprawdzenie kodu , zanim poprosi o jego sprawdzenie.

Po chwili zaproponuj mu przeczytanie kodu, aby wyciągnąć z niego naukę i dlaczego nie zaproponować ulepszeń. Gdy znajdziesz coś do zmiany, zachowaj ostrożność podczas pisania. Zrozumie, że nie ma się czego bać i weźmie tylko pozytywną część procesu recenzowania: uczenie się i poszerzanie swojej wiedzy .

Spróbowałbym pracować w parach - połączyć kogoś, kto nienawidzi tego pomysłu, z kimś, kto mu się podoba, i poprosić, aby sprawdzili sobie nawzajem kod przez kilka tygodni. Oczywiście może to pomóc, ale nie musi, ale znajdowanie się na obu końcach przeglądu da przynajmniej bardziej zaokrąglony obraz procesu. Wspólna praca pary pozwoli im zapoznać się ze stylem i typowymi błędami, a także da im czas, aby rzeczywiście mogli się poprawić, a nie pieczątkę. Może to również pomóc w promowaniu programowania par w środowisku pracy, ponieważ myślę, że możesz zauważyć rosnącą tendencję do nie tylko sprawdzania, ale także przekodowywania, a nawet planowania i kodowania od zera.

Tak długo, jak bezinteresowne strony będą chciały spróbować, może to pomóc. Jeśli odmówią wzięcia tego pod uwagę, niewiele można z tym zrobić, dopóki są w zespole.

Odpowiedź @ Pierre jest na dobrej drodze dla kogoś, kto obawia się przeglądu kodu. Mogę sobie wyobrazić inną sytuację. Gwiezdny programista, który uważa przegląd kodu, to strata czasu, ponieważ kod osiąga akceptowalny standard jakości i poprawności. W tym przypadku mogą czuć, że przegląd kodu jest marnowaniem czasu i polowaniem na czarownice. (Jest to poszukiwanie problemu, gdy nie istnieje).

W takim przypadku zmienię cel przeglądu. Zamiast przeglądu kodu polegającego na znajdowaniu „problemów” w kodzie, należy traktować go jako poszukiwanie celów ponownego faktorowania lub potencjalnych przyszłych ulepszeń lub dodatkowych funkcji projektowych. W ten sposób zarówno koder, jak i recenzent biorą udział w tym procesie i mam nadzieję, że ten sprawny programista poczuje się, jakby dobrze wykorzystano czas.

Szczerze mówiąc, to pytanie nie ma sensu, jeśli masz dobrze zarządzany sklep:

1) Jeśli jest to część pracy, musisz to zrobić lub jesteś niesubordynowany. Ktoś, kto stanowczo odmawia wykonania części pracy, którą są zobowiązani, powinien otrzymać puszkę. Programowanie to rzemiosło i zawód - recenzenci i menedżerowie są po to, aby pomóc w wykonaniu pracy, a nie zajmować się rozpieszczonymi dziećmi (w każdym wieku).

2) Jeśli masz dobrze zarządzany system kontroli źródła (który jest koniecznością w każdym profesjonalnym sklepie z oprogramowaniem), to możesz sprawdzić jego kod, czy mu się to podoba, czy nie. Przejrzyj ich kod:

• Jeśli to dobrze, powiadom ich i poklep ich po plecach - to zachęci do uczestnictwa.

• Jeśli to nie jest dobre, daj im znać. Powinno to skutkować motywowaniem ich do uczestnictwa w celu obrony. Jeśli tak się nie stanie, możesz zastosować środki karne: kary finansowe, obniżenie statusu itp. Jeśli pomimo twoich wysiłków pracownik ten się nie pojawi, IMO ma złego pracownika i należy mu pokazać drzwi.

Czy mają jakieś negatywne doświadczenia w miejscach, w których recenzje kodu nie zostały wykonane prawidłowo? Mogą mieć uzasadnione obawy.

Jeśli absolutnie nie widzą żadnej wartości z tego ćwiczenia, poproś ich o cierpliwość i zobacz, co stanie się z ich kodem, a zwłaszcza z kodem innych (jeśli uważają, że są idealne).

Przegląd kodu „powinien” usprawnić rozwój, ale dopóki nie masz systemu, który faktycznie działa, dlaczego ktoś miałby to robić?

Osobiście uważam, że są walki, których nie da się wygrać ze 100% populacji.

Widzę wystarczająco dużo powodów, dla których programowanie w parach nie działałoby, gdy ktoś był zmuszony to zrobić.

Ale recenzje kodu są różne - zmieniają produktywność, niekoniecznie twoje nawyki pracy.

Kierownictwo może zrobić kilka rzeczy, aby zmniejszyć odporność ze względu na produktywność: 1) Zaakceptuj zmniejszenie prędkości dla wszystkich programistów. 2) Zapewnij odpowiednie narzędzia do zarządzania i łączenia wielu wersji z powodu cykli recenzji (np. Umożliwiając programistom posiadanie lokalnego repozytorium git) 3) Wymuszaj jakąś presję społeczną lub inną, aby zapewnić rozkład obciążenia, jakość i terminowość recenzji.

Jeśli to zrobią, uzasadnione jest wymaganie od wszystkich uczestnictwa, IMHO. Firma, dla której teraz pracuję, zmusza to globalnie - po prostu nie możesz się poddać bez zgody właściciela. I choć spowalnia to wszystko, zapobiega wielu wypadkom.

Zastosowaliśmy środki techniczne, aby przegląd kodu był obowiązkowy.

W sposobie, w jaki wprowadziliśmy przegląd kodu, w naszej kontroli źródła niemożliwe jest scalenie kodu, który nie został podpisany przez kogoś innego niż ten, który go wypchnął.

Zwolnij ich

To takie proste - albo dostają projekt samotnego człowieka, albo muszą iść. Odsuń je od swojego zespołu. Nie tylko nie wywiązują się ze swojej roli, niszczą morale i praktyki zespołu.

Teraz, jeśli wydaje się, że musisz zwolnić 50% swojego zespołu, to ...

Rozumiesz

Dlaczego niektórzy odmawiają? Czy nie mają czasu? Czy są wypaleni? Czy recenzje dotyczą czegoś, z czym nie mają doświadczenia? Czy uważają, że to strata czasu, jeśli tak, to dlaczego?

Pomoże w tym metodyka zwinna - zakładam, że stale pracujesz przeciwko silosom (tj. W celu zmniejszenia współczynnika autobusu), a osoby w twoim zespole są zaangażowane w to, co robią inni.

Pracuj nad tym, aby indywidualne żądania scalania były dość małe. Jeśli jest to więcej niż jeden ekran zmian, potrzebuje rozmowy w trybie stand-up lub błyskawicy, aby wyjaśnić, co się dzieje. Jeśli ma 10 stron, potrzebuje prezentacji ze slajdami i schematami architektury.

Czy wszyscy, o których mowa, pracują nad tym samym projektem?

Czy projekt jest już pochowany pod górą długu technicznego?

Czy wierzą w projekt i ciągłe doskonalenie?