[Feat] 바텀 네비게이션 도입 및 기존 캘린더 및 마이페이지 화면 fragment 로 전환 #293

kangyuri1114 · 2025-05-12T08:28:38Z

Summary

`develop` ~`feat/set-bottom-navigation` 브랜치까지의 작업 내용

MainActivity 내부 3개의 bottomNavi 화면이 도입될 fragment를 넣는 구조로 수정
화면명 수정 및 새 화면 추가

CafeteriaFragment 추가
MapFragment
- MapFragmentComposeView 컴포즈 뷰 추가
MyPageFragment 추가

Fragment 토스트 유틸 함수 추가
바텀네비 관련 menu 및 ui xml 추가

`feat/set-bottom-navigation` 에서 중복 메뉴 호출 및 이전 메뉴 화면 누적되어 반환하던 오류 수정을 위한 브랜치인 feat/fix-bottom-navi-error 작업내용

기존 오류 해결 - 오류 원인 : 기존에는 observe()를 매번 날짜 변경 시마다 호출하며, 메뉴 항목(Section)이 중복으로 totalMenuList에 누적됨

항목	설명
문제	오늘 메뉴 섹션(학식, 도담, 기숙사)이 날짜 변경 시 누적됨
원인	collect 안에서 totalMenuList.add()만 하고 이전 데이터를 제거하지 않음
해결	totalMenuList.removeAll { it.cafeteria == XX } 추가해서 기존 데이터 제거 후 추가
Fixed 메뉴는 왜 괜찮았나?	고정 메뉴는 수집/호출 횟수가 제한적이라 누적되지 않았음

메뉴 데이터 수집 방식 (menuViewModel)에서 LiveData → StateFlow로 수정
collectFixedMenuData, collectMealData 함수로 관심사 분리
viewModel 주입 방식 수정 (수동 Factory 사용 방식 → Hilt DI 적용 방식)
NetworkModule 과 serviceModule 분리
바텀 네비에서 지도 버튼 관련 코드 임시 주석
MenuViewModelFactory , CalendarViewModel 삭제

Describe your changes

default.mp4

Issue

Resolves [Feat] BottomNavigationBar 작업 #259

To reviewers

feat/set-bottom-navigation 브랜치 pr에서 오류가 나고 있기 때문에 해당 pr을 바로 develop으로 머지하는 게 좋을 것 같아 이전 작업 내용도 해당 pr에 같이 정리했습니다.

근데 원래 로딩 속도가 오래걸려 uistate를 넣었는데 서버 고친 후로 너무 빨라져서 uistate에 마땅히 할 로직이 없는데 어떻게 할까요?

… activityviewmodel 사용을 위한 라이브러리 추가 및 프래그먼트용 토스트 확장 함수 생성.

…으나 메뉴 프래그먼트가 보이지 않음.

* add: bottomSheetDialogFragment * refactor: ListAdapter로 변경 * change: bottomSheet clickListener * chore: 전체 리뷰에서 삭제하면 리스트 재호출 안되는거 수정 * chore: 전체 리뷰에서 삭제하면 리스트 재호출 안되는거 수정 * chore: 왼쪽 정렬, 워딩 수정 "리뷰 설정" * chore: 오탈자 수정 * chore: apply로 바로 받아서 쓰기

…식으로 변경

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

kangyuri1114

이전 pr에서 말씀주신대로 하면 이렇게 되는데
너무 패키지가 많아서 바텀 네비 안에 있는 기능들만 main으로 한번 더 감싸면 좋을 것 같다는 의견인데 생각해보고 말씀 주세요~

HI-JIN2 · 2025-05-14T15:23:31Z

이전 pr에서 말씀주신대로 하면 이렇게 되는데
너무 패키지가 많아서 바텀 네비 안에 있는 기능들만 main으로 한번 더 감싸면 좋을 것 같다는 의견인데 생각해보고 말씀 주세요~

아니면 리뷰 쪽 기능들을 cafeteria 안으로 패키징 해야할 것 같아요.
cafeteria 안에 review랑 Info랑 들어가게용

HI-JIN2 · 2025-05-14T15:24:27Z

conflict 한번 확인해주세용! @kangyuri1114

HI-JIN2

정말 정말 수고 많으셨습니다!! 마치 새차한 기분이에요.. 시원해요.

몇가지 코멘트 달아두었습니다! 확인해보시고 반영할만한거는 반영해주세용!

HI-JIN2 · 2025-05-14T15:25:40Z

app/src/main/java/com/eatssu/android/di/ServiceModule.kt

