[4주차 과제] 서버 통신 기초 #9

leeeha · 2023-05-12T14:35:15Z

[4주차 과제] 서버 통신 기초 #7
저번 과제 중에 repo json 데이터 파싱하는 거 HomeFragment에 구현했습니다.
회원가입, 로그인, 로그아웃, 자동 로그인 구현 방법은 이슈에 적어뒀는데, 더 좋은 방법 있으면 알려주세요!
심화 과제는 다음주에 이어서 할 예정입니다!

필수 과제

(진짜) 회원가입/로그인 기능 구현하기
Reqres 샘플 데이터로 Gallery 프래그먼트 꾸미기

심화 과제

SearchFragment에서 카카오 검색 API 사용해보기
코루틴을 활용한 서버 통신 (enqueue 함수 사용 방식과 비교)

도전 과제

Debounce Search에 대해 알아보고 심화과제에 적용해보기

실행 결과

Screen_Recording_20230513_012507_GO.SOPT.Android.mp4

리팩토링 Todo

- MultiViewHolderFactory의 getViewHolder에서 MultiViewHolder 객체를 리턴할 때, `as MultiViewHolder<MultiViewItem>`로 강제 캐스팅을 해야 했음. (그 이유는 무변성 때문에) - Invariance: 형식 매개변수끼리는 하위 타입 관계를 만족시키더라도, 제네릭을 사용하는 클래스나 인터페이스는 하위 타입 관계가 유지되지 않는 것 - 그래서 MultiViewHolder의 형식 매개변수를 제거하면 하위 타입(HearderViewHolder, RepoViewHolder)에서 상위 타입(MultiViewHolder)으로 자동 캐스팅 되어 경고가 발생하지 않음. - MultiViewHolder의 형식 매개변수를 제거한 대신에, 어댑터의 onBindViewHolder에서 holder의 타입을 구분하여 bind 함수를 호출하도록 변경함.

- CoordinatorLayout, AppBarLayout, Toolbar, RecyclerView - 리사이클러뷰에 `appbar_scrolling_view_behavior` 속성을 설정하면, 스크롤 이벤트가 발생할 때마다 AppBarLayout에게 알림이 간다. (AppBarLayout 안에 있는 뷰들은 `layout_scrollFlags`에 설정된 플래그에 따라 동작한다.)

- 회원가입 화면에서 받은 인텐트가 아니라, prefs에 저장된 id, pw와 입력값을 비교하도록 수정 - 서버에 로그인 정보 등록 - prefs에 현재 로그인 상태를 true로 저장 (자동 로그인에 활용) - 마이페이지에는 prefs에 저장된 데이터 표시

- prefs의 모든 데이터를 삭제하지 않고, 로그인 상태를 나타내는 Boolean 값만 조정 - 회원가입 다시 하지 않아도 prefs에 저장된 데이터와 입력값 비교하여 바로 재로그인 가능함.

b1urrrr

이번 주도 과제하시느라 고생하셨습니다 👍👍
코드가 깔끔해서 읽기 너무 좋네요 🤭

b1urrrr · 2023-05-15T05:04:30Z

app/src/main/java/org/android/go/sopt/GoSoptApplication.kt

+class GoSoptApplication : Application() {
    override fun onCreate() {
        super.onCreate()
-        prefs = PreferenceUtil(applicationContext)
+
+        initTimber()
+        initPrefsManager()
+    }


함수화 너무 깔끔하네요 🤩

b1urrrr · 2023-05-15T05:11:03Z

app/build.gradle

+    // retrofit
+    implementation 'com.squareup.retrofit2:retrofit:2.9.0'
+    implementation 'org.jetbrains.kotlinx:kotlinx-serialization-json:1.4.1'
+    implementation 'com.jakewharton.retrofit:retrofit2-kotlinx-serialization-converter:0.8.0'
+
+    // define a BOM and its version
+    implementation(platform("com.squareup.okhttp3:okhttp-bom:4.10.0"))
+
+    // define any required OkHttp artifacts without version
+    implementation("com.squareup.okhttp3:okhttp")
+    implementation("com.squareup.okhttp3:logging-interceptor")


implementation 할 때 Kotlin 스크립트(KTS)와 Groovy 문법이 섞여있는데 통일 시켜주시면 좋을 것 같습니다 :)
공식문서에 따르면 Groovy보다 KTS로 gradle을 작성하는 것이 선호된다고 합니다.

