4차 과제 #8

mjeong21 · 2025-05-09T09:04:35Z

📌 PR 요약

🌱 작업한 내용

TVING 클론 코딩 API 연동
필수 과제
- 회원가입 API 연동
- 로그인 API 연동
- 내 닉네임 조회 API 연동
- UI와 Data 분리
심화 과제
- 닉네임 조회 API 연동
- 닉네임 변경 API 연동
- 자동 로그인 기능 구현
도약 과제
- Debounce Search 구현

🌱 PR 포인트

DataStore 사용 -> userId 저장
type-safety navigation 적용
MVI..패턴을 적용해보고 싶었는데 맞는지 잘 모르겠어요....
코드리뷰 잔뜩 부탁드립니다..!

📸 스크린샷

로그인	마이페이지	검색

📮 관련 이슈

Resolved: [feat] 4차 과제 #7

- DataStore 사용해 userId 저장

- debounce 적용

MinseoSONG

4차 과제 수고했어용 ~~~~~
코드리뷰는 위 파일까지만 남겼습니당
아래는 계속 동일한 코리 반복일거같아서 여기까지만 !
바쁘다고 들엇는데 심화과제, 도전과제까지 .. 진짜 최고다
그래서 코리 조금 .. 아주 조금 많이 달앗는데 화이팅 ! 김혜정 할 수 없 .. 있다 ㅠㅠ!!

MinseoSONG · 2025-05-11T06:57:12Z

app/src/main/java/org/sopt/at/core/component/bottombar/TvingBottomBar.kt

+//    val screen = listOf<BottomNavRoute>(
+//        BottomNavRoute.Home,
+//        BottomNavRoute.Shorts,
+//        BottomNavRoute.Live,
+//        BottomNavRoute.Search,
+//        BottomNavRoute.History
+//    )


이제 쓰지 않는 코드는 주석처리말구 지워주세용 ~

MinseoSONG · 2025-05-11T06:57:36Z

app/src/main/java/org/sopt/at/core/component/bottombar/TvingBottomBar.kt

+        enter = fadeIn() + slideInVertically1 { it },
+        exit = fadeOut() + slideOutVertically { it }


애니메이션까지 야무지다 ~

MinseoSONG · 2025-05-11T07:10:10Z

app/src/main/java/org/sopt/at/core/navigation/RouteModel.kt

진짜 깔끔하다 최고 ~~

MinseoSONG · 2025-05-11T07:12:02Z

app/src/main/java/org/sopt/at/data/model/BaseState.kt

