PLUB 네트워크 로직 리팩토링

[WhiteHyun]

📌 PR 요약

🌱 작업한 내용

• 기존 NetworkResult타입에 존재했던 Error를 PLUBError로 이관하였고, enum case도 일부 수정했습니다.
  • pathError 제거: path가 잘못되면 결국 requestError로 넘어가게 되므로 불필요한 Error라 판단하여 삭제했습니다.
  • decodingError 추가: API 모델을 잘못 구성하여 데이터가 제대로 파싱되지 못하는 경우, success임에도 data프로퍼티가 nil로 처리되었습니다. 이제는 이를 감지하여 디코딩하려 헀던 data값과 함께 Error로 넘겨주도록 구현하였습니다.
  • unknownedError 추가: PLUBError로 직접 해결할 수 없는 에러로, statusCode값을 받아오지 못했거나 statusCode가 비정상적인 값으로 받아졌을 때 해당 Error로 처리됩니다. 이 에러값은 불릴 일이 전혀 없을 것 같지만, 혹시 몰라 처리해두었습니다. 😅

🌱 PR 포인트

• sendRequest와 evaluateStatus는 이제 레거시 코드가 되었습니다..! 새로운 메서드인 sendObservableRequest를 사용해 주세요.

  • sendObservableRequest를 사용하게 되면 sendRequest에서 항상 보내야했던 type 파라미터를 넣을 필요가 없습니다(1번 사진 참고).
  • sendObservableRequest를 사용하면, data값이 Observable로 그대로 리턴되어 보다 깔끔하게 코드를 구성할 수 있습니다.(2번 사진, 3번 사진 참고)

• 에러는 이제 onError에서 처리할 수 있도록 수정했습니다. 기존에는 error도 success로 취급되어 들어갔으나, 이제 subscribe의 onError 클로저에서 처리하시면 됩니다.

// observable
.subscribe { _ in
} onError: { error in
  guard let plubError = error as? PLUBError<GeneralResponse<FeedsContent>> else { return }
  switch plubError {
    // 디코딩 미스인 경우
  case .decodingError(let rawData):
   // 여기서 rawData를 찍어서 어떻게 받아오는 지 확인할 수 있습니다. JSONSerialization을 애용해주세용
    print(rawData)
    
  case .requestError(let response):
    // 요청이 잘못된 경우, ex: statusCode: 9010
    print(response.statusCode)
  default: // 그 외 여러 에러들..
    // Log 처리
    break
  }
}

📸 스크린샷

1번 사진
2번 사진
3번 사진

📮 관련 이슈

• [REFACTOR] Network Model #255

[dlrjswns]

흥 어쩔티비

[soobin-k]

고생하셨습니다 승현님!!👍😊

[WhiteHyun]

흥 어쩔티비

우우우우