4주차 세미나 과제 #9

b1urrrr · 2023-05-12T14:50:03Z

[4차 세미나] 서버 통신 기초, Retrofit2 #8

⛳️ Work Description

Essential

(진짜) 회원가입 /로그인 기능 구현하기
Home 화면 꾸미기

Advanced

Kotlin Coroutines를 활용하여 서버통신

📸 Screenshot

Sign up

Android.Emulator.-.Galaxy_S21_API_31_5554.2023-05-13.01-52-18.mp4

Log in & Follower

Android.Emulator.-.Galaxy_S21_API_31_5554.2023-05-13.01-52-56.mp4

📢 To Reviewers

이번에 특히 얼레벌레 구현한 부분이 많았는데 놓친 부분이나 개선할 부분이 있다면 알려주세요!

…-jeon into seminar4

Dan2dani

채연 언니 코드는 항상 배울 점이 많아용

Dan2dani · 2023-05-14T05:47:50Z

app/src/main/java/org/android/go/sopt/data/entity/remote/response/ResponseGetFollowerListDto.kt

+    fun toFollower() = data.map { follower ->
+        Follower(
+            id = follower.id,
+            name = "${follower.firstName} ${follower.lastName}",


오 이렇게도 할 수 있구낭..!

Dan2dani · 2023-05-14T05:48:23Z

app/src/main/java/org/android/go/sopt/data/entity/remote/response/wrapper/BaseResponse.kt

+import kotlinx.serialization.Serializable
+
+@Serializable
+data class BaseResponse<T>(


우와..! 이거 정말 짱이네용

와 이거 쇽샥해도 되나요 ? ㅋㅋ

배웠습니다 ㅎㅎ

와 이거 쇽샥해도 되나요 ? ㅋㅋ

저희 다 오픈소스잖아요 🤭

Dan2dani · 2023-05-14T05:50:09Z

app/src/main/java/org/android/go/sopt/data/entity/remote/response/ResponseGetFollowerListDto.kt

+
+    fun toFollower() = data.map { follower ->
+        Follower(
+            id = follower.id,


DiffUtill에서 아이템을 비교할 때 혹시 명확한 식별을 위해서 id도 가져오는건가용? (뭔가 나의 말이 이상한데......)

넵 areItemsTheSame 에서 id 값을 비교해주고 싶어서 가져왔습니다!

Dan2dani · 2023-05-14T05:54:05Z

app/src/main/java/org/android/go/sopt/presentation/main/follower/FollowerViewModel.kt

+    val followerList: List<Follower>?
+        get() = _followerList.value
+
+    private val _getFollowerListState = MutableLiveData<RemoteUiState>()


이거 좀 멍청 질문일 수도 있는데 UiState 의 장점은 뭔가영?!

UiState와 같은 클래스를 활용하면 ViewModel에서 수행한 결과(Success, Failure, Error, Loading 등등)에 따라 View에서 적절한 UI 처리를 할 수 있습니다! 효율적인 UI 처리가 가능하고 ViewModel과 View의 역할을 명확하게 구분할 수 있어서 좋은 것 같아요
NowInAndroid 프로젝트에서 이런 Ui 상태 처리를 잘 구현해둔 것 같은데 한번 참고해보셔도 좋을 것 같습니다 :)

leeeha

코드 조금 더 자세히 살펴보고 리뷰 추가하겠습니다! 과제하느라 고생하셨어요!

leeeha · 2023-05-14T06:32:12Z

app/src/main/java/org/android/go/sopt/data/entity/remote/response/wrapper/BaseResponse.kt

+    @SerialName("message")
+    val message: String,
+    @SerialName("data")
+    val data: T? = null,


오오오 Response에서 data의 타입만 달라지니까 제네릭으로 만들 수 있군요! 한수 배워갑니다👍👍

leeeha · 2023-05-14T06:36:41Z

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

+    suspend fun postSignIn(
+        @Body requestPostSignInDto: RequestPostSignInDto,
+    ): BaseResponse<ResponsePostSignInDto>
+}


오호 함수명이랑 Dto 클래스명에 어떤 HTTP 메서드를 사용하는지 명시해주니까 더 좋은 거 같아요!

leeeha · 2023-05-14T06:39:34Z

app/src/main/java/org/android/go/sopt/di/RetrofitModule.kt

+@Module
+@InstallIn(SingletonComponent::class)
+object RetrofitModule {
+    private const val APPLICATION_JSON = "application/json"


아맞다 상수화! 리팩토링 때 반영하겠습니다ㅎㅎ

훨씬 깔끔하네유

leeeha · 2023-05-14T06:44:34Z

app/src/main/java/org/android/go/sopt/di/RetrofitModule.kt

+
+    @Provides
+    @Singleton
+    @Retrofit2(BaseUrlType.SOPT)


오.. base url 타입을 어노테이션으로도 지정할 수 있네요!

amourxyoung

장인이시네요 👏 오늘도 배워갑니당 짱짱

amourxyoung · 2023-05-14T09:29:52Z

app/src/main/java/org/android/go/sopt/data/entity/remote/response/wrapper/BaseResponse.kt

+import kotlinx.serialization.Serializable
+
+@Serializable
+data class BaseResponse<T>(


와 이거 쇽샥해도 되나요 ? ㅋㅋ

amourxyoung · 2023-05-14T09:32:57Z

app/src/main/java/org/android/go/sopt/data/repository/FollowerRepositoryImpl.kt

+) : FollowerRepository {
+    override suspend fun getFollowerList(page: Int): Result<List<Follower>> =
+        runCatching {
+            followerDataSource.getFollowerList(page).toFollower()


오 처음부터 api url 에 page 숫자를 명시해준게 아닌, 인자로 받아서 여러 페이지 접근할 수 있도록 해주셨군요 짱이십니다 👍

amourxyoung · 2023-05-14T09:36:19Z

app/src/main/java/org/android/go/sopt/presentation/main/follower/FollowerFragment.kt

+
+    override fun onDestroyView() {
+        super.onDestroyView()
+        followerAdapter = null


null 초기화 해준 후에 뷰를 소멸시키는게 메모리 누수 방지에 좋을 것 같습니당

onDestroyView는 뷰모로부터 뷰가 제거되기 전에 호출(Internally it is called after the view's state has been saved but before it has been removed from its parent.)되어서 뷰가 파괴된 후에 메모리가 누수되지 않도록 종속적인 자원을 해제하는 역할을 한다고 이해하고 공식문서와 같은 형식으로 작성해보았는데, super.onDestroyView()를 호출하기 전에 자원을 해제하지 않았을 때 메모리가 누수되는 경우가 있을까요?

amourxyoung · 2023-05-14T09:38:05Z

app/src/main/java/org/android/go/sopt/presentation/signup/SignupViewModel.kt

+                .onFailure { t ->
+                    if (t is HttpException) {
+                        when (t.code()) {
+                            CODE_INVALID_INPUT -> _signupState.value = Failure(CODE_INVALID_INPUT)


여러 경우에 대비해서 예외 처리해주신 부분 굉장히 섬세하시네요 👏

lsakee

좋은코드 잘 배워갑니다 ㅎㅎ State를 잘활용하시네요! 저도 배우겠습니다 ㅎ

lsakee · 2023-05-14T15:06:02Z

app/src/main/java/org/android/go/sopt/data/entity/remote/response/wrapper/BaseResponse.kt

+import kotlinx.serialization.Serializable
+
+@Serializable
+data class BaseResponse<T>(


배웠습니다 ㅎㅎ

lsakee · 2023-05-14T15:08:09Z

app/src/main/java/org/android/go/sopt/di/RetrofitModule.kt

+    @Qualifier
+    annotation class Retrofit2(val baseUrlType: BaseUrlType)


오 좋은 방법이네요!

lsakee · 2023-05-14T15:10:32Z

app/src/main/java/org/android/go/sopt/presentation/main/follower/FollowerViewModel.kt

+
+                    _followerList.value = response
+                    _getFollowerListState.value = Success
+                    Timber.d("GET FOLLOWER LIST SUCCESS : $response")


로그찍는 습관 좋네요 !

haeti-dev

눈물과 더불어 빵을 먹어 보지 않은 자는 인생의 참다운 맛을 모른다. -괴테

haeti-dev · 2023-05-16T09:16:21Z

app/src/main/java/org/android/go/sopt/data/repository/AuthRepositoryImpl.kt

 ) : AuthRepository {
+    override suspend fun postSignup(requestPostSignUpDto: RequestPostSignUpDto): Result<ResponsePostSignUpDto?> =


Result 타입을 사용한 이유가 궁금합니다

좀 더 간결하게 이벤트를 처리하기 위해 runCatching을 활용해서 반환 타입을 Result로 지정해주었습니당

haeti-dev · 2023-05-16T09:16:55Z

app/src/main/java/org/android/go/sopt/data/repository/FollowerRepositoryImpl.kt

+) : FollowerRepository {
+    override suspend fun getFollowerList(page: Int): Result<List<Follower>> =
+        runCatching {
+            followerDataSource.getFollowerList(page).toFollower()


haeti-dev · 2023-05-16T09:17:37Z

app/src/main/java/org/android/go/sopt/di/RetrofitModule.kt

+@Module
+@InstallIn(SingletonComponent::class)
+object RetrofitModule {
+    private const val APPLICATION_JSON = "application/json"


훨씬 깔끔하네유

haeti-dev · 2023-05-16T09:19:30Z

app/src/main/java/org/android/go/sopt/presentation/login/LoginViewModel.kt

-    val pwd = MutableLiveData("")
+    val _id = MutableLiveData("")
+    private val id: String
+        get() = _id.value?.trim() ?: ""


센스 좋네요

haeti-dev · 2023-05-16T09:20:27Z

app/src/main/java/org/android/go/sopt/presentation/login/LoginViewModel.kt

+                    if (t is HttpException) {
+                        when (t.code()) {
+                            CODE_INVALID_INPUT -> _loginState.value = Failure(CODE_INVALID_INPUT)
+                            else -> _loginState.value = Error
+                        }
+                        Timber.e("POST SIGNIN FAIL ${t.code()} : ${t.message()}")


고급지네요

haeti-dev · 2023-05-16T09:20:50Z

app/src/main/java/org/android/go/sopt/presentation/main/follower/FollowerAdapter.kt

+            else -> throw ClassCastException(
+                parent.context.getString(
+                    R.string.view_type_error_msg,
+                    viewType,
+                ),


섬세하네요

haeti-dev · 2023-05-16T09:21:10Z

app/src/main/java/org/android/go/sopt/presentation/main/follower/FollowerAdapter.kt

+    override fun getItemCount(): Int {
+        return super.getItemCount() + HEADER_COUNT
+    }


새롭네요

haeti-dev · 2023-05-16T09:22:10Z

app/src/main/java/org/android/go/sopt/presentation/main/follower/FollowerFragment.kt

+    private fun initRecyclerViewLayoutManager() {
+        val layoutManager = GridLayoutManager(activity, 2)
+        layoutManager.spanSizeLookup = object : SpanSizeLookup() {
+            override fun getSpanSize(position: Int): Int {


이 함수가 뭐였죠 하하 기억이 안나네요

position에 해당하는 아이템이 gridLayout에서 몇 개의 폭을 차지하는지 반환해주는 함수입니다!
Header는 화면 너비를 꽉 채우고 싶어서 override 했습니다

haeti-dev · 2023-05-16T09:24:47Z

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

@@ -4,14 +4,28 @@ import android.widget.ImageView
 import androidx.databinding.BindingAdapter
 import coil.load
 import coil.transform.RoundedCornersTransformation
+import org.android.go.sopt.R

 object BindingAdapter {


object 로 생성하신 이유가 궁금합니따이

BindingAdapter 함수들은 프로젝트 전역에서 사용될 것이기 때문에 싱글톤 패턴으로 구현하는 것이 적합할 것 같아 object로 생성했습니다!

b1urrrr added 13 commits May 6, 2023 15:13

[FEAT/#8] Gallery ViewPager 구현

59988f8

[ADD/#8] post sign up 서버통신 함수 설계

fe37b7a

[FEAT/#8] sign up 서버통신 구현

4b7768a

Merge branch 'develop' of https://github.com/GO-SOPT-ANDROID/chaeyeon…

ebaa9b0

…-jeon into seminar4

[MOVE/#8] 패키지 정리

d3c3115

[ADD/#8] placeholder, error, fallback 이미지 추가

2e8e9e5

[ADD/#8] follower 메인 메뉴 추가

e7d9d64

[ADD/#8] 회원가입 서버 통신 예외 처리

a532abb

[MOD/#8] MBTI 입력 기능 제거

a1e8550

[DEL/#8] 프로필 레이아웃에서 mbti 텍스트뷰 제거

95b1324

[FEAT/#8] 로그인 서버 통신 구현

38648d3

[UI/#8] follower 레이아웃 구현

aec7577

[ADD/#8] get follower list 서버통신 함수 설계

386777b

b1urrrr self-assigned this May 12, 2023

b1urrrr added 3 commits May 13, 2023 01:13

[FEAT/#8] get follower list 서버 통신 구현

5e9e42a

[MOD/#8] 서버통신 Failure, Error 예외 처리

3529471

[ADD/#8] placeholder 이미지 추가 및 적용

10a8f42

b1urrrr added essential 필수 과제 advanced 심화 과제 labels May 12, 2023

[CHORE/#8] 회원가입 imeOptions actionDone 적용

33d310c

b1urrrr requested review from amourxyoung, leeeha, Dan2dani, lsakee, haeti-dev, a team and sxunea May 13, 2023 02:00

Dan2dani reviewed May 14, 2023

View reviewed changes

leeeha reviewed May 14, 2023

View reviewed changes

amourxyoung approved these changes May 14, 2023

View reviewed changes

lsakee approved these changes May 14, 2023

View reviewed changes

haeti-dev approved these changes May 16, 2023

View reviewed changes

b1urrrr merged commit f992303 into develop May 19, 2023

		@Qualifier
		annotation class Retrofit2(val baseUrlType: BaseUrlType)

		) : AuthRepository {
		override suspend fun postSignup(requestPostSignUpDto: RequestPostSignUpDto): Result<ResponsePostSignUpDto?> =

4주차 세미나 과제 #9

4주차 세미나 과제 #9

Conversation

b1urrrr commented May 12, 2023 • edited Loading

⛳️ Work Description

Essential

Advanced

📸 Screenshot

Sign up

Log in & Follower

📢 To Reviewers

Dan2dani 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

leeeha 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

leeeha May 14, 2023 • edited Loading

Choose a reason for hiding this comment

amourxyoung 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

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

haeti-dev 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

b1urrrr commented May 12, 2023 •

edited

Loading

leeeha May 14, 2023 •

edited

Loading