habittrackingblog

20240603: Review Backend

• Datum: 03.06.2024
• Startzeit: 15:00 Uhr
• Endzeit: 15:45 Uhr
• Teilnehmende:
  • Paul (Moderation)
  • Kayra (Präsentation)
  • Kai (Protokoll)
  • Jannis (externer Reviewer, Dinder)
• Ziel / Fokus:
  • Ziel des Reviews ist es, einen externen Blick auf die Modellierung der Datenbank und die Implementierung des Backends zu holen, um dadurch Verbesserungen zu finden.
• Komponenten:
  • Datenbankmodell
  • Backend-Modules
• Kriterien:
  • Codequalität
  • Skalierbarkeit
  • Sicherheit
  • Wartbarkeit
• Methodik: Code Review
• Ergebnis: Backend sieht, bis auf ein paar kleine, schnell behebbare, Unschönheiten, gut aus

Review: Datenbankmodell

• User könnte mehr sinnvolle Attribute vertragen
• wenn ein User gelöscht wird, sollten auch die Habits gelöscht werden ➡️ ON DELETE CASCADE hinzufügen
• Codequalität: sieht gut aus (wegen Prismaautoformatierung ist das vorgegeben :) )
• Skalierbarkeit: /
• Sicherheit: /
• Wartbarkeit: ✅ (durch Prisma-CLI einfaches Ändern des Datenmodells und Erstellen von Migrationen)

Review: Backend-Modules

• gute Trennung von Responsibilities
• Debugmethoden (im HabitsService) sollten entfernt werden
• Codequalität:
  • noch einige Artifakte aktiver Entwicklung vorhanden
  • Standard-App-Modul entfernen
• Skalierbarkeit: sieht gut aus (durch nest.js gegeben :) )
• Sicherheit: Authentifizierung und Authorisierung von benutzerspezifischen Endpunkten 👍
• Wartbarkeit: sieht gut aus durch die Modularisierung