feat: 바텀네비게이션에 서재탭 추가 by yeonjeen · Pull Request #631 · Team-WSS/WSS-Android

yeonjeen · 2025-04-13T07:40:07Z

📌𝘐𝘴𝘴𝘶𝘦𝘴

closed feat: 바텀네비게이션에 서재탭 추가 #625

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

바텀네비게이션 서재탭 추가
서재뷰 구현
기존 보관함 불필요한 코드 제거
ux 라이팅 수정(보관함 > 서재)

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

KakaoTalk_Video_2025-04-13-16-37-28.mp4

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

기존 보관함을 '나'와 '다른유저'가 공유하고 사용하면서 분기처리 되어졌던 부분은 삭제를 했습니다.
그 외에는 보관함은 건들이지 않았고 보관함이 기존에는 activity였기 때문에 fragment 형식으로 서재를 만들었습니다!
코드는 기존 보관함과 거의 유사하니 참고해주세요!!

junseo511

고생하셨습니다 👍 몇가지 리뷰만 확인해주세요

junseo511 · 2025-04-14T06:00:31Z

app/src/main/java/com/into/websoso/ui/main/MainActivity.kt

-                entries.find { fragmentView ->
-                    fragmentView.resId == id
-                } ?: throw IllegalArgumentException()
+                entries.find { it.resId == id }


a: 이쪽이 가독성이 더 좋아진 것 같네요 👍

junseo511 · 2025-04-14T06:03:28Z

app/src/main/java/com/into/websoso/ui/main/library/LibraryFragment.kt

