Skip to content

Commit a7c2385

Browse files
committed
refactor: Rq 선언 service -> controller
1 parent 0b90afc commit a7c2385

2 files changed

Lines changed: 9 additions & 6 deletions

File tree

src/main/java/com/back/web7_9_codecrete_be/domain/artists/controller/ArtistsController.java

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,8 @@
88
import com.back.web7_9_codecrete_be.domain.artists.dto.response.SearchResponse;
99
import com.back.web7_9_codecrete_be.domain.artists.service.ArtistService;
1010
import com.back.web7_9_codecrete_be.domain.artists.service.ArtistEnrichService;
11+
import com.back.web7_9_codecrete_be.domain.users.entity.User;
12+
import com.back.web7_9_codecrete_be.global.rq.Rq;
1113
import com.back.web7_9_codecrete_be.global.rsData.RsData;
1214
import io.swagger.v3.oas.annotations.Operation;
1315
import io.swagger.v3.oas.annotations.tags.Tag;
@@ -24,6 +26,7 @@
2426
public class ArtistsController {
2527
private final ArtistService artistService;
2628
private final ArtistEnrichService enrichService;
29+
private final Rq rq;
2730

2831
@Operation(summary = "아티스트 저장", description = "임의의 가수(or 팀)을 DB에 저장합니다.")
2932
@GetMapping("/saved")
@@ -97,7 +100,8 @@ public RsData<List<SearchResponse>> search(
97100
public RsData<Void> artistLikes(
98101
@PathVariable Long id
99102
) {
100-
artistService.likeArtist(id);
103+
User user = rq.getUser();
104+
artistService.likeArtist(id, user);
101105
return RsData.success("아티스트 찜 성공", null);
102106
}
103107

@@ -106,7 +110,8 @@ public RsData<Void> artistLikes(
106110
public RsData<Void> deleteArtistLikes(
107111
@PathVariable Long id
108112
) {
109-
artistService.deleteLikeArtist(id);
113+
User user = rq.getUser();
114+
artistService.deleteLikeArtist(id, user);
110115
return RsData.success("아티스트 찜 해제 성공", null);
111116
}
112117

src/main/java/com/back/web7_9_codecrete_be/domain/artists/service/ArtistService.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -130,8 +130,7 @@ public List<SearchResponse> search(String artistName) {
130130
}
131131

132132
@Transactional
133-
public void likeArtist(Long artistId) {
134-
User user = rq.getUser();
133+
public void likeArtist(Long artistId, User user) {
135134
Artist artist = artistRepository.findById(artistId)
136135
.orElseThrow(() -> new BusinessException(ArtistErrorCode.ARTIST_NOT_FOUND));
137136

@@ -143,8 +142,7 @@ public void likeArtist(Long artistId) {
143142
}
144143

145144
@Transactional
146-
public void deleteLikeArtist(Long artistId) {
147-
User user = rq.getUser();
145+
public void deleteLikeArtist(Long artistId, User user) {
148146
Artist artist = artistRepository.findById(artistId)
149147
.orElseThrow(() -> new BusinessException(ArtistErrorCode.ARTIST_NOT_FOUND));
150148
ArtistLike likes = artistLikeRepository.findByArtistAndUser(artist, user)

0 commit comments

Comments
 (0)