[FEAT] API 버저닝을 위한 ACTargetType 구현, 리뷰 API 수정 (#242)

[yurim830]

🐿️ Pull Requests

🪵 작업 브랜치

• [FEAT] API 버저닝을 위한 ACTargetType 구현, 리뷰 API 수정 #242

🥔 작업 내용

1. API 버저닝을 위해 ACTargetType을 구현했습니다. (Moya의 TargetType 프로토콜 상속)
우리팀 컨벤션상으로는 ACTargetTypeProtocol식으로 네이밍해야하지만,
길이가 길고, Moya의 TargetType이라는 느낌이 약해지는 것 같아 -protocol을 생략했습니다.
혹시 ACTargetTypeProtocol 으로 하는 게 좋겠다고 생각하시면 말씀해주세요!

Code

import Moya

protocol ACTargetType: TargetType {
    var apiVersion: ApiVersionType { get }
}

extension ACTargetType {

    /// API 버전 기본값 (.v1). 필요 시 각 TargetType에서 오버라이드하여 변경 가능.
    var apiVersion: ApiVersionType {
        return .v1
    }

    var utilPath: String {
        return apiVersion.utilPath
    }

}

Usage

extension UploadTargetType: ACTargetType {

    var apiVersion: ApiVersionType {
        switch self {
        case .postReview:
            return .v2
        default:
            return .v1
        }
    }

    var path: String {
        switch self {
        case .getSearchSuggestion:
            return utilPath + "spots/search-suggestions"
    ...

2. 리뷰 API 변경사항을 반영했습니다.

• 리뷰 작성 API
• 리뷰 작성 시 위치 인증 API
• 추천 검색어 목록 조회 API

📸 스크린샷

💥 To be sure

• 모든 뷰가 잘 실행되는지 다시 한 번 체크해주세요 !

🌰 Resolve issue

• Resolved: [FEAT] API 버저닝을 위한 ACTargetType 구현, 리뷰 API 수정 #242

[cirtuare]

리뷰 확인 부탁드립니다 ~~!

[cirtuare]

🐿️🐿️🐿️
굳이 ACTargetType을 따로 만드신 이유가 있나요?? 기존 TargetType+에서 apiVersion을 선언했어도 필요한 경우만 오버라이딩하면 돼서 불필요하다고 생각합니다..!

[yurim830]

b1e8428 에서 처음에 그렇게 구현했었는데,
코드를 호출해보니
protocol extension의 기본 구현은 오버라이딩이 안 되며,
가능하게 하려면 따로 프로토콜을 구현해야함을 알게 되었습니다. (참고)

오버라이딩을 포기하고 각각의 TargetType 내에서
아래처럼 손수 utilPath를 구현하자니 확장성이 너무 떨어져서,
위 코드처럼 ACTargetType 프로토콜을 만들게 되었습니다

    var apiVersion: ApiVersion {
        switch self {
        case .getSearchSuggestion,
            return .v2
        default:
            return .v2
        }
    }

    var utilPath: String {
        return apiVersion.path
    }

   var path: String {
        switch self {
        case .getSearchSuggestion:
            return utilPath + "search-suggestions"
 ....

[cirtuare]

확인했습니다 !!!!