Top-59 feature falling Like UI #82

kwangddang · 2023-06-18T17:19:46Z

Explain this Pull Request 🙏

Like UI 구현

What has changed? 🤔

변경 사항
- AlertDialog가 다양한 형태로 여러 곳에서 재활용 되어서 core-ui에 따로 정의 후 기존에 구현되어 있던 것들 변경
- 투명한 BottomSheet을 이용하여 LikeDetail 화면 구현
- RecyclerView payload 적용
- RecyclerView는 주제와, 주제를 선택한 사람들 목록을 보여주어야 하므로 HashMap 형태로 관리
- 좋아요가 하나도 없을 시 보여줄 Empty 화면 구현

Screenshot 📸

Test Checklist 🚩

cwj-c

고생하셨습니다!!

cwj-c · 2023-06-19T01:30:37Z

core/ui/src/main/java/tht/core/ui/dialog/DialogUtil.kt

+        displayMetrics.heightPixels to displayMetrics.widthPixels
+    }
+
+fun setBottomSheetDialogSize(activity: Activity, bottomSheetDialog: BottomSheetDialog) {


setBottomSheetDialogSize 인데 set되는 값이 매개변수로 전달되거나, 함수 이름에 포함되면 좋을 것 같습니다

매개변수에 bottomSheetDialog를 넘겨주고 있는데 다른 걸 의미하는 걸까요?

어떤 Size로 Set되는지 네이밍에 포함되거나, 사이즈를 매개변수로 전달해주는건 어떨까 생각이 들었었습니다!!

cwj-c · 2023-06-19T02:19:43Z

feature/like/src/main/java/tht/feature/like/constant/LikeConstant.kt

@@ -0,0 +1,5 @@
+package tht.feature.like.constant
+
+object LikeConstant {


LikeFragment에서 LikeDetailFragment로 넘어갈때 argument의 key값으로 사용되는 것 같습니다
네이밍에 해당 의미를 담을 수 있게 하거나, Key값의 위치를 LikeDetailFragment쪽으로 이동하는것도 좋을 것 같다는 생각이 들었습니다!!

그리고 LikeDetailFragment에서 정적 팩토리 메소드 패턴을 사용하면 Key값을 캡슐화 할 수 있어서 더 좋은 방법이라고 생각됩니다

class LikeDetailFragment : Fragment() { companion object { private const val ARGUMENT_KEY = "key" fun getInstance(likeModel: LikeModel): LikeDetailFragment { return LikeDetailFragment().apply { arguments = Bundle().apply { putSerializable(ARGUMENT_KEY, likeModel) } } } } }

좋은 의견 감사합니다!
원래도 정적 팩토리 메서드 패턴을 사용하고 있었으니 더 좋은 방법 같네요!!

cwj-c · 2023-06-19T02:21:56Z

feature/like/src/main/java/tht/feature/like/detail/LikeDetailFragment.kt

+        likeUser = arguments?.let {
+            if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.TIRAMISU)
+                it.getSerializable(LikeConstant.KEY, LikeModel::class.java)
+            else
+                it.getSerializable(LikeConstant.KEY) as LikeModel
+        } ?: LikeModel.getDefaultLikeModel()
+        if (likeUser.nickname.isEmpty()) dismiss()
+        else {
+            binding.apply {
+                svDetail.clipToOutline = true
+                tvCategory.text = likeUser.category
+                tvNickname.text = getString(R.string.nickname, likeUser.nickname, likeUser.age)
+                tvAddress.text = likeUser.address
+                tvIntroduction.text = likeUser.introduce
+                likeUser.idealTypes.forEachIndexed { index, idealType ->
+                    addChip(binding.cgIdealType, idealType.emojiCode, idealType.title, index)
+                }
+                likeUser.interests.forEachIndexed { index, interest ->
+                    addChip(binding.cgInterests, interest.emojiCode, interest.title, index)
+                }
+                val images = listOf(binding.ivOne, binding.ivTwo, binding.ivThree)
+                likeUser.profileImgUrl.forEachIndexed { index, url ->
+                    loadImage(images[index], url)
+                }
+            }
+        }


제 생각에는 if (likeUser.nickname.isEmpty()) dismiss() 같은 판단은 ViewModel에서 수행해주는게 적합하다고 생각됩니다

SavedInstanceHandle로 ViewModel에서 argument를 전달받고, ViewModel에서 데이터나 SideEffect를 방출해주는 식으로 수행하는 방법은 어떠신가요??

cwj-c · 2023-06-19T02:22:50Z

feature/like/src/main/java/tht/feature/like/detail/LikeDetailFragment.kt

+        if (likeUser.nickname.isEmpty()) dismiss()
+        else {


컨벤션상 if else 구문에 모두 중괄호를 붙이거나, 둘다 붙이지 않아야 하는것으로 알고 있습니다!!
else문의 길이가 기니, if문에 중괄호를 추가해주는 방식으로 변경되면 컨벤션에 적합할 것 같습니다

넵 수정하겠습니다 😁

cwj-c · 2023-06-19T02:24:18Z

feature/like/src/main/java/tht/feature/like/detail/LikeDetailFragment.kt

+
+    private fun setOnClickListener() {
+        binding.ivClose.setOnClickListener {
+            dismiss()


간단한 기능이라 Activity에서 직접 dismiss를 호출해주는것도 좋지만, 일관성 있게 dismiss() 또한 ViewModel로 Event를 전달하고 ViewModel에서 State나 SideEffect를 방출해주는 방식의 동작도 좋을 것 같습니다

넵 dismiss()와 같은 행위? 들은 SideEffect를 방출해주는 방식으로 모두 수정하겠습니다!

cwj-c · 2023-06-19T02:38:46Z

feature/like/src/main/java/tht/feature/like/like/LikeViewModel.kt

+    override val _uiStateFlow: MutableStateFlow<LikeUiState> =
+        MutableStateFlow(LikeUiState.Empty)
+
+    private val likeItems = MutableStateFlow<LinkedHashMap<String, List<LikeModel>>>(MockData.data)


<LinkedHashMap<String, List<LikeModel>>> 는 제거되어도 될 것 같습니다!

cwj-c · 2023-06-19T02:43:26Z

feature/like/src/main/res/drawable/ic_notification.xml

Notification같이 Toolbar에 공통적으로 사용되는 아이콘은 core-ui에 배치하면 재사용이 가능할것 같은데... 고민이 드네요..

그렇죠 아무래도 여기저기 쓰이니까 타이틀만 바꿀 수 있도록 core-ui에 배치하면 좋을 것 같긴 합니다..

cwj-c · 2023-06-19T02:45:42Z

feature/like/src/main/java/tht/feature/like/like/adapter/MockData.kt

+import tht.feature.like.like.LikeModel
+
+object MockData {
+    val data = LinkedHashMap<String, List<LikeModel>>().apply {


왜 3명뿐인가요?????

ㅋㅋ.. 귀찮음 이슈..

cwj-c · 2023-06-19T02:51:03Z

feature/like/src/main/java/tht/feature/like/like/adapter/LikeAdapter.kt

+                )
+            )
+
+            else -> LikeContentViewHolder(


ViewHolder에 정적 팩토리 메소드를 추가해준다면, 멀티 뷰 타입 리사이클러 뷰의 Adapter의 onCreateViewHolder의 가독성이 좋아 질 것 같습니다!!

companion object { fun getInstance( parent: ViewGroup ) : LikeContentViewHolder { return LikeContentViewHolder( ItemContentBinding.inflate( LayoutInflater.from( parent.context ), parent, false ) ) } }

위와 같이 팩토리 메소드를 선언해 둔다면

override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): RecyclerView.ViewHolder { return when (viewType) { HEADER -> LikeHeaderViewHolder.getInstance(parent) else -> LikeContentViewHolder.getInstance(parent) } }

onCreateViewHolder가 위와 같이 가독성이 좋아지는 효과가 있어서 사용을 추천드립니다!

역시 팩토레 메소드 패턴의 장점은 가독성이 최고인 것 같네요..! 감사합니다 ㅎㅎ

cwj-c · 2023-06-19T02:51:50Z

feature/like/src/main/java/tht/feature/like/like/adapter/LikeAdapter.kt

+            override fun getChangePayload(oldItem: LikeItem, newItem: LikeItem): Any? {
+                if (oldItem is LikeItem.Content && newItem is LikeItem.Content) {
+                    if (oldItem.item.isNew != newItem.item.isNew)
+                        return newItem.item.isNew
+                }
+
+                return super.getChangePayload(oldItem, newItem)


cwj-c · 2023-06-19T02:54:00Z

KakaoTalk_Video_2023-06-19-11-52-36.mp4

조금 깜박이는 증상이이 있는 것 같습니다
혹시 처음에 크기가 크게 나왔다가, 작게 조절되는 현상인가 싶어서 코멘트 남겼습니다!
환경은 Android Studio Emulator Pixel 5 API 33 입니다

kwangddang added 27 commits June 12, 2023 01:38

feat: TOP-59 heart -> like 이름 변경

75e4086

feat: TOP-59 LikeFragment xml 파일 작성

f1add81

feat: TOP-59 item_like xml 작성

02ef667

feat: TOP-59 item_category.xml 작성

700f0bd

feat: TOP-59 item_like.xml에 new 표시 추가

e375245

feat: TOP-59 RecyclerView를 위해 LikeAdapter 정의

c86cedf

feat: TOP-59 RecyclerView new를 표시하기 위해 payload 적용

37584f3

feat: TOP-59 deleteLike 구현

e65e954

feat: TOP-59 Category와 LikeModel을 한번에 받을 수 있도록 HashMap 정의 후 로직 구현

2ac8693

feat: TOP-59 delete할 때 Category 순서가 유지되도록 LinkedHashMap으로 변경

6b1d35d

feat: TOP-59 LikeDetailFragment 임시 디자인

ec05d18

feat: TOP-59 Like -> LikeDetail 데이터 전달 구현

d562424

feat: TOP-59 BottomSheet 최대 높이, 최소 높이, 투명 설정

7784418

feat: TOP-59 X 버튼 dismiss 구현

b6878e1

feat: TOP-59 groupchip, image 로딩 구현

d81ef58

feat: TOP-59 LikeDetailFragment Button 추가

75e827b

feat: TOP-59 dialog resize

a4b1248

feat: TOP-59 BottomSheet shadow제거, radius 적용

2ec9fc7

feat: TOP-59 좋아요 없을 때 화면 구현

2193780

feat: TOP-59 BottomSheet에서 '다음에' 버튼 클릭 리스너 구현

8882510

feat: TOP-59 ReportDialog 구현

1e8f20c

feat: TOP-59 AlertDialog core-ui로 변경

ba0e37b

feat: TOP-59 Block, Report Dialog 로직 작성

b745281

feat: TOP-59 차단하기 애니메이션 적용

f239ae5

refactor: TOP-59 코드 정렬

11bd78c

refactor: TOP-59 next -> nextChance로 네이밍 변경

60f2e0f

refactor: TOP-59 AlertDialog 모두 core-ui에 정의된 메소드를 통해 생성하도록 변경

619ab63

kwangddang added 최광현 🐱 feat 📎 ui 🖍 labels Jun 18, 2023

kwangddang requested review from cwj-c and taewooyo June 18, 2023 17:19

kwangddang self-assigned this Jun 18, 2023

cwj-c approved these changes Jun 19, 2023

View reviewed changes

kwangddang and others added 9 commits June 28, 2023 17:37

refactor: TOP-59 LikeDetailFragment 정적 팩토리 메소드 패턴으로 변경

ba4ac79

refactor: TOP-59 dismiss() 모두 viewModel에서 sideEffect를 방출하도록 변경

8072b3e

refactor: TOP-59 finishBottomSheet() -> blockUser() 로 네이밍 변경

4394366

refactor: TOP-59 의미 없는 타입 선언 제거

5fcb4d0

refactor: TOP-59 ViewHolder 팩토리 메소드 패턴으로 변경

a64ea28

refactor: TOP-59 viewModelStateHandle 사용

c3f0e21

Merge branch 'develop' into feature/TOP-59_like_ui

fe401e3

refactor: TOP-59 ktLint check

6101ec0

Merge branch 'develop' into feature/TOP-59_like_ui

bbd5ae5

cwj-c merged commit 4a03da9 into develop Jan 12, 2024
3 checks passed

cwj-c deleted the feature/TOP-59_like_ui branch April 21, 2024 14:17

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Top-59 feature falling Like UI #82

Top-59 feature falling Like UI #82

kwangddang commented Jun 18, 2023

cwj-c left a comment

cwj-c Jun 19, 2023

kwangddang Jun 28, 2023

cwj-c Jun 29, 2023

cwj-c Jun 19, 2023

kwangddang Jun 28, 2023

cwj-c Jun 19, 2023

cwj-c Jun 19, 2023

kwangddang Jun 28, 2023

cwj-c Jun 19, 2023

kwangddang Jun 28, 2023

cwj-c Jun 19, 2023

cwj-c Jun 19, 2023

kwangddang Jun 28, 2023

cwj-c Jun 19, 2023

kwangddang Jun 28, 2023

cwj-c Jun 19, 2023

kwangddang Jun 28, 2023

cwj-c Jun 19, 2023

cwj-c commented Jun 19, 2023

		@@ -0,0 +1,5 @@
		package tht.feature.like.constant

		object LikeConstant {

Top-59 feature falling Like UI #82

Top-59 feature falling Like UI #82

Conversation

kwangddang commented Jun 18, 2023

Explain this Pull Request 🙏

What has changed? 🤔

Screenshot 📸

Test Checklist 🚩

cwj-c 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

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

cwj-c commented Jun 19, 2023