Inspekcje kodu - Code Review

Code Review jest jednym z elementów metodologii Extreme Programming (XP), często zastępuje programowanie w parach.

Polega na

  • testowaniu czy dana funkcjonalność działa zgodnie z założeniami
  • przejrzeniu zmian, czy są adekwatne (nazwy zmiennych, funkcji, podział na klasy itd)
  • sprawdzamy tylko zmiany w kodzie dotyczące danego ticketa, bez sprawdzania starego kodu
  • sprawdzamy czy program spełnia wymagania formalne http://wikinew.infocity.pl/Default.aspx?Page=Standardy-kodowania&NS=IT

Zalety

  • każdy z programistów orientuje się w całości projektu
  • dodatkowy poziom testowania
  • lepszy kod, mniej błędów funkcjonalnych, logicznych oraz merytorycznych
  • większy przepływ wiedzy w zespole

Jak zrealizować praktycznie:

  • po zakończeniu prac nad danym ticketem, zmieniamy typ ticketa na code review
  • przypisujemy go do osoby, która powinna sprawdzić kod
  • sprawdzamy zmiany wprowadzone do źródeł (numery rewizji sa w komentarzach na track'u)
  • wpisujemy uwagi jako komentarz i przypisujemy do osoby realizującej dany ticket
  • powtarzamy całą procedurę aż nie będzie uwag