[Fix/#7] fix project setting #18

hyoeunjoo · 2025-01-14T11:52:20Z

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕)
리뷰가 필요한 경우 리뷰어를 지정해 주세요
리뷰는 (아직 미정)에 진행됩니다.
P1 단계의 리뷰는 (아직 미정)까지 반영합니다.
Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다.

📌 𝗜𝘀𝘀𝘂𝗲𝘀

closed [Fix] 프로젝트 기초세팅 수정 #7

📎𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻

바텀네비게이션 + 네비게이션 로직 재구현했습니다.

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

bottom.mp4

💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀

메둥이들아~~!!! 조금만 힘내자!! 다들 넘 잘하고 있으니까!
이거 구조는 이따가 한 번 쫙 설명해주께!!!
다들 메랑해~💗

imtaejugkim

네비게이션 정말 고생 많았습니다!! 인자 넘겨주는 코드도 한번 설명해주시면 감사하드립니다..ㅎ

imtaejugkim · 2025-01-14T15:09:06Z

app/src/main/java/org/memento/presentation/onboarding/navigation/OnboardingNavigation.kt

+fun NavController.navigationOnboarding2(navOptions: NavOptions? = null) {
+    navigate("OnboardingScreen2", navOptions)
+}
+
+fun NavGraphBuilder.onboardingNavGraph(
+    navigateToOnboardingScreen2: () -> Unit,
+    navigateToMainScreen: () -> Unit,
+) {
+    composable("OnboardingScreen1") {
+        OnboardingScreen1(navigateToOnboardingScreen2 = navigateToOnboardingScreen2)
+    }
+    composable("OnboardingScreen2") {
+        OnboardingScreen2(navigateToMainScreen = navigateToMainScreen)
+    }
+}


p3 : 설명하신 것 완벽히 이해했습니다!!

imtaejugkim · 2025-01-14T15:17:58Z

app/src/main/java/org/memento/presentation/todo/navigation/TodoNavigation.kt

+fun NavController.navigationTodo(navOptions: NavOptions) {
+    navigate(
+        route = MainNavigationBarRoute.Todo::class.simpleName.orEmpty(),
+        navOptions = navOptions,
+    )
+}
+


p3 : simplename이 클래스의 동적 이름을 가져오는 방법이군요!

hwidung

너무 너무 고생 많았어요 언니 최고야 진짜!!! 🖤🖤

hwidung · 2025-01-14T15:33:05Z

app/src/main/java/org/memento/presentation/navigator/MainNavigator.kt

p3 : 들어준 온보딩 예시를 잘 활용해 볼게요!! 👍

hwidung · 2025-01-14T15:34:03Z

app/src/main/java/org/memento/presentation/navigator/MainNavigator.kt

+    private val currentDestination: NavDestination?
+        @Composable get() = navHostController.currentBackStackEntryAsState().value?.destination
+
+    val startDestination = "OnboardingScreen1"


p3 : 상수화 필요한지 궁금해요 !

네! 제가 그냥 지금 막 짜느냐고 이렇게 됐는데 사실은 하면 좋죠..
제 생각에 루트는 모두 상수화시키는게 좋은 것 같습니다!

…ID-MEMENTO into fix/#7-fix-project-setting

jihyunniiii

고생하셨습니다!
리뷰 읽어보시고 질문 있으면 언제든 해주셔요!

jihyunniiii · 2025-01-15T13:04:26Z

app/src/main/java/org/memento/presentation/main/MainScreen.kt

+                navigationBarItems = BottomNavigationType.entries,
+                currentNavigationBarItem = navigator.currentMainNavigationBarItem,
+                onNavigationBarItemSelected = { navigator.navigateMainNavigation(it) },
+                onAddButtonClick = { navigator.navigateToBottomSheet() },


바텀 시트도 navigator로 관리한 이유가 궁금해요!
저는 navigator가 앱 내의 서로 다른 화면 간의 이동에 대한 로직을 책임진다고 생각해요.
바텀 시트가 나타나는 것은 서로 다른 화면 간의 이동이 아닌 한 화면 내에서 추가적인 정보를 얻기 위해 UI 요소를 나타나게 하는 것이기 때문에 navigator로 관리하는 것보다는 각 화면의 상태를 변경 시키는 방식으로 구현하는 것이 어떨까 싶네요!

jihyunniiii · 2025-01-15T13:07:03Z

app/src/main/java/org/memento/presentation/navigator/MainNavigator.kt

+    fun navigateToOnboarding2() {
+        navHostController.navigationOnboarding2()
+    }
+


stack 관리를 위해 popBackStack 함수들을 미리 추가해두어도 좋을 것 같아요 !

jihyunniiii · 2025-01-15T13:09:31Z

app/src/main/java/org/memento/presentation/navigator/component/BottomNavigationBar.kt

