PR : 실습 3주차 필수과제 #11

Jokwanhee · 2023-11-08T06:38:49Z

📌𝘐𝘴𝘴𝘶𝘦𝘴

closed ISSUE : 세마나 3주차 과제 #10

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

Use Material3

친구 추가 Dialog (TextField 사용, NumberPicker CustomView 구현)
자동 로그인 Switch (Material3 Switch 사용 + Style color 적용)

Do Refactoring

SharedPreference(로컬 데이터베이스) 모듈화
Login Flow (로그인 흐름 수정)

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

자동로그인 (Switch)

default.mp4

친구추가 (TextField)

default.mp4

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

심화 과제 + 도전 과제 내용이 조금 들어간 것 같지만, 수정이 필요해보입니다.. 나중에 심화, 도전 과제 진행 시 수정하겠습니다.
리뷰 감사합니다!

친구 추가 다이어로그 UI material3 적용하기 - TextField 사용 (닉네임, 상태메시지) - CustomView(NumberPicker 사용) 생일 설정

SharedPreference 모듈화 - 클래스로 만들어 캡슐화 진행 - object 아니고 class 로 한 이유는 객체로 사용함으로써 범용성있게 사용하기 위함. (추가로 생성자로 context를 전달하기 위함)

로그인 확인 오류 개선 (빈 아이디, 빈 패스워드 시 로컬 데이터베이스에서 불러오는 값이 빈 값일 때, 로그인 되는 오류 개선)

M3 Swtich 구현 - 자동 로그인 기능 추가(on/off)

Marchbreeze

진자...완벽하다 이남자

Marchbreeze · 2023-11-09T19:39:17Z

app/src/main/java/org/sopt/dosopttemplate/view/SoptNumberPicker.kt

