Implementierung erster Klasse - Code Qualität in PL/SQL

Größe: px

Ab Seite anzeigen:

Download "Implementierung erster Klasse - Code Qualität in PL/SQL"

Hermann Simen
vor 6 Jahren
Abrufe

1 Implementierung erster Klasse - Code Qualität in PL/SQL Stephan La Rocca Consulting Manager Berlin, April 26 th

2 > % Location Years old Colleagues Customers Oracle Partner USA Germany UK founded in 1999 well trained satisfied highly experienced customers in specialists 40 countries specialized in Oracle technologies 2

3 Motivation Wenn es einen Weg gibt, etwas besser zu machen: finde ihn. Thomas Alva Edison ( ), amerik. Erfinder 3

4 Definition Code Qualität Die Definition von Code-Qualität lässt sich unterscheiden nach: lesbar Lesbarkeit Layout, Darstellung, verständliche Benennungen testbar Code wartbar Testbarkeit Modularisierung, wie leicht lässt sich der Code automatisch testen Wartbarkeit Wie leicht können andere Entwickler die Software bearbeiten 4

5 Code 5

6 Metrics wartbar Eine Softwarequalitätsmetrik ist eine Funktion, die eine Software-Einheit in einen Zahlenwert abbildet, welcher als Erfüllungsgrad einer Qualitätseigenschaft der Software-Einheit interpretierbar ist. IEEE Standard 1061, 1998 [1] Maurice Howard Halstead Thomas J. McCabe 6

7 Halstead Trennung von Operatoren und Operanden wartbar create or replace function get_all_ s(p_adre_id in number) return varchar2 is v_ s varchar2(1000); begin for i in (select adem_ from adem_adressen_ where adem_adre_id=p_adre_id) loop if v_ s is not null then v_ s:=v_ s ';'; end if; v_ s:=v_ s i.adem_ ; end loop; return v_ s; end; 7

8 Halstead wartbar Größe des Vokabulars Unterschiedliche Operatoren (nt) Operanden (nd) Länge des Code Gesamtanzahl Operatoren (Nt) Operanden (Nd) N_Vocal N_Code Halstead-Volume := N_Code * log2(n_vocal) 8

9 Halstead Halstead-Volume := N_Code * log2(n_vocal) Code-Review (Komplexität) CRD := nt/2 * (Nd/nd) wartbar Code-Review (Aufwand) CRA := CRD * HV Zeit (Implementieren/Verstehen) := CRA/18 in Sekunden HV = 31*log2(20) = 92 CRD = 14/2 * (15/6)= 17,5 CRA = 17,5 * 92 = In 1:30 Min. verstanden 9

10 McCabe Anzahl der Binär-Verzweigungen wartbar create or replace function get_all_ s(p_adre_id in number) return varchar2 is v_ s varchar2(1000); begin for i in (select adem_ from adem_adressen_ where adem_adre_id=p_adre_id) loop if v_ s is not null then v_ s:=v_ s ';'; end if; v_ s:=v_ s i.adem_ ; end loop; return v_ s; end; 10

11 Npath-Complexity Anzahl der möglichen Durchläufe wartbar create or replace function get_all_ s(p_adre_id in number) return varchar2 is v_ s varchar2(1000); begin for i in (select adem_ from adem_adressen_ where adem_adre_id=p_adre_id) loop if v_ s is not null then v_ s:=v_ s ';'; end if; v_ s:=v_ s i.adem_ ; end loop; return v_ s; end; 11

12 CRAP testbar Change Risk Analysis and Predictions Cyclomatic Complexity Code Coverage in Prozent 12

13 CRAP testbar Quelle: 13

14 CRAP Code Coverage für PL/SQL testbar 14

15 Coding and Naming Convention lesbar create or replace function gea(p_1 in number) return varchar2 is ve varchar2(1000); begin for i in (select adem_ from adem_adressen_ where adem_adre_id=p_1) loop if ve is not null then ve:=ve ';'; end if; ve:=ve i.adem_ ; end loop; return ve; end; 15

16 Coding and Naming Convention lesbar create or replace function get_all_ s(p_adre_id in number) return varchar2 is v_ s varchar2(1000); begin for i in (select adem_ from adem_adressen_ where adem_adre_id=p_adre_id) loop if v_ s is not null then -- Trennzeichen nicht an die erste Stelle setzen v_ s:=v_ s ';'; end if; v_ s:=v_ s i.adem_ ; end loop; return v_ s; end; 16

