로그인 화면 #46
로그인 화면 #46
Conversation
| viewModel.navigateAction.observeEvent(this) { action -> | ||
| when (action) { | ||
| is RegisterNavigationAction.NavigateToKakaoLogin -> { | ||
| startActivity(MainActivity.getIntent(this)) | ||
| finish() | ||
| } | ||
|
|
||
| is RegisterNavigationAction.NavigateToGoogleLogin -> { | ||
| startActivity(MainActivity.getIntent(this)) | ||
| finish() | ||
| } |
There was a problem hiding this comment.
현재 activity 입장에서는 로그인 성공 여부만 중요한거 같은데, sealed class를 이용해 어떤 로그인인지 알려주는 이유는 추후에 이쪽 로직을 각각 다르게 처리할 거라서 인가요???
There was a problem hiding this comment.
Activity 입장에서는 카카오, 구글 로그인 버튼을 눌렀을 때, Intent를 통해 Kakao, Google SDK를 실행시켜 소셜 로그인을 진행하게 되요!! 그렇기 때문에 각각 다르게 처리를 하기 위해 sealed class를 사용한 거에요!!
그 이후에 소셜 로그인을 통해 필요한 정보(예를 들어 토큰)를 받게 되면, 그 정보를 통해 프로필 설정으로 이동하게 되는 sealed class가 추가될 것 같아요! 아마 벼리가 말한 게 이부분일 것 같네요!!
There was a problem hiding this comment.
작업 흐름을 보니깐 제가 했던 방식이랑 뭔가 다르네요..? 이건 직접 만나서 얘기해보면 좋을거 같아용
넘넘 수고많았습니다!!!
There was a problem hiding this comment.
저도 이 부분에 대해서 궁금해요! 더 이야기 나누면 좋을 것 같아요!
| viewModel.navigateAction.observeEvent(this) { action -> | ||
| when (action) { | ||
| is RegisterNavigationAction.NavigateToKakaoLogin -> { | ||
| startActivity(MainActivity.getIntent(this)) | ||
| finish() | ||
| } | ||
|
|
||
| is RegisterNavigationAction.NavigateToGoogleLogin -> { | ||
| startActivity(MainActivity.getIntent(this)) | ||
| finish() | ||
| } |
| @@ -0,0 +1,7 @@ | |||
| package com.woowacourse.friendogly.presentation.ui.register | |||
|
|
|||
| sealed interface RegisterNavigationAction { | |||
There was a problem hiding this comment.
패키지 구조에 XXXNavigateAction이랑 XXXActionHadler 차이좀 알 수 있을까요?
There was a problem hiding this comment.
XXXNavigateAction에는 특정 화면으로 이동하거나 다른 액션을 트리거하기 위한 동작을 정의합니다. 예를 들어, 화면 전환을 위한 네비게이션 액션, 특정 데이터를 가지고 다음 화면으로 이동하는 경우 등이 포함될 수 있습니다!! 해당 액티비티/프래그먼트에서 화면 전환은 XXXNavigateAction에 정의되어 있는 sealed interface를 통해 이동되는거죠!!
XXXActionHadler는 현재 저희 프로젝트에서 사용하지는 않았지만, 특정 액션을 처리하고, 그에 따른 비즈니스 로직을 ViewModel에서 실행시키기 위한 interface가 될 것 같습니다!!
| class RegisterViewModel : BaseViewModel() { | ||
| private val _navigateAction: MutableLiveData<Event<RegisterNavigationAction>> = | ||
| MutableLiveData(null) | ||
| val navigateAction: LiveData<Event<RegisterNavigationAction>> get() = _navigateAction |
There was a problem hiding this comment.
SingleLiveData랑 비슷해보이는데 다른점이 궁금해요
There was a problem hiding this comment.
동일합니다!! 다른 점이라면 LiveData를 직접 커스텀하는 것보다 더 간단하고 가독성이 좋은 것 같습니다!
https://medium.com/androiddevelopers/livedata-with-snackbar-navigation-and-other-events-the-singleliveevent-case-ac2622673150
위 LiveData 공식문서의 참고 블로그를 보면 이해하기 쉬울 것 같아요!!
| viewModel.navigateAction.observeEvent(this) { action -> | ||
| when (action) { | ||
| is RegisterNavigationAction.NavigateToKakaoLogin -> { | ||
| startActivity(MainActivity.getIntent(this)) | ||
| finish() | ||
| } | ||
|
|
||
| is RegisterNavigationAction.NavigateToGoogleLogin -> { | ||
| startActivity(MainActivity.getIntent(this)) | ||
| finish() | ||
| } |
There was a problem hiding this comment.
저도 이 부분에 대해서 궁금해요! 더 이야기 나누면 좋을 것 같아요!
| fun executeKakaoLogin() { | ||
| _navigateAction.emit(RegisterNavigationAction.NavigateToKakaoLogin) | ||
| } | ||
|
|
||
| fun executeGoogleLogin() { | ||
| _navigateAction.emit(RegisterNavigationAction.NavigateToGoogleLogin) | ||
| } |
There was a problem hiding this comment.
emit 함수를 처음보는데 무슨 역할을 하는 함수인지 궁금해요!
There was a problem hiding this comment.
현재 코들르 보면 LiveData가 Event라는 wrapper class를 사용하여 일회성 이벤트를 처리하도록 구성되어 있습니다. 즉, emit 함수는 MutableLiveData에 이벤트를 발생시키는 함수입니다!
| <!--회원가입 화면--> | ||
| <string name="register_now">간편하게 로그인 할래요</string> | ||
|
|
There was a problem hiding this comment.
strings.xml 활용 좋은 것 같아요!
|
|
||
| <TextView | ||
| android:id="@+id/tv_register_title" | ||
| style="@style/Theme.AppCompat.TextView.Regular.Gray07.Size16" |
There was a problem hiding this comment.
style 리소스 활용 좋은 것 같아요! 저도 한번 사용해봐야 겠어요!
이슈
개발 사항
전달 사항
RegisterActivity로 변경했습니다!