Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RemoteSource와 Firestore 사이의 Layer 생성 #15

Open
malibinYun opened this issue Sep 14, 2023 · 0 comments
Open

RemoteSource와 Firestore 사이의 Layer 생성 #15

malibinYun opened this issue Sep 14, 2023 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@malibinYun
Copy link
Owner

RemoteSource와 Firestore 사이에 하나의 계층을 두고자 함.

interface FirestoreClient {
    suspend fun getCategories(): List<CategoriesResponse>
   ...
}

각 Remote Source에서 collection 경로에 대해 알 필요가 없다고 생각함.
이 때문에 RemoteSource에서 중복된 Collection 경로를 알고 있는 경우가 생김.

또, 응답 값에 대해 String 키 값을 모두 RemoteSource가 알고 있음.

중간 계층을 둠으로써 collection 경로가 중복되는 것을 막을 수 있으며, 응답에 대한 키 값의 관심사를 분리해낼 수 있을 것으로 기대됨.


중간 Layer를 두면 Remote Source는 사실상 값을 꺼내 중개해주는 역할이 되어버림. 깡통 객체가 될 가능성이 높음.
중복되는 값을 줄이고, 각 객체간 역할이 적절히 분배된 다른 방법을 생각해봐도 좋겠음.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant