스트리밍 - 광고 조회수 업데이트 메서드 안되는 문제 해결

chobeebee · Jul 3, 2024 · 04791bb · 04791bb
1 parent 61b300d
commit 04791bb
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 18 deletions.
diff --git a/src/main/java/com/sparta/binplay/controller/VideoController.java b/src/main/java/com/sparta/binplay/controller/VideoController.java
@@ -74,8 +74,8 @@ public ResponseEntity<StreamResponseDto> videoStop(@PathVariable("video-id") Lon
     }
 
     //광고 시청
-    @PostMapping("/play/ads/{ad-id}")
-    public ResponseEntity<VideoAdResponseDto> viewAd(@PathVariable("ad-id") Long adId, @RequestBody Long videoId) {
-        return ResponseEntity.ok().body(videoAdService.updateAdCount(adId, videoId));
+    @PostMapping("/play/ads/{video-ad-id}")
+    public ResponseEntity<VideoAdResponseDto> viewAd(@PathVariable("video-ad-id") Long videoAdId) {
+        return ResponseEntity.ok().body(videoAdService.updateAdCount(videoAdId));
     }
 }
diff --git a/src/main/java/com/sparta/binplay/repository/VideoAdRepository.java b/src/main/java/com/sparta/binplay/repository/VideoAdRepository.java
@@ -1,13 +1,8 @@
 package com.sparta.binplay.repository;
 
-import com.sparta.binplay.entity.Ads;
 import com.sparta.binplay.entity.VideoAd;
-import com.sparta.binplay.entity.Videos;
 import org.springframework.data.jpa.repository.JpaRepository;
 import org.springframework.stereotype.Repository;
-
-import java.util.Optional;
 @Repository
 public interface VideoAdRepository extends JpaRepository<VideoAd, Long> {
-    Optional<VideoAd> findByVideoAndAd(Videos video, Ads ad);
 }
diff --git a/src/main/java/com/sparta/binplay/repository/VideoRepository.java b/src/main/java/com/sparta/binplay/repository/VideoRepository.java
@@ -6,6 +6,7 @@
 import org.springframework.stereotype.Repository;
 
 import java.util.List;
+import java.util.Optional;
 
 @Repository
 public interface VideoRepository extends JpaRepository<Videos, Long> {
@@ -20,5 +21,5 @@ public interface VideoRepository extends JpaRepository<Videos, Long> {
 
     List<Videos> findByUserUserId(Long userId);
 
-    Videos findByVideoId(Long videoId);
+    Optional<Videos> findByVideoId(Long videoId);
 }
diff --git a/src/main/java/com/sparta/binplay/service/VideoAdService.java b/src/main/java/com/sparta/binplay/service/VideoAdService.java
@@ -1,9 +1,7 @@
 package com.sparta.binplay.service;
 
 import com.sparta.binplay.dto.response.VideoAdResponseDto;
-import com.sparta.binplay.entity.Ads;
 import com.sparta.binplay.entity.VideoAd;
-import com.sparta.binplay.entity.Videos;
 import com.sparta.binplay.repository.AdRepository;
 import com.sparta.binplay.repository.VideoAdRepository;
 import com.sparta.binplay.repository.VideoRepository;
@@ -18,13 +16,8 @@ public class VideoAdService {
     private final AdRepository adRepository;
 
     // 특정 광고 재생 횟수 업데이트
-    public VideoAdResponseDto updateAdCount(Long adId, Long videoId) {
-        Ads ad = adRepository.findById(adId).orElseThrow(() -> new RuntimeException("광고를 찾을 수 없음"));
-        Videos video = videoRepository.findById(videoId).orElseThrow(() -> new RuntimeException("비디오를 찾을 수 없음"));
-        VideoAd videoAd = videoAdRepository.findByVideoAndAd(video, ad).orElse(VideoAd.builder()
-                .ad(ad)
-                .video(video)
-                .build());
+    public VideoAdResponseDto updateAdCount(Long videoAdId) {
+        VideoAd videoAd = videoAdRepository.findById(videoAdId).orElseThrow(() -> new RuntimeException("영상을 찾을 수 없습니다."));
 
         videoAd.countAd();
         videoAdRepository.save(videoAd);