Schön is anders 2 Jahre Getränkeautomaten als Einstellungstest

Transkript

1 Schön is anders 2 Jahre Getränkeautomaten als Einstellungstest Jan Leßner jan.lessner@bertelsmann.de arvato.de Schön is anders Jan Leßner 1

2 Inhalt Die Aufgabe Schulnoten aus zwei Jahren Live-Review (Eure Benotung ist gefragt) Erstaunliche wiederkehrende Schwachpunkte (Vorsicht: Experimente!) Ganz finstere Abgründe (Keine Namen!) Was tun? (Ideen? Vorschläge? Erfahrungen?) Schön is anders Jan Leßner 2

3 Die Aufgabe Zentrale Geschäftslogik eines Getränkeautomaten mit Wechselgeldfunktion GetränkUndWechselgeld kaufen (Getränkewunsch auswahl, Münze... einzahlung); Elegante, objektorientierte, an fachlichen Begriffen orientierte Lösung gewünscht Nur augenscheinlich notwendige Flexibilität einbauen, kein Over-Engineering Keine Persistenz, keine Oberfläche 2-4 Stunden Zeit (auf Wunsch auch länger, Studenten sowieso) Bewusst gewählte Herausforderungsmischung: Enterprise, nur JSE, OO, ein bisschen Algorithmik Schön is anders Jan Leßner 3

4 * Studenten wurden nach gleichem Maßstab bewertet. Sehr gute Noten waren daher nicht die Erwartung. ** Schwere Macke an einer Stelle nicht durch vorbildliches Design an anderer Stelle wieder gut zu machen Schulnoten: 32 Personen aus 2 Jahren ** Studenten* (8 Personen) Professionelle Entwickler Schön is anders Jan Leßner 4

5 Live-Review was meint ihr? Entwickler 06 Lösung von März 2015 Software-Entwickler mit mindestens 10 Jahren Berufserfahrung Schon von früher bekannt Eigentlich guter Ruf Bewandert in vielen Java-Technologien Briefing vorab: Wir brauchen mehr Qualität als früher Mal schauen Schön is anders Jan Leßner 5

6 Wiederkehrende Macken Schön is anders Jan Leßner 6

7 Wiedergänger: Getränke-Enum Entwickler 08 public enum Getraenk { COLA, SPRITE, FANTA, BIER } Code muss bei Änderung des Sortiments und für unterschiedlich bestückte Automaten geändert werden. Passt nicht zur Aufgabenstellung Für die Münzen ist das OK Schön is anders Jan Leßner 7

8 Widergänger: Fehlerbehandlung Entwickler 01 Keine Exceptions oder schlechte Ausnutzung des Konzepts Statt dessen: Teilinitialisierte Rückgabeobjekte String oder boolean als Rückgabe Fehler nur auf Stdout / Stderr public class AutomatAusgabe { private Fehler fehler; private Getraenk getraenk; private List<Muenze> rueckgeld; } public enum Fehler { AUTOMAT_LEER("Automat ist leer"), KEIN_WECHSELGELD("Kein Wechselgeld"),... } Schön is anders Jan Leßner 8