+
+    private val libraryAdapter: LibraryViewPagerAdapter by lazy {
+        LibraryViewPagerAdapter(
+            novels = emptyList(),


c: 기본 생성자로 emptyList()를 두는 것과 초기화시 emptyList()를 넣어주는 것에는 어떤 차이점이 있을까요?

emptyList()를 기본 생성자로 두는 것과 초기화시 두는 것은 동일한 동작을 하는 것으로 알고 있습니다! 하지만 설계적 관점에서는 차이가 있을 수 있는데 생성장에 기본값으로 두는 경우 해당 클래스가 내부적으로 어떤 초기 상태를 갖는지를 스스로 정의하는 것이기 때문에 빈리스트로 시작하다는 것을 명시적으로 전달할 수 있습니다! 반면 초기화시 두는 것은 생성하는 쪽에서 초기 상태를 명시적으로 결정할 수 있어서 책임의 위치가 보다 외부로 분리된다는 장점이 있다고 알고 있습니다! 이는 어댑터를 재사용할 때 더 유연하게 동작하게 만듭니다! 현재와 같이 외부에서 초기화를 관리하는 것이 어댑터를 보다 명시적으로 제어할 수 있는 장점이 있다고 생각합니다

junseo511 · 2025-04-14T06:03:55Z

app/src/main/java/com/into/websoso/ui/main/library/LibraryFragment.kt

+        super.onViewCreated(view, savedInstanceState)
+        super.onViewCreated(view, savedInstanceState)


c: onViewCreated를 두 번이나 실행시킨 이유가 있으실까요?

으앗 이건 정신없는 복붙에 향연 속 실수입니다...ㅎ

junseo511 · 2025-04-14T06:04:34Z

app/src/main/java/com/into/websoso/ui/main/library/LibraryFragment.kt

+    private fun setupViewPagerAndTabLayout() {
+        setupViewPager()
+        setupTabLayoutWithViewPager()
+        setupObserver()


c: 함수명에 비해 많은 일을 하고 있는 것은 아닌지 생각해봅시다!

으음 그러고 보니 그렇네염! 리뷰 감사합니다! 저는 setupObserver()만 따로 onViewCreated()만 분리해도 좋을 것 같은데 어떻게 생각하시나염

으음 그러고 보니 그렇네염! 리뷰 감사합니다! 저는 setupObserver()만 따로 onViewCreated()만 분리해도 좋을 것 같은데 어떻게 생각하시나염

그런 방향도 괜찮게 보입니다 👍

junseo511 · 2025-04-14T06:06:02Z

app/src/main/java/com/into/websoso/ui/main/library/LibraryFragment.kt

+
+    private fun setupTabLayoutWithViewPager() {
+        TabLayoutMediator(binding.tlLibrary, binding.vpLibrary) { tab, position ->
+            tab.text = StorageTab.fromPosition(position).title


c: fromPosition(position) 이라는 식이 조금 어색한데 fromPosition은 기본으로 제공되는 확장함수인가요?

기본으로 제공되는 확장함수는 아닙니다! 탭 이넘 클래스에서 직접 정의한 정적 함수인데요! 처음 보시면 조금 어색하게 느껴지실 수 있다고 생각합니다 아마 저도 다른 분들의 코드를 참고하면서 사용했던 걸로 알고 이넘에서는 관용적으로 사용되는 함수명인 것 같아요!!

junseo511 · 2025-04-14T06:06:52Z

app/src/main/java/com/into/websoso/ui/main/library/LibraryFragment.kt

+        binding.tlLibrary.addOnTabSelectedListener(
+            object : TabLayout.OnTabSelectedListener {
+                override fun onTabSelected(tab: TabLayout.Tab?) {
+                    val selectedTab = requireNotNull(tab) { "Tab must not be null" }


c: require을 통해 강제하신 이유가 있으신가요?

require로 둔 이유는 탭 파라미터가 null일 가능성이 없기 때문에 명시적으로 null이 될 수 없는 값이라는 것을 표시하고 싶었습니다!

junseo511 · 2025-04-14T06:07:12Z

app/src/main/java/com/into/websoso/ui/main/library/LibraryFragment.kt

+        }.attach()
+
+        binding.tlLibrary.addOnTabSelectedListener(
+            object : TabLayout.OnTabSelectedListener {


a: 명시적인 리스너 활용 좋습니다 👍

junseo511 · 2025-04-14T06:07:52Z

app/src/main/java/com/into/websoso/ui/main/library/LibraryFragment.kt

+            binding.clLibraryNull.visibility =
+                if (uiState.userNovelCount == 0L) View.VISIBLE else View.GONE
+
+            binding.vpLibrary.visibility =
+                if (uiState.userNovelCount > 0L) View.VISIBLE else View.GONE
+
+            binding.btnLibraryGoToSearchNovel.visibility =
+                if (uiState.userNovelCount == 0L) View.VISIBLE else View.GONE


c: 반복되는 상수는 const로 뺌은 어떠신지 궁금합니다~~

앗 그러네염 반영하겠습니다~

junseo511 · 2025-04-14T06:10:46Z

app/src/main/java/com/into/websoso/ui/main/library/adapter/LibraryViewPagerAdapter.kt

+
+    @SuppressLint("NotifyDataSetChanged")
+    fun updateNovels(newNovels: List<StorageNovelModel>) {
+        this.novels = newNovels


c: 여기서만 this를 명시해주신 이유가 있으실까요? 위에서는 그대로 사용하는 것으로 보이는데요!

바로 위에서 newNovels라는 매개변수와 클래스의 novels 프로퍼티 간 이름이 유사해서 양쪽을 명확히 구분해 의도를 더 분명히 전달하기 위함이었는데 혹시 다른 의견있으신가여?

저는 그냥 사용하는 다른 위치에서의 일관성을 생각했어요!! 본인이 좋으신 방향으로 고고하시죠~~

m6z1

고생하셨어요 ~~ ! 👍
소소한 코멘트 확인 부탁드립니당

m6z1 · 2025-04-14T13:34:44Z

app/src/main/java/com/into/websoso/ui/main/library/LibraryFragment.kt

+        setupViewPagerAndTabLayout()
+        setupInitialReadStatusTab()
+        onSortTypeButtonClick()
+        onExploreButton()


r: 요것도 postfix 에 Click 붙여주세요!

m6z1 · 2025-04-14T13:37:24Z

app/src/main/java/com/into/websoso/ui/main/library/LibraryFragment.kt

+    }
+
+    private fun setupObserver() {
+        libraryViewModel.uiState.observe(requireActivity()) { uiState ->


r: 요거 requireActivity 말고 viewLifecycleOwner 사용해주세요!!

네에엡!!

m6z1 · 2025-04-14T13:39:26Z

app/src/main/java/com/into/websoso/ui/main/library/adapter/LibraryViewPagerViewHolder.kt

+        LibraryItemAdapter(emptyList(), novelClickListener)
+    }
+
+    private val layoutManager: GridLayoutManager by lazy {


a: 이거 xml 에서 세팅해도 되지 않나욥 ??

맞아욤! 지금은 고려하지 않아도 되는 사항인 것 같아서 xml에서 관리하도록 하겠습니담!

# Conflicts: # app/src/main/res/values/strings.xml

junseo511

고생하셨슴당

yeonjeen added 13 commits April 13, 2025 16:16

feat: 서재 xml 구현

0cf3e43

feat: 서재 구현

a8cb940

feat: 바텀네비게이션 서재 탭 추가

b66cf0e

refactor: 프레그먼트 서재 타입 임포트

dc96079

add: 서재 아이콘 추가

de1aa7a

feat: MyLibraryFragment > LibraryFragment로 이동 함수 구현

d9c189f

refactor: 불필요한 인자 제거

76388a5

refactor: 린트 개행

2ad9068

refactor: 린트 개행

50e1bdd

refactor: 불필요한 코드 제거

cd00192

refactor: 불필요한 코드 제거

df4c80a

refactor: 불필요한 코드 제거

8328b76

refactor: requireActivity() 적용

5ebfe5f

yeonjeen added 🍯 [FEAT] 새로운 기능을 개발합니다. 👛 황후 연진 labels Apr 13, 2025

yeonjeen added this to the 부활소소 milestone Apr 13, 2025

yeonjeen requested a review from a team April 13, 2025 07:40

yeonjeen self-assigned this Apr 13, 2025

junseo511 suggested changes Apr 14, 2025

View reviewed changes

m6z1 approved these changes Apr 14, 2025

View reviewed changes

s9hn approved these changes Apr 14, 2025

View reviewed changes

yeonjeen added 5 commits April 15, 2025 01:04

refactor: 린트 반영

5656628

refactor: it을 명시적 변수명으로 개선

214c2f4

refactor: setupViewPagerAndTabLayout()에서 setupObserver()함수 분리

193b64a

refactor: 불필요한 GridLayoutManager 제거

d791c10

Merge remote-tracking branch 'origin/develop' into feat/625

54d3e13

# Conflicts: # app/src/main/res/values/strings.xml

junseo511 approved these changes Apr 14, 2025

View reviewed changes

yeonjeen added 2 commits April 15, 2025 01:15

refactor: 린트 수정

32a43b1

feat: 내 서재> 내 통계로 ux 라이팅 변경

f61eb2a

yeonjeen merged commit 8e2b50e into develop Apr 14, 2025
1 check passed

m6z1 deleted the feat/625 branch April 19, 2025 17:12

		super.onViewCreated(view, savedInstanceState)
		super.onViewCreated(view, savedInstanceState)

Conversation

yeonjeen commented Apr 13, 2025

📌𝘐𝘴𝘴𝘶𝘦𝘴

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

📷𝘚𝘤𝘳𝘦𝘦𝘯𝘴𝘩𝘰𝘵

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

Uh oh!

junseo511 left a comment

Choose a reason for hiding this comment

Uh oh!

junseo511 Apr 14, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

m6z1 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

junseo511 left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

junseo511 Apr 14, 2025 •

edited

Loading