[Bruno_]

#brunouczysieprogramowac #java #spring #programowanie

Takie query działa (ale wywala blad ze nie moze zaladowac danych bo fetchtype.Lazy)

Select u.body from Post u where u.user = ?1

Natomiast przy tym wywala blad ze nie moze sparsowac query:

Select u.body from Post u join fetch u.user where u.user = ?1

Jakie bedzie poprawne query? Chce wydostac Post.body z tabeli body danego usera (probowalem obiektem user i user_id filtrowac)

[S.....s]

wywala blad ze nie moze zaladowac danych bo fetchtype.Lazy

@Bruno_: pobrać w tej samej sesji

[Bruno_]

@SuppressWarnings: to nie wchodzi w gre, nie bede ladowal 200 postow usera z bazy danych tylko po to zeby dostac np jego email

[Bruno_]

@Bruno_: w sumie jest opcja zeby pobrac usera z bazy danych, potem z obiektu odczytac jego id i zapytac repo o posty z danym id, ale to bedzie chyba slabe wydajnosciowo (bo 2 zapytania bazy danych zamiast jednego)

[S.....s]

@Bruno_: nie rozumiem w takim razie co ty chcesz zrobić

[juniordev]

@Bruno_: a w transakcji to robisz?

[Bruno_]

@juniordev: a co to? ( ͡° ͜ʖ ͡°)

[Bruno_]

@juniordev: kurde zle napisalem we wpisie i juz nie mam opcji zeby zedytowac, wiec napisze jeszcze raz

Mam dwie encje: User i Post. W encji Post mam relacje ManyToOne i joincolumn z userem w postaci jego id (kolumna user_id). FetchType.Lazy (czemu? patrz drugi komentarz od gory), czyli jak wczytuje z bazy danych usera i chce odwolac sie do jego postow to wywala blad bo polaczenie z baza danych jakby jest juz

[juniordev]

@Bruno_: kurła usunąłem niechcący. Ogólnie wyżej pytałem co dokładnie chcesz osiągnąć tym zapytaniem?

[Bruno_]

@juniordev: @SuppressWarnings: eh w sumie to niewazne, usuwam zaraz xD i tak musze wczytac usera i tak (╯︵╰,)

[juniordev]

@Bruno_: Musisz to poprostu wykonać w transakcji. Oznacz swoje repo albo metode anntoacją @Transactional i będzie gites

[juniordev]

@Bruno_: zajrzałem jeszcze w twoje repo i widzę, że posty w użytkowniku trzymasz w liście. Zastanów się czy rzeczywiście potrzebujesz listy i doczytaj o konsekwencjach używania list w encjach w JPA(a dokladniej Hibernate)

[Bruno_]

@juniordev: a jak to zrobic bez listy? transactional nic nie dalo i nie widze w sumie powodu zeby cos dalo

[juniordev]

@Bruno_ Uczysz się więc chcę cię tylko nakierować. Można użyć innej kolekcji.
Jeśli wykonujesz operacje na leniwie ładowanych zależnościach to hibernate je podciągnie w locie. Powinno pomóc chyba, że wychodzisz z transakcji i tam próbujesz pobrać tego usera.

[Bruno_]

@juniordev: w przypadku innej kolekcji (chodzilo ci o set?) problem jest ten sam:

org.hibernate.LazyInitializationException: failed to lazily initialize a collection of role: pl.karolcz.springforum.user.User.posts, could not initialize proxy - no Session

[juniordev]

@Bruno_: set nie rozwiąże tego problemu. Chodziło mi o inne konsekwencje używania listy. Wrzuć swoją kod na repo i wtedy zobaczymy, ok?

[Bruno_]

@juniordev: ja rozumiem w czym jest problem, w teorii wiem jak go rozwiazac ale rozwiazania ze stacka nie przynosza pozadanego efektu - nie wiem czym to jest spowodowane

[juniordev]

@Bruno_: dlatego pokaż kod to nie będziemy zgadywać

[Bruno_]

@juniordev: duzo sie w sumie nie zmienilo, jest to wersja robocza, dlatego moze byc w miejscach w ktorych cos zmienialem syf
https://github.com/Karolcz125/spring-forum/tree/dev

[juniordev]

@Bruno_: jeśli to metoda findAllByUser z PostServicu rzuca wyjątkiem to ja oznacz jako transakcyjna a nie metodę w repozytorium. Tak jak pisałem wyżej, to metoda w której próbujesz wydobyć leniwie ładowane pole musi być wykonana w transakcji

[Bruno_]

@juniordev: tego transactionala wkladalem wszedzie, gdzie sie da - bez skutku

[juniordev]

@Bruno_: jestem na telefonie więc nie sprawdzę tego, ale wygląda na to, że serializacja Jacksona wykonuje się nie w transakcji.
Gdybyś zrobił DTO dla tego obiektu i przepisał z encji do niego wartości w transakcji to powinno działać.
Chociaż w tym wypadku najlepiej byłoby stworzyć query z joinem żeby uniknąć problemu n+1.

