Jakie są moje najlepsze możliwości przeprowadzenia przeglądu kodu, ponieważ jestem "armią 1" w pracy? Najlepsze, co zrobiłem do tej pory, jest uruchamianie Sonar na moich projektach, ale myślę, że można zrobić więcej.Przegląd kodu podczas pracy solo
Chciałbym wyjść poza publikację i poprosić o opinię na temat małych fragmentów kodu. Dzięki.
Jedną z technik, którą możesz wypróbować, jest uruchomienie pliku różnic w każdym z plików źródłowych przed ich sprawdzeniem. Wystarczy, że uruchomisz narzędzie do porównywania w każdym pliku źródłowym i przejrzysz zmiany kodu. Poświęć chwilę, aby zastanowić się nad każdą zmianą tak jak byś gdybyś przeglądu kodu innego autora, a krótko zadać sobie pytanie:
Po chwili otrzymasz w zwyczaju przebiegającej przez tych szybkich opinii kodowych i może zagwarantować, że co najmniej spojrzenie na każdy z twoich zmian więcej niż raz.
Ta metoda działa lepiej, gdy zachowujesz kod i wprowadzasz małe zmiany - trudniej jest zmienić duże bloki kodu - ale ten poziom dyscypliny jest nadal cenny.
Zgadzam się, że następujące zalecenie może być pomocne, może być bardziej czymś, co robisz jako część drugiej natury, jeśli masz kod pod kontrolą wersji i masz zwyczaj przeprowadzania zmian za pomocą programu różnicowego przed ich wprowadzeniem. Wiem, że dla mnie, nawet gdy wiem, jakie dokładnie dokonałem zmian, przynajmniej raz sprawdziłem wyjście diff przed wprowadzeniem zmian. – ayaz
Rozwiązania nietechniczne: Może mógłbyś poszukać innej "armii 1" i wymieniać kod?
Alternatywnie, utwórz ogólne części o otwartym kodzie źródłowym, prześlij je i zaczekaj na raporty o błędach (prawdopodobnie dadzą lepsze wyniki).
Wątpię w "prawdopodobnie": twoja "część ogólna" musi być bardzo użyteczna dla innych programistów, a to nie jest naturalne w przypadku większości kodów. – gimpf
Chciałbym, gdybym mógł, jakieś sugestie? – vector
Uważam, że podejście oparte na otwartym kodzie źródłowym lepiej by działało dla niezależnych projektów z pewną wartością dla innych. W pracy jest to typ "ho-ham" :-( – vector
Istnieją społeczności przeglądu kodu, zobacz niektóre z moich starych question.
+1 Aby zaoszczędzić ci otwarcia 'pytanie', istnieje tylko jedna odpowiedź z tej strony: http: //www.refactormycode Dobrze wygląda –
@ Chris Knight - Ale oczywiście upvote moje pytanie tak czy inaczej :) – willcodejavaforfood
... Rzeczywiście, refactormycode może to zrobić :-), na pewno spróbuj! – vector
Nie sprecyzowałeś w swoich pytaniach, ale jeśli korzystasz z Javy, konfiguracja ściśle kontrolowana Checkstyle pomoże ci pozostać na linii prostej i wąskiej, jak również zidentyfikować obszary, w których twój projekt jest słaby, a refaktoryzacja byłaby korzystny.
... to jest Java i używam NB, ale wtyczka checkstyle jest naprawdę stara, uważam, że Sonar robi to samo. – vector
Wystarczy przejrzeć różnice, po pewnym czasie, korzystając z narzędzia do weryfikacji kodu, które pomoże Ci zidentyfikować, śledzić i naprawić problemy.
Podczas sprawdzania musisz założyć czapkę "recenzenta" i próbować krytykować swoje zmiany najlepiej jak potrafisz.
Z mojego osobistego doświadczenia wynika, że samoocena może być równie skuteczna jak samo-testowanie w identyfikowaniu problemów.
Uwaga: Jestem powiązany z firmą, która buduje wyżej powiązany produkt.
Proszę przejrzeć [FAQ] (http://stackoverflow.com/faq#promotion). "Ponadto, jeśli znaczna część twoich postów zawiera wzmiankę o twoim produkcie, jesteś tutaj z niewłaściwych powodów." 13 na 14 jest dość wysokie, szczególnie dla nowego użytkownika. Twoje posty są już oznaczane jako spam, więc staraj się wnosić swój wkład w inny sposób. –
Sprawdź http://area51.stackexchange.com/proposals/11464/code-review Możemy uzyskać przewagę, jeśli otrzymamy wystarczającą liczbę zatwierdzeń. – greatwolf