Code Quality in Practice

Größe: px

Ab Seite anzeigen:

Download "Code Quality in Practice"

Helmut Rosenberg
vor 5 Jahren
Abrufe

1 Code Quality in Practice Orientation in Objects GmbH Entwicklertag Karlsruhe Mai 2017 Version: 1.2 Weinheimer Str Mannheim

2 Ihr Sprecher Thorsten Maier Trainer, Berater, Entwickler Orientation in Objects GmbH Code Quality in Practice 2

3 Code Quality 3

4 TEAM 4

5 5

6 Ein gut funktionierendes Team aus adäquaten Mitarbeitern wird ein Projekt beinahe unabhängig vom Verfahren und der Technologie [ ] fertig stellen. Alistair Cockburn (Agiles Manifest) Orientation in Objects GmbH Code Quality in Practice 6

7 Der Gute-Laune- Macher Der Kommunikator Der Schaffer Das Organisationstalent und der Realist Der Technikfreak und Visionär 7

8 ARCHITEKTUR 8

9 Architektur Fundamentale Strukturen, Konzepte, Entscheidungen und Lösungsansätze... die man nicht mehr leicht los bekommt! 9

10 10

11 11

12 12

13 13

14 Theorie: Architekturen früh vergleichbar Praxis: Analyse wird während Schadensbegrenzung spät im Projekt durchgeführt 14

15 Szenariobasierte Architekturbewertung SAAM Software architecture analysis method ATAM Architecture tradeoff analysis method ACDM Architecture-centric design method 15

16 (1) Szenarien erheben und priorisieren (2) Erstellen der Architektur(en) (3) Bewertung aus dem Blickwinkel der Szenarien 16

17 Stakeholder Neues Krankenversicherungsprodukt Architektur 1 Architektur 2 Komponenten Aufwand Komponenten Aufwand 3 28d 2 20d DHL statt Hermes d Ergebnis 3 28d 6 32d 17

18 DOKU 18

19 19

20 Speziallogik Rechtekonzept Architektur ~10 Seiten Betrieb Starten Stoppen ~5 Seiten Endbenutzer ~50 Seiten DevOp DevOp Endbenutzer Orientation in Objects GmbH Code Quality in Practice 20

21 Markdown yuml.me Graphviz Umlet 21

22 Markdown Überschrift 1 ============= Dies ist ein Text. Es folgt eine Aufzählung: - Punkt 1 - *Kursiver* Punkt 2 - **Fetter** Punkt 3 Überschrift Spalte 1 Spalte Zeile 1... Zeile Orientation in Objects GmbH Code Quality in Practice 22

23 yuml.me [Company]<>-1>[Location] [Location]+->[Point] Orientation in Objects GmbH Code Quality in Practice 23

24 CODE 24

25 UserController 25

26 CRUDController UserController 26

27 ReadController CRUDController UserController 27

28 ReadController CRUDController CRUDButNotDeleteController UserController 28

29 CreateController ReadController UpdateController UserCreateController UserReadController UserUpdateController 29

30 Testarten Unit Test Integration Test UI Test 30

31 Testarten Unit Test Integration Test UI Test 30s+ Funktioniert mein Mock? 31

32 Testarten Unit Test Integration Test UI Test 30s+ Funktioniert mein Mock? 30s+ Das sieht der User 32

33 Testarten Unit Test Integration Test UI Test 1s Aussage über Codequalität! 30s+ Funktioniert mein Mock? 30s+ Das sieht der User 33

34 REVIEWS 34

35 Wer kennt das? Das ist historisch gewachsen. 35

36 noch ein Klassiker Thomas ist leider gerade im Urlaub 36

37 Es bleibt nie Zeit für das große Refactoring Code-Reviews als kontinuierliche Lösung 37

38 Bugs finden 38

39 Wissen 39

40 Entwicklungskosten in % Code-Review Entwicklung 20 0 ohne Code-Review mit Code-Review 40

41 Wissen Bugfixing Code-Review Entwicklung ohne Code-Review mit Code-Review 41

42 Jeder mit jedem! 42

43 Zeilen Gefundene Fehler 1 2 Stunden Dauer des Reviews 43

44 44

45 RETRO 45

46 Entwicklungsteam Code Qualität Management Kunde 46

47 ANALYSE 47

48 48

49 49

50 public class DemoServiceImpl implements DemoService private DemoController democontroller; Controller ServiceImpl } public void superservicemethod() { democontroller.dosomething(); } Orientation in Objects GmbH Code Quality in Practice 50

51 public class DepencencyTesterApplicationTests private DefaultListableBeanFactory public void checkdependencies() { for (String beanname : beanfactory.getbeandefinitionnames()) { for (String depname : beanfactory.getdependenciesforbean(beanname)) { checknotallowedconnection(beanname, depname, ".*ServiceImpl", ".*Controller"); checknotallowedconnection(beanname, depname, ".*Controller", ".*DaoImpl"); checknotallowedconnection(beanname, depname, ".*DaoImpl", ".*Controller"); } } } Controller ServiceImpl DaoImpl } private void checknotallowedconnection(string beanname, String depname, String beannamepattern, String dependencynamepattern) { String msg = "Connection not allowed: " + beanname + " -> " + depname; assertfalse(msg, beanname.matches(beannamepattern) && depname.matches(dependencynamepattern)); } 51

52 52

53 (Service) - [DEPENDS_ON] -> (Controller) 53

54 MATCH (s:service)-[:depends_on]->(c:controller) return s,c 54

55 55

56 17 Findings 56

57 git ratchet check 57

58 tl;dr; Die richtigen Leute Klare Struktur(en) 4-Augen-Prinzip Miteinander reden Automatische Codetests Automatische Strukturtests 58

59 ??? Fragen?? Orientation in Objects GmbH? Weinheimer Str Mannheim 59

60 Vielen Dank für Ihre Aufmerksamkeit! Orientation in Objects GmbH Weinheimer Str Mannheim

Ähnliche Dokumente

Code Quality in Practice

Code Quality in Practice Version: 1.0 Orientation in Objects GmbH Weinheimer Str. 68 68309 Mannheim www.oio.de info@oio.de Ihr Sprecher Thorsten Maier Trainer, Berater, Entwickler Feedback @ThorstenMaier