search

phlobbi / prog3

Repository für die Projektarbeit
1 stars 1 forks source link

PrePersist in DB-Klassen einbauen #51

Closed phlobbi closed 1 year ago

phlobbi commented 1 year ago

Über PrePersist soll sichergestellt werden, dass null-Attribute (wie über den leeren Konstruktor möglich) nicht in der Datenbank landen können (wo sinnvoll). Zum Beispiel kann das realDropOffDate beim Erstellen von Booking nicht bekannt sein, darf also auch null sein.

Bei einem fehlgeschlagenen Check soll eine IllegalArgumentException geworfen werden mit entsprechender Nachricht (dafür auch gerne die Methode in Utils verwenden).

Darüber hinaus könnte man die Datenbank auch weiter absichern, in dem über die Definition verhindert wird, dass überhaupt dort null-Werte landen können.

gitroba commented 1 year ago

@phlobbi Schau mal bitte, ob das so ist, wie du es dir vorgestellt hast. Hab die Prüfung in Validations-Klassen ausgelagert. Ich prüfe die Attribute als Stream und geb bei null eine allgemeine Message aus. Könnte das auch wahlweise noch mit ifs aufdröseln und dann ne spezifische Message geben, welcher Wert jetzt genau null war (notwendig?)

phlobbi commented 1 year ago

Soweit hatte ich nicht gedacht dass man das auf die Art lösen kann, dachte die Methode wird in die Klasse selbst eingebaut. Aber wenns so funktioniert ist auch gut. Denke mal wir müssen das nicht noch weiter aufdröseln.