+
+@Module
+@InstallIn(SingletonComponent::class)
+object ServiceModule {


아아 xxxService만 따로 뺀거군요. 뭔가 하고 한참 봤네요

HI-JIN2 · 2025-05-14T15:26:08Z

app/src/main/java/com/eatssu/android/presentation/main/MainActivity.kt

+//    override fun onCreateOptionsMenu(menu: Menu?): Boolean {
+//        menuInflater.inflate(R.menu.menu_main, menu)
+//        return true
+//    }
+
+//    override fun onOptionsItemSelected(item: MenuItem): Boolean {
+//        return when (item.itemId) {
+//            R.id.action_setting -> {
+//                val intent = Intent(this, MyPageActivity::class.java)  // 인텐트를 생성해줌,
+//                startActivity(intent)  // 화면 전환을 시켜줌
+//                true
+//            }
+//
+//            else -> super.onOptionsItemSelected(item)
+//        }
+//    }
+
+    // Permission --


이거 안쓰는거면 지워도 될 것 같아용

HI-JIN2 · 2025-05-14T15:26:26Z

app/src/main/java/com/eatssu/android/presentation/main/MainViewModel.kt

+
+            _uiState.update {
+                it.copy(
+                    toastMessage = "로그아웃 되었습니다.",


요거 R.string으로 바꿔주셍용

HI-JIN2 · 2025-05-14T15:27:16Z

app/src/main/java/com/eatssu/android/presentation/main/MainViewModel.kt

+    private val data = MutableLiveData<LocalDate>()
+
+    fun setData(dataToSend: LocalDate) {
+        data.value = dataToSend
+
+        Log.d("setdata", dataToSend.toString())
+    }
+
+    fun getData(): LiveData<LocalDate> {
+        return data
    }
+
 }


이게 아마 오늘 날짜 받아오는것 같은데.. mainviewmodel에서 하긴 좀 애매해보이네요..

허허.. 저도 이 날짜 받아오는 부분은 리팩하고싶은데.. 다음 pr로 넘겨보겠슴다...

HI-JIN2 · 2025-05-14T15:28:20Z

app/src/main/java/com/eatssu/android/presentation/main/menu/MenuFragment.kt

+    private val mainViewModel by activityViewModels<MainViewModel>()
+    private val infoViewModel: InfoViewModel by activityViewModels()
+    private val menuViewModel by viewModels<MenuViewModel>()


요거 형식 통일하면 좋을 것 같아용~

HI-JIN2 · 2025-05-14T15:29:31Z

app/src/main/java/com/eatssu/android/presentation/main/menu/MenuFragment.kt

+                menuViewModel.uiState.collect { state ->
+                    when (state) {
+                        is UiState.Init -> {
+                            Log.d("Debug", "UI State: Init")


로그는 Timber로 바꿔주세용~ 의미 없는 로그면 지워주시고용

HI-JIN2 · 2025-05-14T15:30:07Z

app/src/main/java/com/eatssu/android/presentation/main/menu/MenuViewModel.kt


                        if (response.isSuccessful) {
-                            Log.d("post", "onResponse 성공" + response.body())
+                            Timber.tag("post").d("onResponse 성공" + response.body())


이거 제가 작성한 코드인것 같긴한데, Timber면은 tag 지워주세욥!

HI-JIN2 · 2025-05-14T15:32:01Z

app/src/main/java/com/eatssu/android/presentation/mypage/MyPageViewModel.kt

-    fun loginOut() {
-        viewModelScope.launch {
-            logoutUseCase() //Todo 반환값이 쓰이는게 아니면 이렇게 해도 되나?
-
-            _uiState.update {
-                it.copy(
-                    toastMessage = "로그아웃 되었습니다.",
-                    isLoginOuted = true
-                )
-            }
-        }
-    }
-
-    fun signOut() {
-        viewModelScope.launch {
-            signOutUseCase().onStart {
-                _uiState.update { it.copy(loading = true) }
-            }.onCompletion {
-                _uiState.update { it.copy(loading = false, error = true) }
-            }.catch { e ->
-                _uiState.update { it.copy(error = true, toastMessage = "정보를 불러올 수 없습니다.") }
-                Timber.e(e.toString())
-            }.collectLatest { result ->
-                Timber.d(result.toString())
-                if (result.result == true) {
-                    logoutUseCase()
-                    _uiState.update {
-                        it.copy(
-                            isSignOuted = true,
-                            toastMessage = "탈퇴가 완료되었습니다."
-                        )
-                    }
-                }
-            }
-        }
-    }
-


어??? 로그인이랑 로그아웃 코드는 어디갔어요??

회원탈퇴는 회원탈퇴뷰모델이 따로있어 여기 회원탈퇴 함수를 사용하지 않아 지웠습니다

HI-JIN2 · 2025-05-14T15:34:08Z

app/src/main/java/com/eatssu/android/presentation/main/MainViewModel.kt

+    fun logOut() {
+        viewModelScope.launch {
+            logoutUseCase() //Todo 반환값이 쓰이는게 아니면 이렇게 해도 되나?
+
+            _uiState.update {
+                it.copy(
+                    toastMessage = "로그아웃 되었습니다.",
+                    isLoggedOut = true
+                )
+            }
+        }
+    }


로그아웃이 왜 여기있죠?!

아 로그아웃 하고 인텐트로 화면 이동하려면 메인 액티비티에서 로그아웃 함수 실행해야 해서 여기로 옮겼어욤
근데 로그아웃하고 로그인화면으로 인텐트하는거 여기서 까먹고 안 넣어서 추가해서 다시 푸시해놨어욤

kangyuri1114 · 2025-05-18T14:55:58Z

탑바에서 마이페이지 이미지 삭제 안했던거 같은데 그거 꼭 반영해야 합니다.. 제가 까먹을까봐 남겨두요

HI-JIN2

main.menu랑 cafeteria를 합치는게 어떠할까요?
결국 cafeteria가 menu에 속하는 개념인 거 같아서요!
모듈을 나눈다면 리뷰 관련이랑 지도 관련으로 크게 나눌 것 같아, 크게 피쳐 별로 잡아주시면 좋을 것 같습니다!! 그리고 멀티모듈 도입도 논의해보고 싶어요..!!

그리고 map도 main 하위로 가야할 것 같아요!

상단에 점 3개 꼭 지우고!!! 병합해주세요! 그리고 지효님한테 바텀네비 이렇게 해서 배포할거라고 확인? 받으면 좋을 것 같습니다!!

kangyuri1114 · 2025-05-21T04:14:52Z

main.menu 빈 패키지였어서 요렇게 바꿨습니당

HI-JIN2

굿굿입니다!

HI-JIN2 · 2025-05-23T08:04:22Z

app/src/main/java/com/eatssu/android/presentation/MainActivity.kt

+    // 아래 함수를 View 에서는 사용이 어려워 util 로 빼지 않음
+    private fun BottomNavigationView.setOnSingleItemSelectedListener(
+        minInterval: Long = 2000L,
+        onSingleItemSelected: (item: MenuItem) -> Boolean
+    ) {
+        var lastClickTime = 0L
+
+        setOnItemSelectedListener { item ->
+            val currentClickTime = SystemClock.uptimeMillis()
+            if (currentClickTime - lastClickTime > minInterval) {
+                lastClickTime = currentClickTime
+                onSingleItemSelected(item)
+            } else {
+                false // 너무 빠른 클릭 무시
+            }
+        }
+    }
+


오호 이거 baseActivity에 할 줄 알았는데, mainActivity에 하셨군요!👍

근데 이제 compose로 짜면 base activity도 의미 없을 것 같아요 그죠?

kangyuri1114 and others added 30 commits March 18, 2025 17:26

[feat] 바텀 네비 세팅 중 임시 커밋

63fba88

[feat] 바텀 네비 UI 세팅

4a5f5c0

[feat] 바텀 네비 constraint 수정

a779c8e

[feat] 바텀 네비 아이템 클릭 리스너 함수 추가

18b23e2

Merge branch 'develop' into feat/set-bottom-navigation

334cac4

[feat] 현 메인 액티비티 명을 cefeteriaActivity로 변경

0dc0340

[feat] 현 메인 액티비티 명을 cefeteriaActivity로 변경

1a7ac30

[feat] Mypage, Cafeteria Fragment 로 전환 후 MainActivity에 있어야 하는 로직 옮기기.…

cbaa8aa

… activityviewmodel 사용을 위한 라이브러리 추가 및 프래그먼트용 토스트 확장 함수 생성.

[chore] 주석 추가 및 불필요 코드 삭제

30d644e

[chore] Mypage, Cafeteria Fragment를 MainActivity ViewPager 로 넣기는 성공 했…

a494fe9

…으나 메뉴 프래그먼트가 보이지 않음.

[feat] CalendarViewModel 로직을 MainViewModel로 옮김

f514999

[chore] 불필요한 코드 삭제

12a89d5

[fix] 이전, 다음 주차 이동 버튼 클릭 시 앱터짐 이슈 해결 (데이터바인딩을 뷰바인딩으로)

f803c08

[chore] 불필요한 코드 삭제

7a5894c

[feat] 마이페이지 toolbar 추가

8bbea17

[feat] map 프래그먼트 컴포즈 뷰 띄우기 성공 및 임시로 주석처리 진행

6d534b9

Merge branch 'develop' into feat/set-bottom-navigation

1ae1bd1

release 2.1.7 (#278)

bda5973

feat: splash 기존 이미지로 변경 및 스플래쉬 실행 시간 2초로 복구 (#281)

39c8996

feat: splash 배경 색 변경 (#282)

c5879dd

Merge branch 'develop' into feat/set-bottom-navigation

a1c32f7

feat: BottomNavigation 에서 viewPager로 화면 전환하던 방식을 jetpack navigation 방…

7f69207

…식으로 변경

Merge branch 'develop' into feat/set-bottom-navigation

af06c6b

Merge branch 'develop' into feat/set-bottom-navigation

15f58b9

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

feat: menu 정보 liveData -> flow 수정

c491207

feat: uistate 세팅

3eda238

feat: 로그 삭제

ef8fd0f

feat: 디버깅 로그 추가

8706b6b

Merge branch 'develop' into feat/fix-bottom-navi-error

eac5ef9

kangyuri1114 self-assigned this May 12, 2025

kangyuri1114 added this to 2025년 1학기 May 12, 2025

kangyuri1114 added ✨ feature 기능 추가 ⭐️ refactor 코드를 개선시킵니다. labels May 12, 2025

kangyuri1114 changed the base branch from feat/set-bottom-navigation to develop May 12, 2025 08:28

jayn2u moved this to Todo in 2025년 1학기 May 12, 2025

kangyuri1114 changed the title ~~Feat/fix bottom navi error~~ [Feat] 바텀 네비게이션 도입 및 기존 캘린더 및 마이페이지 화면 fragment 로 전환 May 12, 2025

kangyuri1114 commented May 13, 2025

View reviewed changes

HI-JIN2 requested changes May 14, 2025

View reviewed changes

HI-JIN2 reviewed May 14, 2025

View reviewed changes

kangyuri1114 added 6 commits May 16, 2025 09:37

[feat] repackaging

e50e7e9

[feat] merge develop

b8f7040

[feat] 불필요 코드 삭제

ae46236

[feat] string 추출

4018cb6

[feat] viewmodel 선언 형식 통일

9d4f32f

[feat] 로그아웃 후 로그인 화면으로 이동 코드 추가 및 log -> timber

4bed0da

HI-JIN2 approved these changes May 19, 2025

View reviewed changes

kangyuri1114 added 2 commits May 21, 2025 13:09

[feat] 마이페이지 이미지 버튼 삭제

c17656f

[feat] 불필요 이미지 삭제

6b12449

kangyuri1114 added 2 commits May 21, 2025 13:17

[feat] 패키징 정리, 불필요 파일 삭제

d8170e9

[feat] 연타 방지용 setOnSingleItemSelectedListener 추가

fab649e

HI-JIN2 approved these changes May 23, 2025

View reviewed changes

kangyuri1114 merged commit d487550 into develop May 26, 2025
1 check passed

kangyuri1114 deleted the feat/fix-bottom-navi-error branch May 26, 2025 08:52

github-project-automation bot moved this from Todo to Done in 2025년 1학기 May 26, 2025

HI-JIN2 mentioned this pull request May 31, 2025

[Fix] 날짜를 바뀌어도 이전에 선택했던 시간대로 보여지도록 수정 (iOS와 통일) #299

Closed

[Feat] 바텀 네비게이션 도입 및 기존 캘린더 및 마이페이지 화면 fragment 로 전환 #293

[Feat] 바텀 네비게이션 도입 및 기존 캘린더 및 마이페이지 화면 fragment 로 전환 #293

Uh oh!

Conversation

kangyuri1114 commented May 12, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

develop ~feat/set-bottom-navigation 브랜치까지의 작업 내용

feat/set-bottom-navigation 에서 중복 메뉴 호출 및 이전 메뉴 화면 누적되어 반환하던 오류 수정을 위한 브랜치인 feat/fix-bottom-navi-error 작업내용

Describe your changes

Issue

To reviewers

Uh oh!

kangyuri1114 left a comment

Choose a reason for hiding this comment

Uh oh!

HI-JIN2 commented May 14, 2025

Uh oh!

HI-JIN2 commented May 14, 2025

Uh oh!

HI-JIN2 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!

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!

kangyuri1114 commented May 18, 2025

Uh oh!

HI-JIN2 left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

상단에 점 3개 꼭 지우고!!! 병합해주세요! 그리고 지효님한테 바텀네비 이렇게 해서 배포할거라고 확인? 받으면 좋을 것 같습니다!!

Uh oh!

kangyuri1114 commented May 21, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

HI-JIN2 left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

kangyuri1114 commented May 12, 2025 •

edited

Loading

`develop` ~`feat/set-bottom-navigation` 브랜치까지의 작업 내용

`feat/set-bottom-navigation` 에서 중복 메뉴 호출 및 이전 메뉴 화면 누적되어 반환하던 오류 수정을 위한 브랜치인 feat/fix-bottom-navi-error 작업내용

HI-JIN2 left a comment •

edited

Loading

kangyuri1114 commented May 21, 2025 •

edited

Loading