피드 상세화면에 필요한 Domain영역과 Data 영역을 정의해 보았습니다. #13

GeonH0 · 2024-07-03T13:03:15Z

RecipeDetailFetchService 정의
피드 상세화면에 필요한 네트워크 서비스를 정의했습니다.
recipeId를 인스턴스로 받아서 URL 컴포넌트로 사용합니다.
RecipeDetailDTO 정의
서버로부터 받은 JSON 데이터를 디코딩하고 도메인 모델로 변환할 DTO(Data Transfer Object)를 정의했습니다.
RecipeDetailRepository 정의
서버로부터 데이터를 받아오는 레포지토리를 정의했습니다.
RecipeDetailFetchService를 사용하여 데이터를 가져옵니다.
RecipeDetailInteractor 정의
피드 상세화면에 알맞는 비즈니스 로직을 전달하는 인터랙터를 정의했습니다.
FetchRecipeDetailUseCase를 통해 비즈니스 로직을 처리합니다.
FetchRecipeDetailUseCase 정의
피드 상세화면에 필요한 비즈니스 로직을 담은 유즈 케이스를 정의했습니다.
레포지토리와 상호작용하여 데이터를 가져오고 가공합니다.

f-lab-barry · 2024-07-03T14:56:15Z

HomeCafeRecipes/HomeCafeRecipes/Data/Network/RecipeDetailFetchService.swift