17 Coding and Naming Convention lesbar Naming Convention Lesbarkeit, Reg. Expression, Kürzel, Sonderzeichen, Wiedererkennung Best Practise Globale Variablen, Magische Konstanten, vermeide DeadCode Number Precission Performance Advises System Protection Context Switches, Subqueries, Union SQL Injection, Potentielle Fehlerquellen 17

18 Coding and Naming Convention lesbar 18

19 Messung con Code Qualität Messbare Größen helfen für die Verbesserung: lesbar Lesbarkeit Audit Rules Code Testbarkeit Crap, Code Coverage Wartbarkeit Halstead, McCabe, testbar wartbar 19

20 Container für PL/SQL Datenbank Apex Forms Reports Skripts Packages, Prozesse Trigger, Units Triggers, Inside file Prozeduren, Libraries, Units system, *.sql Funktionen 20

21 SECTION ONE Maßnahmen für bessere Qualität 21

22 Maßnahmen für bessere Qualität 22

23 Maßnahmen für bessere Qualität Versionskontrolle Nutzen von Branches, Compiler Flags, gesicherten Ständen Kommentare Ein Entwickler verbringt mehr Zeit mit Source Code lesen als schreiben Coding Standards Etablieren Sie neben den Best Practise eigene Unternehmensstandards Qualität Analyse Dead Code und Redundanzen entfernen. Nutzen von Refakturierungen Profiling Performance Analysen, Kontext Switche, Unit Tests SQL Developer, OATS, Aufzeichnung von Testfällen, Testabdeckung 23

24 Best Practise Steven_Feuerstein_Best_Practice_PLSQL/ _Steven_Feuerstein_Best_Practice_PLSQL.pd f 24

25 Exemplarische Maßnahmen unterschiedlichster Ebene Bedeutsame Namen Kommentare Error Handling Unit Tests Verwende suchbare Warnen vor Error Handling ist Error FIRST, Fast, Independent, Repeatable, Namen Konsequenzen Handling nicht die Logik Self-Validating, Timely Funktionen Formatierung Abgrenzung Systems Eine Funktion für eine Guter Code ist wie ein Verlasse Dich auf nichts Erweitern, Refakturieren, Aufgabe guter Zeitungsartikel spezifische Erweiterung vs. Global Picture 25

26 Performance Checks Besser: FORALL I IN statt: INSERT INTO bulk_bind_beispiel VALUES (feld1(i), feld2(i), feld3(i)); FOR I IN LOOP INSERT INTO bulk_bind_beispiel VALUES (feld1(i), feld2(i), feld3(i)); END LOOP; Besser: IF v_kreditlimit > c_max_limit AND check_kreditlimit(v_kreditlimit) THEN... statt: IF check_kreditlimit(v_kreditlimit) AND v_kreditlimit > c_max_limit THEN... 26

27 Entwicklungsrichtlinien für das Unternehmen 27

28 Messung von Code Qualität Messbare Größen helfen für die Verbesserung: lesbar Lesbarkeit Audit Rules Code Testbarkeit Crap, Code Coverage Wartbarkeit Halstead, McCabe, testbar wartbar 28

29 Einbindung in Nightly Build Command Line Interface für Laden, Parsen, Bewerten 29

30 Einbindung in Nightly Build Select Script innerhalb HUDSON Auslösen von Fehlern für Benachrichtigung 30

31 Kontinuierliches Monitoring 31

32 Boy Scout Rule Try and leave this world a little better than you found it Robert Stephenson Smyth Baden-Powell 32

33 Prozess zur Qualität Standards Code Convention & Guidelines Ausbildung Übernahme vorhandener und entwickeln unternehmenseigener Vorgaben Mitarbeiter, Teams Kontinuierliche Auseinandersetzung mit dem Thema Qualität Automatisierung KPIs, Nightly Checks Aufzeichnen, visualisieren und sinnvolle Ziele definieren Company Goal Kontinuierliche Verbesserung der Software-Qualität Reviews Als Verbesserung Und nicht um die Mitarbeiter an den Pranger zu stellen 33

34 Einstieg 34

35 Aktivitäten der DOAG 35

36 Definition Code Qualität 36

Ähnliche Dokumente

Oracle 12c: Neuerungen in PL/SQL. Roman Pyro DOAG 2014 Konferenz

Oracle 12c: Neuerungen in PL/SQL Roman Pyro DOAG 2014 Konferenz Herrmann & Lenz Services GmbH Herrmann & Lenz Solutions GmbH Erfolgreich seit 1996 am Markt Firmensitz: Burscheid (bei Leverkusen) Beratung,