+
+@Composable
+fun CustomNavigationBarItem(
+    context: Context,


사용되지 않는 인자같아요!
확인 후 지워주시면 좋을 것 같네요 ~

jihyunniiii · 2025-01-15T13:10:06Z

app/src/main/java/org/memento/presentation/navigator/component/BottomNavigationBar.kt

+        bottomNavigationType.label?.let {
+        }


요거 없어도 될 것 같은데!
마찬가지로 확인 후 수정하면 좋을 것 같아요 ~

jihyunniiii · 2025-01-15T13:16:35Z

app/src/main/java/org/memento/presentation/navigator/component/BottomNavigationBar.kt

+            CustomNavigationBarItem(
+                context = context,
+                bottomNavigationType = navigationBarItems[0],
+                isSelected = currentNavigationBarItem == navigationBarItems[0],
+                onClick = { onNavigationBarItemSelected(navigationBarItems[0]) },
+                modifier = Modifier.weight(1f),
+            )
+
+            Box(
+                modifier =
+                    Modifier
+                        .size(48.dp)
+                        .background(Color.LightGray, shape = RoundedCornerShape(24.dp))
+                        .clickable { onAddButtonClick() },
+                contentAlignment = Alignment.Center,
+            ) {
+                Icon(
+                    painter = painterResource(id = R.drawable.add_white_24),
+                    contentDescription = "Add Button",
+                    tint = Color.White,
+                )
+            }
+
+            CustomNavigationBarItem(
+                context = context,


이 부분 조금 더 확장성있게 관리하는 방법을 고민해봐도 좋을 것 같아요!
지금 생각나는 방법은,,,

중간 + 버튼까지 navigationBarItems에 넣기

navigationBarItems enum 내부에 분기처리를 할 수 있는 값 넣기

추가 한 값으로 분기처리 하여 다른 UI 및 로직 실행하기

고민해보시고 추후에라도 리팩토링 해보면 효율적인 코드가 될 것 같습니다! (추후 바텀 네비의 요소가 늘어나는 경우도 정말 많기에 변화에 유연하게 대처할 수 있도록 !!)

jihyunniiii · 2025-01-15T13:18:31Z

app/src/main/java/org/memento/presentation/navigator/component/MainNavHost.kt

+    modifier: Modifier = Modifier,
+    navigator: MainNavigator,
+    padding: PaddingValues,


이거 순서 맞춰줍시다!!
필수 인자 > Modifier > 선택 인자 순서 !!

jihyunniiii · 2025-01-15T13:19:40Z

app/src/main/java/org/memento/presentation/navigator/component/MainNavHost.kt

+            )
+
+            todayNavGraph(
+                navigateToReqres = { navigator.navigateToReqres() },


:: (더블콜론)을 통해서 작성할 수도 있어요!

jihyunniiii · 2025-01-15T13:21:52Z

app/src/main/java/org/memento/presentation/navigator/route/MainNavigationBarRoute.kt

나중에 세미나 때 다뤘던 것처럼 TypeSafety한 방법을 사용해 보아도 좋을 것 같아요.

jihyunniiii · 2025-01-15T13:22:55Z

app/src/main/java/org/memento/presentation/onboarding/OnboardingScreen1.kt

+import androidx.compose.ui.graphics.Color
+
+@Composable
+fun OnboardingScreen1(navigateToOnboardingScreen2: () -> Unit) {


OnboardingScreen을 2개를 만드신 이유가 있나요?

UI를 보지 않아서 잘 모르겠지만 pager 등으로 이동하는 것이 필요한 화면이라면 하나의 Screen을 통해서 구현하셔도 충분할 것 같아요!

jihyunniiii · 2025-01-15T13:28:05Z

app/src/main/java/org/memento/presentation/todo/navigation/TodoNavigation.kt

+    composable(TodoRoute.TODO) {
+        TodoScreen()
+    }
+}
+
+object TodoRoute {
+    const val TODO = "Todo"
+}


이 화면 잘 이동 하나요??
navigate()의 route 값과 composable()의 route 값이 동일해야 오류 없이 정상적으로 navigation 됩니다!

hyoeunjoo added 6 commits January 14, 2025 20:42

[del/#17] 불필요 파일 삭제

ceda773

[del/#17] 불필요 파일 삭제

9f2a57c

[fix/#17] BottomNavigation 재구현

17fe208

[fix/#17] MainNavigation 로직 재구현

87459e2

[feat/#17] 각 화면 네비게이션 구현

8d5f556

[mod/#17] ui인자 변경

71f4d77

hyoeunjoo added the 🔧 [FIX] 버그 및 오류 해결 label Jan 14, 2025

hyoeunjoo requested review from imtaejugkim and hwidung January 14, 2025 11:52

hyoeunjoo self-assigned this Jan 14, 2025

imtaejugkim approved these changes Jan 14, 2025

View reviewed changes

hwidung approved these changes Jan 14, 2025

View reviewed changes

hyoeunjoo added 2 commits January 15, 2025 01:57

[mod/#7] 루트 상수화

a8b6baa

Merge branch 'develop' of https://github.com/SOPT-all/35-APPJAM-ANDRO…

4eb2689

…ID-MEMENTO into fix/#7-fix-project-setting

hyoeunjoo merged commit 50cdae8 into develop Jan 14, 2025
1 check passed

hyoeunjoo deleted the fix/#7-fix-project-setting branch January 14, 2025 17:07

jihyunniiii reviewed Jan 15, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Fix/#7] fix project setting #18

[Fix/#7] fix project setting #18

hyoeunjoo commented Jan 14, 2025

imtaejugkim left a comment

imtaejugkim Jan 14, 2025

imtaejugkim Jan 14, 2025

hwidung left a comment

hwidung Jan 14, 2025

hwidung Jan 14, 2025

hyoeunjoo Jan 14, 2025

jihyunniiii left a comment

jihyunniiii Jan 15, 2025

jihyunniiii Jan 15, 2025

jihyunniiii Jan 15, 2025

jihyunniiii Jan 15, 2025

jihyunniiii Jan 15, 2025

jihyunniiii Jan 15, 2025

jihyunniiii Jan 15, 2025

jihyunniiii Jan 15, 2025

jihyunniiii Jan 15, 2025

jihyunniiii Jan 15, 2025

jihyunniiii Jan 15, 2025

[Fix/#7] fix project setting #18

[Fix/#7] fix project setting #18

Conversation

hyoeunjoo commented Jan 14, 2025

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

📌 𝗜𝘀𝘀𝘂𝗲𝘀

📎𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀

imtaejugkim left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

hwidung 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

jihyunniiii 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