[FEAT/#19] Week6 / 심화 과제 #22

kangyuri1114 · 2023-12-06T20:08:14Z

📌𝘐𝘴𝘴𝘶𝘦𝘴

closed [FEAT] Week6 심화 과제 #19

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

양방향 데이터 바인딩 사용해보기 (필수 과제에서 적용함)
코루틴 suspend 사용해보기 (회원가입, 로그인, open API 친구리스트)
BindingAdaptor 사용 (타이틀 글자 키우기)

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

_2023_12_07_04_59_12_784.mp4

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

양방향 데이터 바인딩은 필수 과제에서 구현 했습니다.
회원가입, 로그인, open API 친구리스트 세 부분에서 API 통신 구현한 곳 모두 코루틴 suspend로 변경 했습니다.
근데............................... 자신은 없네여ㅠㅠ 잘 한건가... 뭔가 쓸데없는 코드 있다 싶으면 바로 코리 부탁해염
BindingAdaptor 사용해서 회원가입 시 모든 로그인 조건 만족 시 값자기 타이틀 글씨가 커지게 했습니다.
왜냐면... 그냥 뭔가 애니메이션 느낌이 있으면 재밌을거 같아서여 (저만 재밌나여)
엄... 스낵바랑 토스트랑 다 섞여서 너무 보기 안 좋은데 이 부분은 차차 수정하겠습니당 ㅎㅎ;;
그리고 영상 보면 로그인 성공 토스트가 두번 뜨는데, 토스트 코드를 두번 썼나 하고 찾아보니까 한번만 썼는데 왜 두번 뜰까요? 두번이 뜬다는 건 로그인 액티비티에서 UiState.Success가 두번 간 거 같은데.... 왜...지..? 로딩도 아니고 왜 성공이 두번이지..
API 통신 로그인, 회원가입만 신경써서 자동로그인 부분이랑 유저정보 불러오는 부분이 지금 다 엉망입니다 ㅠㅠ 이것도 유저정보 API 붙이게 될... 시간이 있다면 붙이면서 리팩토링 해볼게요..

적용 코드로 수정

Marchbreeze

진차... 야무지네요...

Marchbreeze · 2023-12-07T08:54:44Z

app/src/main/java/org/sopt/dosopttemplate/data/service/AuthService.kt

@@ -10,13 +10,13 @@ import retrofit2.http.POST
 interface AuthService {
    // 회원가입
    @POST("api/v1/members")
-    fun signUp(
+    suspend fun signUp(
        @Body request: RequestSignupDto,
    ): Call<Unit>


요 부분의 Call<>도 빼주셔도 될듯함다! 그 전에 Call이 여기서 어떤 역할을 했었고, 왜 이제 안 감싸주어도 작동이 되는지 공부해보면 더 좋을 듯 합니다 ~~

Marchbreeze · 2023-12-07T08:55:37Z

app/src/main/java/org/sopt/dosopttemplate/presentation/auth/LoginActivity.kt


-            loginViewModel.loginResult.observe(
+            loginViewModel.getLoginInfo.observe(


옵저버는 onClickListener보다는 바깥쪽에 위치하는 게 더 좋을 것 같아요~!

오 생각해보니 그렇네염.,,,,,맞슴니다

Marchbreeze · 2023-12-07T09:00:23Z

app/src/main/java/org/sopt/dosopttemplate/presentation/auth/LoginViewModel.kt

+    private val _getLoginInfo = MutableLiveData<UiState<User>>()
+    val getLoginInfo: LiveData<UiState<User>> get() = _getLoginInfo


변수명이 get으로 시작하면 너무 함수명같지 않나요..? ㅎㅎ

저 이제 변수명 정하는거 오류났어요
최종_진짜최종_찐최종 느낌..;; 좀 더 고민해볼게요 ㅎㅎ,,

Marchbreeze · 2023-12-07T09:02:11Z

app/src/main/java/org/sopt/dosopttemplate/presentation/auth/LoginActivity.kt

                        }
+
+                        val userId = uiState.data?.userId
+                        showShortToast("로그인 성공, 유저 아이디 : $userId")


그냥 슥 봐서는 저는 두번 토스트 뜨는 이유가 안보이네욥... 로그 찍어봐야할듯 !

ㅋㅋㅋㅋㅋ 제가 버튼을 두번 누르는 이슈였습니다 ..ㅎㅎ^^

@kangyuri1114 제 코드 중에 util에 setOnSingleClickListener 라구 있거덩요? 그거 가져와서 대신 쓰면 더블 클릭에 대응하지 않게 된답니다~~!

Marchbreeze · 2023-12-07T09:04:15Z

app/src/main/java/org/sopt/dosopttemplate/presentation/main/peoplelist/PeopleListFragment.kt

+                    val peopleList = uiState.data
+                    peopleListAdapter.setUsers(ArrayList(peopleList))


불필요한 객체 생성은 줄이는게 좋을 듯 합니다! 사실 제 강박관념임 ㅋ

네~~ 주의할게요!

Marchbreeze · 2023-12-07T09:05:53Z

app/src/main/java/org/sopt/dosopttemplate/util/UiState.kt

@@ -0,0 +1,13 @@
+package org.sopt.dosopttemplate.util
+
+sealed interface UiState<out T> {


혹시 제 UiState 가져오셧나요? ㅎㅎ 아님말구~

합세때 쓰던 거 가져왔어욤 ㅎㅎ

leeeyubin

너무 잘하시는데요!!! 심화과제까지 수고하셨습니당

leeeyubin · 2023-12-07T16:39:13Z

app/src/main/java/org/sopt/dosopttemplate/presentation/auth/LoginActivity.kt

+            ) { uiState ->
+                when (uiState) {
+                    is UiState.Success -> {
+                        if (binding.cbLoginAutologin.isChecked) {
+                            signUpUser?.let {
+                                loginViewModel.saveUserForAutoLogin(this, it)


저도 UiSate 써보고 싶었는데,, 다음에 할 때 참고해야게써요 최곱니다,,,

leeeyubin · 2023-12-07T16:40:43Z

app/src/main/java/org/sopt/dosopttemplate/presentation/auth/SignUpViewModel.kt

+                RequestSignupDto(
+                    inputId.value ?: "",
+                    inputPw.value ?: "",
+                    inputNickname.value ?: "",
+                ),


예외처리 이렇게 해주는군요! 효율적이네용

저는 orEmpty를 썼는데 inputId.value.orEmpty로 하는 것과 inputId.value ?: "" 하는 것 중에 어느게 더 효율적인가요..?

음... 제 생각에는 orEmpty가 더 깔끔해서 시각적으로는 좋을 것 같네요 ㅎㅎ..성능부분까지는 모르겠슴다 ㅎㅎ..

DoReMinWoo

고생하셨습니다~

DoReMinWoo · 2023-12-08T08:11:51Z

app/src/main/res/layout/activity_signup.xml

+        <!--        android:layout_width="0dp"-->
+        <!--        android:layout_height="wrap_content"-->
+        <!--        android:layout_marginHorizontal="30dp"-->
+        <!--        android:layout_marginTop="12dp"-->
+        <!--        android:text="@string/tv_signup_age"-->
+        <!--        android:textSize="20sp"-->
+        <!--        app:layout_constraintStart_toStartOf="parent"-->
+        <!--        app:layout_constraintTop_toBottomOf="@id/et_signup_nickname" />-->
+
+        <!--    <EditText-->
+        <!--        android:id="@+id/et_signup_age"-->
+        <!--        android:layout_width="0dp"-->
+        <!--        android:layout_height="wrap_content"-->
+        <!--        android:layout_marginHorizontal="30dp"-->
+        <!--        android:layout_marginTop="5dp"-->
+        <!--        android:hint="@string/et_age_hint"-->
+        <!--        android:inputType="number"-->
+        <!--        android:singleLine="true"-->
+        <!--        app:layout_constraintEnd_toEndOf="parent"-->
+        <!--        app:layout_constraintStart_toStartOf="parent"-->
+        <!--        app:layout_constraintTop_toBottomOf="@id/tv_signup_age" />-->


사용하지 않는 코드는 지워주면 좋을거같아요!

DoReMinWoo · 2023-12-08T08:24:45Z

app/src/main/java/org/sopt/dosopttemplate/presentation/auth/SignUpActivity.kt

+
+    private fun observeValid() {
+        signUpViewModel.idFlag.observe(this) { idFlag ->
+            binding.telSignupId.error = if (idFlag) null else getString(R.string.id_layout_title)


오홍 이런식으로 if문 쓰는방법도 좋네요

조장이 고쳐줬어염 ㅎㅅㅎ

DoReMinWoo · 2023-12-08T10:49:27Z

app/src/main/java/org/sopt/dosopttemplate/presentation/auth/SignUpViewModel.kt

+                RequestSignupDto(
+                    inputId.value ?: "",
+                    inputPw.value ?: "",
+                    inputNickname.value ?: "",
+                ),


저는 orEmpty를 썼는데 inputId.value.orEmpty로 하는 것과 inputId.value ?: "" 하는 것 중에 어느게 더 효율적인가요..?

kangyuri1114 added 30 commits December 4, 2023 03:23

[UI/#18] TextInputLayout 적용

a594083

[UI/#18] TextInputLayout 길이 제한

428b628

[UI/#18] TextInputLayout 길이 제한

5bb053d

[MOD/#18] 네이밍 수정

cae165a

[MOD/#18] 네이밍 수정

50eec15

[MOD/#18] 에러 문구 추가

36ce63c

[MOD/#18] 아이디, 비밀번호 패턴 상수 추가

6e886a8

[MOD/#18] 정규식 처리 라이브데이터 추가

ecb6e57

[MOD/#18] TextWatcher 추가

d6e0f7a

[MOD/#18] 정규식은 성공.. 버튼활성화랑 빈 데이터가 들어가는 이슈..

acf6872

[MOD/#18] 정규식은 성공.. 버튼활성화랑 빈 데이터가 들어가는 이슈..

574b07a

[MOD/#18] 조건에 따라 회원가입 하기 성공

d060b9a

[MOD/#18] 조건에 따라 회원가입 하기 성공

8049716

[MOD/#18] 버튼 클릭 로직 함수화

c0e9b3a

[MOD/#18] 함수 위치 수정

c91896a

[MOD/#18] 함수화

1504823

[MOD/#18] 양방향 데이터 바인딩 적용 성공, Transformations.map

541335a

적용 코드로 수정

[MOD/#18] 주석 제거, string 추가

c37c4b0

[DOCS/#18] 리드미 수정

d6841fc

[DOCS/#18] 버전 업데이트

1eecf85

[ADD/#18] 바인딩 어댑터 추가

b2135ad

[FEAT/#18] 바인딩 어댑터로 텍스트 크기 변환하기

49569bc

[ADD/#19] UiState 파일 추가

fa11e83

[MOD/#19] enqueue로 통신 했던 것을 코루틴 suspend로 대체 - 회원가입 성공

d00240b

[MOD/#19] enqueue로 통신 했던 것을 코루틴 suspend로 대체 - 로그인 성공, response 받아오는 건 못함

363da8f

[MOD/#19] enqueue로 통신 했던 것을 코루틴 suspend로 대체 - 로그인 성공, response 받아오기 성공

149c5b6

[COMMENT/#19] 주석 제거, 추가

ce865f7

[DELETE/#19] 중복 로직 삭제

d17c116

[MOD/#19] 스낵바 수정

140599e

[MOD/#19] enqueue로 통신 했던 것을 코루틴 suspend로 대체 - open API 친구 리스트 성공

213e623

kangyuri1114 requested review from DoReMinWoo, Marchbreeze, sohyun127, taeheeL and leeeyubin December 6, 2023 20:08

kangyuri1114 self-assigned this Dec 6, 2023

kangyuri1114 added 🌱필수 필수 과제 🌳심화 심화 과제 and removed 🌱필수 필수 과제 labels Dec 6, 2023

kangyuri1114 added 2 commits December 7, 2023 05:10

[MOD/#19] 코드리뷰 반영

c8ce94b

[MOD/#19] 함수화

fe951ff

Marchbreeze approved these changes Dec 7, 2023

View reviewed changes

kangyuri1114 changed the title ~~Feat/#19 week6 advanced~~ [FEAT/#19] Week6 / 도전 과제 Dec 7, 2023

leeeyubin approved these changes Dec 7, 2023

View reviewed changes

kangyuri1114 changed the title ~~[FEAT/#19] Week6 / 도전 과제~~ [FEAT/#19] Week6 / 심화 과제 Dec 8, 2023

DoReMinWoo approved these changes Dec 8, 2023

View reviewed changes

kangyuri1114 merged commit 968cd95 into feat/#18-week6-essential Dec 12, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[FEAT/#19] Week6 / 심화 과제 #22

[FEAT/#19] Week6 / 심화 과제 #22

kangyuri1114 commented Dec 6, 2023

Marchbreeze left a comment

Marchbreeze Dec 7, 2023

Marchbreeze Dec 7, 2023

kangyuri1114 Dec 7, 2023

Marchbreeze Dec 7, 2023

kangyuri1114 Dec 7, 2023

Marchbreeze Dec 7, 2023

kangyuri1114 Dec 7, 2023

Marchbreeze Dec 7, 2023

Marchbreeze Dec 7, 2023

kangyuri1114 Dec 7, 2023

Marchbreeze Dec 7, 2023

kangyuri1114 Dec 7, 2023

leeeyubin left a comment

leeeyubin Dec 7, 2023

leeeyubin Dec 7, 2023

DoReMinWoo Dec 8, 2023

kangyuri1114 Dec 8, 2023

DoReMinWoo left a comment

DoReMinWoo Dec 8, 2023

DoReMinWoo Dec 8, 2023

kangyuri1114 Dec 8, 2023

DoReMinWoo Dec 8, 2023


		loginViewModel.loginResult.observe(
		loginViewModel.getLoginInfo.observe(

		private val _getLoginInfo = MutableLiveData<UiState<User>>()
		val getLoginInfo: LiveData<UiState<User>> get() = _getLoginInfo

		val peopleList = uiState.data
		peopleListAdapter.setUsers(ArrayList(peopleList))

		@@ -0,0 +1,13 @@
		package org.sopt.dosopttemplate.util

		sealed interface UiState<out T> {

[FEAT/#19] Week6 / 심화 과제 #22

[FEAT/#19] Week6 / 심화 과제 #22

Conversation

kangyuri1114 commented Dec 6, 2023

📌𝘐𝘴𝘴𝘶𝘦𝘴

📎𝘞𝘰𝘳𝘬 𝘋𝘦𝘴𝘤𝘳𝘪𝘱𝘵𝘪𝘰𝘯

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

💬𝘛𝘰 𝘙𝘦𝘷𝘪𝘦𝘸𝘦𝘳𝘴

Marchbreeze left a 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

leeeyubin left a 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

DoReMinWoo left a 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