+import org.sopt.dosopttemplate.databinding.CustomNumberPickerBinding
+import java.text.SimpleDateFormat
+
+class SoptNumberPicker @JvmOverloads constructor(


이 부분에서 @jvmoverloads는 무슨 용도인가요?

커스텀뷰에는 3가지 생성자를 default로 구현하게 되는데 하나의 생성자 내에서 처리 가능하게 끔 하는 어노테이션입니다

어노테이션을 굳이 안사용해도 무리없이 잘 작동하겠지만, 생성자를 오버로딩하고 싶지 않아서 그만..

Marchbreeze · 2023-11-09T19:39:59Z

app/src/main/res/color/m3_switch_thumb.xml

@@ -0,0 +1,26 @@
+<?xml version="1.0" encoding="utf-8"?>
+<selector xmlns:android="http://schemas.android.com/apk/res/android">


아웅 야무지네요 셀렉터

kez-lab

엄청나고 굉장한 코드 잘보구 갑니다!!

kez-lab · 2023-11-10T04:36:45Z

app/src/main/java/org/sopt/dosopttemplate/db/local/Preference.kt

+class Preference(val context: Context) {
+    fun getId(): String = context.getSharedPreferences().getString(ID, "") ?: ""
+    fun getPassword(): String = context.getSharedPreferences().getString(PWD, "") ?: ""
+    fun getNickname(): String = context.getSharedPreferences().getString(NICKNAME, "") ?: ""
+    fun getMBTI(): String = context.getSharedPreferences().getString(MBTI, "") ?: ""
+    fun getAutoLogin(): Boolean = context.getSharedPreferences().getBoolean(AUTO_LOGIN, false)
+


?:"" 도 좋지만 .orEmpty() 라는 것도 있습니다 참고한번 해보세욥!

kez-lab · 2023-11-10T04:38:48Z

app/src/main/java/org/sopt/dosopttemplate/db/local/Preference.kt

+    fun setAutoLogin(auto: Boolean) {
+        context.getSharedPreferences().edit(commit = true) {
+            putBoolean(AUTO_LOGIN, auto)
+        }
+    }
+
+    fun setUser(user: User?) {
+        context.getSharedPreferences().edit(commit = true) {
+            user?.let {
+                putString(ID, it.id)
+                putString(PWD, it.password)
+                putString(NICKNAME, it.nickname)
+                putString(MBTI, it.mbti.toString())
+            }
+        }
+    }
+
+    private fun Context.getSharedPreferences(): SharedPreferences =
+        getSharedPreferences(getString(R.string.preference_file_key), Context.MODE_PRIVATE)


매번 SharedPreferences 객체를 선언하시는 것 같은데요, 혹시 더 좋은 방법 없을까요?

이번에 싱글톤으로 작성해보겠습니다!

kez-lab · 2023-11-10T04:45:11Z

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

+    private val preference: Preference by lazy {
+        Preference(this)
+    }


lazy하게 초기화 하시는 이유가 있을까요?!!?

불변의 객체 (읽기 전용)으로 선언하기 위함과 adapater 와 같은 경우에는 말씀해주신대로 view 를 가지고 컨트롤할 수 있어서 지연 초기화(lateinit)가 좋아보이지만 해당 클래스에서는 지연 초기화의 필요성을 못느꼈습니다! (주저리주저리 말씀드렸지만.. 맞나용? ㅎㅎ..)

kez-lab · 2023-11-10T04:46:33Z

app/src/main/java/org/sopt/dosopttemplate/view/SoptNumberPicker.kt

+import org.sopt.dosopttemplate.databinding.CustomNumberPickerBinding
+import java.text.SimpleDateFormat
+
+class SoptNumberPicker @JvmOverloads constructor(


커스텀뷰에는 3가지 생성자를 default로 구현하게 되는데 하나의 생성자 내에서 처리 가능하게 끔 하는 어노테이션입니다

yeonjeen

항상 완벽하게 구현하셔서 ㅎㅎ 오늘도 많이 배우고 갑니다! 친구추가 하기 기능으로 저도 추가해주세요 ㅋㅋㅋ

yeonjeen · 2023-11-10T06:12:19Z

app/src/main/java/org/sopt/dosopttemplate/db/local/Preference.kt

+    fun getAutoLogin(): Boolean = context.getSharedPreferences().getBoolean(AUTO_LOGIN, false)
+
+    fun setAutoLogin(auto: Boolean) {
+        context.getSharedPreferences().edit(commit = true) {


apply함수가 아니라 commit 함수를 사용하신 이유가 있을까요?

edit 내부 인자로 commit = true 인자를 사용한 것에 대한 질문이 맞으시다면,

context.getSharedPreferences().edit().apply()

일반적으로 사용하겠지만, SharedPreference KTX 를 사용하게된다면, 제가 작성한 코드처럼 작성할 수 있습니다.
여기서 KTX 가 궁금하실텐데, 간단하게 코틀린스럽게 라이브러리를 확장해서 사용할 수 있는 방법입니다.
https://developer.android.com/kotlin/ktx?hl=ko 해당 링크를 참조해보시면 좋을 것 같아요!

SharedPreference 객체 선언 시 생성을 싱글톤 패턴으로 구현하기

엘비스 연산자 대신 사용하는 orEmpty 함수 사용햅괴

진짜 싱글톤패턴 적용

emjayMJkim

수고했어!! 담부터는 코리 빨리 할께 ㅎㅎ

emjayMJkim · 2023-11-12T10:52:19Z

app/src/main/java/org/sopt/dosopttemplate/db/local/PreferenceManager.kt

+
+        @Synchronized
+        private fun getInstance(context: Context) {
+            if (::sharedPreferencesInstance.isInitialized) {


저번에 같이 봤던 부분이네여 저는 좀 더 공부해봐야 할 듯요ㅎㅎ

같이 화이팅합시다~

emjayMJkim · 2023-11-12T10:54:07Z

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

+
+                <com.google.android.material.textfield.TextInputLayout
+                    android:id="@+id/text_field_description"
+                    style="@style/TextFieldCounterAndEndClear"


오 style까지 따로 지정해주셨군요..!

Jokwanhee added 4 commits November 8, 2023 11:30

[add/#10] HomeAddDialog UI

a8be150

친구 추가 다이어로그 UI material3 적용하기 - TextField 사용 (닉네임, 상태메시지) - CustomView(NumberPicker 사용) 생일 설정

[refactor/#10] SharedPreference

0c5f798

SharedPreference 모듈화 - 클래스로 만들어 캡슐화 진행 - object 아니고 class 로 한 이유는 객체로 사용함으로써 범용성있게 사용하기 위함. (추가로 생성자로 context를 전달하기 위함)

[mod/#10] Login Flow

b3259da

로그인 확인 오류 개선 (빈 아이디, 빈 패스워드 시 로컬 데이터베이스에서 불러오는 값이 빈 값일 때, 로그인 되는 오류 개선)

[add/#10] material3 switch

775013f

M3 Swtich 구현 - 자동 로그인 기능 추가(on/off)

Jokwanhee added the 🐣 필수 과제 필수 과제 label Nov 8, 2023

Jokwanhee requested a review from a team November 8, 2023 06:38

Jokwanhee self-assigned this Nov 8, 2023

Marchbreeze self-requested a review November 9, 2023 19:37

Marchbreeze reviewed Nov 9, 2023

View reviewed changes

kez-lab approved these changes Nov 10, 2023

View reviewed changes

yeonjeen reviewed Nov 10, 2023

View reviewed changes

Jokwanhee added 3 commits November 10, 2023 16:46

[review] 의진 리뷰 피드백

6598cba

SharedPreference 객체 선언 시 생성을 싱글톤 패턴으로 구현하기

[review] 의진 리뷰 피드백

69fd5e4

엘비스 연산자 대신 사용하는 orEmpty 함수 사용햅괴

[review] 의진 리뷰 피드백

3c3909e

진짜 싱글톤패턴 적용

Jokwanhee merged commit 3bcf660 into develop Nov 11, 2023

emjayMJkim reviewed Nov 12, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

PR : 실습 3주차 필수과제 #11

PR : 실습 3주차 필수과제 #11

Jokwanhee commented Nov 8, 2023

Marchbreeze left a comment

Marchbreeze Nov 9, 2023

kez-lab Nov 10, 2023

Jokwanhee Nov 10, 2023

Marchbreeze Nov 9, 2023

kez-lab left a comment

kez-lab Nov 10, 2023

kez-lab Nov 10, 2023

Jokwanhee Nov 10, 2023

kez-lab Nov 10, 2023

Jokwanhee Nov 10, 2023

kez-lab Nov 10, 2023

yeonjeen left a comment

yeonjeen Nov 10, 2023

Jokwanhee Nov 10, 2023

emjayMJkim left a comment

emjayMJkim Nov 12, 2023

Jokwanhee Nov 12, 2023

emjayMJkim Nov 12, 2023

		@@ -0,0 +1,26 @@
		<?xml version="1.0" encoding="utf-8"?>
		<selector xmlns:android="http://schemas.android.com/apk/res/android">

PR : 실습 3주차 필수과제 #11

PR : 실습 3주차 필수과제 #11

Conversation

Jokwanhee commented Nov 8, 2023

📌𝘐𝘴𝘴𝘶𝘦𝘴

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

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

자동로그인 (Switch)

친구추가 (TextField)

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

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

kez-lab 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

yeonjeen left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

emjayMJkim 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