+import kotlinx.coroutines.flow.flow
+import retrofit2.Response
+
+sealed class BaseState<out T> {


State까지 고려햇다고 벌써?
넌 파트장해라 ~

MinseoSONG · 2025-05-11T07:13:02Z

app/src/main/java/org/sopt/at/data/model/ErrorResponse.kt

ErrorResponse를 따로 준비한 이유가 있을까요 ??

BaseResponse를 만들어두고 받아오는 응답의 success, code에 따라 에러와 같은 응답 처리를 해주는 것이 에러 응답을 따로 준비하는 것보다 좋아보입니다 !

MinseoSONG · 2025-05-11T07:56:04Z

app/src/main/java/org/sopt/at/feature/editnickname/EditNicknameViewModel.kt

+        viewModelScope.launch {
+            authPreferences.userId.collect {
+                repository.patchNickname(token = it, request = request).collect { result ->
+                    when (result) {
+                        is BaseState.Error -> {
+                            _effect.send(EditNicknameEffect.ShowSnackbar(result.message))
+                        }
+
+                        is BaseState.Success -> {
+                            _effect.send(EditNicknameEffect.NavigateToMy)
+                        }
+                        BaseState.Idle -> {
+                            _uiState.value = BaseState.Idle
+                        }
+                    }
+                }


세미나 때 배운 enqueue가 아닌 viewModelScope를 쓰셨군요 !!
진짜 최고임 ㅠㅠ 이렇게 코루틴 기반으로 구성해주는게 훨씬 더 모던하고 안정적이랍니다.

enqueue는 콜백 기반 비동기 처리라 콜백 지옥으로 이어지기도 쉽고, 테스트나 에러 핸들링도 어려워요.
근데? viewModelScope.launch를 사용하면 코루틴 + flow 기반으로 선언적이고 간결하게 비동기 처리가 가능해지죠.
또, viewModel이 lifecycle과 함께 자동으로 취소되기 때문에 메모리 누수 방지에도 효과적이랍니다 ~~

또 ! 지금 MVI를 도입하려는 시도에 아주 자연스럽게 연결이 되고 있는 것 같아요.
정말 너무 좋은 시도와 코드 ... 너 오비해라 !

와 진짜 구조 예쁘게 잘 짜셨네요!!
덕분에 저도 많이 배워 갑니다 총총 ..

MinseoSONG · 2025-05-11T07:58:22Z

app/src/main/java/org/sopt/at/feature/home/HomeScreen.kt

+@Composable
+fun HomeRoute(modifier: Modifier = Modifier) {
+    HomeScreen(
+
+    )
+}


이거 머임 !

MinseoSONG · 2025-05-11T07:58:45Z

app/src/main/java/org/sopt/at/feature/home/navigation/HomeNavigation.kt

+        HomeRoute(
+
+        )


MinseoSONG · 2025-05-11T08:00:35Z

app/src/main/java/org/sopt/at/feature/main/MainActivity.kt

-import org.sopt.at.core.navigation.LocalNavController
 import org.sopt.at.ui.theme.TvingTheme

 // Application 클래스에 Hilt를 설정하고 애플리케이션 수준 구성요소를 사용할 수 있게 되면 Hilt는 이 주석이 있는 다른 Android 클래스에 종송 항목 제공 가능


공부해놓는거 짱이다
합세 아티클공유에 hilt 아티클 적어주라 ! ㅎㅎ

MinseoSONG · 2025-05-11T08:01:14Z

app/src/main/java/org/sopt/at/feature/main/MainNavHost.kt

+            )
+            myNavGraph(


너무 다닥다닥 붙어잇는거 같아요
줄바꿈해주라 !

seungjunGong

한주간 고생많았습니다!
전체적으로 코드를 보면서 새로운 기술들을 무리없이 잘 적용한 거 같아 인상 깊었습니다
저 또한 많은 도움 되었어요~@@!

남은 합세기간 동안도 코리 잘 부탁해요~ 파이팅bb

seungjunGong · 2025-05-11T11:16:16Z

app/src/main/java/org/sopt/at/core/component/textfield/BasicTextField.kt

-fun IdTextField(
-    id: String,
-    onIdChange: (String) -> Unit,
+fun BasicTextField(


공통 컴포넌트로 뺀거 좋네용

seungjunGong · 2025-05-11T11:25:06Z

app/src/main/java/org/sopt/at/data/model/BaseState.kt

+                emit(BaseState.Error("응답이 비어 있습니다", "EMPTY"))
+            }
+        } else {
+            val error = Gson().fromJson(response.errorBody()?.string(), ErrorResponse::class.java)


Gson 으로 error message 추출하니 깔끔하고 좋네요~

seungjunGong · 2025-05-11T12:03:48Z

app/src/main/java/org/sopt/at/feature/editnickname/EditNicknameViewModel.kt

+        viewModelScope.launch {
+            authPreferences.userId.collect {
+                repository.patchNickname(token = it, request = request).collect { result ->
+                    when (result) {
+                        is BaseState.Error -> {
+                            _effect.send(EditNicknameEffect.ShowSnackbar(result.message))
+                        }
+
+                        is BaseState.Success -> {
+                            _effect.send(EditNicknameEffect.NavigateToMy)
+                        }
+                        BaseState.Idle -> {
+                            _uiState.value = BaseState.Idle
+                        }
+                    }
+                }


와 진짜 구조 예쁘게 잘 짜셨네요!!
덕분에 저도 많이 배워 갑니다 총총 ..

seungjunGong · 2025-05-11T12:16:23Z

app/src/main/java/org/sopt/at/feature/editnickname/navigation/EditNicknameNavigation.kt

+            modifier = modifier
+        )
+    }
+}


화면 등록과 이동을 분리해주신 구조가 깔끔하네요!!

seungjunGong · 2025-05-11T12:35:18Z

app/src/main/java/org/sopt/at/feature/main/MainNavigator.kt

+    private inline fun <reified T : Route> isSameCurrentDestination(): Boolean =
+        navController.currentDestination?.route == T::class.qualifiedName


이건 어디서 사용되는 건가요??

seungjunGong · 2025-05-11T12:46:51Z

app/src/main/java/org/sopt/at/feature/search/SearchViewModel.kt

+    @OptIn(FlowPreview::class)
+    private fun search() {
+        viewModelScope.launch {
+            queryFlow.debounce(300)


debounce 처리까지 대박이네용!

seungjunGong · 2025-05-11T12:53:01Z

app/src/main/java/org/sopt/at/feature/search/SearchViewModel.kt

Ui 상태관리 이렇게 하는거구나..
잘 작성해주어서 많은 공부 되었어요!!

mjeong21 added 30 commits May 8, 2025 00:01

build: network 의존성 추가

f2cf289

build: network 의존성 추가

ebdc0c3

build: network 의존성 추가

2118c08

chore: 패키지 변경

bebcf4c

build: gson 의존성 추가

0ce3441

feat: 회원가입 API 연동

eec8ef7

feat: 로그인 API 연동

20d6d39

chore: 네이밍 수정

65613fa

fix: 회원가입 뒤로 가기 버튼 수정

7fe87f0

fix: 로그인 이펙트 수정

eca2ccb

fix: 로그인에서 홈으로 이동 추가

eee0c2f

chore: TvingTopBar 수정

d307305

chore: AuthRepository 네이밍 수정

72c2202

chore: AuthService 네이밍 수정

abf898f

build: datastore 의존성 추가

dd4c75c

feat: 내 닉네임 조회 API 연동

ef6c302

- DataStore 사용해 userId 저장

del: 불필요한 파일 삭제

755ae39

feat: 자동 로그인 구현

20490db

fix: 바텀 네비게이션 오류 수정

ccf0d55

feat: 로그아웃 구현

ccb57bb

del: 불필요한 코드 삭제

62e7e6b

chore: 파일 경로 수정

dc12c49

chore: 파일 경로 수정

c2e358c

feat: 검색 화면 구현

4d9345a

chore: 네이밍 수정

ea75ffd

feat: 닉네임 조회 API 연동

cf75643

- debounce 적용

del: 불필요한 코드 삭제

9662045

feat: 닉네임 변경 API 연동

c137855

chore: 네이밍 수정

e68c854

fix: 로그아웃 시 API 호출 방지

f097e8f

mjeong21 requested review from MinseoSONG, Sangwook123 and seungjunGong May 9, 2025 09:04

mjeong21 self-assigned this May 9, 2025

mjeong21 added ✨ 필수 과제 필수 과제 👊 심화 과제 심화 과제 labels May 9, 2025

mjeong21 linked an issue May 9, 2025 that may be closed by this pull request

[feat] 4차 과제 #7

Closed

12 tasks

mjeong21 added the 💪 도전 과제 도전 과제 label May 9, 2025

feat: type-safety navigation 적용

6a977e5

MinseoSONG approved these changes May 11, 2025

View reviewed changes

seungjunGong approved these changes May 11, 2025

View reviewed changes

mjeong21 merged commit f8b12d3 into develop Jun 8, 2025

mjeong21 deleted the feat/#7 branch June 8, 2025 08:12

		enter = fadeIn() + slideInVertically1 { it },
		exit = fadeOut() + slideOutVertically { it }

		private inline fun <reified T : Route> isSameCurrentDestination(): Boolean =
		navController.currentDestination?.route == T::class.qualifiedName

4차 과제 #8

4차 과제 #8

Uh oh!

Conversation

mjeong21 commented May 9, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📌 PR 요약

📸 스크린샷

📮 관련 이슈

Uh oh!

MinseoSONG left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

seungjunGong left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

mjeong21 commented May 9, 2025 •

edited

Loading

MinseoSONG left a comment •

edited

Loading