+import RxSwift
+
+protocol RecipeDetailFetchService {
+    func fetchRecipeDetail(recipeId: Int) -> Single<Recipe>


Suggested change

func fetchRecipeDetail(recipeId: Int) -> Single<Recipe>

func fetchRecipeDetail(recipeID: Int) -> Single<Recipe>

이 파일 전체적으로 고쳐주세요.

[6178184] 수정했습니다

f-lab-barry · 2024-07-03T14:56:31Z

HomeCafeRecipes/HomeCafeRecipes/Data/Network/RecipeDetailFetchService.swift

+    func fetchRecipeDetail(recipeId: Int) -> Single<Recipe>
+}
+
+class RecipeDetailFetchServiceImpl: RecipeDetailFetchService {


여기엔 final 필요 없을까요?

필요 한거 같아요 final 추가 하겠습니다

[e117629] 수정했습니다

f-lab-barry · 2024-07-03T14:57:42Z

HomeCafeRecipes/HomeCafeRecipes/Data/Network/RecipeDetailFetchService.swift

+
+    func fetchRecipeDetail(recipeId: Int) -> Single<Recipe> {
+        guard let URL = makeURL(recipeId: recipeId) else {
+            return Single.error(NSError(domain: "URLComponentsError", code: -1, userInfo: [NSLocalizedDescriptionKey: "Invalid URL"]))


에러를 정의해보는거 어때요?

[8f79c55],[c91393f] 정의후 적용시켰습니다

f-lab-barry · 2024-07-03T14:59:26Z

HomeCafeRecipes/HomeCafeRecipes/Data/Repositories/RecipeDetailRepository.swift

+import Foundation
+import RxSwift
+
+protocol RecipeDetailRepository {


RecipeDetailFetchService 랑 분리한 이유가 있나요..?

RecipeDetailFetchService는 네트워크 통신만을 담당하고 RecipeDetailRepository는 데이터 접근 로직을 추상화 할수 있습니다.

사실상 서비스에서는 url을 만드는 것 외에 복잡한 통신로직은 없는 것 같은데 이에 비해 레이어가 많은 것 같아요.
어떻게 생각하시나요~?

[cda676a] service를 삭제후 repository에 service가 하는역할을 추가했습니다

f-lab-barry · 2024-07-03T15:00:49Z

HomeCafeRecipes/HomeCafeRecipes/Domain/Interactor/RecipeDetailInteractor.swift

+        return recipeDetailSubject.asObservable()
+    }
+
+    init(fetchRecipeDetailUseCase: FetchRecipeDetailUseCase, recipeID: Int) {


개행 부탁드려요

[9ba8620] 수정했습니다

f-lab-barry · 2024-07-03T15:02:55Z

HomeCafeRecipes/HomeCafeRecipes/Domain/Interactor/RecipeDetailInteractor.swift

+protocol OutputRecipeDetailInteractor {
+    var recipe: Observable<Result<Recipe, Error>> { get }
+}


이건 어떻게 활용되는거에요?

recipeDetailSubject를 Observable로 반환함으로써 외부에서 데이터를 읽기 전용으로 사용할 수 있습니다

f-lab-barry · 2024-07-03T15:03:41Z

HomeCafeRecipes/HomeCafeRecipes/Domain/Interactor/RecipeDetailInteractor.swift

+        recipe
+            .subscribe(onNext: { [weak self] result in
+                self?.delegate?.fetchedRecipe(result: result)
+            })
+            .disposed(by: disposeBag)


바로 델리게이트를 호출해도 될 것 같은데 recipe를 통하는 이유가 궁금해요.
output과 delegate가 같은 역할 같아서요.

recipe를 통해서 비동기 데이터를 전달하고, Delegate는 이벤트만을 처리하기 위해 만들어 둔것 입니다.

f-lab-barry · 2024-07-05T12:03:11Z

HomeCafeRecipes/HomeCafeRecipes/Domain/Entities/RecipeError.swift

+
+import Foundation
+
+enum RecipeError: Error {


레시피상세에서 발생할 수 있는 에러인지, 리스트에서 발생할 수있는 에러인지 모호한 것 같은데 건호님 생각은 어때요?

구분지어야 할거 같아요 RecipeDeatilError로 네이밍 수정하겠습니다!

[6f3afb4] 네이밍 수정했습니다!

f-lab-barry · 2024-07-09T11:18:04Z

HomeCafeRecipes/HomeCafeRecipes/Data/Repositories/RecipeDetailRepository.swift

-    private let networkService: RecipeDetailFetchService
+class RecipeDetailRepositoryImpl: RecipeDetailRepository {
+    private let networkService: NetworkService
+    private static let baseURL: URL = URL(string: "https://meog0.store/api")!


공통 객체로 분리해보기
-> baseURL이 변경되었을 때 수정범위가 넓음

f-lab-barry · 2024-07-09T11:22:18Z

HomeCafeRecipes/HomeCafeRecipes/Data/Network/RecipeDetailFetchService.swift

@@ -26,15 +26,15 @@ final class RecipeDetailFetchServiceImpl: RecipeDetailFetchService {

    func fetchRecipeDetail(recipeID: Int) -> Single<Recipe> {
        guard let url = makeURL(recipeID: recipeID) else {
-            return Single.error(RecipeError.invalidURL)
+            return Single.error(RecipeDetailError.invalidURL)


usecase
func foo() {
fetchRecipeDetail(recipeID: 100)... { onFailure: { [weak self] error in ..
var errorMessage: String {
switch error {
case .inbalidUR: "페이지를 찾을 수 없어요."
}
}
} }
}

f-lab-barry · 2024-07-09T11:37:53Z

HomeCafeRecipes/HomeCafeRecipes/Domain/Interactor/RecipeDetailInteractor.swift

+protocol InputRecipeDetailInteractor {
+    func viewDidLoad()
+}


InputRecipeDetailInteractor -> RecipeDetailInteractor

RecipeDetailInteractorImpl: RecipeDetailInteractor
// vc에서 행동에 따른 비즈니스 로직 호출 ex) viewDidLoad
// 비즈니스 로직 -> return 도메인 모델 -> delegate로 vc에 전달 -> mapper -> return 뷰 모델 -> 뷰 업데이트

output 빼고

f-lab-barry · 2024-07-10T13:30:01Z

HomeCafeRecipes/HomeCafeRecipes/Data/Network/RecipeFetchService.swift


-    init(networkService: NetworkService) {
+    init(networkService: NetworkService, apiConfig: APIConfig) {


baseURL은 주입받아서 들고있을 필요 없이 public으로 열어두고 쓰는건 어때요?
레포지토리에서 무조건 봐야하는 모델인데 주입받을 필요는 없을 것 같아서요!

sonarqubecloud · 2024-07-11T04:16:00Z

Quality Gate passed

Issues
5 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

GeonH0 added 5 commits July 3, 2024 21:42

Feat: RecipeDetailDTO정의

6eb97ea

Feat: RecipeDetailFetchService 정의

bab43de

Feat: RecipeDetailRepository 정의

4b09f04

Feat: FetchRecipeDetailUseCase 정의

3f7f570

Feat: RecipeDetailInteractor 정의

429a962

f-lab-barry reviewed Jul 3, 2024

View reviewed changes

GeonH0 added 5 commits July 5, 2024 13:43

Fix: recipeID로 네이밍 수정

6178184

Fix: RecipeDetailFetchServiceImpl final 추가

e117629

Feat: RecipeError 정의

8f79c55

Fix: 정의된 Error 적용

c91393f

Fix: 개행 적용

9ba8620

f-lab-barry reviewed Jul 5, 2024

View reviewed changes

GeonH0 added 2 commits July 9, 2024 16:33

Fix: RecipeDetailError로 네이밍 변경

ed07510

Fix: RecipeDetailFetchService삭제후 Repository에 Service를 적용

cda676a

GeonH0 force-pushed the feature/FeedDetailDomainData branch from 2fb942f to cda676a Compare July 9, 2024 07:34

f-lab-barry reviewed Jul 9, 2024

View reviewed changes

f-lab-barry reviewed Jul 10, 2024

View reviewed changes

GeonH0 force-pushed the feature/FeedDetailDomainData branch from 90fa58c to a388cf6 Compare July 11, 2024 04:09

Fix: APIConfig생성후 각 네트워크 서비스에 적용

dbf2111

GeonH0 force-pushed the feature/FeedDetailDomainData branch from a388cf6 to dbf2111 Compare July 11, 2024 04:14

Merge branch 'main' into feature/FeedDetailDomainData

02a63aa

f-lab-barry approved these changes Jul 11, 2024

View reviewed changes

GeonH0 merged commit 2916d59 into main Jul 11, 2024
2 checks passed

GeonH0 deleted the feature/FeedDetailDomainData branch September 12, 2024 01:00

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

피드 상세화면에 필요한 Domain영역과 Data 영역을 정의해 보았습니다. #13

피드 상세화면에 필요한 Domain영역과 Data 영역을 정의해 보았습니다. #13

GeonH0 commented Jul 3, 2024

f-lab-barry Jul 3, 2024

f-lab-barry Jul 3, 2024

GeonH0 Jul 5, 2024

f-lab-barry Jul 3, 2024

GeonH0 Jul 5, 2024

GeonH0 Jul 5, 2024

f-lab-barry Jul 3, 2024

GeonH0 Jul 5, 2024

f-lab-barry Jul 3, 2024

GeonH0 Jul 5, 2024

f-lab-barry Jul 5, 2024

GeonH0 Jul 9, 2024

f-lab-barry Jul 3, 2024

GeonH0 Jul 5, 2024

f-lab-barry Jul 3, 2024

GeonH0 Jul 5, 2024

f-lab-barry Jul 3, 2024

GeonH0 Jul 5, 2024

f-lab-barry Jul 5, 2024

GeonH0 Jul 7, 2024

GeonH0 Jul 9, 2024

f-lab-barry Jul 9, 2024

f-lab-barry Jul 9, 2024

f-lab-barry Jul 9, 2024

f-lab-barry Jul 10, 2024

sonarqubecloud bot commented Jul 11, 2024

	func fetchRecipeDetail(recipeId: Int) -> Single<Recipe>
	func fetchRecipeDetail(recipeID: Int) -> Single<Recipe>


		init(networkService: NetworkService) {
		init(networkService: NetworkService, apiConfig: APIConfig) {

피드 상세화면에 필요한 Domain영역과 Data 영역을 정의해 보았습니다. #13

피드 상세화면에 필요한 Domain영역과 Data 영역을 정의해 보았습니다. #13

Conversation

GeonH0 commented Jul 3, 2024

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sonarqubecloud bot commented Jul 11, 2024

Quality Gate passed