[Feat] #271 - 서비스 레이어 추가했습니다.

[wjdalswl]

🩵 Issue

close #271

💙 변경된 내용

서비스 레이어 추가했습니다.

• SearchService(API 3개)
• MypageService(API 2개)
• AnnouncementsService(API 1개)

🅿️ PR Point

각각의 ServiceProtocol의 func의 파라미터와 반환값은 기존에 있던 API 호출 코드 그대로 반영해서 작성했습니다!
Service안의 func은 전부 API 호출한 후 응답 DTO 변환하여 반환하는 모습으로 작성되었습니다.
서비스는 호출만 하는 역할이라고 생각해서, 딱히 다른 코드를 추가하지 않았는데 더 필요한게 있을까요?

Service안의 func 이름들은 전부 TargetType의 case이름과 통일 시켰습니다. 공고 상세 조회하기의 경우에만 TargetType의 case이름이 getAnnouncements이길래, 다른 공고 조회 API과 헷갈릴 것 같아서 getDetailAnnouncements로 바꿔서 작성했습니다.

예시) 기존 코드

extension SearchResultViewModel {
    func fetchJobCards(keyword: String, sortBy: String, page: Int, size: Int) -> Observable<SearchResultModel> {
        return Observable.create { observer in
            let request = self.searchProvider.request(
                .getSearchResult(
                    keyword: keyword,
                    sortBy: sortBy,
                    page: page,
                    size: size
                )
            ) { result in
                switch result {
                case .success(let response):
                    let status = response.statusCode
                    if 200..<300 ~= status {
                        do {
                            let responseDto = try response.map(BaseResponse<SearchResultModel>.self)
                            if let data = responseDto.result {
                                observer.onNext(data)
                                observer.onCompleted()
                            } else {
                                observer.onError(NSError(domain: "", code: -1, userInfo: [NSLocalizedDescriptionKey: "No data available"]))
                            }
                        } catch {
                            print(error.localizedDescription)
                            observer.onError(error)
                        }
                    } else if status >= 400 {
                        observer.onError(NSError(domain: "", code: status, userInfo: [NSLocalizedDescriptionKey: "Error with status code: \(status)"]))
                    }
                case .failure(let error):
                    print(error.localizedDescription)
                    observer.onError(error)
                }
            }
            return Disposables.create {
                request.cancel()
            }
        }
    }
}

예시) 서비스 코드

func getSearchResult(keyword: String, sortBy: String, page: Int, size: Int) -> Observable<SearchResultModel> {
        return provider.rx.request(.getSearchResult(keyword: keyword, sortBy: sortBy, page: page, size: size))
            .filterSuccessfulStatusCodes()
            .map(BaseResponse<SearchResultModel>.self)
            .compactMap { $0.result }
            .asObservable()
    }

[thingineeer]

Service로 정리한 이 코드와, 변경하기 전(PR에 적은 긴 서버 통신 코드) 코드는 어떤 차이점이 있을까요?

[wjdalswl]

ViewModel에 존재하던 이전 서버 통신 코드에서 데이터 가공이나 UI 업데이트 코드 등을 제외하고 네트워크 호출 부분만 가져와서 Service에 넣었어요..!

[thingineeer]

아 제가 말한 거는 예외 처리에서 차이가 있다!라는 것을 말씀드리려고 했습니다.

현재 코드는 filterSuccessfulStatusCodes
200에서 299번까지 성공 코드일 때만 필터가 되기 때문에
mapper 문제인지, 서버 문제인지 어디서 문제가 발생하는지 구분하는 데에 어려울 수 있다는 점 말하려고 했어요!

저도 어떻게 해야 서버 통신을 좋게 할 수 있을지 고민하고 있기 때문에 회의 때 이 부분도 말하면 좋을 것 같습니다~

[thingineeer]

고생하셨습니다~

[thingineeer]

아 제가 말한 거는 예외 처리에서 차이가 있다!라는 것을 말씀드리려고 했습니다.

현재 코드는 filterSuccessfulStatusCodes
200에서 299번까지 성공 코드일 때만 필터가 되기 때문에
mapper 문제인지, 서버 문제인지 어디서 문제가 발생하는지 구분하는 데에 어려울 수 있다는 점 말하려고 했어요!

저도 어떻게 해야 서버 통신을 좋게 할 수 있을지 고민하고 있기 때문에 회의 때 이 부분도 말하면 좋을 것 같습니다~