Continuous Integration i jakość kodu Michał Prajs

Agenda

Michał Prajs SMT Software Prowadzący Continuous Integration Jenkins CI Statyczna analiza kodu Checkstyle Pokrycie kodu testami jednostkowymi Cobertura

SMT Software Na rynku od 2002 roku Ponad 500 specjalistów IT 7 oddziałów na terenie kraju: Wrocław (siedziba główna), Warszawa, Poznań, Kraków, Gliwice, Katowice, Białystok Oddziały w Holandii, Francji, Wielkiej Brytanii Część grupy kapitałowej Grupa SMT S.A. notowanej na GPW Outsourcing IT Specjaliści Zespoły Usługi zarządzane Projekty informatyczne Dedykowane Online Mobilne Testy i audyty

Michał Prajs Współpracuje z SMT od ponad dwóch lat Programista i techniczny leader zespołu Specjalista Javy i frameworków na niej opartych Wprowadza do projektów praktyki pozwalające na łatwy rozwój i utrzymanie oprogramowania

Continuous Integration

Firma software’owa Czym się zajmuje? Kto tworzy software? Jak?

Zamierzchłe czasy Rozdzielamy projekt na zadania niezależne Integrujemy części wytworzone przez zespoły Same wady: Konieczność wczesnego zdefiniowania punktów styku między modułami Trudność wprowadzania zmian – mogą dotyczyć kilku zespołów Czasochłonność integracji i wysokie prawdopodobieństwo porażki

Obecnie, ale bez CI (z wykorzystaniem SCM) Rozdzielamy zadania tak jak poprzednio Centralne repozytorium kodu Kilka wad zostało Mała odporność projektu na zmiany Nadal trzeba zintegrować moduły

Continuous Integration Częste zmiany wymagań …a nawet samej architektury systemu Wybredni klienci Korzyści: Częste budowanie całego systemu Każde wgranie kodu do repozytorium sprawdzane (serwer CI robi update przed każdym buildem) Automatyczne uruchamianie testów jednostkowych i integracyjnych Gotowe artefakty na testy akceptacyjne, gotowy produkt Prowadzenie metryk „U mnie działa”? Pfffff…..

Build CI Sukces Archiwizacja kodu Archiwizacja wytworzonych artefaktów Statystyki metryk Porażka Informacja dla zespołu Poprawki na gorąco

No to do dzieła Repozytorium (SVN, CVS, TFS, GitHub, itp.) Narzędzie do budowania (Maven, Ant) Testy jednostkowe, integracyjne (JUnit, Mockito, PowerMock) Analiza statyczna kodu (Checkstyle, PMD) Serwer CI (Jenkins, Apache CI) Archiwizacja kodu Archiwizacja artefaktów Zadania powykonawcze (auto deploy)

Analiza statyczna kodu Checkstyle

Analiza statyczna kodu „Czystość kodu” Jednolity kod Czytelność kodu Rozszerzalność Udokumentowany kod Łatwość utrzymania/serwisowania Wskazuje miejsca nadające się do refactoringu

Checkstyle Konfiguracja kontroli standardowych Integracja z narzędziami do budowania (Maven plugin) Integracja z narzędziami do programowania (Eclipse plugin) Integracja z CI (Jenkins plugin) 133 standardowe czujki

Design klasy VisibilityModifier Wymusza enkapsulację FinalClass Wymusza modyfikator final w klasie, która ma tylko prywatne konstruktory InterfaceIsType Interface nie powinien jedynie definiować stałych HideUtilityClassConstructor Instancja takiej klasy nie ma sensu DesignForExtension Wymusza styl programowania, gdzie klasy nadrzędne pozostawiają miejsca, które mogą być nadpisane przez klasy podrzędne, czyli nieprywatna, niestatyczna metoda w klasie nadrzędnej musi być: Abstrakcyjna lub Finalna lub Mieć pustą implementację

Design klasy cd. MutableException Wykrywa gdy wyjątek może zmienić swój stan w trakcie cyklu życia ThrowsCount Gdy metoda rzuca wyjątki z różnych grup, prowadzi to do złych praktyk, jak łapanie ogólnych wyjątków, jak catch (Exception) InnerTypeLast

Metryki BooleanExpressionComplexity ClassDataAbstractionCoupling ClassFanOutComplexity CyclomaticComplexity NPathComplexity JavaNCSS

Pozostałe grupy Konwencje nazewnictwa Wymuszenie rozmiaru pliku/klasy/metody/linii Długość linii Ilość parametrów metody Polityka pustych znaków Wyrównanie

Testy jednostkowe

Testy jednostkowe Test Driven Development Testy jednostkowe Mockowanie obiektów Testy integracyjne

Motywacja Eliminacja błędów już podczas programowania Ograniczenie czasu potrzebnego na bugfixing Poprawność modyfikacji

Pokrycie kodu Procent w jakim kod jest sprawdzony przez testy jednostkowe Wszystkie rozgałęzienia Sytuacje wyjątkowe

Pytania? „Wolałbyś walczyć z kaczką wielkości konia, czy ze stoma końmi wielkości kaczki?”