๐ 3๋จ๊ณ - ์ฆ๊ฒจ์ฐพ๊ธฐ ๊ธฐ๋ฅ ๊ตฌํ #352
Conversation
| .orElseThrow(IllegalArgumentException::new); | ||
|
|
||
| Member member = memberService.findByEmail(loginMember.getEmail()); | ||
| member.removeFavorite(favorite); |
There was a problem hiding this comment.
member.removeFavorite(favorite); ํด์ฃผ์
จ๋๋ฐ์! ํ์ธํด๋ณด๋ favorites ์์ ์ต์
์ด orphanRemoval = true ๋ก ๋์ด ์๋๊ฑธ ๋ดค์ต๋๋ค! ๊ณ ์๊ฐ์ฒด๊ฐ ๋ favorite ๋ ๊ฐ์ด delete ๋ ๊ฒ์ผ๋ก ์์ ๋๋๋ฐ์!
55 line ์ favoriteRepository.delete(favorite); ๋ถ๋ถ์ด ํ์ํ ๊น์? ๐ง
| import static nextstep.member.domain.exception.FavoriteException.SAME_SOURCE_AND_TARGET; | ||
|
|
||
| @Entity | ||
| public class Favorite { |
There was a problem hiding this comment.
ํ์ฌ member ์ ๋ํ ์ ๋ณด๋ Favorite ์์ ์ ์ ์๋ค๊ณ ๋ณด๋๋ฐ์!
JPA ์์ ์ธ๋ํค๊ฐ ์์ด๋ OneToMany ์ ManyToOne ๊ด๊ณ๊ฐ ๋งบ์ด์ง ์ ์๋์ง ๊ถ๊ธํฉ๋๋ค!
์ง์ ์ฐธ์กฐ๋ ๊ฐ์ ์ฐธ์กฐ๋ฅผ ํ์ง ์์๋ ๋ ๊น์? ๐ง
ํ
์คํธ์์ ๋์ค๋ ์ฟผ๋ฆฌ๋ฅผ ํ์ธํด๋ณธ ๊ฒฐ๊ณผ
update
favorite
set
member_id=?
where
id=?
๋ก ์
๋ฐ์ดํธ ์ฟผ๋ฆฌ๋ฅผ ๋ ๋ฆฌ๊ณ ์๋๋ฐ์. favorite ์๋ memberId ๊ฐ ์์์๋ ์ฟผ๋ฆฌ๋ฅผ ๋ ๋ฆฌ๊ณ ์์ต๋๋ค!
ํ๋ฒ ํ์ธํด๋ณด์๋ฉด ์ข์ ๊ฒ ๊ฐ์ต๋๋ค ๐
| } | ||
|
|
||
| public Favorite addFavorite(Station source, Station target) { | ||
| Favorite favorite = new Favorite(source, target); |
There was a problem hiding this comment.
member domain ์์ Favorite ์์ฑ ์ญํ ์ ๊ฐ์ง๊ณ ์๋๊ฒ ๋ง์๊น์? ์ด๋ป๊ฒ ์๊ฐํ์๋์? ๐
| @@ -0,0 +1,10 @@ | |||
| package nextstep.member.domain.exception; | |||
|
|
|||
| public class FavoriteException extends IllegalArgumentException { | |||
| */ | ||
| @DisplayName("์ ํจํ์ง ์์ ๋ก๊ทธ์ธ ์ ๋ณด๋ก ์ญ์ ํ ์ ์๋ค") | ||
| @Test | ||
| void ์ ํจํ์ง_์์_ํ ํฐ์ผ๋ก_๋ก๊ทธ์ธ_์ญ์ _์์ฒญ() { |
์๋ ํ์ธ์! ๊ธฐ๊ฐ์ด ๋๋ฌ๋๋ฐ๋ ๋ถ๊ตฌํ๊ณ ๋ฆฌ๋ทฐํด์ฃผ์ ์ ๊ฐ์ฌํฉ๋๋ค :)
ATDD๊ฐ์์ ๋ง์ง๋ง ๋ฆฌ๋ทฐ ์์ฒญ์ด๋ค์..!
์๊ฐ ๋์ค๋ ์ฒ์ฒํ ํด์ฃผ์๋ฉด ๊ฐ์ฌ๋๋ฆฌ๊ฒ ์ต๋๋ค. ๐