CampingOn / CampingOn-BE

0 stars 1 forks source link

Fix: #127 LazyInitializationException ํ•ด๊ฒฐ #128

Closed jhYun505 closed 1 day ago

jhYun505 commented 2 days ago

๐Ÿ“Œ ๋ชฉ์ 

์ด PR์ด ํ•ด๊ฒฐํ•˜๋ ค๋Š” ๋ฌธ์ œ๋‚˜ ์ถ”๊ฐ€ํ•˜๋ ค๋Š” ๊ธฐ๋Šฅ์˜ ๋ชฉ์ ์„ ๊ฐ„๋‹จํžˆ ์„ค๋ช…ํ•ด์ฃผ์„ธ์š”.

Lazy loading์œผ๋กœ ๋ฐœ์ƒํ•˜๋Š” LazyInitializationException ํ•ด๊ฒฐ

๐Ÿ› ๏ธ ์ž‘์—… ์ƒ์„ธ ๋‚ด์šฉ

์ž‘์—…ํ•œ ๋‚ด์šฉ์— ๋Œ€ํ•œ ์„ค๋ช…์„ ์ฒดํฌ๋ฆฌ์ŠคํŠธ ํ˜•์‹์œผ๋กœ ์ž‘์„ฑํ•ด์ฃผ์„ธ์š”.

  • [x] @ToString ์–ด๋…ธํ…Œ์ด์…˜ ์‚ฌ์šฉํ•˜์ง€ ์•Š๊ณ  overrideํ•ด์„œ ์ง์ ‘ ๊ตฌํ˜„

๐Ÿ” ๋ณ€๊ฒฝ ์‚ฌํ•ญ

์ฝ”๋“œ ๋ณ€๊ฒฝ ์‚ฌํ•ญ์„ ์š”์•ฝํ•˜๊ณ  ์ค‘์š”ํ•œ ๋ถ€๋ถ„์„ ์„ค๋ช…ํ•ด์ฃผ์„ธ์š”.

  • User.java: toString overriding
  • toString ๋‚ด๋ถ€์—์„œ Lazy Loading ๋˜๋Š” ์†์„ฑ ์ œ๊ฑฐ
  • UserKeyword.java: toString override

โœ… ํ…Œ์ŠคํŠธ ๋ฐฉ๋ฒ•

๋ณ€๊ฒฝ๋œ ์ฝ”๋“œ๊ฐ€ ์–ด๋–ป๊ฒŒ ํ…Œ์ŠคํŠธ๋˜์—ˆ๋Š”์ง€ ์„ค๋ช…ํ•ด์ฃผ์„ธ์š”.

  1. [ ] ํ…Œ์ŠคํŠธ ์ผ€์ด์Šค ์ž‘์„ฑ
  2. [x] ๋กœ์ปฌ ํ™˜๊ฒฝ์—์„œ ์ง์ ‘ ํ…Œ์ŠคํŠธ
  3. [ ] ๊ธฐํƒ€:

โš ๏ธ ์ฃผ์˜ ์‚ฌํ•ญ

์ฝ”๋“œ ๋ณ€๊ฒฝ ์‹œ ๊ณ ๋ คํ•ด์•ผ ํ•  ์ ์ด๋‚˜ ๋ฆฌ๋ทฐ์–ด๊ฐ€ ์ฃผ์˜ํ•ด์•ผ ํ•  ์ ์ด ์žˆ์œผ๋ฉด ์ž‘์„ฑํ•ด์ฃผ์„ธ์š”.

๐Ÿ“ธ ์Šคํฌ๋ฆฐ์ƒท (์„ ํƒ ์‚ฌํ•ญ)

๋ณ€๊ฒฝ๋œ ๊ธฐ๋Šฅ์ด ์žˆ์œผ๋ฉด ์Šคํฌ๋ฆฐ์ƒท์ด๋‚˜ ๋™์˜์ƒ์„ ์ถ”๊ฐ€ํ•ด์ฃผ์„ธ์š”.

๐Ÿ”— ์ฐธ๊ณ  ์‚ฌํ•ญ

๊ด€๋ จ๋œ ์ด์Šˆ ๋ฒˆํ˜ธ๋ฅผ ์–ธ๊ธ‰ํ•ด์ฃผ์„ธ์š”. ์ถ”๊ฐ€ ์ฐธ๊ณ ํ•  ๋งŒํ•œ ์ž๋ฃŒ๋‚˜ ๋งํฌ๊ฐ€ ์žˆ์œผ๋ฉด ์ž‘์„ฑํ•ด์ฃผ์„ธ์š”.

  • Closes #127, #109