b1urrrr · 2023-05-15T05:17:05Z

app/src/main/java/org/android/go/sopt/data/local/model/MockRepoDto.kt

+    val name: String,
+    @SerialName("full_name")
+    val fullName: String,
+    val `private`: Boolean,


private은 소프트 키워드가 아닌 키워드(예약어)에 해당하기 때문에 @SerialName을 통해 다른 이름으로 바꿔주는 것이 헷갈리지 않아 가독성과 유지보수성이 좋아질 것 같습니다 :)

허걱 플러그인으로 자동 생성했는데 이렇게 변환했는지 몰랐네요! 자동 생성한 코드도 유심히 다시 살펴보겠습니다!

b1urrrr · 2023-05-15T05:24:21Z

app/src/main/java/org/android/go/sopt/ui/main/gallery/GalleryFragment.kt

 class GalleryFragment : BindingFragment<FragmentGalleryBinding>(R.layout.fragment_gallery) {
-    private val listAdapter by lazy { MyListAdapter() }
+    private val followerAdapter by lazy { FollowerAdapter() }


Adapter 변수도 GalleryFragment의 뷰에 종속된 리소스이기 때문에 메모리 누수를 예방하기 위해 onDestroyView()에서 해제시켜주는 것이 좋을 것 같습니다!

b1urrrr · 2023-05-15T05:29:39Z

app/src/main/java/org/android/go/sopt/ui/signup/SignUpActivity.kt

+    private fun checkDuplicateId() {
+        val userId = binding.etId.text.toString()
+        AuthFactory.ServicePool.authService.getUserInfo(userId)
+            .enqueue(object : retrofit2.Callback<ResLoginDto> {
+                override fun onResponse(call: Call<ResLoginDto>, response: Response<ResLoginDto>) {
+                    // 이미 등록된 유저인 경우
+                    if (response.isSuccessful) {
+                        response.body()?.let {
+                            Timber.d(it.status.toString())
+                            Timber.d(it.message)
+                            Timber.d(it.data.id)
+                            Timber.d(it.data.name)
+                            Timber.d(it.data.skill)
+                        }
+                        showSnackbar(binding.root, getString(R.string.id_duplicate_error_msg))
+                        return
+                    }
+
+                    // 새로운 유저 등록
+                    registerNewUser()
+                }


sign-up api의 요청으로 중복된 정보를 입력할 경우 status code 409를 반환하고 있기 때문에 이에 대한 예외 처리를 하고 유저 정보 조회는 통신을 하지 않는 것이 서버 통신 횟수를 줄일 수 있어 효율적일 것 같습니다!

b1urrrr · 2023-05-15T05:34:33Z

app/src/main/java/org/android/go/sopt/util/PreferenceManager.kt

+    fun putUserData(user: User) {
+        val json = Gson().toJson(user)
+        putString(Intent.EXTRA_USER, json)
    }

-    fun deleteAllData() {
-        sharedPreferences.edit().clear().apply()
+    fun getUserData(): User? {
+        val json = getString(Intent.EXTRA_USER, "")
+        return Gson().fromJson(json, User::class.java)
    }


json 문자열로 객체를 저장하고 반환하도록 구현까지 하셨네요 크 👍👍

b1urrrr · 2023-05-15T05:35:36Z

app/src/main/java/org/android/go/sopt/util/binding/BindingAdapter.kt

+object BindingAdapter {
+    @JvmStatic
+    @BindingAdapter("app:imgUrl", "app:placeholder", "app:error")
+    fun ImageView.loadImage(
+        imgUrl: String,
+        loadingImg: Drawable,
+        errorImg: Drawable
+    ) {
+        Glide.with(this.context)
+            .load(imgUrl)
+            .placeholder(loadingImg)
+            .error(errorImg)
+            .apply(RequestOptions().fitCenter())
+            .into(this)
+    }
+}


BindingAdapter 활용 너무 좋습니다 👍
코드랩에서 활용하고 있는 Coil 라이브러리도 한번 찾아보시면 좋을 것 같습니다!

- 기존 방식: 서버에서 GET 요청으로 유저 정보 조회 성공하면 중복 id로 판단 - 바꾼 방식: 서버에 POST로 회원가입 요청을 보냈는데 409 에러가 뜨면 중복 id로 판단하여 에러 스낵바 띄우기 - 회원가입에서 로그인으로 다시 돌아올 때 회원가입 성공 스낵바 띄우도록 변경

sxunea

이번주도고생하셨습니다 💟 합세 화이팅

sxunea · 2023-05-15T15:25:23Z

app/src/main/java/org/android/go/sopt/data/local/source/RepoDataSource.kt

+    }
+
+    companion object {
+        private const val FILE_MOCK_REPO_LIST = "mock_repo_list.json"


json 파일명까지 상수화 꼼꼼하십니다 👍

sxunea · 2023-05-15T15:32:35Z

app/src/main/java/org/android/go/sopt/ui/login/LoginActivity.kt

+    }
+
+    private fun compareInputWithPrefs(): Boolean {
+        val id = binding.etId.text.toString()


User라는 데이터 클래스가 있으니 binding.~~.text.toString() 을 반복하는 것 보다는 savedUser처럼 User형을 하나 더 만들어서 재사용하는건 어떨까요 ?

sxunea · 2023-05-15T15:34:34Z

app/src/main/java/org/android/go/sopt/ui/login/LoginActivity.kt

    }

    private fun initSignUpButtonClickListener() {
        val signUpResultLauncher = registerForActivityResult(
            ActivityResultContracts.StartActivityForResult()
        ) { result ->
            if (result.resultCode == RESULT_OK) {
-                handleSignUpResult(result)
+                showSnackbar(binding.root, getString(R.string.sign_up_success_msg))
            }
        }

        binding.btnSignUp.setOnClickListener {


버튼을 클릭하고 나면 초기화되는것까지 너무 좋은데요 ? !

sxunea · 2023-05-15T15:36:44Z

app/src/main/java/org/android/go/sopt/ui/signup/SignUpActivity.kt

+            return true
+        }
+
+        if (response.code() == CODE_DUPLICATE_ID) {


아이디 중복확인까지 👍

sxunea · 2023-05-15T15:37:50Z

app/src/main/res/values/strings.xml

    <string name="sign_up_success_msg">회원가입이 완료되었습니다.</string>
    <string name="login_success_msg">로그인에 성공했습니다.</string>
    <string name="login_fail_msg">로그인에 실패했습니다.</string>
-    <string name="sign_up_invalid_input_err">모든 항목에 유효한 값을 입력해주세요.</string>
+    <string name="invalid_input_error">모든 항목에 유효한 값을 입력해주세요.</string>

    <!-- login -->


상황별 꼼꼼한 처리 멋집니다

leeeha added 15 commits May 6, 2023 16:23

[FEAT/#7] SearchFragment에 뷰페이저 적용

2e791c0

[FEAT/#7] 회원가입 레트로핏 구현

44f5f83

[DESIGN/#5] 상태바 투명하게 만들기

4ec6e2a

[MERGE] Squash merge week3 to week4 (week3의 커밋 내용을 합쳐서 week4에 병합)

e5307d3

[CHORE/#7] 서버 URL 주소는 버전 관리 하지 않도록 설정

88e76cf

[FEAT/#7] Glide with DataBinding

b5b8a78

[FEAT/#7] JSON 데이터 파싱해서 홈 프래그먼트의 리사이클러뷰에 보여주기

8e38b36

[FEAT/#7] 회원가입 레트로핏 코드 수정

4dbb0ad

[FEAT/#7] 로그인 레트로핏 코드 구현

f6dd17e

[FEAT/#7] 갤러리 프래그먼트에 Reqres API 적용

0486c97

[DESIGN/#7] 갤러리 프래그먼트의 리사이클러뷰 vertical margin 삭제

d411c56

[FEAT/#7] 갤러리 프래그먼트에 스크롤 되지 않는 헤더 추가

a659f64

[FIX/#7] 회원가입 시 이미 등록된 id를 입력하면 예외 처리

563bea8

leeeha requested review from b1urrrr, sxunea, lsakee and a team May 12, 2023 14:36

leeeha self-assigned this May 12, 2023

leeeha added Essential 필수 과제 Advanced 심화 과제 labels May 12, 2023

leeeha added 3 commits May 13, 2023 00:54

[FIX/#7] 로그아웃 코드 수정

c14f135

- prefs의 모든 데이터를 삭제하지 않고, 로그인 상태를 나타내는 Boolean 값만 조정 - 회원가입 다시 하지 않아도 prefs에 저장된 데이터와 입력값 비교하여 바로 재로그인 가능함.

[REMOVE/#7] 사용하지 않는 prefs 함수 삭제

be2df13

b1urrrr requested changes May 15, 2023

View reviewed changes

leeeha added 4 commits May 15, 2023 19:48

[REFACTOR/#7] 모듈 단위 Gradle 파일에서 Groovy 대신 KTS (Kotlin Scriopt) 문법으로 통일

0167c98

[FIX/#7] REQRES_BASE_URL 주소 변경

86b6650

[FIX/#7] MockRepoDto 변수명 변경

8a7500b

[REFACTOR/#7] FollowerAdapter 메모리 누수 방지를 위해 프래그먼트 뷰가 사라질 때 같이 null 처리

e4dbd38

sxunea reviewed May 15, 2023

View reviewed changes

leeeha added 11 commits May 22, 2023 11:57

[RENAME/#9] remote 요청, 응답 Dto 클래스명에 HTTP 메서드 이름 포함하도록 변경

fcce377

[REFACTOR/#9] BaseResponse 이용해서 서버 응답 data를 제네릭으로 사용하기

7d83b70

[RENAME/#9] 패키지 이름 변경 (ui -> presentation)

bad8595

[REFACTOR/#9] 회원가입에 ViewModel, LiveData 적용

3534b46

[REFACTOR/#9] 회원가입, 로그인 데이터바인딩 성공

03c942b

[FIX/#9] 갤러리 뷰모델에서 레트로핏 데이터 바로 가져오도록 변경 (DataSource, Repository 미사용)

498cff2

[REFACTOR/#9] 홈 프래그먼트의 레포 리스트 가져오는 로직을 뷰 모델에서 담당하도록 리팩토링

7acd6e9

[REMOVE/#9] 불필요한 코드 삭제

a57d3db

[REFACTOR/#9] 회원가입 로직을 뷰모델에서 담당하도록

9dbfca4

[REFACTOR/#9] 로그인, 자동 로그인 로직을 뷰 모델에서 담당하도록

0e8293a

[FEAT/#9] Glide 대신 Coil 라이브러리 적용해보기

7ff542d

leeeha requested a review from b1urrrr May 23, 2023 06:59

[FEAT] ActivityResultContracts 사용하여 갤러리 이미지 로딩하기

e1fe05a

leeeha linked an issue May 29, 2023 that may be closed by this pull request

[4주차 과제] 서버 통신 기초 #7

Open

6 tasks

leeeha added 4 commits May 30, 2023 15:08

[FEAT] multi-part 이미지 서버에 업로드

43d0248

[CHORE] 패키지 이름 변경

8a8bf99

[CHORE] 레트로핏 생성 팩토리 객체를 data/api 패키지로 이동

224608c

[MERGE] develop, week4 브랜치 충돌 해결

4867d56

leeeha merged commit 201b899 into develop May 30, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[4주차 과제] 서버 통신 기초 #9

[4주차 과제] 서버 통신 기초 #9

leeeha commented May 12, 2023 •

edited

Loading

b1urrrr left a comment

b1urrrr May 15, 2023

b1urrrr May 15, 2023

b1urrrr May 15, 2023

leeeha May 15, 2023

b1urrrr May 15, 2023

b1urrrr May 15, 2023

b1urrrr May 15, 2023

b1urrrr May 15, 2023

sxunea left a comment

sxunea May 15, 2023

sxunea May 15, 2023

sxunea May 15, 2023

sxunea May 15, 2023

sxunea May 15, 2023

[4주차 과제] 서버 통신 기초 #9

[4주차 과제] 서버 통신 기초 #9

Conversation

leeeha commented May 12, 2023 • edited Loading

필수 과제

심화 과제

도전 과제

실행 결과

리팩토링 Todo

b1urrrr 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

sxunea 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

leeeha commented May 12, 2023 •

edited

Loading