🔀 :: (#416) - update expocreatescreen location input #422

answad · 2025-02-09T16:16:51Z

💡 개요

2025-02-09.235223.mp4

2025-02-10.011859.mp4

주소검색을 expoScreen이 아닌 AddressSearchScreen으로 이동하여 수행하도록 변경했습니다

📃 작업내용

ExpoCreateRoute에서 주소검색 및 변환 기능 삭제
ExpoAddressSearchViewModel 생성 및 주소검색, 좌표 변환 관련 기능 추가
ExpoAddressSearchScreen 및 ExpoAddressSearchRoute 관련 UI 구현 및 연결
ExpoCreateRoute 및 ExpoModifyScreen에서 ExpoAddressSearchRoute로 이동하도록 수정
위치 정보와 좌표 상태를 관리하기 위한 로직 개선
viewModel을 AddressSearchScreen, ExpoCreateScreen, ExpoModifyScreen에서 공유하도록 수정, 공유로 인해 발생하는 오류 해결

🔀 변경사항

ExpoCreateRoute에서 주소검색 및 변환 기능 삭제
ExpoAddressSearchRoute에서 관련 UI 및 Navigation 처리 추가
텍스트 필드에 위에 장소 텍스트를 추가하고, ExpoAddressSearchScreen UI 개선
ExpoCreateRoute 및 ExpoModifyScreen에서 주소 검색 화면으로의 네비게이션 연결

🙋‍♂️ 질문사항

개선할 점, 오타, 코드에 이상한 부분이 있다면 Comment 달아주세요.

🍴 사용방법

ExpoAddressSearchRoute를 호출하여 주소 검색 기능을 사용할 수 있습니다.
ExpoAddressSearchScreen에서 위치를 입력하고 자동으로 좌표를 변환할 수 있습니다.
ExpoCreateRoute와 ExpoModifyScreen에서 주소 검색 기능을 사용할 수 있습니다.

Summary by CodeRabbit

신규 기능
- Expo 앱 내 주소 검색 화면이 추가되어, 주소 입력 시 주소 검색 기능으로 원활하게 이동할 수 있습니다.
- 검색 아이콘이 포함된 새로운 입력 필드가 도입되어, 검색 인터페이스가 한층 강화되었습니다.
개선 사항
- 내비게이션 흐름이 확장되어 화면 전환 및 오류 알림 등 사용자 피드백이 개선되었습니다.
- 주소와 좌표 관리 로직이 향상되어 전반적인 주소 입력 경험이 개선되었습니다.

…arch 로 변경

…수 추가

…sUiState 를 초기화 하도록 기능 추가

…인스턴스를 공유하도록 수정

…록 수정

…가, 핸들링함수 추가

…블럭 추가

coderabbitai · 2025-02-09T16:16:57Z

Walkthrough

이번 변경 사항은 앱 내 네비게이션 기능을 확장하기 위한 수정으로, 주소 검색 화면을 추가하고 해당 기능으로 이동할 수 있는 네비게이션 메서드를 도입합니다. ExpoNavHost와 ExpoNavigation 파일에서 기존 expoModifyScreen과 expoCreateScreen 함수에 새로운 네비게이션 파라미터가 추가되었으며, design-system에서는 검색 아이콘을 포함한 새로운 텍스트 필드가 도입되었습니다. 또한, feature/expo의 ExpoAddressSearchScreen, ExpoCreateScreen, ExpoModifyScreen 및 ExpoViewModel에 주소 검색 관련 로직과 상태 관리가 추가되었습니다.

Changes

파일(들)	변경 요약
app/.../ExpoNavHost.kt, feature/expo/.../ExpoNavigation.kt	네비게이션 기능 확장을 위해 `expoAddressSearchScreen` 추가, `navigateToExpoAddressSearch` 네비게이션 함수 도입 및 기존 `expoModifyScreen`, `expoCreateScreen`에 해당 파라미터 추가
core/.../ExpoTextField.kt	`ExpoSearchIconTextField` 컴포저블 함수 추가(검색 아이콘 사용), 관련 `SearchIcon` 임포트 추가
feature/expo/.../ExpoAddressSearchScreen.kt	주소 검색 기능을 위한 `ExpoAddressSearchRoute` 및 `ExpoAddressSearchScreen` 컴포저블 함수 추가, UI 및 상태 처리 로직 구현
feature/expo/.../ExpoCreateScreen.kt, feature/expo/.../ExpoModifyScreen.kt	주소 검색 네비게이션 기능 도입으로 컴포저블 함수 파라미터(`navigateToExpoAddressSearch`) 업데이트, 주소 관련 UI 요소 및 상태 제거 또는 수정
feature/expo/.../ExpoViewModel.kt	주소 검색을 위한 상수(`SEARCHED_COORDINATEX`, `SEARCHED_COORDINATEY`, `SEARCHED_LOCATION`)와 상태 변수(`searched_location`, `searched_coordinateX`, `searched_coordinateY`) 추가, 관련 로직 업데이트

Possibly related PRs

🔀 :: (#199) - 박람회 -> 박람회 상세 -> 수정하기 화면을 퍼블리싱하였습니다. #201: ExpoNavHost.kt 파일의 네비게이션 기능 강화와 관련된 변경 사항입니다.
🔀 :: (#366) - expo creation map communication implementation #400: expoModifyScreen 및 expoCreateScreen 함수에 navigateToExpoAddressSearch 파라미터 추가와 관련된 변경 사항입니다.
🔀 :: (#218) - 디자인 변동사항을 따릅니다. #219: expoModifyScreen 및 expoCreateScreen 함수의 업데이트와 관련된 변경 사항입니다.

Suggested labels

1️⃣ Priority: High, ✨ Feature, ♻️ Refactor

Suggested reviewers

diejdkll
Chaejongin12
kimjeakwan
audgns10

Poem

나는 귀여운 토끼, 코드의 숲을 달리며
새로운 주소 검색 길목을 찾았네,
화면 속에서 길을 잃어버려도
네비게이션의 빛이 길을 비추네.
코드와 함께 뛰노는 우리의 발자취,
🐇 오늘도 즐겁게 달려가리라!
작은 변화에 큰 행복을 담아!

📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e8b6a9f and a09ff41.

📒 Files selected for processing (2)

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoCreateScreen.kt (10 hunks)
feature/expo/src/main/java/com/school_of_company/expo/view/ExpoModifyScreen.kt (7 hunks)

🚧 Files skipped from review as they are similar to previous changes (1)

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoCreateScreen.kt

⏰ Context from checks skipped due to timeout of 90000ms (1)

GitHub Check: build

🔇 Additional comments (4)

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoModifyScreen.kt (4)

90-92: ViewModel 인스턴스화 방식이 개선되었습니다.

ComponentActivity를 사용하여 ViewModel을 인스턴스화하는 방식으로 변경된 것이 좋습니다. 이는 메모리 누수를 방지하고 수명 주기를 더 잘 관리할 수 있게 해줍니다.

128-133: 상태 초기화 로직이 개선되었습니다.

DisposableEffect를 사용하여 화면이 dispose될 때 상태를 초기화하는 것이 좋습니다. 이는 메모리 누수를 방지하고 다음 사용을 위해 깨끗한 상태를 보장합니다.

535-541: 주소 검색 UX가 개선되었습니다.

텍스트 필드를 비활성화하고 주소 검색 화면으로 이동하는 방식으로 변경된 것이 좋습니다. 이는 사용자가 주소를 더 정확하게 입력할 수 있도록 도와줍니다.

631-664: Preview 함수가 업데이트되었습니다.

Preview 함수에 새로운 navigateToExpoAddressSearch 파라미터가 추가되어 일관성이 유지되었습니다.

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

Review comments: Directly reply to a review comment made by CodeRabbit. Example:
- I pushed a fix in commit <commit_id>, please review it.
- Generate unit testing code for this file.
- Open a follow-up GitHub issue for this discussion.
Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
- @coderabbitai generate unit testing code for this file.
- @coderabbitai modularize this function.
PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
- @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
- @coderabbitai read src/utils.ts and generate unit testing code.
- @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
- @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai generate docstrings to generate docstrings for this PR. (Beta)
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (`.coderabbit.yaml`)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

audgns10

pr 컨벤션에 맞춰서 다시 작성해주세여

coderabbitai

Actionable comments posted: 2

🧹 Nitpick comments (2)

core/design-system/src/main/java/com/school_of_company/design_system/component/textfield/ExpoTextField.kt (1)

444-498: 코드 중복을 제거하기 위한 리팩토링 제안

구현은 정확하지만, ExpoLocationIconTextField와 거의 동일한 코드가 중복되어 있습니다. 아이콘만 다른 두 컴포넌트를 하나로 통합하는 것이 좋겠습니다.

다음과 같이 리팩토링하는 것을 제안합니다:

-@Composable
-fun ExpoLocationIconTextField(
-    modifier: Modifier = Modifier,
-    value: String,
-    onValueChange: (String) -> Unit,
-    placeholder: String,
-    isDisabled: Boolean,
-    onButtonClicked: () -> Unit,
-) {
-    ExpoAndroidTheme { colors, typography ->
-        Box {
-            OutlinedTextField(
-                // ... existing implementation
-                trailingIcon = {
-                    IconButton(onClick = onButtonClicked) {
-                        LocationIcon(
-                            tint = colors.gray500,
-                            modifier = Modifier.size(24.dp)
-                        )
-                    }
-                }
-            )
-        }
-    }
-}
-
-@Composable
-fun ExpoSearchIconTextField(
-    modifier: Modifier = Modifier,
-    value: String,
-    onValueChange: (String) -> Unit,
-    placeholder: String,
-    isDisabled: Boolean,
-    onButtonClicked: () -> Unit,
-) {
-    ExpoAndroidTheme { colors, typography ->
-        Box {
-            OutlinedTextField(
-                // ... existing implementation
-                trailingIcon = {
-                    IconButton(onClick = onButtonClicked) {
-                        SearchIcon(
-                            tint = colors.gray500,
-                            modifier = Modifier.size(24.dp)
-                        )
-                    }
-                }
-            )
-        }
-    }
-}

+enum class ExpoIconType {
+    LOCATION,
+    SEARCH
+}
+
+@Composable
+fun ExpoIconTextField(
+    modifier: Modifier = Modifier,
+    value: String,
+    onValueChange: (String) -> Unit,
+    placeholder: String,
+    isDisabled: Boolean,
+    onButtonClicked: () -> Unit,
+    iconType: ExpoIconType
+) {
+    ExpoAndroidTheme { colors, typography ->
+        Box {
+            OutlinedTextField(
+                // ... existing implementation
+                trailingIcon = {
+                    IconButton(onClick = onButtonClicked) {
+                        when (iconType) {
+                            ExpoIconType.LOCATION -> LocationIcon(
+                                tint = colors.gray500,
+                                modifier = Modifier.size(24.dp)
+                            )
+                            ExpoIconType.SEARCH -> SearchIcon(
+                                tint = colors.gray500,
+                                modifier = Modifier.size(24.dp)
+                            )
+                        }
+                    }
+                }
+            )
+        }
+    }
+}
+
+@Composable
+fun ExpoLocationIconTextField(
+    modifier: Modifier = Modifier,
+    value: String,
+    onValueChange: (String) -> Unit,
+    placeholder: String,
+    isDisabled: Boolean,
+    onButtonClicked: () -> Unit,
+) {
+    ExpoIconTextField(
+        modifier = modifier,
+        value = value,
+        onValueChange = onValueChange,
+        placeholder = placeholder,
+        isDisabled = isDisabled,
+        onButtonClicked = onButtonClicked,
+        iconType = ExpoIconType.LOCATION
+    )
+}
+
+@Composable
+fun ExpoSearchIconTextField(
+    modifier: Modifier = Modifier,
+    value: String,
+    onValueChange: (String) -> Unit,
+    placeholder: String,
+    isDisabled: Boolean,
+    onButtonClicked: () -> Unit,
+) {
+    ExpoIconTextField(
+        modifier = modifier,
+        value = value,
+        onValueChange = onValueChange,
+        placeholder = placeholder,
+        isDisabled = isDisabled,
+        onButtonClicked = onButtonClicked,
+        iconType = ExpoIconType.SEARCH
+    )
+}

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoCreateScreen.kt (1)

513-534: 생성 버튼의 활성화 조건이 복잡합니다.

버튼 활성화 조건이 한 줄에 너무 길게 작성되어 있어 가독성이 떨어집니다.

다음과 같이 개선하는 것을 추천드립니다:

-state = if (
-    modifyTitleState.isNotEmpty() &&
-    startedDateState.isNotEmpty() &&
-    endedDateState.isNotEmpty() &&
-    introduceTitleState.isNotEmpty() &&
-    addressState.isNotEmpty() &&
-    locationState.isNotEmpty() &&
-    trainingProgramTextState.isNotEmpty() &&
-    standardProgramTextState.isNotEmpty() &&
-    startedDateState.isValidDateSequence(endedDateState)
-) {
-    ButtonState.Enable
-} else {
-    ButtonState.Disable
-}
+state = run {
+    val isRequiredFieldsFilled = listOf(
+        modifyTitleState,
+        startedDateState,
+        endedDateState,
+        introduceTitleState,
+        addressState,
+        locationState
+    ).all { it.isNotEmpty() }
+
+    val isProgramsNotEmpty = trainingProgramTextState.isNotEmpty() &&
+        standardProgramTextState.isNotEmpty()
+
+    val isDateValid = startedDateState.isValidDateSequence(endedDateState)
+
+    if (isRequiredFieldsFilled && isProgramsNotEmpty && isDateValid) {
+        ButtonState.Enable
+    } else {
+        ButtonState.Disable
+    }
+}

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 7f1b1de and e2d41c3.

📒 Files selected for processing (7)

app/src/main/java/com/school_of_company/expo_android/navigation/ExpoNavHost.kt (2 hunks)
core/design-system/src/main/java/com/school_of_company/design_system/component/textfield/ExpoTextField.kt (2 hunks)
feature/expo/src/main/java/com/school_of_company/expo/navigation/ExpoNavigation.kt (5 hunks)
feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt (1 hunks)
feature/expo/src/main/java/com/school_of_company/expo/view/ExpoCreateScreen.kt (10 hunks)
feature/expo/src/main/java/com/school_of_company/expo/view/ExpoModifyScreen.kt (7 hunks)
feature/expo/src/main/java/com/school_of_company/expo/viewmodel/ExpoViewModel.kt (11 hunks)

⏰ Context from checks skipped due to timeout of 90000ms (1)

GitHub Check: build

🔇 Additional comments (21)

feature/expo/src/main/java/com/school_of_company/expo/navigation/ExpoNavigation.kt (5)

19-19: 주소 검색 라우트 상수가 적절히 정의되었습니다.

라우트 상수의 이름이 명확하고 일관된 네이밍 컨벤션을 따르고 있습니다.

53-55: 주소 검색 화면으로의 네비게이션 함수가 잘 구현되었습니다.

기존 네비게이션 함수들과 일관된 패턴으로 구현되어 있습니다.

89-93: ExpoModifyScreen의 네비게이션 파라미터가 적절히 추가되었습니다.

주소 검색 기능 추가에 따른 네비게이션 파라미터가 잘 통합되어 있습니다.

Also applies to: 97-101

105-108: ExpoCreateScreen의 네비게이션 파라미터가 적절히 추가되었습니다.

주소 검색 기능 추가에 따른 네비게이션 파라미터가 잘 통합되어 있습니다.

Also applies to: 110-113

125-135: 주소 검색 화면 컴포저블이 잘 구현되었습니다.

오류 처리와 백 네비게이션이 적절히 구현되어 있습니다.

app/src/main/java/com/school_of_company/expo_android/navigation/ExpoNavHost.kt (4)

9-9: 주소 검색 관련 import문이 적절히 추가되었습니다.

필요한 import문들이 잘 정리되어 있습니다.

Also applies to: 15-15

144-146: ExpoModifyScreen의 네비게이션 파라미터가 적절히 연결되었습니다.

주소 검색 네비게이션이 navController를 통해 잘 연결되어 있습니다.

149-152: ExpoCreateScreen의 네비게이션 파라미터가 적절히 연결되었습니다.

주소 검색 네비게이션이 navController를 통해 잘 연결되어 있습니다.

158-161: 주소 검색 화면이 네비게이션 그래프에 잘 추가되었습니다.

오류 처리와 백 네비게이션이 적절히 연결되어 있습니다.

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt (3)

41-47: ExpoAddressSearchRoute 컴포저블이 잘 구현되었습니다.

뷰모델과 상태 관리가 적절히 구현되어 있습니다.

56-61: 초기 상태 초기화가 적절히 구현되었습니다.

LaunchedEffect를 사용한 상태 초기화가 잘 되어 있습니다.

63-72: 오류 처리가 체계적으로 구현되었습니다.

좌표 변환과 주소 검색 실패에 대한 오류 처리가 잘 되어 있습니다.

Also applies to: 74-83

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoCreateScreen.kt (2)

120-130: 상태 초기화가 적절히 구현되었습니다.

LaunchedEffect를 사용한 상태 초기화와 동기화가 잘 되어 있습니다.

494-500: 주소 입력 UI가 적절히 구현되었습니다.

사용자 입력을 비활성화하고 주소 검색 화면으로 리다이렉션하는 로직이 잘 구현되어 있습니다.

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoModifyScreen.kt (4)

91-93: 뷰모델 인스턴스화 방식이 개선되었습니다.

ComponentActivity를 사용하여 뷰모델을 인스턴스화하는 것은 좋은 방식입니다. 이는 메모리 누수를 방지하고 수명 주기를 더 잘 관리할 수 있게 해줍니다.

130-134: 리소스 정리 로직이 추가되었습니다.

DisposableEffect를 사용하여 화면이 dispose될 때 Expo 정보를 초기화하는 것은 좋은 방식입니다.

192-200: 검색된 위치 정보 처리 로직이 개선되었습니다.

검색된 위치가 있을 경우 해당 정보를 우선적으로 사용하고, 없을 경우 기존 정보를 사용하는 방식으로 구현되었습니다.

536-542: 위치 입력 필드가 주소 검색 화면과 연동되었습니다.

위치 입력 필드가 비활성화되고 주소 검색 화면으로 이동하는 방식으로 변경되었습니다.

이는 사용자 경험을 개선하고 일관된 주소 데이터 입력을 보장합니다.

feature/expo/src/main/java/com/school_of_company/expo/viewmodel/ExpoViewModel.kt (3)

80-82: 검색된 위치 정보를 위한 상수가 추가되었습니다.

위치 검색 관련 상태를 관리하기 위한 상수들이 적절하게 정의되었습니다.

154-158: 검색된 위치 정보를 위한 상태 변수가 추가되었습니다.

SavedStateHandle을 사용하여 검색된 위치 정보의 상태를 관리하는 것은 구성 변경 시에도 데이터를 유지할 수 있어 좋은 방식입니다.

420-433: 위치 검색 로직이 개선되었습니다.

검색 시작 시 좌표를 초기화하고, 검색 결과가 없을 경우 적절한 예외 처리를 하는 방식으로 구현되었습니다.

coderabbitai · 2025-02-09T16:19:52Z

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt

+                if (
+                    coordinateX.isNotBlank()
+                    && coordinateY.isNotBlank()
+                ) {
+                    HomeKakaoMap(
+                        locationX = coordinateX.toDouble(),
+                        locationY = coordinateY.toDouble(),
+                    )
+                }
+            }


🛠️ Refactor suggestion

지도 표시 로직에 대한 개선이 필요합니다.

좌표값 검증이 문자열 비교로만 되어 있어 잠재적인 문제가 있을 수 있습니다.

다음과 같이 개선하는 것을 추천드립니다:

-if ( - coordinateX.isNotBlank() - && coordinateY.isNotBlank() -) { +if (coordinateX.isNotBlank() && coordinateY.isNotBlank()) { + try { + val x = coordinateX.toDouble() + val y = coordinateY.toDouble() + if (x.isFinite() && y.isFinite()) { HomeKakaoMap( - locationX = coordinateX.toDouble(), - locationY = coordinateY.toDouble(), + locationX = x, + locationY = y, ) + } + } catch (e: NumberFormatException) { + // 좌표 변환 실패 처리 + } }

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

if (

coordinateX.isNotBlank()

&& coordinateY.isNotBlank()

) {

HomeKakaoMap(

locationX = coordinateX.toDouble(),

locationY = coordinateY.toDouble(),

)

}

}

if (coordinateX.isNotBlank() && coordinateY.isNotBlank()) {

try {

val x = coordinateX.toDouble()

val y = coordinateY.toDouble()

if (x.isFinite() && y.isFinite()) {

HomeKakaoMap(

locationX = x,

locationY = y,

)

}

} catch (e: NumberFormatException) {

// 좌표 변환 실패 처리

}

}

}

coderabbitai · 2025-02-09T16:19:52Z

feature/expo/src/main/java/com/school_of_company/expo/viewmodel/ExpoViewModel.kt

+                        onSearchedCoordinateChange(
+                            x = result.data.x.toDoubleOrNull()?.let { "%.6f".format(it) } ?: "0.000000",
+                            y = result.data.y.toDoubleOrNull()?.let { "%.6f".format(it) } ?: "0.000000"
+                        )


🛠️ Refactor suggestion

좌표 정밀도 처리가 개선되었습니다.

좌표값을 소수점 6자리로 제한하는 것은 좋은 방식입니다. 하지만 좌표 변환 실패 시 "0.000000"을 사용하는 것은 위험할 수 있습니다.

다음과 같이 수정하는 것을 추천드립니다:

-x = result.data.x.toDoubleOrNull()?.let { "%.6f".format(it) } ?: "0.000000", -y = result.data.y.toDoubleOrNull()?.let { "%.6f".format(it) } ?: "0.000000" +x = result.data.x.toDoubleOrNull()?.let { "%.6f".format(it) } ?: throw IllegalArgumentException("Invalid coordinate value"), +y = result.data.y.toDoubleOrNull()?.let { "%.6f".format(it) } ?: throw IllegalArgumentException("Invalid coordinate value")

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

onSearchedCoordinateChange(

x = result.data.x.toDoubleOrNull()?.let { "%.6f".format(it) } ?: "0.000000",

y = result.data.y.toDoubleOrNull()?.let { "%.6f".format(it) } ?: "0.000000"

)

onSearchedCoordinateChange(

x = result.data.x.toDoubleOrNull()?.let { "%.6f".format(it) } ?: throw IllegalArgumentException("Invalid coordinate value"),

y = result.data.y.toDoubleOrNull()?.let { "%.6f".format(it) } ?: throw IllegalArgumentException("Invalid coordinate value")

)

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt

audgns10 · 2025-02-09T16:22:31Z

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt

+    LaunchedEffect(getCoordinatesUiState) {
+        when (getCoordinatesUiState) {
+            is GetCoordinatesUiState.Error -> onErrorToast(
+                (getCoordinatesUiState as GetCoordinatesUiState.Error).exception,
+                R.string.get_address_convert_fail
+            )
+
+            else -> Unit
+        }
+    }
+
+    LaunchedEffect(getAddressUiState) {
+        when (getAddressUiState) {
+            is GetAddressUiState.Error -> onErrorToast(
+                (getAddressUiState as GetAddressUiState.Error).exception,
+                R.string.get_address_fail
+            )
+
+            else -> Unit
+        }
+    }


Error 상태일때 말고 다른 Success Loading 상태도 나타내주면 좋을 것 같아여

불러오기 기능은 Screen단에서 두개 한번에 처리를 한 후 각 상태에 맞게 화면을 띄워주는 방법도 있을 것 같아여

♻️ :: getCoordinatesUiState, getAddressUiState 상태에 따라서 success 시에도 토스…
수정했습니다

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoCreateScreen.kt

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt

…트를 띄우도록 수정

coderabbitai

Actionable comments posted: 1

♻️ Duplicate comments (1)

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt (1)

176-185: ⚠️ Potential issue

좌표값 검증 로직 개선 필요

좌표값 변환 시 예외 처리가 누락되어 있어 잠재적인 크래시가 발생할 수 있습니다.

-if (
-    coordinateX.isNotBlank()
-    && coordinateY.isNotBlank()
-) {
+if (coordinateX.isNotBlank() && coordinateY.isNotBlank()) {
+    try {
+        val x = coordinateX.toDouble()
+        val y = coordinateY.toDouble()
+        if (x.isFinite() && y.isFinite()) {
             HomeKakaoMap(
-                locationX = coordinateX.toDouble(),
-                locationY = coordinateY.toDouble(),
+                locationX = x,
+                locationY = y,
             )
+        }
+    } catch (e: NumberFormatException) {
+        // 좌표 변환 실패 처리
+    }
}

🧹 Nitpick comments (2)

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt (2)
56-61: 매직 스트링 상수화 필요

"InitSearched" 문자열을 상수로 분리하여 관리하는 것이 좋습니다.
+private const val INIT_SEARCHED_KEY = "InitSearched"

@Composable
internal fun ExpoAddressSearchRoute(
    // ...
) {
-    LaunchedEffect("InitSearched") {
+    LaunchedEffect(INIT_SEARCHED_KEY) {
        // ...
    }
}
63-87: 로딩 상태 UI 피드백 추가 필요

현재 Loading 상태에서 사용자에게 시각적 피드백이 없습니다. 프로그레스 바나 로딩 인디케이터를 추가하여 사용자 경험을 개선하는 것이 좋습니다.
@Composable
private fun ExpoAddressSearchScreen(
    // ...
+    isLoading: Boolean,
) {
    ExpoAndroidTheme { colors, typography ->
        Column {
+            if (isLoading) {
+                CircularProgressIndicator(
+                    modifier = Modifier.align(Alignment.CenterHorizontally)
+                )
+            }
            // ...
        }
    }
}

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e2d41c3 and 88fa06b.

📒 Files selected for processing (1)

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms (1)

GitHub Check: build

🔇 Additional comments (1)

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt (1)

117-123: 키보드 포커스 해제 처리 필요

사용자가 텍스트 필드나 키보드 영역 외의 화면을 터치할 때 키보드가 자동으로 닫히도록 처리가 필요합니다.
Column(
    modifier = modifier
        .fillMaxSize()
        .background(color = colors.white)
        .padding(16.dp)
+        .pointerInput(Unit) {
+            detectTapGestures(
+                onTap = {
+                    LocalFocusManager.current.clearFocus()
+                }
+            )
+        }
) {

coderabbitai · 2025-02-11T06:08:48Z

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt

+            ExpoStateButton(
+                text = "완료",
+                state = if (
+                    location.isNotEmpty() &&
+                    coordinateX.isNotBlank() &&
+                    coordinateY.isNotBlank()
+                ) {
+                    ButtonState.Enable
+                } else {
+                    ButtonState.Disable
+                },
+                modifier = Modifier
+                    .fillMaxWidth()
+                    .padding(14.dp),
+                onClick = popUpBackStack
+            )


🛠️ Refactor suggestion

버튼 활성화 조건 개선 필요

현재 버튼 활성화 조건이 단순 문자열 검사에만 의존하고 있습니다. 좌표값의 유효성도 함께 검증하는 것이 안전합니다.

+private fun isValidCoordinate(value: String): Boolean { + return try { + val num = value.toDouble() + num.isFinite() + } catch (e: NumberFormatException) { + false + } +} ExpoStateButton( text = "완료", state = if ( location.isNotEmpty() && - coordinateX.isNotBlank() && - coordinateY.isNotBlank() + isValidCoordinate(coordinateX) && + isValidCoordinate(coordinateY) ) { ButtonState.Enable } else { ButtonState.Disable }, // ... )

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change

ExpoStateButton(

text = "완료",

state = if (

location.isNotEmpty() &&

coordinateX.isNotBlank() &&

coordinateY.isNotBlank()

) {

ButtonState.Enable

} else {

ButtonState.Disable

},

modifier = Modifier

.fillMaxWidth()

.padding(14.dp),

onClick = popUpBackStack

)

// Add this helper function at the appropriate scope in the file

private fun isValidCoordinate(value: String): Boolean {

return try {

val num = value.toDouble()

num.isFinite()

} catch (e: NumberFormatException) {

false

}

}

ExpoStateButton(

text = "완료",

state = if (

location.isNotEmpty() &&

isValidCoordinate(coordinateX) &&

isValidCoordinate(coordinateY)

) {

ButtonState.Enable

} else {

ButtonState.Disable

},

modifier = Modifier

.fillMaxWidth()

.padding(14.dp),

onClick = popUpBackStack

)

coderabbitai

Actionable comments posted: 0

🧹 Nitpick comments (1)

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt (1)
60-65: LaunchedEffect 키 값 개선 필요

문자열 "InitSearched"를 키로 사용하는 것은 권장되지 않습니다. 대신 객체 참조나 상수를 사용하는 것이 더 안전합니다.

다음과 같이 수정하는 것을 추천드립니다:
-LaunchedEffect("InitSearched") {
+private object InitSearchedKey
+LaunchedEffect(InitSearchedKey) {
     with(viewModel) {
         onSearchedLocationChange("")
         onSearchedCoordinateChange("", "")
     }
}

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 88fa06b and d035e2d.

📒 Files selected for processing (1)

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms (1)

GitHub Check: build

🔇 Additional comments (3)

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt (3)

188-197: 지도 표시 로직에 대한 개선이 필요합니다.

좌표값 검증이 문자열 비교로만 되어 있어 잠재적인 문제가 있을 수 있습니다.

128-134: 포커스 관리 구현이 잘 되었습니다!

화면 탭 감지를 통한 키보드 포커스 해제가 적절하게 구현되었습니다.

201-216: 버튼 활성화 조건 개선 필요

현재 버튼 활성화 조건이 단순 문자열 검사에만 의존하고 있습니다. 좌표값의 유효성도 함께 검증하는 것이 안전합니다.

…iState 함수 추가

initializeWithSearchedData 추가

coderabbitai

Actionable comments posted: 0

🧹 Nitpick comments (2)

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt (2)

61-95: 상태 관리 로직 개선이 필요합니다.

LaunchedEffect 블록들이 분산되어 있어 코드의 가독성과 유지보수성이 저하될 수 있습니다.

다음과 같이 개선하는 것을 추천드립니다:

-LaunchedEffect(getCoordinatesUiState) {
-    when (getCoordinatesUiState) {
-        is GetCoordinatesUiState.Loading -> Unit
-        is GetCoordinatesUiState.Success -> onErrorToast(
-            null,
-            R.string.get_address_convert_success
-        )
-        is GetCoordinatesUiState.Error -> onErrorToast(
-            (getCoordinatesUiState as GetCoordinatesUiState.Error).exception,
-            R.string.get_address_convert_fail
-        )
-    }
-}
-
-LaunchedEffect(getAddressUiState) {
-    when (getAddressUiState) {
-        is GetAddressUiState.Loading -> Unit
-        is GetAddressUiState.Success -> onErrorToast(null, R.string.get_address_success)
-        is GetAddressUiState.Error -> onErrorToast(
-            (getAddressUiState as GetAddressUiState.Error).exception,
-            R.string.get_address_fail
-        )
-    }
-}
+LaunchedEffect(getCoordinatesUiState, getAddressUiState) {
+    when (getCoordinatesUiState) {
+        is GetCoordinatesUiState.Loading -> Unit
+        is GetCoordinatesUiState.Success -> onErrorToast(
+            null,
+            R.string.get_address_convert_success
+        )
+        is GetCoordinatesUiState.Error -> onErrorToast(
+            getCoordinatesUiState.exception,
+            R.string.get_address_convert_fail
+        )
+    }
+    
+    when (getAddressUiState) {
+        is GetAddressUiState.Loading -> Unit
+        is GetAddressUiState.Success -> onErrorToast(
+            null,
+            R.string.get_address_success
+        )
+        is GetAddressUiState.Error -> onErrorToast(
+            getAddressUiState.exception,
+            R.string.get_address_fail
+        )
+    }
+}

175-190: 주소 검색 결과에 대한 사용자 피드백 개선이 필요합니다.

현재 구현에서는 로딩 상태나 에러 상태를 사용자에게 시각적으로 표시하지 않고 있습니다.

다음과 같은 개선사항을 제안드립니다:

 LazyColumn(modifier.zIndex(1f)) {
+    when {
+        addressList.isEmpty() && getAddressUiState is GetAddressUiState.Loading -> {
+            item {
+                LoadingIndicator()
+            }
+        }
+        addressList.isEmpty() && getAddressUiState is GetAddressUiState.Error -> {
+            item {
+                ErrorMessage(
+                    message = "주소를 찾을 수 없습니다.",
+                    onRetry = onLocationSearch
+                )
+            }
+        }
+        addressList.isEmpty() -> {
+            item {
+                EmptyMessage(message = "검색 결과가 없습니다.")
+            }
+        }
+        else -> {
             itemsIndexed(addressList) { index, address ->
                 // ... existing code ...
             }
+        }
+    }
 }

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d035e2d and e8b6a9f.

📒 Files selected for processing (3)

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt (1 hunks)
feature/expo/src/main/java/com/school_of_company/expo/view/ExpoCreateScreen.kt (10 hunks)
feature/expo/src/main/java/com/school_of_company/expo/viewmodel/ExpoViewModel.kt (11 hunks)

🚧 Files skipped from review as they are similar to previous changes (2)

feature/expo/src/main/java/com/school_of_company/expo/viewmodel/ExpoViewModel.kt
feature/expo/src/main/java/com/school_of_company/expo/view/ExpoCreateScreen.kt

⏰ Context from checks skipped due to timeout of 90000ms (1)

GitHub Check: build

🔇 Additional comments (2)

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt (2)

192-201: 지도 표시 로직에 대한 개선이 필요합니다.

좌표값 검증이 문자열 비교로만 되어 있어 잠재적인 문제가 있을 수 있습니다.

205-220: 버튼 활성화 조건 개선 필요

현재 버튼 활성화 조건이 단순 문자열 검사에만 의존하고 있습니다.

…-expocreatescreen-location-input

audgns10

Good

answad

👏 감사합니다~~

answad added 30 commits February 7, 2025 23:51

🔥 :: ExpoCreateRoute 에서 주소검색 기능 삭제

231ce3a

🔥 :: ExpoViewModel 에서 주소검색, 변환 기능 삭제

f6ba337

✨ :: ExpoAddressSearchViewModel 생성

62c891c

✨ :: coordinateX 추가, coordinateY 추가

1dcf8a6

✨ :: convertJibunToXY 으로 받아온 xy 값을 소수점 6자리까지만 저장하도록 변환로직 추가

c48fce7

🔥 :: ExpoAddressSearchViewModel 에서 x, y 값을 변환함으로 변환로직 삭제

6f78226

💄 :: 위치를 알려주세요 텍스트필드 위에 장소 text 추가

74da6d9

💄 :: ExpoSearchIconTextField 퍼블리싱

b4996ad

💄 :: ExpoAddressSearchScreen 퍼블리싱

29a4d35

✨ :: ExpoAddressSearchRoute viewModel과 기능 연결

d25422b

✨ :: ExpoAddressSearchRoute의 navigation 함수 생성

33e56a1

✨ :: ExpoAddressSearchRoute navHost에 등록

36bc022

✨ :: navigateToExpoAddressSearchRoute 함수의 이름을 navigateToExpoAddressSe…

d96f433

…arch 로 변경

✨ :: ExpoCreateRoute 에 navigateToExpoAddressSearch 전달

01f5512

♻️ :: 위치를 알려주세요. textField 의 버튼을 눌렀을때 ExpoAddressSearchRoute로 이동하도록 함…

470db80

…수 추가

💄 :: ExpoAddressSearchScreen 의 최상위 column에 padding 16.dp 추가

cda4994

✨ :: searchLocation 실행시 xy 좌표를 초기화 하고 convertJibunToXY 실행시 _getAddres…

9312c92

…sUiState 를 초기화 하도록 기능 추가

🔥 :: _getCoordinatesUiState, _getAddressUiState삭제

a7005ca

🔥 :: ExpoAddressSearchViewModel 삭제

0247df7

♻️ ::viewModel을 ExpoViewModel로 변경

b8f8903

♻️ :: ExpoAddressSearchViewModel과 ExpoViewModel 병합

5e35789

♻️ :: ExpoAddressSearchRoute 과 ExpoAddressSearchRoute 이 같은 viewModel …

9ee6094

…인스턴스를 공유하도록 수정

♻️ :: 하단버튼의 택스트를 생성하기 -> 완료로 변경

1450b52

♻️ :: expoModifyScreen 에 navigateToExpoAddressSearch 전달

780ad34

♻️ :: ExpoLocationIconTextField 에서 navigateToExpoAddressSearch 을 사용하도…

ee2e985

…록 수정

♻️ :: SEARCHED_COORDINATEX, SEARCHED_COORDINATEY, SEARCHED_LOCATION 추…

63519d4

…가, 핸들링함수 추가

♻️ :: Searched 함수로 변경

11356b7

♻️ :: DisposableEffect 에 resetExpoInformation 추가

4f56426

♻️ :: searchedState로 location 과 coordinate 상태를 초기화 하는 launchedEffect …

b6217bd

…블럭 추가

♻️ :: ExpoAddressSearchRoute 에서 location과 xy 좌표를 초기화하는 LaunchedEffect블럭

9b6f9b1

♻️ :: searched_location 상태에 따라 사용하는 값을 다르게 하도록 수정

e2d41c3

answad requested a review from audgns10 February 9, 2025 16:16

answad self-assigned this Feb 9, 2025

answad linked an issue Feb 9, 2025 that may be closed by this pull request

expoCreateScreen 장소 입력 부분을 변경해야 합니다 #416

Closed

audgns10 reviewed Feb 9, 2025

View reviewed changes

coderabbitai bot reviewed Feb 9, 2025

View reviewed changes

audgns10 reviewed Feb 9, 2025

View reviewed changes

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt Outdated Show resolved Hide resolved

audgns10 reviewed Feb 9, 2025

View reviewed changes

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoCreateScreen.kt Outdated Show resolved Hide resolved

audgns10 reviewed Feb 9, 2025

View reviewed changes

feature/expo/src/main/java/com/school_of_company/expo/view/ExpoAddressSearchScreen.kt Outdated Show resolved Hide resolved

♻️ :: getCoordinatesUiState, getAddressUiState 상태에 따라서 success 시에도 토스…

88fa06b

…트를 띄우도록 수정

coderabbitai bot reviewed Feb 11, 2025

View reviewed changes

♻️ :: unfocus 를 하기 쉽도록 pointerInput 추가

d035e2d

coderabbitai bot reviewed Feb 11, 2025

View reviewed changes

answad added 5 commits February 11, 2025 19:55

✨ :: SearchedLocation과 SearchedCoordinate를 초기화 하는 initSearchedData 함수 추가

6cb4dbb

✨ :: _getAddressUiState와 _getCoordinatesUiState 를 초기화하는 initSearchedU…

0f8a47c

…iState 함수 추가

✨ :: initSearchedUiState 함수와 initSearchedData 함수가 적절한 타이밍에 작동 되도록 추가

f9208a7

✨ :: searched 값으로 Coordinate 과 location을 초기화 하는 함수

7c33397

initializeWithSearchedData 추가

♻️ :: initializeWithSearchedData 함수 적용

e8b6a9f

coderabbitai bot reviewed Feb 11, 2025

View reviewed changes

Merge remote-tracking branch 'origin/develop' into feature/416-update…

a09ff41

…-expocreatescreen-location-input

audgns10 approved these changes Feb 11, 2025

View reviewed changes

answad commented Feb 11, 2025

View reviewed changes

answad merged commit b53352b into develop Feb 11, 2025
2 checks passed

answad deleted the feature/416-update-expocreatescreen-location-input branch February 11, 2025 13:08

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

🔀 :: (#416) - update expocreatescreen location input #422

🔀 :: (#416) - update expocreatescreen location input #422

answad commented Feb 9, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Feb 9, 2025 •

edited

Loading

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (`.coderabbit.yaml`)

Documentation and Community

audgns10 left a comment

coderabbitai bot left a comment

coderabbitai bot Feb 9, 2025

coderabbitai bot Feb 9, 2025

audgns10 Feb 9, 2025

audgns10 Feb 9, 2025

answad Feb 11, 2025

coderabbitai bot left a comment

coderabbitai bot Feb 11, 2025

coderabbitai bot left a comment

coderabbitai bot left a comment

audgns10 left a comment

answad left a comment

🔀 :: (#416) - update expocreatescreen location input #422

🔀 :: (#416) - update expocreatescreen location input #422

Conversation

answad commented Feb 9, 2025 • edited by coderabbitai bot Loading

💡 개요

📃 작업내용

🔀 변경사항

🙋‍♂️ 질문사항

🍴 사용방법

Summary by CodeRabbit

coderabbitai bot commented Feb 9, 2025 • edited Loading

Walkthrough

Changes

Possibly related PRs

Suggested labels

Suggested reviewers

Poem

Chat

CodeRabbit Commands (Invoked using PR comments)

Other keywords and placeholders

CodeRabbit Configuration File (.coderabbit.yaml)

Documentation and Community

audgns10 left a comment

Choose a reason for hiding this comment

coderabbitai bot left a comment

Choose a reason for hiding this comment

coderabbitai bot Feb 9, 2025

Choose a reason for hiding this comment

coderabbitai bot Feb 9, 2025

Choose a reason for hiding this comment

audgns10 Feb 9, 2025

Choose a reason for hiding this comment

audgns10 Feb 9, 2025

Choose a reason for hiding this comment

answad Feb 11, 2025

Choose a reason for hiding this comment

coderabbitai bot left a comment

Choose a reason for hiding this comment

coderabbitai bot Feb 11, 2025

Choose a reason for hiding this comment

coderabbitai bot left a comment

Choose a reason for hiding this comment

coderabbitai bot left a comment

Choose a reason for hiding this comment

audgns10 left a comment

Choose a reason for hiding this comment

answad left a comment

Choose a reason for hiding this comment

answad commented Feb 9, 2025 •

edited by coderabbitai bot

Loading

coderabbitai bot commented Feb 9, 2025 •

edited

Loading

CodeRabbit Configuration File (`.coderabbit.yaml`)