Merge pull request #137 from dnd-side-project/dev

fix(PlaylistSaveService): 플레이리스트 수정 시 곡 중복 가능하도록 수정

Author: gzhan0226

api-server/src/main/java/com/example/demo/domain/songs/service/YouTubeSongServiceImpl.java

@@ -54,7 +54,9 @@ private Map<String, String> extractValidLinkToVideoId(List<String> links) {
                 .filter(this::isValidYouTubeUrl)
                 .collect(Collectors.toMap(
                         Function.identity(),
-                        this::extractVideoId
+                        this::extractVideoId,
+                        (exist, dup) -> exist,      // 중복 시 첫 값 유지
+                        LinkedHashMap::new          // 삽입 순서 보존
                 ));
     }
 

main-server/src/main/java/com/example/demo/domain/playlist/service/PlaylistSaveService.java

@@ -88,34 +88,56 @@ public PlaylistWithSongsResponse editPlaylistWithSongs(String usersId, Long play
 
         playlistRepository.save(playlist); // playlist부터 수정
 
+        // 기존 곡
         List<Song> existingSongs = songRepository.findSongsByPlaylistId(playlistId);
 
-        // 요청 링크 세트(검증된 링크라고 가정)
-        Set<String> requestedLinks = request.youTubeVideoInfo().stream()
+        // 새로 요청된 곡
+        List<String> requestedLinks = request.youTubeVideoInfo().stream()
                 .map(YouTubeVideoInfoDto::link)
-                .collect(Collectors.toCollection(LinkedHashSet::new));
+                .map(String::trim)
+                .toList();
 
-        // 기존 링크 세트
-        Set<String> existingLinks = existingSongs.stream()
-                .map(Song::getYoutubeUrl)
-                .collect(Collectors.toSet());
+        Map<String, Long> existingCount = existingSongs.stream()
+                .collect(Collectors.groupingBy(Song::getYoutubeUrl, Collectors.counting()));
 
-        // 1) 삭제: 기존엔 있으나 요청엔 없는 링크 -> id 수집 후 deleteAllByIdIn
-        List<Long> deleteIds = existingSongs.stream()
-                .filter(s -> !requestedLinks.contains(s.getYoutubeUrl()))
-                .map(Song::getId)
-                .toList();
+        Map<String, Long> requestedCount = requestedLinks.stream()
+                .collect(Collectors.groupingBy(link -> link, Collectors.counting()));
 
+        // 삭제 로직
+        List<Long> deleteIds = new ArrayList<>();
+        for (Song song : existingSongs) {
+            String url = song.getYoutubeUrl();
+            long reqCnt = requestedCount.getOrDefault(url, 0L);
+            long existCnt = existingCount.getOrDefault(url, 0L);
+
+            if (existCnt > reqCnt) {
+                // 삭제 필요 → 하나씩 줄여나감
+                deleteIds.add(song.getId());
+                existingCount.put(url, existCnt - 1); // 카운트 감소
+            }
+        }
         if (!deleteIds.isEmpty()) {
             songRepository.deleteAllByIdIn(deleteIds);
         }
 
-        // 2) 추가: 요청엔 있으나 기존엔 없는 링크만 저장
-        List<Song> toAdd = request.youTubeVideoInfo().stream()
-                .filter(dto -> !existingLinks.contains(dto.link()))
-                .map(dto -> SongMapper.toEntity(dto, playlist))
-                .toList();
-
+        // 추가
+        List<Song> toAdd = new ArrayList<>();
+        for (Map.Entry<String, Long> entry : requestedCount.entrySet()) {
+            String url = entry.getKey();
+            long reqCnt = entry.getValue();
+
+            long existCnt = existingSongs.stream()
+                    .filter(s -> s.getYoutubeUrl().equals(url))
+                    .count();
+
+            if (reqCnt > existCnt) {
+                long needToAdd = reqCnt - existCnt;
+                request.youTubeVideoInfo().stream()
+                        .filter(dto -> dto.link().equals(url))
+                        .limit(needToAdd) // 필요한 만큼만 생성
+                        .forEach(dto -> toAdd.add(SongMapper.toEntity(dto, playlist)));
+            }
+        }
         if (!toAdd.isEmpty()) {
             songRepository.saveAll(toAdd);
         }
@@ -133,9 +155,6 @@ public PlaylistWithSongsResponse editPlaylistWithSongs(String usersId, Long play
         return new PlaylistWithSongsResponse(playlist.getId(), songDtos);
     }
 
-    /**
-     * 기존 대표 해제 → 새 대표 설정 → RepresentativePlaylist 갱신
-     */
     public void replaceRepresentativePlaylist(Users user, Playlist newRepPlaylist) {
         String userId = user.getId();