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?
Dziękujemy za uwagę Dywizja Południe SMT Software Spółka z ograniczoną odpowiedzialnością SKA ul. Piłsudskiego 13 50-048 Wrocław tel. +48 71 769 59 00 fax +48 71 769 59 01 Dywizja Centrum SMT Software Spółka z ograniczoną odpowiedzialnością SKA ul. Jutrzenki 183 02-231 Warszawa tel. +48 22 380 47 50 fax +48 22 380 47 51 Wrocław Warszawa Poznań Kraków Gliwice Katowice Białystok