Refactor/#413 코드리뷰 반영

[juri123123]

🔗 연결된 이슈

• Connected: [Refactor] 코드리뷰 반영 #413

📄 작업 내용

• Feat/#387 ai api 연동 #394 해당 PR의 코드리뷰를 반영했습니다.
• task의 취소를 관리할 수 있도록 변수로 선언하고 view가 사라질 때 cancel 및 메모리 해제 시켜주었습니다.
• AI 답장 존재 여부에 따라 분기하던 로직을 usecase로 변경했습니다.

💻 주요 코드 설명

AIAnswerViewModel

• viewDidDisappear에서 cancelTask를 호출해줬습니디ㅓㅏ!

private var fetchAIAnswerTask: Task<Void, Never>?

private func fetchAIAnswer() {
        fetchAIAnswerTask?.cancel()
        
        fetchAIAnswerTask = Task {
            do {
                AILoadingSubject.send(true)
                
                let answer = try await fetchAIAnswerUseCase.execute(
                    questID: questID,
                    isAnswerExists: isAnswerExists
                ).AIAnswer
                AIResultSubject.send(.success(answer))
                
                AILoadingSubject.send(false)
            } catch {
                guard let error = error as? ByeBooError else {
                    return
                }
                AIResultSubject.send(.failure(error))
                AILoadingSubject.send(false)
            }
        }
    }
    
    private func cancelTask() {
        fetchAIAnswerTask?.cancel()
        fetchAIAnswerTask = nil
    }

[y-eonee]

고생하셨습니당

[y-eonee]

요 주석은 무엇인가용??

[juri123123]

요 주석은 error를 던지는 경우를 테스트하기 위함입니닷!!!!

[y-eonee]

우와..저도 이번 스프린트부터 적용해보겠습니다!!

[dev-domo]

Task 취소 관리 좋네요!

[dev-domo]

답장 존재 유무와 상관없이 계속 create을 하게 되나요?

[juri123123]

답장 존재 유무를 판단하고 fetch 할지 create 할지 결정하는 역할을 usecase로 넘겼습니다 !

[dev-domo]

fetchAIAnswerTask?.cancel() 대신 만들어놓은 cancelTask를 호출하는 방법도 괜찮을 것 같아요!

[juri123123]

좋은 의견 감사합니다 !!!
추후 fetch하는 task가 많아질 경우 cancelTask 내부에 task들을 취소하는 로직을 모두 넣을 예정이라 통일성을 위해 분리해서 사용할 수 있도록 이렇게 작성했습니닷 !