#91/enhancement/jsp/app enhancement #96

pknujsp · 2023-06-01T15:16:45Z

📌 관련 이슈

#91

✨ 과제 내용

검색 목록 저장/로드/삭제 로직 구현 완료, Room사용, core:database모듈
BaseNavArgs개선 인자를 전달받지 않은 경우에도 오류없이 도착지에서 처리가능
댓글 전송 버튼 클릭 시 BindingAdapter로 ViewModel에 데이터 전달하여 댓글 전송 로직 동작
로딩 다이얼로그 레이아웃 개선
HomeFragment내 RecentPenaltiesFragment에서 NewsScreen으로 이동가능
AI 카메라 Fragment 오류 수정, 로직 최적화하였고, 캡처된 이미지 클릭 시 이미지가 표시됨
위의 이미지에 확대/축소 기능 추가하기 위해서 PhotoView라이브러리 추가

📸 스크린샷(선택)

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

…chHistoryRepository, RoomDb, RoomModule등 Hilt로 의존성 주입, Room 라이브러리 의존성 추가, :core:database 모듈 추가

pknujsp · 2023-06-01T15:19:33Z

app/src/main/java/com/android/mediproject/MainActivity.kt

@@ -60,10 +61,26 @@ class MainActivity :
            viewModel = activityViewModel.apply {
                repeatOnStarted { eventFlow.collect { handleEvent(it) } }
            }
