search

SajariS / JuomaPeli

WineRed-Juomapeli
MIT License
0 stars 0 forks source link

Katselmointipyyntö #64

Open Samwes13 opened 6 months ago

SajariS commented 6 months ago

https://github.com/SajariS/JuomaPeli/blob/dev/src/main/java/RW/JuomaPeli/web/RestCardController.java https://github.com/SajariS/JuomaPeli/blob/dev/src/main/java/RW/JuomaPeli/domain/Card.java

jinblo commented 6 months ago

Card.java: -selkeä muuttujien nimeämiskäytäntö -pakettinimet ei pitäisi alkaa isolla, myöskään CamelCase ei yleensä käytössä -taulun uudelleen nimeäminen (@Table(name...)) mahdollisesti turha? -turhia rivinvaihtoja attribuuttien luettelossa, vaikeuttaa hahmottamista -Constructorit epätavallisesti tiedoston lopussa, yleensä heti attrubuuttien jälkeen -epätavallisen pitkä sisennys

RestCardController.java: -pakettinimessä sama kuin Card.java -Get- ja PostMappingin käytössä, helpommin luettava kuin @RequestMapping -ei turhia kommentteja -selkeä muuttujien nimeämiskäytäntö