[feat] 위니피드, 마이피드 / Paging3 적용 + 로딩 프로그레스바 구현 #108

sxunea · 2023-08-11T20:43:42Z

closed [feat] 위니피드, 마이피드 / Paging3 적용, 로딩 인디케이터 구현 #99

📝 Work Description

피드에 paging3 적용
페이지 사이에 로딩 프로그레스바 구현
좋아요,삭제 등 기존 로직 페이징, 소셜로그인에 맞게 수정

📣 To Reviewers

내일 오후까지 리뷰해주시면 감사하겠습니다 :)
구현방법을 여러가지로 해보느라 커밋별로 보기보단 전체변경사항으로 바꿔서 보는걸 추천드립니다
로딩 바 무슨색 할까요

…inifinitescroll # Conflicts: # app/src/main/java/com/android/go/sopt/winey/data/repository/AuthRepositoryImpl.kt # app/src/main/java/com/android/go/sopt/winey/presentation/main/feed/WineyFeedFragment.kt

…gress bar 구현

This reverts commit e72dce8.

…ding progress bar 구현" - 방식 변경 위해 revert This reverts commit 945290f.

sxunea · 2023-08-11T20:51:48Z

기록용 : 페이징된 recyclerview에 페이지 끝마다 로딩 프로그레스바 넣기

다이얼로그로 표현 (2440b52)
MultiviewType ViewHolder + 추상화 + type으로 로딩 파악 (945290f) (e72dce8)
loadState에 따라 visibility 조절
loadStateAdapter 도입, addLoadStateListener 이용해 탐지 (ff6ddf1) (535cfdc) -> 최종 !

…inifinitescroll # Conflicts: # app/src/main/java/com/android/go/sopt/winey/data/repository/AuthRepositoryImpl.kt

leeeha

페이징, LoadStateAdapter, SwipeRefresh, API 분리까지 넘넘 고생많았어!!!
백혜선 폼 미쳐따~~~!!! 💘💘

leeeha · 2023-08-13T11:01:28Z

app/src/main/java/com/android/go/sopt/winey/data/source/pagingSource/BasePagingSource.kt

@@ -0,0 +1,43 @@
+package com.android.go.sopt.winey.data.source.pagingSource


저희 패키지 컨벤션 문서 보면, 패키지 네임은 모두 소문자로 작성한다고 되어있어요! source 패키지 아래에 있으니까 paging 만 적어도 괜찮을 거 같습니다 :)

감사합니다 수정할게요 :)

leeeha · 2023-08-13T11:04:50Z

app/src/main/java/com/android/go/sopt/winey/data/source/pagingSource/BasePagingSource.kt

+        return state.anchorPosition?.let { position ->
+            state.closestPageToPosition(position)?.prevKey?.plus(1)
+                ?: state.closestPageToPosition(position)?.nextKey?.minus(1)
+        }


중복되는 부분은 아래 코드처럼 변수를 만들어서 사용해도 좋을 듯 하네요!

return state.anchorPosition?.let { position -> val anchorPage = state.closestPageToPosition(position) anchorPage?.prevKey?.plus(1) ?: anchorPage?.nextKey?.minus(1) }

leeeha · 2023-08-13T11:05:47Z

app/src/main/java/com/android/go/sopt/winey/data/source/pagingSource/BasePagingSource.kt

+
+    override suspend fun load(params: LoadParams<Int>): LoadResult<Int, WineyFeed> {
+        val position = params.key ?: STARTING_KEY
+        if (position != STARTING_KEY) delay(DELAY_MILLIS)


상수화 👍

leeeha · 2023-08-13T11:14:32Z

app/src/main/java/com/android/go/sopt/winey/data/service/FeedService.kt

+        @Query("page") page: Int
+    ): ResponseGetWineyFeedListDto
+
+    @GET("feed/myFeed")


오호 이 부분만 다르고, 위니피드, 마이피드 요청/응답 데이터는 모두 같은 타입이군요!

leeeha · 2023-08-13T11:16:54Z

app/src/main/java/com/android/go/sopt/winey/data/source/pagingSource/WineyFeedPagingSource.kt