[juniordev]

Query powinno byc coś koło tego
Select p from Post p JOIN FETCH p.user u

[kernelpan1c]

@Bruno_: select p.body from User u JOIN FETCH Post as p on u.pk = p.user where u.id =?1 - spróbuj w te stronę; syntax może leżeć, piszę z telefonu

[juniordev]

Kurla ten wykop... Ucięło mi
where u.id=?1

Ogólnie przesyłanie encji po całym systemie to trochę słaby design. Jak zmieni Ci się encja to od razu zmieni Ci się zwracany json a to może złamać jakiś kontrakt. Powinieneś oddzielić warstwę prezentacji od biznesu i persystencji.

[Bruno_]

@juniordev: query nie chce sie sparsowac
@kernelpan1c: dziala ale nie #!$%@?, caly czas failed to lazily initialize a collection of role

[juniordev]

@Bruno_: zbudowałem twój projekt i moje query działa.

Select p from Post p JOIN FETCH p.user u where u=?1

Tylko teraz będziesz mial problem z cykliczną zależnością:
Post bedzie mial usera->user będzie miał liste postów -> kazdy z postów bedzie mial usera itd..

Ale query działa i nie ma problemu z lazy loadingiem.

[Bruno_]

@juniordev: ehhh ( ͡° ʖ̯ ͡°)

[juniordev]

@Bruno_: nie masz juz problemy z tym query. masz problem z testem. odpal sobie debug w tym tescie i zobacz co tam dostajesz. Normalny http get request nie wywali Ci tego exceptiona.

Teraz exception leci z :

posts.toString()

[Bruno_]

@juniordev: ehhh nie moge uwierzyc ze nie ogarnalem tego xD myslalem ze to jakis blad ide ze akurat w tamtej linii mi blad daje xD
dziekuje (｡◕‿‿◕｡)

[juniordev]

@Bruno_: spoczko. Zacząłem obserwować twój tag, czasami jeszcze może pomogę :)

[Bruno_]

@juniordev: to juz cale dwie osoby (ʘ‿ʘ)

[juniordev]

@Bruno_: jeszcze jedno. Aktualnie dostajesz ten exception bo z jakiegoś powodu metoda @BeforeAll nie obsluguje transakcji. Gdy zrobisz tak: https://pastebin.com/9HnbZ1uW to będzie działać

[Bruno_]

@juniordev: w sumie dzialalo i bez tego, no ale i tak musialem zmienic na BeforeEach tam zeby testy byly od siebie niezalezne (bo tak to zmieniala sie np ilosc postow i musialaby byc okreslona kolejnosc tych testow, a tak to moze asynchronicznie kiedys bede je robil, chociaz nawet nie wiem czy junit wspiera takie cos)

[Przegrywek123]

@Bruno_: Po co tworzysz konstruktor kontrolera

PostController(UserService userService) i go oplatasz?
Nie łatwiej dać @Autowired na tym private UserService userService?

[Bruno_]

@Przegrywek123
Punkt 3.3
https://www.baeldung.com/spring-autowire

Taka konwencja, nic nie poradzisz ¯_(ツ)_/¯

[juniordev]

@Przegrywek123: Może ja odpowiem. Ogólnie łatwiej ale
1. Wrzucanie @Autowired na pola klasy w konsekwencji prowadzi do tworzenia duzych, skomplikowanych obiektów z mnóstwem zależności
2. Jeśli masz jedyny konstruktor na klasie i nie jest to konstruktor domyślny, to znaczy, że nie jesteś w stanie utworzyć tej klasy bez tych zależności, a to znaczy, że ta klasa prawdopodobnie nie może istnieć bez tych zależności

[Bruno_]

@Przegrywek123 https://stackoverflow.com/questions/40620000/spring-autowire-on-properties-vs-constructor

[Przegrywek123]

@juniordev: @Bruno_: Znaczy ja rozumiem, tylko ten Controller raczej nie będzie używany nigdzie indziej. Zawsze widziałem injekcję na pole w kontrolerach.
https://www.baeldung.com/building-a-restful-web-service-with-spring-and-java-based-configuration <- 5 rozdział

[Bruno_]

@Przegrywek123 może dla uproszczenia, ale jak wkleisz klasę z tego 5 rozdziału do intellij to ci samo będzie chciało poprawić na constructor injection

[juniordev]

@Przegrywek123: ogólnie to nie można zakładać, że nie będzie nigdzie inaczej używany.
Będzie chociażby w testach. Poza tym, jeśli raz ustalisz jakieś zasady na projekcie to musisz ich przestrzegać wszędzie.

[Bruno_]

@Przegrywek123 Efekt ten sam, na bardzo podobnej zasadzie używasz getterow i setterow a nie publicznych zmiennych w klasach

[Przegrywek123]

@juniordev: @Bruno_: , okej, czyli dobra praktyka ( ͡° ͜ʖ ͡°)