Merge pull request #345 from boostcampwm2023/iOS/task/Background-Task

[iOS] 백그라운드 업데이트

iOS/Features/Home/Sources/Home/Presentation/HomeViewController.swift

@@ -5,6 +5,7 @@
 //  Created by 이창준 on 2023.12.01.
 //
 
+import ActivityKit
 import Combine
 import UIKit
 
@@ -33,6 +34,7 @@ public final class HomeViewController: HomeBottomSheetViewController {
     private enum Metric {
 
         static let startButtonBottomInset: CGFloat = 16.0
+        static let recordJourneyButtonBottomInset: CGFloat = 50.0
 
         enum RefreshButton {
             static let topSpacing: CGFloat = 80.0
@@ -133,7 +135,7 @@ public final class HomeViewController: HomeBottomSheetViewController {
         self.viewModel.state.startedJourney
             .receive(on: DispatchQueue.main)
             .sink { [weak self] startedJourney in
-                self?.contentViewController.journeyShouldStarted(startedJourney)
+                self?.contentViewController.recordingShouldStart(startedJourney)
             }
             .store(in: &self.cancellables)
 
@@ -146,13 +148,14 @@ public final class HomeViewController: HomeBottomSheetViewController {
 
         self.viewModel.state.isRecording
             .removeDuplicates()
+            .dropFirst()
             .receive(on: DispatchQueue.main)
             .sink { [weak self] isRecording in
                 if isRecording {
                     self?.hideBottomSheet()
                 } else {
                     self?.showBottomSheet()
-                    self?.contentViewController.journeyShouldStopped(isCancelling: false)
+                    self?.contentViewController.recordingShouldStop(isCancelling: false)
                 }
                 self?.updateButtonMode(isRecording: isRecording)
             }
@@ -187,22 +190,14 @@ public final class HomeViewController: HomeBottomSheetViewController {
     private func updateButtonMode(isRecording: Bool) {
         UIView.transition(with: startButton, duration: 0.5,
                           options: .transitionCrossDissolve,
-                          animations: {
-            self.startButton.isHidden = isRecording
+                          animations: { [weak self] in
+            self?.startButton.isHidden = isRecording
         })
         UIView.transition(with: recordJourneyButtonStackView, duration: 0.5,
                           options: .transitionCrossDissolve,
-                          animations: {
-            self.recordJourneyButtonStackView.isHidden = !isRecording
+                          animations: { [weak self] in
+            self?.recordJourneyButtonStackView.isHidden = !isRecording
         })
-
-        if isRecording {
-            self.setUserLocationToCenter()
-        }
-    }
-
-    private func setUserLocationToCenter() {
-
     }
 
 }
@@ -213,27 +208,34 @@ extension HomeViewController: RecordJourneyButtonViewDelegate {
 
     private func configureAction() {
         let startButtonAction = UIAction { [weak self] _ in
-            guard let userLocation = self?.contentViewController.userLocation else { return }
-
-            let coordinate = Coordinate(latitude: userLocation.coordinate.latitude,
-                                        longitude: userLocation.coordinate.longitude)
-            self?.viewModel.trigger(.startButtonDidTap(coordinate))
+            self?.startButtonDidTap()
         }
         self.startButton.addAction(startButtonAction, for: .touchUpInside)
 
         let refreshButtonAction = UIAction { [weak self] _ in
-            guard let coordinates = self?.contentViewController.visibleCoordinates else { return }
-
-            self?.viewModel.trigger(.refreshButtonDidTap(visibleCoordinates: coordinates))
+            self?.refreshButtonDidTap()
         }
         self.refreshButton.addAction(refreshButtonAction, for: .touchUpInside)
     }
 
+    private func startButtonDidTap() {
+        guard let userLocation = self.contentViewController.userLocation else { return }
+
+        let coordinate = Coordinate(latitude: userLocation.coordinate.latitude,
+                                    longitude: userLocation.coordinate.longitude)
+        self.viewModel.trigger(.startButtonDidTap(coordinate))
+    }
+
+    private func refreshButtonDidTap() {
+        let coordinates = self.contentViewController.visibleCoordinates
+        self.viewModel.trigger(.refreshButtonDidTap(visibleCoordinates: coordinates))
+    }
+
     public func backButtonDidTap(_ button: MSRectButton) {
         guard self.viewModel.state.isRecording.value == true else { return }
 
         self.viewModel.trigger(.backButtonDidTap)
-        self.contentViewController.journeyShouldStopped(isCancelling: true)
+        self.contentViewController.recordingShouldStop(isCancelling: true)
     }
 
     public func spotButtonDidTap(_ button: MSRectButton) {
@@ -289,7 +291,7 @@ private extension HomeViewController {
         NSLayoutConstraint.activate([
             self.recordJourneyButtonStackView.centerXAnchor.constraint(equalTo: self.view.centerXAnchor),
             self.recordJourneyButtonStackView.bottomAnchor.constraint(equalTo: safeAreaBottomAnchor,
-                                                                      constant: -Metric.startButtonBottomInset)
+                                                                      constant: -Metric.recordJourneyButtonBottomInset)
         ])
 
         self.view.insertSubview(self.refreshButton, belowSubview: self.bottomSheetViewController.view)

iOS/Features/Home/Sources/Home/Presentation/HomeViewModel.swift

@@ -5,6 +5,7 @@
 //  Created by 이창준 on 2023.12.06.
 //
 
+import ActivityKit
 import Combine
 import Foundation
 

iOS/Features/Home/Sources/NavigateMap/Presentation/Common/MapViewController+EventListener.swift

@@ -27,7 +27,7 @@ extension MapViewController {
 
 extension MapViewController {
 
-    public func journeyShouldStarted(_ startedJourney: RecordingJourney) {
+    public func recordingShouldStart(_ startedJourney: RecordingJourney) {
         guard self.viewModel is NavigateMapViewModel else {
             MSLogger.make(category: .home).error("여정이 시작되어야 하지만 이미 Map에서 RecordJourneyViewModel을 사용하고 있습니다.")
             return
@@ -39,9 +39,16 @@ extension MapViewController {
                                                             userRepository: userRepository,
                                                             journeyRepository: journeyRepository)
         self.swapViewModel(to: recordJourneyViewModel)
+
+        self.locationManager.startUpdatingLocation()
+        self.locationManager.allowsBackgroundLocationUpdates = true
+
+        #if DEBUG
+        MSLogger.make(category: .home).debug("여정 기록이 시작되었습니다.")
+        #endif
     }
 
-    public func journeyShouldStopped(isCancelling: Bool) {
+    public func recordingShouldStop(isCancelling: Bool) {
         guard let viewModel = self.viewModel as? RecordJourneyViewModel else {
             MSLogger.make(category: .home).error("여정이 종료되어야 하지만 이미 Map에서 NavigateMapViewModel을 사용하고 있습니다.")
             return
@@ -54,6 +61,13 @@ extension MapViewController {
         let journeyRepository = JourneyRepositoryImplementation()
         let navigateMapViewModel = NavigateMapViewModel(repository: journeyRepository)
         self.swapViewModel(to: navigateMapViewModel)
+
+        self.locationManager.stopUpdatingLocation()
+        self.locationManager.allowsBackgroundLocationUpdates = false
+
+        #if DEBUG
+        MSLogger.make(category: .home).debug("여정 기록이 종료되었습니다.")
+        #endif
     }
 
 }

iOS/Features/Home/Sources/NavigateMap/Presentation/Common/MapViewController.swift

@@ -21,8 +21,8 @@ public final class MapViewController: UIViewController {
 
     private enum Typo {
 
-        static let locationAlertTitle = "위치 권한"
-        static let locationAlertMessage = "위치 권한을 허용하시지 않아서 위치 접근이 불가능합니다."
+        static let locationAlertTitle = "위치 권한이 허용되지 않음"
+        static let locationAlertMessage = "MusicSpot은 위치 권한을 필요합니다. 설정에서 \"앱을 사용하는 동안\" 이상의 권한을 허용해주세요."
         static let locationAlertCancel = "취소"
         static let locationAlertSettings = "설정"
 
@@ -65,7 +65,7 @@ public final class MapViewController: UIViewController {
     public weak var delegate: MapViewControllerDelegate?
     var viewModel: (any MapViewModel)?
 
-    private let locationManager = CLLocationManager()
+    internal let locationManager = CLLocationManager()
 
     private var cancellables: Set<AnyCancellable> = []
 
@@ -118,7 +118,7 @@ public final class MapViewController: UIViewController {
 
         self.configureLayout()
         self.configureCoreLocation()
-        self.bind(viewModel)
+        self.bind(self.viewModel)
     }
 
     // MARK: - Combine Binding
@@ -169,6 +169,7 @@ public final class MapViewController: UIViewController {
                 }
                 return (previousCoordinate, currentCoordinate)
             }
+            .receive(on: DispatchQueue.main)
             .sink { [weak self] previousCoordinate, currentCoordinate in
                 let points = [previousCoordinate, currentCoordinate]
                 self?.drawPolylineToMap(using: points)
@@ -294,10 +295,17 @@ private extension MapViewController {
         ])
     }
 
-    func configureCoreLocation() {
+    private func configureCoreLocation() {
         self.locationManager.delegate = self
-        self.locationManager.requestWhenInUseAuthorization()
-        self.locationManager.desiredAccuracy = kCLLocationAccuracyNearestTenMeters
+        switch self.locationManager.authorizationStatus {
+        case .restricted, .denied:
+            self.presentLocationAuthorizationAlert()
+        case .notDetermined:
+            self.locationManager.requestWhenInUseAuthorization()
+        default:
+            return
+        }
+        self.locationManager.desiredAccuracy = kCLLocationAccuracyBestForNavigation
     }
 
 }
@@ -341,21 +349,8 @@ extension MapViewController: CLLocationManagerDelegate {
         case .notDetermined:
             manager.requestWhenInUseAuthorization()
         case .restricted, .denied:
-            let sheet = UIAlertController(title: Typo.locationAlertTitle,
-                                          message: Typo.locationAlertMessage,
-                                          preferredStyle: .alert)
-            let cancelAction = UIAlertAction(title: Typo.locationAlertCancel, style: .cancel)
-            let settingsAction = UIAlertAction(title: Typo.locationAlertSettings, style: .default) { _ in
-                guard let url = URL(string: UIApplication.openSettingsURLString) else { return }
-                UIApplication.shared.open(url)
-            }
-            sheet.addAction(cancelAction)
-            sheet.addAction(settingsAction)
-            DispatchQueue.main.async {
-                self.present(sheet, animated: true)
-            }
+            self.presentLocationAuthorizationAlert()
         case .authorizedAlways, .authorizedWhenInUse:
-            manager.startUpdatingLocation()
             self.checkAccuracyAuthorizationStatus(manager)
         @unknown default:
             MSLogger.make(category: .home).error("잘못된 위치 권한입니다.")
@@ -382,6 +377,22 @@ extension MapViewController: CLLocationManagerDelegate {
         return 5 <= location1.distance(from: location2) && location1.distance(from: location2) <= 50
     }
 
+    private func presentLocationAuthorizationAlert() {
+        let sheet = UIAlertController(title: Typo.locationAlertTitle,
+                                      message: Typo.locationAlertMessage,
+                                      preferredStyle: .alert)
+        let cancelAction = UIAlertAction(title: Typo.locationAlertCancel, style: .cancel)
+        let settingsAction = UIAlertAction(title: Typo.locationAlertSettings, style: .default) { _ in
+            guard let url = URL(string: UIApplication.openSettingsURLString) else { return }
+            UIApplication.shared.open(url)
+        }
+        sheet.addAction(cancelAction)
+        sheet.addAction(settingsAction)
+        DispatchQueue.main.async {
+            self.present(sheet, animated: true)
+        }
+    }
+
 }
 
 // MARK: - MKMapView