feat: 코스 발견 성능 최적화 및 UX 개선

thingineeer · thingineeer · commit 7650e4e019a3 · 2026-02-13T17:19:47.000+09:00
- 탭 재탭 시 스크롤 투 탑만 수행 (불필요한 API 호출 제거)
- reloadData() → reloadSections() 전환 (Section 4만 갱신)
- 배너 캐러셀 셀 재사용 최적화 (viewWithTag 패턴)
- 배너 가로 스와이프 시 세로 스크롤 충돌 방지
- 배너 영속성 (prepareForReuse 상태 유지, skipReload)
- pull-to-refresh 추가 (Section 4 전용)
- iOS 26 Liquid Glass 탭바 대응
- 탭 아이콘 rendering intent template 전환
- pull-to-refresh 시 LoadingIndicator 이중 표시 제거
- courseList 사전 삭제 제거 (data source 불일치 방지)
diff --git a/Runnect-iOS/Runnect-iOS/Global/Extension/UIKit+/UITabBar+.swift b/Runnect-iOS/Runnect-iOS/Global/Extension/UIKit+/UITabBar+.swift
@@ -8,7 +8,7 @@
 import UIKit
 
 extension UITabBar {
-    
+
     static func clearShadow() {
         UITabBar.appearance().shadowImage = UIImage()
         UITabBar.appearance().backgroundImage = UIImage()
diff --git a/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_course_discove_fill.imageset/Contents.json b/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_course_discove_fill.imageset/Contents.json
@@ -21,6 +21,6 @@
     "version" : 1
   },
   "properties" : {
-    "template-rendering-intent" : "original"
+    "template-rendering-intent" : "template"
   }
 }
diff --git a/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_course_discover.imageset/Contents.json b/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_course_discover.imageset/Contents.json
@@ -21,6 +21,6 @@
     "version" : 1
   },
   "properties" : {
-    "template-rendering-intent" : "original"
+    "template-rendering-intent" : "template"
   }
 }
diff --git a/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_course_draw.imageset/Contents.json b/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_course_draw.imageset/Contents.json
@@ -21,6 +21,6 @@
     "version" : 1
   },
   "properties" : {
-    "template-rendering-intent" : "original"
+    "template-rendering-intent" : "template"
   }
 }
diff --git a/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_course_draw_fill.imageset/Contents.json b/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_course_draw_fill.imageset/Contents.json
@@ -21,6 +21,6 @@
     "version" : 1
   },
   "properties" : {
-    "template-rendering-intent" : "original"
+    "template-rendering-intent" : "template"
   }
 }
diff --git a/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_mypage.imageset/Contents.json b/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_mypage.imageset/Contents.json
@@ -21,6 +21,6 @@
     "version" : 1
   },
   "properties" : {
-    "template-rendering-intent" : "original"
+    "template-rendering-intent" : "template"
   }
 }
diff --git a/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_mypage_fill.imageset/Contents.json b/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_mypage_fill.imageset/Contents.json
@@ -21,6 +21,6 @@
     "version" : 1
   },
   "properties" : {
-    "template-rendering-intent" : "original"
+    "template-rendering-intent" : "template"
   }
 }
diff --git a/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_storage.imageset/Contents.json b/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_storage.imageset/Contents.json
@@ -21,6 +21,6 @@
     "version" : 1
   },
   "properties" : {
-    "template-rendering-intent" : "original"
+    "template-rendering-intent" : "template"
   }
 }
diff --git a/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_storage_fill.imageset/Contents.json b/Runnect-iOS/Runnect-iOS/Global/Resource/Assets.xcassets/ic_storage_fill.imageset/Contents.json
@@ -21,6 +21,6 @@
     "version" : 1
   },
   "properties" : {
-    "template-rendering-intent" : "original"
+    "template-rendering-intent" : "template"
   }
 }
diff --git a/Runnect-iOS/Runnect-iOS/Presentation/CourseDiscovery/Views/AdImageCollectionViewCell.swift b/Runnect-iOS/Runnect-iOS/Presentation/CourseDiscovery/Views/AdImageCollectionViewCell.swift
@@ -166,23 +166,8 @@ final class AdImageCollectionViewCell: UICollectionViewCell {
     override func prepareForReuse() {
         super.prepareForReuse()
         stopAutoScroll()
-        cancelShimmerTimeout()
-
-        // 상태 초기화
-        adsResponseCount = 0
-        carouselShown = false
-        isAd1Loaded = false
-        isAd2Loaded = false
-        currentPage = 0
-        pages = imgBanners.map { .image($0) }
-
-        // UI 초기 상태로 리셋
-        bannerCollectionView.isHidden = true
-        gradientView.isHidden = true
-        pageControlStack.isHidden = true
-        adLabelContainer.alpha = 0
-        shimmerView.isHidden = false
-        shimmerView.alpha = 1
+        // 이미 캐러셀이 표시된 상태면 리셋하지 않음
+        // 새 광고 로드가 필요할 때만 리셋 (setRootViewController에서 처리)
     }
 
     deinit {
@@ -195,13 +180,20 @@ final class AdImageCollectionViewCell: UICollectionViewCell {
 
 extension AdImageCollectionViewCell {
 
-    func setRootViewController(_ viewController: UIViewController) {
+    /// 이미 로드된 광고가 있으면 shimmer 없이 바로 표시
+    func setRootViewController(_ viewController: UIViewController, skipReload: Bool = false) {
         guard shouldShowAds else {
+            if skipReload && carouselShown { return }
             skipAdsAndShowCarousel()
             return
         }
         bannerView1.rootViewController = viewController
         bannerView2.rootViewController = viewController
+
+        if skipReload && carouselShown {
+            // 이미 로드된 상태 — shimmer 없이 바로 표시
+            return
+        }
         loadAdMobBanners()
     }
 
@@ -456,6 +448,11 @@ extension AdImageCollectionViewCell {
         return Self.imagePageInterval
     }
 
+    func resumeAutoScrollIfNeeded() {
+        guard carouselShown, autoScrollWork == nil else { return }
+        scheduleNextScroll()
+    }
+
     private func startAutoScroll() {
         stopAutoScroll()
 
@@ -533,22 +530,36 @@ extension AdImageCollectionViewCell {
 
 extension AdImageCollectionViewCell: UIScrollViewDelegate {
 
+    func scrollViewWillBeginDragging(_ scrollView: UIScrollView) {
+        guard scrollView === bannerCollectionView else { return }
+        // 배너 가로 스와이프 시 상위 세로 스크롤 잠금
+        findParentScrollView()?.isScrollEnabled = false
+    }
+
+    func scrollViewDidEndDragging(_ scrollView: UIScrollView, willDecelerate decelerate: Bool) {
+        guard scrollView === bannerCollectionView else { return }
+        if !decelerate {
+            findParentScrollView()?.isScrollEnabled = true
+        }
+    }
+
     func scrollViewDidEndDecelerating(_ scrollView: UIScrollView) {
         guard scrollView === bannerCollectionView, scrollView.frame.width > 0 else { return }
+
+        findParentScrollView()?.isScrollEnabled = true
+
         currentPage = Int(scrollView.contentOffset.x / scrollView.frame.width)
 
         // 무한 스크롤: 양 끝에 도달하면 중간 범위로 리셋
         if !pages.isEmpty {
             if currentPage < pages.count {
-                // 왼쪽 끝 → 중간 범위로 점프
                 currentPage += pages.count
                 bannerCollectionView.scrollToItem(
                     at: IndexPath(item: currentPage, section: 0),
                     at: .centeredHorizontally,
                     animated: false
                 )
             } else if currentPage >= pages.count * 2 {
-                // 오른쪽 끝 → 중간 범위로 점프
                 currentPage -= pages.count
                 bannerCollectionView.scrollToItem(
                     at: IndexPath(item: currentPage, section: 0),
@@ -562,6 +573,17 @@ extension AdImageCollectionViewCell: UIScrollViewDelegate {
         updateAdPillVisibility()
         scheduleNextScroll()
     }
+
+    private func findParentScrollView() -> UIScrollView? {
+        var view = superview
+        while let v = view {
+            if let scrollView = v as? UIScrollView, scrollView !== bannerCollectionView {
+                return scrollView
+            }
+            view = v.superview
+        }
+        return nil
+    }
 }
 
 // MARK: - Carousel Setup
@@ -651,31 +673,34 @@ extension AdImageCollectionViewCell: UICollectionViewDelegate, UICollectionViewD
 
     func collectionView(_ collectionView: UICollectionView, cellForItemAt indexPath: IndexPath) -> UICollectionViewCell {
         let cell = collectionView.dequeueReusableCell(withReuseIdentifier: "BannerCell", for: indexPath)
-        cell.contentView.subviews.forEach { $0.removeFromSuperview() }
-
         guard !pages.isEmpty else { return cell }
 
         let pageIndex = indexPath.item % pages.count
         switch pages[pageIndex] {
         case .image(let image):
             cell.contentView.backgroundColor = .clear
-            let imageView = UIImageView().then {
-                $0.image = image
-                $0.contentMode = .scaleAspectFill
-                $0.clipsToBounds = true
-            }
-            cell.contentView.addSubview(imageView)
-            imageView.snp.makeConstraints {
-                $0.edges.equalToSuperview()
+            // 기존 UIImageView 재사용 — 매번 생성/제거 방지
+            if let imageView = cell.contentView.viewWithTag(100) as? UIImageView {
+                imageView.image = image
+            } else {
+                cell.contentView.subviews.forEach { $0.removeFromSuperview() }
+                let imageView = UIImageView()
+                imageView.tag = 100
+                imageView.contentMode = .scaleAspectFill
+                imageView.clipsToBounds = true
+                imageView.image = image
+                cell.contentView.addSubview(imageView)
+                imageView.snp.makeConstraints { $0.edges.equalToSuperview() }
             }
         case .ad(let adBannerView):
             cell.contentView.backgroundColor = .w1
-            adBannerView.removeFromSuperview()
-            cell.contentView.addSubview(adBannerView)
-            adBannerView.snp.makeConstraints {
-                $0.centerX.equalToSuperview()
-                $0.centerY.equalToSuperview()
-                $0.leading.trailing.equalToSuperview()
+            if adBannerView.superview !== cell.contentView {
+                cell.contentView.subviews.forEach { $0.removeFromSuperview() }
+                cell.contentView.addSubview(adBannerView)
+                adBannerView.snp.makeConstraints {
+                    $0.centerY.equalToSuperview()
+                    $0.leading.trailing.equalToSuperview()
+                }
             }
         }
         return cell
diff --git a/Runnect-iOS/Runnect-iOS/Presentation/CourseDiscovery/Views/VC/CourseDiscoveryVC.swift b/Runnect-iOS/Runnect-iOS/Presentation/CourseDiscovery/Views/VC/CourseDiscoveryVC.swift
@@ -44,6 +44,10 @@ final class CourseDiscoveryVC: UIViewController {
     /// 남은 아이템이 이 수 이하일 때 다음 페이지 프리페치 시작
     private let prefetchThreshold = 6
 
+    // MARK: - Banner Ad Cache
+
+    private var bannerAdsLoaded = false
+
     // MARK: - Native Ad Properties
 
     private static let nativeAdInterval = 10
@@ -105,6 +109,7 @@ final class CourseDiscoveryVC: UIViewController {
         setLayout()
         setAddTarget()
         setCombineEvent()
+        setupRefreshControl()
         loadNativeAds()
         self.getCourseData(pageNo: pageNo)
     }
@@ -215,12 +220,28 @@ extension CourseDiscoveryVC {
         return (item + 1) / adInterval - 1
     }
 
-    func refresh() {
-        print("refresh")
+    private func setupRefreshControl() {
+        let refreshControl = UIRefreshControl()
+        refreshControl.tintColor = .m1
+        refreshControl.addTarget(self, action: #selector(handlePullToRefresh), for: .valueChanged)
+        mapCollectionView.refreshControl = refreshControl
+    }
+
+    @objc private func handlePullToRefresh() {
+        refreshCourseList()
+    }
+
+    /// Section 4(courseList)만 새로고침 — courseList를 API 응답 전에 비우지 않아 data source 불일치 방지
+    func refreshCourseList() {
         pageNo = 1
         isFetchingData = false
-        self.courseList = []
-        self.getCourseData(pageNo: pageNo)
+        getCourseData(pageNo: pageNo)
+    }
+
+    /// 탭 재탭 시 스크롤만 — API 호출 없음
+    func scrollToTop() {
+        guard mapCollectionView.contentOffset.y > 0 else { return }
+        mapCollectionView.setContentOffset(.zero, animated: true)
     }
 }
 
@@ -346,7 +367,10 @@ extension CourseDiscoveryVC: UICollectionViewDelegate, UICollectionViewDataSourc
         switch indexPath.section {
         case Section.adImage:
             guard let cell = collectionView.dequeueReusableCell(withReuseIdentifier: AdImageCollectionViewCell.className, for: indexPath) as? AdImageCollectionViewCell else { return UICollectionViewCell() }
-            cell.setRootViewController(self)
+            cell.setRootViewController(self, skipReload: bannerAdsLoaded)
+            if !bannerAdsLoaded {
+                bannerAdsLoaded = true
+            }
             return cell
         case Section.marathonTitle:
             guard let cell = collectionView.dequeueReusableCell(withReuseIdentifier: MarathonTitleCollectionViewCell.className, for: indexPath) as? MarathonTitleCollectionViewCell else { return UICollectionViewCell() }
@@ -450,6 +474,10 @@ extension CourseDiscoveryVC: UICollectionViewDelegateFlowLayout {
     }
 
     func collectionView(_ collectionView: UICollectionView, willDisplay cell: UICollectionViewCell, forItemAt indexPath: IndexPath) {
+        if indexPath.section == Section.adImage, let bannerCell = cell as? AdImageCollectionViewCell {
+            bannerCell.resumeAutoScrollIfNeeded()
+        }
+
         guard indexPath.section == Section.courseList else { return }
 
         let totalItems = totalItemCount()
@@ -590,7 +618,7 @@ extension CourseDiscoveryVC: ScrapStateDelegate {
         // 왜??? 이미 데이터는 삭제가 되어서 $0.id 랑 publicCourseId 가 같은게 매치가 될 수 없어!!!
         // 네트워크 성공하기 전에 didRemoveCourse(publicCourseId:) 를 호출 해야 해당 부분 확인하고 지운다음, 서버측에서 지워야 1페이지부터 시작 안하고 지울 수 있음
         // ⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️⭐️
-        self.refresh()
+        self.refreshCourseList()
     }
 }
 
@@ -599,7 +627,7 @@ extension CourseDiscoveryVC: ScrapStateDelegate {
 extension CourseDiscoveryVC: UploadSuccessDelegate {
     func didUploadSuccess() {
         print("여기서 didUploadSuccess 함수 호출\n MyCourseSelectVC -> CourseDiscoveryVC 이벤트 전달")
-        self.refresh()
+        self.refreshCourseList()
         print("코스 발견 피드 새로고침 완료 되었음")
     }
 }
@@ -629,7 +657,7 @@ extension CourseDiscoveryVC: GADAdLoaderDelegate, GADNativeAdLoaderDelegate {
 
     func adLoaderDidFinishLoading(_ adLoader: GADAdLoader) {
         if !nativeAds.isEmpty {
-            mapCollectionView.reloadData()
+            mapCollectionView.reloadSections(IndexSet(integer: Section.courseList))
         }
     }
 
@@ -644,19 +672,22 @@ extension CourseDiscoveryVC {
     private func getCourseData(pageNo: Int) {
         isFetchingData = true
 
-        // 첫 페이지 로드 시에만 로딩 인디케이터 표시 (페이지네이션은 백그라운드 로딩)
         let isFirstPage = (pageNo == 1)
-        if isFirstPage {
+        let isRefreshing = mapCollectionView.refreshControl?.isRefreshing == true
+
+        // pull-to-refresh 중이면 자체 스피너 사용, 아니면 로딩 인디케이터
+        if isFirstPage && !isRefreshing {
             LoadingIndicator.showLoading()
         }
 
         publicCourseProvider.request(.getCourseData(pageNo: pageNo, sort: sort)) { [weak self] response in
             guard let self = self else { return }
 
-            if isFirstPage {
+            if isFirstPage && !isRefreshing {
                 LoadingIndicator.hideLoading()
             }
             self.isFetchingData = false
+            self.mapCollectionView.refreshControl?.endRefreshing()
 
             switch response {
             case .success(let result):
@@ -673,16 +704,13 @@ extension CourseDiscoveryVC {
                         let newCourses = data.publicCourses
 
                         if isFirstPage {
-                            // 첫 페이지(초기 로드 또는 정렬 변경): reloadData 사용
                             self.courseList = newCourses
-                            self.mapCollectionView.reloadData()
+                            // Section 4(courseList)만 갱신 — 배너·마라톤 등 불필요한 재구성 방지
+                            self.mapCollectionView.reloadSections(IndexSet(integer: Section.courseList))
                             self.emptyView.isHidden = !newCourses.isEmpty
                         } else {
-                            // 페이지네이션: performBatchUpdates + insertItems 사용
                             self.insertNewCourses(newCourses)
                         }
-
-                        print("pageNo= \(pageNo), isEnd= \(self.isEnd), totalPageNum= \(self.totalPageNum)")
                     } catch {
                         print(error.localizedDescription)
                     }
@@ -759,7 +787,6 @@ extension CourseDiscoveryVC: TitleCollectionViewCellDelegate {
         pageNo = 1
         isFetchingData = false
         sort = ordering
-        self.courseList.removeAll()
         getCourseData(pageNo: pageNo)
         
         switch ordering {
diff --git a/Runnect-iOS/Runnect-iOS/Presentation/TabBar/TabBarController.swift b/Runnect-iOS/Runnect-iOS/Presentation/TabBar/TabBarController.swift

Original file line number	Diff line number	Diff line change
`@@ -21,6 +21,6 @@`
`21`	`21`	`"version" : 1`
`22`	`22`	`},`
`23`	`23`	`"properties" : {`
`24`		`- "template-rendering-intent" : "original"`
	`24`	`+ "template-rendering-intent" : "template"`
`25`	`25`	`}`
`26`	`26`	`}`