search

Geend / HshHelper

Hannover University of Applied Sciences and Arts - Master Project - Security competition to make a secure filesharing website.
GNU General Public License v3.0
0 stars 1 forks source link

Streamlining der Visibility unserer Model-Variabeln #56

Closed eloquenza closed 6 years ago

eloquenza commented 6 years ago

Julius hatte das eigentlich schon gemacht, aber Philip moechte unbedingt das ganze durch Lombok ersetzen, obwohl alles davon eigentlich nutzlos ist, weil unsere Getter/Setter niemals Invarianten pruefen oder sonstigen Quatsch machen, und somit public access eigentlich reichen sollte.

Mir ist dennoch aufgefallen, dass userID public ist, waehrend die IDs der anderen Models private sind.

Geend commented 6 years ago

Das ist von 2012, aber wir sollten prüfen ob das ein Problem ist: https://stackoverflow.com/questions/11062167/lombok-with-play-2 tl;dr: Lombok getter/setter sind nicht in Scala Templates verfügbar

Flips01 commented 6 years ago

Scheint auch noch mit der aktuellen Version zu sein. Schade. Dann lassen wir das. :(

@eloquenza Lombok hat nix mit Invarianten o.Ä. zu tun. Du machst einfach ein @Data über eine Klasse und Lombok bewirkt, dass 1:1 das gleiche passiert wie beim IntelliJ auto generierten code. Der Unterschied ist, dass der Getter/Setter Code nicht mehr sichtbar ist und so wahnsinnig viel nutzloser Boilerplate Code verschwinden würde.