러닝탭 화면 구현 (현재 러너 카운트 받아오기) #11
Conversation
Co-authored-by: yonghanJu <[email protected]>
Co-authored-by: yonghanJu <[email protected]>
Co-authored-by: yonghanJu <[email protected]>
Co-authored-by: soopeach <[email protected]>
Co-authored-by: yonghanJu <[email protected]>
Co-authored-by: yonghanJu <[email protected]>
Co-authored-by: yonghanJu <[email protected]>
Co-authored-by: yonghanJu <[email protected]>
Co-authored-by: yonghanJu <[email protected]>
Co-authored-by: yonghanJu <[email protected]>
| class RunningDataSourceImpl(private val db: FirebaseFirestore) : RunningDataSource { | ||
|
|
||
| override fun getCurrentRunnerCount(): Flow<Int> = callbackFlow { | ||
| db.collection("Runners") | ||
| .document("runnersId") | ||
| .addSnapshotListener { snapshot, _ -> | ||
| snapshot?.let { | ||
| val count = it.data?.size ?: -1 | ||
| trySend(count) | ||
| } | ||
| } | ||
|
|
||
| awaitClose() | ||
| } | ||
|
|
||
| override suspend fun startRunning(uid: String): Boolean { | ||
| return suspendCoroutine { continuation -> | ||
| db.collection("Runners") | ||
| .document("runnersId") | ||
| .update(uid, uid) | ||
| .addOnSuccessListener { | ||
| continuation.resume(true) | ||
| } | ||
| .addOnFailureListener { | ||
| continuation.resume(false) | ||
| } | ||
| } | ||
| } | ||
|
|
||
| override suspend fun finishRunning(uid: String): Boolean { | ||
| return suspendCoroutine { continuation -> | ||
| db.collection("Runners") | ||
| .document("runnersId") | ||
| .update(uid, FieldValue.delete()) | ||
| .addOnSuccessListener { | ||
| continuation.resume(true) | ||
| } | ||
| .addOnFailureListener { | ||
| continuation.resume(false) | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
하드 코딩된 "Runners", "runnersId", -1 등을 변수로 관리하면 가독성이 더 좋을 것 같아요!
| override fun getCurrentRunnerCount(): Flow<Int> = callbackFlow { | ||
| db.collection("Runners") | ||
| .document("runnersId") | ||
| .addSnapshotListener { snapshot, _ -> |
There was a problem hiding this comment.
저는 error를 사용하지 않음에도 불구하고 error로 표기해놔서 승민님께 리뷰를 받았는데 _ 로 안쓰는 변수임을 잘 표현해주셨네요..!! error가 어떠한 상황에 발생하는 값인지 확인하고 예외처리가 필요하다면 예외처리해주고 그렇지 않다면 _ 로 사용하지 않음을 명시해주는 것이 좋아보이네요!! 나중에 같이 논의해보면 좋을 것 같습니다
| .document("runnersId") | ||
| .addSnapshotListener { snapshot, _ -> | ||
| snapshot?.let { | ||
| val count = it.data?.size ?: -1 |
There was a problem hiding this comment.
달리는 사람이 없을 때 -1이라는 값을 플로우로 방출하는 것 같은데 이에 대한 처리가 보이지 않는 것 같습니다! (제가 못 찾는 것일 수도 있지만요..^^) -> 달리고 있는 사람이 -1일 수는 없으니까요!
-1로 에러 케이스임을 명시해준 것은 분명히 좋지만 -1대신 0을 사용하면 별도의 케이스 처리를 해줄 필요도 없을 것 같고 승민님이 리뷰하신 것처럼 0이라는 값을 상수로 관리하면 보기도 좋고 관리하기도 좋아보입니다!
| @Provides | ||
| @Singleton | ||
| fun provideRunningRepository(runningDataSource: RunningDataSource): RunningRepository { | ||
| return RunningRepositoryImpl(runningDataSource) | ||
| } |
There was a problem hiding this comment.
저희 이번에 인터페이스주입도 그냥 Binds 대신 Provides 사용하기로 했었나요?? 기억이 잘 안나서요!
저도 항상 Provides만 써오긴 했습니다 ㅎㅎ
There was a problem hiding this comment.
확정하지는 않았던거 같은데 오늘 리뷰때 이야기 해보면 좋을거 같아요!
There was a problem hiding this comment.
#13 : Provides 대신 Binds 사용하기로 결정
| super.onViewCreated(view, savedInstanceState) | ||
| binding.vm = viewModel | ||
|
|
||
| lifecycleScope.launch { |
There was a problem hiding this comment.
flow를 collect하는 최상위 스코프는 fragment의 lifecycleScope네요..!! (물론 내부에서 viewLifecycleOwner에서 collect를 하는 범위를 컨트롤 해주시긴하지만) 저는 최상위 스코프도 viewLifecycleOwner.lifecycleScope로 두고 사용했거든요!
무한히 collect하는 스코프의 수집 범위를 컨트롤하는 결과는 동일하기도하고 이에 관련해서 차이가 없을 것 같다고 생각이 들긴하는데 어떻게 생각하시나요??
There was a problem hiding this comment.
#13 : 달리는 사람 수 라이프사이클 변경
| android:layout_width="0dp" | ||
| android:layout_height="0dp" | ||
| app:cardBackgroundColor="@color/mogakrun_primary_light" | ||
| app:cardCornerRadius="999dp" |
There was a problem hiding this comment.
이거 원모양으로 하려면 999dp가 공식적인 가이드라인인건가요?
😎 작업 내용
🧐 변경된 내용
🥳 동작 화면
🤯 이슈 번호