+) : BasePagingSource(feedService) {
+    override suspend fun getFeedList(position: Int): List<WineyFeed> {
+        return feedService.getWineyFeedList(position).toWineyFeed()
+    }


무분별한 상속은 지양해야 하지만, 지금은 중복되는 코드를 줄이기 위해 BaseXXX 을 사용하는 게 괜찮아 보입니다!
더 나은 다른 방식이 있는지는 좀 더 고민해봐야 할 거 같네요!

leeeha · 2023-08-13T12:14:02Z

app/src/main/java/com/android/go/sopt/winey/presentation/main/mypage/myfeed/MyFeedFragment.kt

+    private fun initGetFeedStateObserver() {
+        viewLifecycleOwner.lifecycleScope.launch {
+            viewLifecycleOwner.repeatOnLifecycle(Lifecycle.State.STARTED) {
+                launch {


요기도 launch 블록으로 감싸지 않아도 될 거 같아요!

leeeha · 2023-08-13T12:17:54Z

app/src/main/java/com/android/go/sopt/winey/presentation/main/mypage/myfeed/MyFeedFragment.kt

+                                myFeedAdapter.submitData(state.data)
+                                myFeedAdapter.addLoadStateListener { loadState ->
+                                    wineyFeedLoadAdapter.loadState = loadState.refresh
+                                    if (loadState.append.endOfPaginationReached) {


오호 페이징의 endOfPaginationReached 이용하니까 좋네요!!

leeeha · 2023-08-13T12:20:08Z

...java/com/android/go/sopt/winey/presentation/main/mypage/myfeed/MyFeedDeleteDialogFragment.kt

@@ -45,7 +48,7 @@ class MyFeedDeleteDialogFragment(private val feedId: Int, private val userLevel:
    }

    private fun initDeleteFeedStateObserver() {
-        myFeedViewModel.deleteMyFeedState.observe(viewLifecycleOwner) { state ->
+        myFeedViewModel.deleteMyFeedState.flowWithLifecycle(viewLifeCycle).onEach { state ->


엇 여기서 launchIn() 함수를 블록 끝에 작성해줘야 되지 않을까요??

leeeha · 2023-08-13T12:20:56Z

app/src/main/java/com/android/go/sopt/winey/presentation/main/mypage/myfeed/MyFeedFragment.kt

            }
        }
    }

    private fun initPostLikeStateObserver() {
-        viewModel.postMyFeedLikeState.observe(viewLifecycleOwner) { state ->
+        viewModel.postMyFeedLikeState.flowWithLifecycle(viewLifeCycle).onEach { state ->


요기도 블록 끝에 launchIn 함수 호출해줘야 할 거 같아요!

leeeha · 2023-08-13T12:23:19Z

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

+            android:layout_height="wrap_content"
+            app:layout_constraintEnd_toEndOf="parent"
+            app:layout_constraintStart_toStartOf="parent"
+            app:layout_constraintTop_toTopOf="parent" />


로딩 프로그레스바 색상은,, 인스타처럼 그레이 색상 어떨까요?!

눈물머금고 형광노랑 보내줍니다 ,,, ❤️

leeeha · 2023-08-14T12:29:04Z

app/src/main/java/com/android/go/sopt/winey/presentation/main/feed/WineyFeedAdapter.kt

            }
-            notifyItemChanged(index)
        }
    }



여기 아래 코드에서 궁금한 점이 있는데요! DiffUtil.ItemCallback의 areItemsTheSame 함수를 오버라이딩 할 때는 두 객체가 동일한지 비교하게 되는데, 이때 보통 아이템을 유일하게 식별할 수 있는 id 값이나 === 연산자로 주소 값을 비교하는 거 같아요! 그런데 아래 코드에서 feedId 가 아니라, isLiked 로 WineyFeed 아이템을 식별한 이유가 있을까요??

만약 areItemsTheSame를 잘못 정의하면 모든 Item을 다 지우고 새 아이템을 Insert 하게 되므로, 기존의 notifyDataSetChanged()와 다를 것이 없어지기 때문에 주의해야 한다고 해요!

참고) https://jminie.tistory.com/146?category=1040997

어 마침 방금 푸쉬한거에 해당 부분 수정되어있습니다 ㅎㅎㅎ
앱잼기간에 좋아요 상태를 반영할때 해봤던건데 수정이안되어있었네요 ... ! 감사합니다 👍

leeeha

어푸! LGTM 👍👍

leeeha · 2023-08-16T02:06:11Z

app/src/main/java/com/android/go/sopt/winey/presentation/main/mypage/myfeed/MyFeedFragment.kt

+                                wineyFeedLoadAdapter.loadState = loadState.refresh
+                                checkAndSetEmptyLayout()
+                            }
+                            myFeedAdapter.submitData(state.data)


이 부분에서 뭔가 로직이 잘못 되었는지 '아직 없사옵니다' 이미지가 잠깐 나타났다 사라지는 이슈가 다시 발생하더라구요! 저번에 이 문제 때문에 Loading 상태에서 해당 로직을 처리했던 거 같은데, 확인 한번 부탁드려요!

sxunea added 17 commits August 8, 2023 17:25

[add] #99 AuthPagingSource 추가

2923553

[feat] #90 viewModel에서 PagingData 가져오도록 구현

52a06ae

[feat] #99 PagingAdapter로 Paging된 데이터 넘겨 구현

d56bc26

[feat] #99 로딩 인디케이터 구현

6d900e3

[feat] #99 마지막페이지면 로딩 인디케이터 없도록 처리

2440b52

Merge remote-tracking branch 'origin/develop' into feature/feat-feed-…

cc8657e

…inifinitescroll # Conflicts: # app/src/main/java/com/android/go/sopt/winey/data/repository/AuthRepositoryImpl.kt # app/src/main/java/com/android/go/sopt/winey/presentation/main/feed/WineyFeedFragment.kt

[feat] #99 페이징로직에 맞게 좋아요,삭제 코드 정상화

6241f91

[feat] #99 피드 로딩/성공 MultiViewType 으로 변경, 그에 따른 모든 로직 수정하여 loading pro…

945290f

…gress bar 구현

[feat] #99 multiviewtype viewholder 추상화

e72dce8

Revert "[feat] #99 multiviewtype viewholder 추상화"

b512489

This reverts commit e72dce8.

Revert "[feat] #99 피드 로딩/성공 MultiViewType 으로 변경, 그에 따른 모든 로직 수정하여 loa…

59ef0fc

…ding progress bar 구현" - 방식 변경 위해 revert This reverts commit 945290f.

[feat] #99 로딩 프로그레스 바 LoadStateAdapter로 state 받아와 구현, 관련 페이징 로직 수정

ff6ddf1

[chore] #99 누락된 파일 커밋

535cfdc

[feat] #99 pagingSource 재사용 위해 추상화

c134619

[feat] #99 wineyFeed, myFeed pagingSource 분리

5dafc6f

[feat] #99 마이피드에도 페이징, 로딩 프로그레스바 적용

51a186b

[refactor] #99 리팩토링, 불필요 파일 및 코드 제거

d0d1610

sxunea requested review from leeeha and Sangwook123 and removed request for leeeha August 11, 2023 20:43

sxunea self-assigned this Aug 11, 2023

sxunea added feat ✨ 새로운 기능 구현 혜선 🐱 labels Aug 11, 2023

sxunea added this to the 기능 구현 ✨ milestone Aug 11, 2023

sxunea added 3 commits August 13, 2023 17:13

Merge remote-tracking branch 'origin/develop' into feature/feat-feed-…

f60a494

…inifinitescroll # Conflicts: # app/src/main/java/com/android/go/sopt/winey/data/repository/AuthRepositoryImpl.kt

[mod] #99 피드 불러오기 uiState 활용 후 emit 하도록 변경

bb0a1b6

[chore] #99 module,repository,service,datastore 용도별 분리

a433551

leeeha requested changes Aug 13, 2023

View reviewed changes

leeeha reviewed Aug 14, 2023

View reviewed changes

sxunea added 4 commits August 14, 2023 22:49

[mod] #99 코드리뷰 반영 및 간단 수정

37decea

[fix] #99 마이 피드 데이터 없을 시 로직 오류 해결

b459072

[feat] #99 좋아요 아이콘 바인딩 어댑터로 처리

2d78901

[chore] #99 ktlint 수정사항 반영

3699534

sxunea requested a review from leeeha August 14, 2023 16:48

leeeha approved these changes Aug 14, 2023

View reviewed changes

sxunea merged commit 0e389af into develop Aug 14, 2023
1 check passed

sxunea deleted the feature/feat-feed-inifinitescroll branch August 14, 2023 16:59

leeeha reviewed Aug 16, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat] 위니피드, 마이피드 / Paging3 적용 + 로딩 프로그레스바 구현 #108

[feat] 위니피드, 마이피드 / Paging3 적용 + 로딩 프로그레스바 구현 #108

sxunea commented Aug 11, 2023 •

edited

Loading

sxunea commented Aug 11, 2023 •

edited

Loading

leeeha left a comment

leeeha Aug 13, 2023

sxunea Aug 13, 2023

leeeha Aug 13, 2023

leeeha Aug 13, 2023

leeeha Aug 13, 2023

leeeha Aug 13, 2023

leeeha Aug 13, 2023

leeeha Aug 13, 2023

leeeha Aug 13, 2023

leeeha Aug 13, 2023

leeeha Aug 13, 2023

sxunea Aug 13, 2023

leeeha Aug 14, 2023

sxunea Aug 14, 2023

leeeha left a comment

leeeha Aug 16, 2023

		@@ -0,0 +1,43 @@
		package com.android.go.sopt.winey.data.source.pagingSource

[feat] 위니피드, 마이피드 / Paging3 적용 + 로딩 프로그레스바 구현 #108

[feat] 위니피드, 마이피드 / Paging3 적용 + 로딩 프로그레스바 구현 #108

Conversation

sxunea commented Aug 11, 2023 • edited Loading

📝 Work Description

📣 To Reviewers

sxunea commented Aug 11, 2023 • edited Loading

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

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

sxunea commented Aug 11, 2023 •

edited

Loading

sxunea commented Aug 11, 2023 •

edited

Loading