Aktywne Wpisy
Sepia +38
Czy są w ogóle jakieś rozwinięte państwa na świecie z wysoką dzietnością? Otóż są. A raczej jest. Jedno. Izrael. Tylko, że tam posiadanie trójki i więcej dzieci to moralny, patriotyczny, religijny, obywatelski, rodzinny i osobisty OBOWIĄZEK. Mocno wspierany i propagowany na każdym kroku. Oprócz tego rodzicielstwo jest promowane jako coś po prostu fajnego, modnego, dającego szczęście i przynoszącego zyski (xd). Słowem, presja na posiadanie dzieci jest w Izraelu ogromna, dodatkowo żydzi mają
#famemma #bitasmietanka #!$%@? przeczytalem moze z 40 wpisow o reprezentantach waszego portalu dla stulejek, i nie moge z beki #!$%@? wytrzymac. lizecie dupe temu teemo czy jakos tak, piszecie ze powinien dostac walke w prime czy clout. a przypomnie ze chlop wazyl pewnie tyle co jaszczurowy i nunu razem wzieci, ciosy i kopniecia nunu zadawaly mu -1hp a jaszczurowego -2hp. taka prawda ze to zwykly gostek o normalnej wadze, chodzacy przynajmniej 2razy/tydzien
https://github.com/Sampeteq/cinema-app
#java #naukaprogramowania #programowanie
*
domain/Room.java
nie powinien mieć w sobie bazodanowego śmiecia. Powinieneś mieć osobny "czysty" typ domenowy zwracany przez repozytorium a to co masz teraz powinno iść tam gdzie jest implementacja* brak serwisów domenowych
* dwustrona relacja pomiędzy appllication a domain: application używa domain a domain oczekuje całą tranzakcyjność od springa.
Klasy które mają po jednej metodzie? Pakiety, podpakiety, podpodpodpakiety a w środku po 3 pliki? Klasy, które nic nie robią a tylko przekazują sterowanie gdzieś dalej? Po ch*j to? Nie cierpię takiego stylu kodowania.
Projekt tej wielkości powinien być płaski. Jeden pakiet, kilka klas reprezentujących dane wyciągane z bazy, jeden serwis zawierający całą logikę.
1. UserPasswordResetHandlerService
log.info("Mail sent");
Ten logger niczego nie mówi, dodaj kontekst (np. typ maila)
2. SeatDto
boolean isFree
Unikałbym booleanów jak ognia. Przyjdzie Ci zaraz product owner i powie że chce miec nowy typ - częsciowo free itp. Nawet jak mamy 2
@Krolik: ma to sens jak chcesz sobie pocwiczyć. W praktyce taki kod nie ma sensu przy tej skali i zerowej logice
Nie lepiej po prostu zrobić jeden serwis dla danej funkcjonalności, np. ogólny UserService w którym możesz zrobić tworzenie, pobieranie, edytowanie usera i jego składowych
@Nofenak
https://github.com/Sampeteq/cinema-app/blob/main/src/main/java/com/cinema/users/application/services/UserAdminCreateService.java
nie wstrzykuj w ten sposób wartości z propertek,lepiej jest to robić prze konstruktor, wtedy masz robisz adnotację nad tym Stringiem i spring ci to ogarnia, aktualnie ta klasa jest nietestowalna jednostkowo, bo żeby wartości wstrzyknąć musisz mieć kontekst springa podniesiony, żeby wczytać dane z pliku properties
https://github.com/Sampeteq/cinema-app/blob/main/src/main/java/com/cinema/rooms/application/services/RoomAvailableService.java
to jest strasznie słabe, co jeśli będziesz miał milion pokoi w bazie?
https://github.com/Sampeteq/cinema-app/blob/main/src/main/java/com/cinema/rooms/application/services/RoomConfigService.java
tutaj dwie rzeczy w metodzie readRoomsConfigAsJson robisz coś
jak gdzieś w klasie używasz pola to nie musisz pisać this.pole tylko samo pole