[FEAT] 상상의 닉네임 뷰 구현 및 리팩토링(?) #48
Open
+470
−56
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Open
sozohoy
reviewed
Jun 5, 2024
| import Moya | ||
|
|
||
|
|
||
| final class HomeViewModel: ObservableObject { |
There was a problem hiding this comment.
- viewModel에서 콜백지옥이 우려된다면 Moya를 async / await 방식으로 사용해보려고 구현해보아도 괜찮을 지도..?!
| func request<T: Decodable>(_ target: Target, completion: @escaping (Result<T, Error>) -> Void) | ||
| } | ||
|
|
||
| class NetworkService<T: TargetType>: NetworkServiceProtocol { |
There was a problem hiding this comment.
- 추상화 되게 깔끔하게 잘 한 것 같은데?? 나도 Rx를 많이 써보지 않아서, Rx를 사용하면 어떤 추가적인 장점이 있을 것 같아?
Comment on lines
+40
to
+52
| networkService.getQuestionDescription(requestDTO: request) { [weak self] result in | ||
| DispatchQueue.main.async { | ||
| switch result { | ||
| case .success(let response): | ||
| let data = response.data | ||
| self?.questionTitle = data.question | ||
| self?.answerText = data.answer | ||
| self?.questionDescription = data.solution | ||
| case .failure(let error): | ||
| print(error) | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
- 이런 부분이 걱정된다면, 레포지토리를 중간에 놔둬도 괜찮을 것 같아!
- 지희가 생각하기에 이 부분에서 Rx를 사용할 경우 어떤 이점이 존재할 것 같아?
There was a problem hiding this comment.
사실 좀 편하게 해보려고 rx 도입 생각한건데, 뭔가 다시 생각해보니까 라이브러리 도입 없이도 충분히 고칠 수 있을 것 같다!
아키텍처부터 좀 다시 고민해볼게요!!
| @@ -10,21 +10,26 @@ import Foundation | |||
| extension DTO { | |||
| import Moya | ||
|
|
||
| // MARK: Attempt | ||
| class AttemptNetworkService: NetworkService<APITarget.Attempts> { |
There was a problem hiding this comment.
- 이 부분도 requestDTO를 제한하지 않고 추상화 시킬 수 있을 것 같아!!! 하지만 따로 제한해놓은 이유가 있다면 괜찮은 것 같고!!
There was a problem hiding this comment.
음음 서비스를 이미 추상화해서 여기까지 추상화할 생각은 안해봤네
여기도 한번 다시 살펴 보겠습니다~~
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#️⃣연관된 이슈
#47
📝작업 내용 요약
우선 매우매우 많은 파일이 바뀐 점, PR 단위가 너무 커진 점 양해 부탁드립니다. 😢
닉네임 뷰
처음 닉네임을 생성하고 userId를 가지고 있어야 가능한 통신이 있어서 닉네임 생성 뷰를 만들어 두었습니다.
임시로
UserDefaults로 관리합니다.❗️전체 연결이 아니기 때문에 일부 로직에서 서버 통신 실패 일어나는 부분 있습니다❗️
ex. 사용자 퀴즈 풀이가 없는데 questionId 이용해서 서버 통신 -> 404 발생
💭 기술적 고민
Moya 네트워크 계층을 추상화 해보려 했는데, 뷰모델까지 콜백 지옥에 빠져서 Rx 사용 고민 중 입니다.
MVVM 으로 리팩토링 중인데 더 좋은 방법이 있다면 꼭 코드리뷰 부탁드립니다!
스크린샷 (선택)