9 Widergänger: Englisch Entwickler 04 Ca. ¼ der Lösungen liegen auf Englisch vor Alle Entwickler sind aber Deutsche Kein intuitives Gefühl, ob der Code für andere lesbar ist private List<Coin> calcreturnmoney(int sum) throws ProcessException { int remainingmoney = sum; List<Coin> coins = new ArrayList<Coin>(); for (CoinStorage storage : coinstorages) { int value = storage.getcoin().getvalue(); if (remainingmoney < value) {... Und jetzt ein Experiment Schön is anders Jan Leßner 9

10 Weitere Wiedergänger Entwickler 02 Hybrides Getränk : Getränk? Getränkeart? Getränkewunsch? Fehlende Transaktionalität OO-Armut Schlechte Namen Redundanz Komplizierte Methoden Spagetti-Code Mehr davon jetzt beim Schön is anders Jan Leßner 10

11 Schön is anders Jan Leßner

12 Entwickler 05 Effective Java des Grauens Schön is anders Jan Leßner 12

13 Student 04 Automatisches Automatenprogramm Schön is anders Jan Leßner 13

14 Entwickler 11 Die Geisterimplementierung Schön is anders Jan Leßner 14

15 Reden in Zungen Schön is anders Jan Leßner 15

16 Entwickler 16 public GetraenkUndWechselgeld kaufen(getraenk getraenk, Muenze... einzahlung) { if (getraenkebestand.aktuell(getraenk) <= 0) { // genug Bestand throw new BestandZuNiedrigException(getraenk); }... Schön is anders Jan Leßner 16

17 Student 01 Schön is anders Jan Leßner 17

18 Entwickler 11 public void flaschenentleeren() {... } Schön is anders Jan Leßner 18

19 Stunden 01 public void muenzenentleeren() {... } Schön is anders Jan Leßner 19

20 Entwickler 18 public void emptycoins() {... } Schön is anders Jan Leßner 20

21 Entwickler 18 public void emptycoins() { coins.clear(); } Schön is anders Jan Leßner 21

22 public AutomatAusgabe kaufen(getraenk getraenkwunsch, Muenze... einwurf) { if (getraenkwunsch == null) return new AutomatAusgabe(KEIN_GETRAENK_GEWAEHLT); Entwickler 18 if (einwurf == null einwurf.length == 0) return new AutomatAusgabe(Fehler.KEIN_GELD_EINGEZAHLT); if (this.waren == null this.waren.isempty()) return new AutomatAusgabe(AUTOMAT_LEER); // Getraenk im Automat vorhanden? AutomatPosten ware = null; for (AutomatPosten p : this.waren) if (p.getraenk().equals(getraenkwunsch)) ware = p; if (ware == null) return new AutomatAusgabe(Fehler.GETRAENK_UNBEKANNT); if (ware.menge() <= 0) return new AutomatAusgabe(Fehler.GETRAENK_AUSVERKAUFT); final int summeeinwurf = summeeingeworfenermuenzen(einwurf); if (ware.preis().cents() > summeeinwurf) return new AutomatAusgabe(Fehler.BETRAG_GERING); if (ware.preis().cents() == summeeinwurf) { ware.mengereduzieren(); return new AutomatAusgabe(ware.getraenk()); } // Muenzen absteigend sortieren Collections.sort(muenzen, Collections.reverseOrder()); // Wechselgeld berechnen int rest = summeeinwurf - ware.preis().cents(); ArrayList<Muenze> wechselgeld = new ArrayList<Muenze>(); ArrayList<Muenze> uebrigemuenzen = new ArrayList<Muenze>(); for (int i = 0; i < muenzen.size() && rest > 0; i++) { Muenze muenze = muenzen.get(i); if (muenze.cents() <= rest) { rest = rest - muenze.cents(); wechselgeld.add(muenze); } else { uebrigemuenzen.add(muenze); } } if (rest!= 0) return new AutomatAusgabe(Fehler.KEIN_WECHSELGELD); this.muenzen = uebrigemuenzen;// Ÿbrig gebliebenes Wechselgeld ware.mengereduzieren(); Schön is anders Jan Leßner return new AutomatAusgabe(ware.getraenk(), wechselgeld); 22 }

23 Was tun? Wege bei arvato Schön is anders Jan Leßner 23

24 Ausbildung für Studenten Programmierhandwerk an Unis nicht gelernt ABER: Auch 5 er-kandidaten haben teilweise Potential Mehrwöchige Grundausbildung außerhalb der Projektarbeit Getränkeautomat zur Perfektion bringen Kampf der Killerphrase Wieso, geht doch Mehrere Review- und Verfeinerungszyklen Gleichzeitig Vermittlung von Design Prinzipien Pair Programming gegen Motivationslöcher und bei Verständnishürden Lernerfolg an Folgeaufgaben beweisen Glanzstücke sind Referenz für Produktivarbeit Schön is anders Jan Leßner 24

25 Ausbildung für Entwickler: Clean Code Initiative (seit Juli 2014) CCD definiert Grade Jeder Grad hat Themenschwerpunkte Diese bilden zeitlich begrenzt den Mittelpunkt der Aufmerksamkeit Nach einiger Zeit wechselt man zum nächsten Grad Ist man oben angekommen, fängt man unten wieder an Schrittweise stellen sich höherwertige Erkenntnisse ein Schön is anders Jan Leßner 25

26 Was macht ihr? Schön is anders Jan Leßner 26

27 Fragen? Vielen Dank! Schön is anders Jan Leßner 27