[FEAT/#233] 뷰 동적 크기 조정 및 버튼 연결, 제목 조정 #234

chattymin · 2024-03-06T18:23:36Z

closed [UI] 여행 친구들 프로필 뷰 / ui 구현 #223

⛳️ Work Description

페이지 크기 동적 조정
버튼 연결
다양한 분기처리

📸 Screenshot

no_smooth.mp4

yes.mp4

📢 To Reviewers

예시사진으론 안나왔는데 내 계정이 성향검사 안했을 경우 버튼도 다 나오고 분기처리 돼있서용
정말...많은일이 있었어요
이걸 해냄~~~~ 우하하하하하하하하하하하
행복하다

Marchbreeze

진짜 미친사람 ~~~~~~~~~~

Marchbreeze · 2024-03-06T18:29:39Z

data/src/main/java/com/going/data/repositoryImpl/ProfileRepositoryImpl.kt

+        participantProfileRequestModel: ParticipantProfileRequestModel
+    ): Result<ParticipantProfileResponseModel> =
+        runCatching {
+            profileDataSource.getParticipantProfile(participantProfileRequestModel.toParticipantRequestDto()).data.toParticipantProfileModel()


어제 코딩하면서 든 생각인데
얘네 그냥 다 toModel(), toDto()로 둘까 고민했었삼 어처피 특정 모델에 대한 함수라소
그치만 귀찮아서 건드리지는~ 않았다~

넘나 맞는 말인거 같네욤...
생각해보니까 우리는 맵퍼를 같이 묶어두니까 진짜 그렇게 네이밍 해도 될거같음!!!

매퍼를 따로 놔두면 풀네임하는게 맞겠지만....

Marchbreeze · 2024-03-06T18:32:20Z

...ation/src/main/java/com/going/presentation/profile/participant/ParticipantProfileActivity.kt

+    private val participantId: Long by lazy {
+        intent.getLongExtra(PARTICIPANT_ID, 0)
+    }


이거 내 뷰에서도 바꿀라고 봤는데,
저는 아이디값들을 모두 뷰모델에 저장해두고 있더라구요 !

여기 액티비티에서도 아이디값을 활용하는 부분은 모두 서버통신인 것 같은데,
뷰모델에서 저장하고, 서버통신 실행하는 함수의 파라미터를 지우고 뷰모델에서 실행 시 넣어주는 방법이 비즈니스 로직을 줄이는 방법일 것 같다고 생각했습니다 !

사실 여기 뷰는 그렇게 많이 안쓰여서 그렇게 상관은 없어보이는데, 투두 수정뷰는 아디값들이 계속 쓰이길래...ㅎㅎ

생각해보니까 데이터값을 activity에서 저장하고있다는게 안맞는거 같네요..!!
viewModel에서 데이터를 가지고 있는게 맞는것 같습니당!

Marchbreeze · 2024-03-06T18:32:53Z

...ation/src/main/java/com/going/presentation/profile/participant/ParticipantProfileActivity.kt

+                when (state) {
+                    is UiState.Loading -> return@onEach
+                    is UiState.Success -> bindData(state.data)
+                    is UiState.Failure -> toast(state.msg)


요거는 일단 기본 토스트로?

아

스낵바적용해야하는데 깜빡했다....
일단 스낵바부터 좀 손보고 적용도 하겠슴다....

일단 토스트로 하고... 기능부터 구현할게요....
마지막 QA기간이나 이때 일괄 적용하겠습니다

Marchbreeze · 2024-03-06T18:33:51Z

...ation/src/main/java/com/going/presentation/profile/participant/ParticipantProfileActivity.kt

+                    behavior?.setDragCallback(object : DragCallback() {
+                        override fun canDrag(appBarLayout: AppBarLayout): Boolean {
+                            return !this@with
+                        }
+                    })
+                    setFragmentHeight(this)
+
+                    if (this) binding.appbarTripProfile.setExpanded(true)


오...흥미로운 친구들

Marchbreeze · 2024-03-06T18:35:09Z

.../main/java/com/going/presentation/profile/participant/ParticipantProfileCharacterFragment.kt

+        val participantViewModel =
+            ViewModelProvider(requireActivity()).get(ParticipantProfileViewModel::class.java)


요 친구도 by Lazy로 처리하는건 안되려나?

가능가능!!!
수정햇심다

Marchbreeze · 2024-03-06T18:35:55Z

.../main/java/com/going/presentation/profile/participant/ParticipantProfileCharacterFragment.kt

+            when (state) {
+                is UiState.Success -> bindData(state.data.isOwner, state.data.result)
+                else -> return@onEach
+            }


이런 경우에는 UiState 대신 Boolean값으로 처리하는것도 괜찮을 것 같아요! SharedFlow를 활용하면 더 간편해질듯

SharedFlow를 사용하는게 맞는 것 같아서 수정했습니다!

Marchbreeze · 2024-03-06T18:36:06Z

.../main/java/com/going/presentation/profile/participant/ParticipantProfileCharacterFragment.kt

+            btnTripProfileRestart.isVisible = isOwner
+            btnParticipantProfileEmptyOwner.isVisible = isOwner
+            tvParticipantProfileEmptyOwner.isVisible = isOwner


leeeyubin

진짜 천재만재 개발자,,,,,,최고~!!

leeeyubin · 2024-03-06T20:47:24Z

...ation/src/main/java/com/going/presentation/profile/participant/ParticipantProfileActivity.kt

+    private fun setFragmentHeight(temp: Boolean = true) {
+        val displayHeight = getWindowHeight()
+        val toolbarHeight = binding.tbTripProfile.height
+        val appBarHeight = binding.appbarTripProfile.totalScrollRange
+        val tabHeight = binding.tabTripProfile.height
+
+        binding.vpTripProfile.layoutParams = binding.vpTripProfile.layoutParams.also {
+            it.height =
+                if (temp) displayHeight - toolbarHeight - appBarHeight - tabHeight else displayHeight


와,,,,너무 신기합니다,,

이렇게 계산해서 구하다니.................멋진데요..

leeeyubin · 2024-03-06T20:55:22Z

...ation/src/main/java/com/going/presentation/profile/participant/ParticipantProfileActivity.kt

+    private fun observeParticipantProfileState() {
+        participantProfileViewModel.participantProfileState.flowWithLifecycle(lifecycle)
+            .onEach { state ->
+                when (state) {
+                    is UiState.Loading -> return@onEach
+                    is UiState.Success -> bindData(state.data)
+                    is UiState.Failure -> toast(state.msg)


이거 말씀하신거군요! 잘 사용하겠습니다!!

crownjoe

입이 안다물어짐.이걸어케함...역시 오비.....>!! 수고하셧슴니다!!

crownjoe · 2024-03-07T12:15:40Z

...ation/src/main/java/com/going/presentation/profile/participant/ParticipantProfileActivity.kt

+    private fun setFragmentHeight(temp: Boolean = true) {
+        val displayHeight = getWindowHeight()
+        val toolbarHeight = binding.tbTripProfile.height
+        val appBarHeight = binding.appbarTripProfile.totalScrollRange
+        val tabHeight = binding.tabTripProfile.height
+
+        binding.vpTripProfile.layoutParams = binding.vpTripProfile.layoutParams.also {
+            it.height =
+                if (temp) displayHeight - toolbarHeight - appBarHeight - tabHeight else displayHeight


이렇게 계산해서 구하다니.................멋진데요..

chattymin added 17 commits March 6, 2024 17:53

[ADD/#233] Intent 추가

29211f4

[ADD/#233] domain request 관련 정의

5ecbb34

[ADD/#233] domain response 관련 정

bdc26aa

[ADD/#233] domain, data 서버통신 관련 repository 패턴 코드 구현

c1f7d81

[ADD/#233] guest img 추가

9a77963

[FEAT/#233] api 통신 1차 연결

a4e1d98

[UI/#233] UI 수정. img 기본값 추가

324da5f

[FEAT/#233] ViewModel 공유를 통한 activity와 fragment data 연결

54ae8e6

[FIX/#233] error 상황을 고려한 btn default invisible 설정

4d765db

[FEAT/#233] 불필요한 Null 체크 제거

e2322bc

[CHORE/#233] 네이밍 변경

a8447cc

[FEAT/#233] 동적 페이지 이동 구현

729ef7d

[ADD/#233] 관련 string, img 처리

66c56d4

[ADD/#233] 친구 이름 분기처리

a333a45

[FEAT/#233] 이미지 다운로드 구현

1f200fa

[FEAT/#233] navigate 구현

9ac3c15

[CHORE/#233] 테스트용 변수 제거

77008c2

chattymin added 동민 🐥 ADD ➕ 부수적인 코드 추가 및 라이브러리 추가, 새로운 파일 생성 FEAT ✨ 새로운 기능 구현 labels Mar 6, 2024

chattymin added this to the 2차 스프린트 milestone Mar 6, 2024

chattymin requested review from Marchbreeze, leeeyubin and crownjoe March 6, 2024 18:23

chattymin self-assigned this Mar 6, 2024

Marchbreeze approved these changes Mar 6, 2024

View reviewed changes

leeeyubin approved these changes Mar 6, 2024

View reviewed changes

[FIX/#233] viewModel by lazy 선언

c43bc2b

crownjoe approved these changes Mar 7, 2024

View reviewed changes

[FIX/#233] viewModel sharedFlow로 수정

c232065

chattymin added 4 commits March 8, 2024 00:36

[FIX/#233] isEmpty 변수 viewModel로 이동

33aa96c

[FIX/#233] exported false

3d8ff2c

Merge branch 'develop' into feat/#233-trip-friend-api

ad74d5b

[FIX/#233] 최종 수정

a0b56e0

chattymin merged commit 00b693c into develop Mar 7, 2024
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[FEAT/#233] 뷰 동적 크기 조정 및 버튼 연결, 제목 조정 #234

[FEAT/#233] 뷰 동적 크기 조정 및 버튼 연결, 제목 조정 #234

chattymin commented Mar 6, 2024 •

edited

Loading

Marchbreeze left a comment

Marchbreeze Mar 6, 2024

chattymin Mar 6, 2024

Marchbreeze Mar 6, 2024

chattymin Mar 7, 2024

Marchbreeze Mar 6, 2024

chattymin Mar 6, 2024

chattymin Mar 7, 2024

Marchbreeze Mar 6, 2024

Marchbreeze Mar 6, 2024

chattymin Mar 7, 2024

Marchbreeze Mar 6, 2024

chattymin Mar 7, 2024

Marchbreeze Mar 6, 2024

leeeyubin left a comment

leeeyubin Mar 6, 2024

crownjoe Mar 7, 2024

leeeyubin Mar 6, 2024

crownjoe left a comment

crownjoe Mar 7, 2024

		val participantViewModel =
		ViewModelProvider(requireActivity()).get(ParticipantProfileViewModel::class.java)

[FEAT/#233] 뷰 동적 크기 조정 및 버튼 연결, 제목 조정 #234

[FEAT/#233] 뷰 동적 크기 조정 및 버튼 연결, 제목 조정 #234

Conversation

chattymin commented Mar 6, 2024 • edited Loading

⛳️ Work Description

📸 Screenshot

📢 To Reviewers

Marchbreeze 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

leeeyubin 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

crownjoe left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

chattymin commented Mar 6, 2024 •

edited

Loading