2021 LightningTalk: Naming is hard

Was kann schon schiefgehen, wenn man im Vier-Augen-System Legacy-Code refactort, um Redundanzen darin abzubauen? Aus Elenas Erfahrung hängt das stark davon ab, ob der Code tatsächlich redundant ist oder aufgrund von Variablennamen nur so aussieht.

Transcript

1. Elena Neuschild | elena.neuschild@cosee.biz
2. • Suchmaschine, die Suchwörter und Ortssuchen unterstützt. • User können Ergebnis-E-Mails abonnieren, die sie zugeschickt bekommen, wenn es neue Ergebnisse gibt. Der Kontext https://www.ebay-kleinanzeigen.de/
3. • Suchlogik ◦ Legacy-Code ◦ Suchergebnisse bestehen aus “Dokumenten” • Code Review → kleines Refactoring Der Kontext
4. Der Kontext Ein DocumentIdSet bildet eine Menge von Suchergebnis-Dokumenten ab.
5. Der Kontext Die Berechnung eines Suchergebnisses verwendet mehrere DocumentIdSets. Der Kontext
6. Das Refactoring vorher:
7. Das Refactoring vorher: nachher:
8. "Warum kriege ich Ergebnis-E-Mails, die überhaupt nicht zu meiner Ortssuche passen???????" -- anonymous User
9. Der Hintergrund Die Berechnung eines Suchergebnisses verwendet mehrere DocumentIdSets: • zuerst einen pro Suchparameter, • dann einen weiteren als Endergebnis.
10. Der Hintergrund: Optimierung Wenn ein Suchparameter nicht angefordert ist, soll danach nicht gefiltert werden. Optimierung: • Den Filterungsteil der Logik lassen wir weg, geht schneller.
11. Der Hintergrund: Optimierung
12. Zur Erinnerung: Das Refactoring vorher: nachher:
13. Das Refactoring: vorher
14. Das Refactoring: nachher
15. Der Fuckup Durch das Refactoring fällt ein wichtiger Unterschied weg: • es gibt keine Ortssuche ◦ “Finde alle Spiegelschränke, egal von wo!” • die Ortssuche produziert keine Ergebnisse ◦ “Finde alle Spiegelschränke im Lautertal!” ◦ “Es gibt aber keine Angebote im Lautertal. 🙁”
16. Der Fuckup Der Fuckup
17. Der Fix: Undo and Rename
18. Take-Aways • Legacy-Code ist doof. • Dinge benennen ist schwierig, aber wichtig. • Wenn du mit Legacy-Code arbeitest, lass dich nicht von Namen verwirren... ◦ … oder schlimmer, in falscher Sicherheit wiegen!
19. Elena Neuschild | elena.neuschild@cosee.biz