+
+            root.viewTreeObserver.addOnPreDrawListener(object : ViewTreeObserver.OnPreDrawListener {


MainActivity의 루트 레이아웃이 Coordinator이라서 FragmentContainer의 높이가 MATCH_PARENT가 되어 BottomNav와 겹치는데 이 문제를 수정하기 위한 코드입니다. FragmentContainer의 높이를 수정합니다

뭐할때 호출되는 리스너인가요?.?

mainactivity의 뷰가 그려지기 직전에 호출되고 BottomNav높이를 가져와서 fragmentcontainerview높이 설정해줍니다

pknujsp · 2023-06-01T15:20:17Z

core/common/src/main/java/com/android/mediproject/core/common/adapter/GodBindListAdapter.kt

일단 만들어놨는데 아직 쓰질 않았네요

pknujsp · 2023-06-01T15:22:42Z

core/common/src/main/java/com/android/mediproject/core/common/bindingadapter/BindingAdapter.kt

+     */
+    @BindingAdapter("onClickSend", "inputText")
+    @JvmStatic
+    fun setOnClick(view: View, iSendText: ISendText, editText: EditText) {


댓글 전송 버튼 클릭 시 돌아가는 코드입니다.
editText의 값을 비우고,
ViewModel에 ISentText인터페이스를 구현하여 직접 ViewModel을 참조하지 않도록 했습니다.
iSendText.onClickedSendButton()을 호출하면 댓글을 전송하는 코드를 ViewModel에서 돌립니다.

오 뷰모델을 직접 안넣어주고 인터페이스로 쓰면 편리하네요

pknujsp · 2023-06-01T15:28:22Z

core/common/src/main/java/com/android/mediproject/core/common/dialog/ProgressDialog.kt

로딩 다이얼로그 레이아웃 모서리가 사각형이 되어서 라운드로 되게 머티리얼 다이얼로그로 바꿨습니다

pknujsp · 2023-06-01T15:28:58Z

core/common/src/main/java/com/android/mediproject/core/common/paging/PagingUtil.kt

여기에서 페이징 상태에 따라 리스트뷰, 텍스트뷰, 로딩 바 보여줄지 여부를 처리합니다

pknujsp · 2023-06-01T15:30:51Z

core/common/src/main/java/com/android/mediproject/core/common/util/UriBuilder.kt

+
+
+fun NavController.deepNavigate(


deepNavigate로 새로 만들었고,
기존에는 Argument가 XML에 실제로 붙여지면서 처리되는 거였는데,
이번에는 XMLd에 Argument를 추가하지 않고 딥링크를 새로 추가해서 돌립니다.
데이터 전달 안받는 프래그먼트에서도 오류없이 잘 돌아가네요.
by navArgs로 하면됩니다.

이건 실제로 써봐야 알 것 같아요!

pknujsp · 2023-06-02T02:10:38Z

core/domain/src/main/java/com/android/mediproject/core/domain/CommentsUseCase.kt

댓글 처리하는 유스케이스입니다.
내 댓글 목록 처리나 댓글 등록 등 관련한 모든 작업은 이걸로 하면 될거 같습니다

pknujsp · 2023-06-02T02:12:47Z

feature/camera/src/main/res/layout/fragment_detected_image_dialog.xml

캡처된 사진을 보여주는 다이얼로그의 레이아웃 입니다.
뒤로가기 버튼, 확대, 축소 버튼 구성입니다

pknujsp · 2023-06-02T02:13:48Z

...va/com/android/mediproject/feature/comments/commentsofamedicine/MedicineCommentsViewModel.kt

+ * @property NONE 초기 상태
+ */
+@Suppress("ClassName")
+sealed class CommentActionState {


댓글 관련 처리 로직 상태입니다.
좋아요 버튼 클릭 댓글 등록 버튼 클릭 등의 처리를 flow에 저장합니다

tgyuuAn · 2023-06-02T05:56:35Z

app/src/main/java/com/android/mediproject/MainActivity.kt

@@ -60,10 +61,26 @@ class MainActivity :
            viewModel = activityViewModel.apply {
                repeatOnStarted { eventFlow.collect { handleEvent(it) } }
            }
+
+            root.viewTreeObserver.addOnPreDrawListener(object : ViewTreeObserver.OnPreDrawListener {


뭐할때 호출되는 리스너인가요?.?

tgyuuAn · 2023-06-02T06:08:53Z

core/common/src/main/java/com/android/mediproject/core/common/bindingadapter/BindingAdapter.kt

+     */
+    @BindingAdapter("onClickSend", "inputText")
+    @JvmStatic
+    fun setOnClick(view: View, iSendText: ISendText, editText: EditText) {


오 뷰모델을 직접 안넣어주고 인터페이스로 쓰면 편리하네요

tgyuuAn · 2023-06-02T06:11:50Z

core/common/src/main/java/com/android/mediproject/core/common/util/UriBuilder.kt

+
+
+fun NavController.deepNavigate(


이건 실제로 써봐야 알 것 같아요!

tgyuuAn · 2023-06-02T06:13:20Z

...tabase/src/androidTest/java/com/android/mediproject/core/database/ExampleInstrumentedTest.kt

+@RunWith(AndroidJUnit4::class)
+class ExampleInstrumentedTest {
+    @Test
+    fun useAppContext() {


유닛테스트 해보신거에욬ㅋㅋㅋㅋㅋㅋㅋㅋㅋㅋ?

안했는데 자동으로 만들어졌나보네욬ㅋ

tgyuuAn · 2023-06-02T06:16:25Z

...medicine/src/main/java/com/android/mediproject/feature/medicine/main/MedicineInfoFragment.kt

+                topAppBar.removeOnOffsetChangedListener(null)
+                // smoothly hide medicinePrimaryInfoViewgroup when collapsing toolbar
+                topAppBar.addOnOffsetChangedListener { appBarLayout, verticalOffset ->
+                    // 스크롤 할 때 마다 medicinePrimaryInfoViewgroup의 투명도 조정


setOnSCrollChangedListner랑 addOnOffsetChangedListner랑 뭐가 다른거죠?.?

addOnOffsetChangedListner이거는 스크롤될때마다 topAppBar가 최초 원래 y좌표에서 얼마나 멀어지거나 가까워지는지를 받는 리스너인데 이거 쓰면 리스너를 단 뷰의 높이 등을 바꿀수 있어요.

onScroll은 스크롤 할때마다 최초 스크롤 좌표랑 현재 스크롤 중인 좌표를 받을수있는데 이거는 화면 스크롤 하면 FAB버튼이 사라졌다 나타났다 같은 기능을 만들수 있습니다. 이거는 아마 스크롤 뷰같은거에만 달수 있는걸로 알고있는데 암튼 이런 차이가 있습니당

pknujsp added 11 commits May 29, 2023 18:48

#91 검색 목록 저장/로드/삭제 하는 로직 제작, SearchHistoryDao, SearchHistoryDto, Sear…

bf49823

…chHistoryRepository, RoomDb, RoomModule등 Hilt로 의존성 주입, Room 라이브러리 의존성 추가, :core:database 모듈 추가

#91 BaseNavArgs개선(NavArgs로 가능한 거의 대부분의 타입 호환)

d0cfa8e

#91 수동 검색 로직 수정 중

a83ad40

#91 수동 검색 로직 작업 완료, 검색 기록 데이터 베이스 저장 로직 제작 완료

652c2c1

#91 검색 기록 저장/수정하는 SearchHistoryRepository내 로직 최적화, 불필요한 suspend 제거

5531eed

#91 댓글 전송 시 로직을 BindingAdapter로 통합해 제작, 댓글 등록/수정/좋아요/삭제 비즈니스 로직 교체,

d8517f0

#91 로딩 다이얼로그 라운드 코너 미표시 오류 수정

4a17b86

#91 HomeFragment에서 회수폐기 아이템 클릭 시 NewsScreen으로 이동하는 로직 구현 중

b1e113b

#91 navigateByDeepLink 개선, arguments를 전달받지 않은 경우에 빈 객체를 받도록 개선

9d130af

#91 AI카메라 프래그먼트 로직 오류 수정, 캡처된 이미지 보여주는 dialog수정중

d6b1a43

#91 AI카메라 화면 오류 수정 완료

dad2251

pknujsp added enhancement 기능 박준성 안드로이드 🔨refactor🔨 코드 리팩터링! labels Jun 1, 2023

pknujsp requested a review from tgyuuAn June 1, 2023 15:16

pknujsp commented Jun 1, 2023

View reviewed changes

pknujsp commented Jun 2, 2023

View reviewed changes

tgyuuAn approved these changes Jun 2, 2023

View reviewed changes

pknujsp merged commit 7e18ba4 into android_develop Jun 2, 2023

tgyuuAn assigned pknujsp Jun 15, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

#91/enhancement/jsp/app enhancement #96

#91/enhancement/jsp/app enhancement #96

pknujsp commented Jun 1, 2023

pknujsp Jun 1, 2023

tgyuuAn Jun 2, 2023

pknujsp Jun 2, 2023

pknujsp Jun 1, 2023

pknujsp Jun 1, 2023

tgyuuAn Jun 2, 2023 •

edited

Loading

pknujsp Jun 1, 2023

pknujsp Jun 1, 2023

pknujsp Jun 1, 2023

tgyuuAn Jun 2, 2023

pknujsp Jun 2, 2023

pknujsp Jun 2, 2023

pknujsp Jun 2, 2023

tgyuuAn Jun 2, 2023

tgyuuAn Jun 2, 2023 •

edited

Loading

tgyuuAn Jun 2, 2023

tgyuuAn Jun 2, 2023

pknujsp Jun 2, 2023

tgyuuAn Jun 2, 2023

pknujsp Jun 2, 2023 •

edited

Loading

#91/enhancement/jsp/app enhancement #96

#91/enhancement/jsp/app enhancement #96

Conversation

pknujsp commented Jun 1, 2023

📌 관련 이슈

✨ 과제 내용

📸 스크린샷(선택)

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

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

tgyuuAn Jun 2, 2023 • edited Loading

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

tgyuuAn Jun 2, 2023 • edited Loading

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

pknujsp Jun 2, 2023 • edited Loading

Choose a reason for hiding this comment

tgyuuAn Jun 2, 2023 •

edited

Loading

tgyuuAn Jun 2, 2023 •

edited

Loading

pknujsp Jun 2, 2023 •

edited

Loading