Feature/week6 essential #17

stellar-halo · 2023-12-07T15:39:07Z

📌𝘐𝘴𝘴𝘶𝘦𝘴

closed 6주차 필수과제 #13

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

삼항 연산자와 livedata를 이용해 실시간 상태 반영
비밀번호, ID 조건을 만족하지 않거나 닉네임, mbti 필드를 채우지 않으면 버튼 클릭 불가(색상 변경)

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

-.Clipchamp.mp4

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

제 함수명에 잔뜩 짜증나고,, 제 더러운 코드에 2배 더 짜증나네요. 다들 시험 기간 화이팅

chattymin

이번주도 그저 배워갑니다!!

chattymin · 2023-12-09T01:00:51Z

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

+    private val _isPasswordMeetCriteria: MutableLiveData<Boolean> = MutableLiveData()
+    val isPasswordMeetCriteria: LiveData<Boolean> = _isPasswordMeetCriteria
+    private val _isMeetCriteria: MutableLiveData<Boolean> = MutableLiveData(false)
+    val isMeetCriteria: LiveData<Boolean> = _isMeetCriteria


하나는 초기값을 설정하고 하나는 안했는데 어떤 기준으로 선택하시는 건가요??
저는 전부다 초기값 없이 사용하는 편인데 어느게 더 좋은지 궁금합니다!

저도 이부분 궁금합니다!!

초기값을 설정하게 되면 화면에 들어간 순간부터 경고문이 뜨기 때문입니다!
아무것도 아닌 상태, 실패한 상태, 성공한 상태 세 가지가 필요했기 때문에 초기값 설정을 해주지 않았습니다.
그때 그때 상황에 따라 다르게 설정하는 것 같습니다 :)

chattymin · 2023-12-09T01:02:23Z

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

+            android:backgroundTint="@{vm.isMeetCriteria == true ? @color/baby_blue : @color/gray}"
+            android:clickable="@{vm.isMeetCriteria == true ? true : false}"


backgroundTint에서 3항 연산자 활용 멋지네요!!
근데 clickable은 3항연산자가 아니라 바로 vm.isMeetCriteria 써도 될것 같기도 합니다 ㅎㅎ

클릭은 그냥 vm.isMeetCriteria 를 안쓰고 저렇게 하신이유가 있나욥>

없습니다 ㅎ.. 바꿀게요👍🏻

jihyun0v0 · 2023-12-10T10:59:53Z

군더더기 없이 깔끔한 강희언니 코드 본받고 싶습니다!!

lsakee

LGTM

lsakee · 2023-12-10T11:31:06Z

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

@@ -64,13 +64,26 @@
            android:layout_width="match_parent"
            android:layout_height="wrap_content"
            android:layout_marginTop="10dp"
+            android:backgroundTint="@{vm.isPasswordMeetCriteria != null &amp;&amp; vm.isPasswordMeetCriteria == false ? @color/red : @color/black}"


데바마스터이신가

이방법이 참 좋은데 가독성을 떨구는 부문인거 같아요 ㅠ 저도 이번 과제는 데바를 활용해야 해서 사용하였는데 평소에도 애용하시나요?

이런 식으로 값 변경에 즉각즉각 바뀌는 부분엔 애용하는 것 같아요

lsakee · 2023-12-10T11:31:52Z

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

+            android:backgroundTint="@{vm.isMeetCriteria == true ? @color/baby_blue : @color/gray}"
+            android:clickable="@{vm.isMeetCriteria == true ? true : false}"


클릭은 그냥 vm.isMeetCriteria 를 안쓰고 저렇게 하신이유가 있나욥>

crownjoe

이번 주도 수고 많으셨슴다 역시 최고 코드입니다람쥐 많이 배워 가고 눈에 담아갑니다 👍👍😎😮

crownjoe · 2023-12-10T13:14:36Z

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

+            android:visibility="@{vm.isPasswordMeetCriteria != null &amp;&amp; vm.isPasswordMeetCriteria == false ? View.VISIBLE : View.INVISIBLE}"
+            app:layout_constraintStart_toStartOf="@id/tv_sign_up_id"


이렇게 가시성 설정할 수 있군요/,... 오늘도 배워갑니다 최고 👍👍

crownjoe · 2023-12-10T13:15:04Z

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

+    private val _isPasswordMeetCriteria: MutableLiveData<Boolean> = MutableLiveData()
+    val isPasswordMeetCriteria: LiveData<Boolean> = _isPasswordMeetCriteria
+    private val _isMeetCriteria: MutableLiveData<Boolean> = MutableLiveData(false)
+    val isMeetCriteria: LiveData<Boolean> = _isMeetCriteria


저도 이부분 궁금합니다!!

stellar-halo added 11 commits December 6, 2023 00:50

#15 [chore] color값 추가

7f15da7

#15 [add] id, password 패턴 변수 변경 및 조건 만족 변수 생성

2ed9bdf

#15 [add] id, password 변경 때마다 조건 충족하는지 확인 후 변수 값 update

13c2c80

#15 [add]isMeetCriteria에 따른 삼항 연산자를 이용해 버튼 click과 색상 변경

81a65e1

#15 [add] password error 시 띄울 경고 문구 추가

de925ea

#15 [add] password error 시 password editText 밑줄 색상 변경

4561cb2

#15 [fix] false로 초기값 주는 것 삭제 -> init 상태를 감지하기 위함

280a0d0

#15 [add] 비밀번호에 맞지 않을 경우, 경고문 뜨게 작성

5662d55

#15 [fix] 함수 분리

17c96bb

#15 [fix] livedata 업데이트 함수 분리

84b786f

#15 [fix] viewModel 함수 분리에 따른 수정

d15345d

stellar-halo requested review from chattymin, jihyun0v0, lsakee, taeheeL, crownjoe and a team December 7, 2023 15:39

stellar-halo self-assigned this Dec 7, 2023

chattymin reviewed Dec 9, 2023

View reviewed changes

lsakee approved these changes Dec 10, 2023

View reviewed changes

crownjoe reviewed Dec 10, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/week6 essential #17

Feature/week6 essential #17

stellar-halo commented Dec 7, 2023

chattymin left a comment

chattymin Dec 9, 2023

crownjoe Dec 10, 2023

stellar-halo Dec 11, 2023

chattymin Dec 9, 2023

lsakee Dec 10, 2023

stellar-halo Dec 11, 2023

jihyun0v0 commented Dec 10, 2023

lsakee left a comment

lsakee Dec 10, 2023

lsakee Dec 10, 2023

stellar-halo Dec 11, 2023

lsakee Dec 10, 2023

crownjoe left a comment

crownjoe Dec 10, 2023

crownjoe Dec 10, 2023

		android:backgroundTint="@{vm.isMeetCriteria == true ? @color/baby_blue : @color/gray}"
		android:clickable="@{vm.isMeetCriteria == true ? true : false}"

		android:visibility="@{vm.isPasswordMeetCriteria != null && vm.isPasswordMeetCriteria == false ? View.VISIBLE : View.INVISIBLE}"
		app:layout_constraintStart_toStartOf="@id/tv_sign_up_id"

Feature/week6 essential #17

Are you sure you want to change the base?

Feature/week6 essential #17

Conversation

stellar-halo commented Dec 7, 2023

📌𝘐𝘴𝘴𝘶𝘦𝘴

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

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

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

chattymin 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

jihyun0v0 commented Dec 10, 2023

lsakee 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

crownjoe left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment