[UI/YAF-74] 알람 설정 시 알람 미루기를 설정할 수 있다 #46

DongChyeon · 2025-01-19T08:36:29Z

Related issue 🛠

closed #45

어떤 변경사항이 있었나요?

🐞 BugFix Something isn't working
🎨 Design Markup & styling
📃 Docs Documentation writing and editing (README.md, etc.)
✨ Feature Feature
🔨 Refactor Code refactoring
⚙️ Setting Development environment setup
✅ Test Test related (Junit, etc.)

CheckPoint ✅

PR이 다음 요구 사항을 충족하는지 확인하세요.

PR 컨벤션에 맞게 작성했습니다. (필수)
merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕) (필수)
Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다. (필수)
BugFix의 경우, 버그의 원인을 파악하였습니다. (선택)

Work Description ✏️

default.mp4

선택된 날짜가 없다면 공휴일 알람 끄기 스위치 비활성화
알람 남은 시간 디바운싱 없이 실시간 반영
알람 미루기 설정 바텀 시트 연동

Uncompleted Tasks 😅

N/A

To Reviewers 📢

data class State(
    val timeState: AlarmTimeState = AlarmTimeState(),
    val daySelectionState: AlarmDaySelectionState = AlarmDaySelectionState(),
    val holidayState: AlarmHolidayState = AlarmHolidayState(),
    val snoozeState: AlarmSnoozeState = AlarmSnoozeState(),
) : UiState

data class AlarmTimeState(
    val currentAmPm: String = "오전",
    val currentHour: Int = 1,
    val currentMinute: Int = 0,
    val alarmMessage: String = "",
)

data class AlarmDaySelectionState(
    val days: Set<AlarmDay> = enumValues<AlarmDay>().toSet(),
    val isWeekdaysChecked: Boolean = false,
    val isWeekendsChecked: Boolean = false,
    val selectedDays: Set<AlarmDay> = setOf(),
)

data class AlarmHolidayState(
    val isDisableHolidayEnabled: Boolean = false,
    val isDisableHolidayChecked: Boolean = false,
)

data class AlarmSnoozeState(
    val isSnoozeEnabled: Boolean = true,
    val snoozeIntervalIndex: Int = 2,
    val snoozeCountIndex: Int = 1,
    val isBottomSheetOpen: Boolean = false,
    val snoozeIntervals: List<String> = listOf("1분", "3분", "5분", "10분", "15분"),
    val snoozeCounts: List<String> = listOf("1회", "3회", "5회", "10회", "무한"),
)

화면에 표시할 정보가 너무 많아서 컴포넌트 단위로 상태 정의해서 관리하는데
나중에 다른 화면에도 써먹을 수 있을 것 같아요

이거보다 더 좋은 방식 있을 것 같음 코멘트 달아주세용

MoonsuKang

수고하셨습니다.
컴포넌트 단위로 State처리한거에 대한 생각:
지금은 기능단위로 처리된걸 Context 단위로도 분리할 수 있을 것 같다?
예를 들어서 알람 화면 전체에 관련된 상태를 하나의 AlarmState로 통합하고,
필요시 하위 컴포넌트에서 일부 상태를 가져다 쓰는 방식으로 설계할 수도 있을 것 같습니다.

MoonsuKang · 2025-01-19T09:25:25Z

feature/home/src/main/java/com/yapp/alarm/AlarmAddEditViewModel.kt

-            is AlarmAddEditContract.Action.ToggleWeekendsChecked -> toggleWeekendsChecked()
-            is AlarmAddEditContract.Action.ToggleDaySelection -> toggleDaySelection(action.day)
-            is AlarmAddEditContract.Action.ToggleDisableHolidayChecked -> toggleDisableHolidayChecked()
+        viewModelScope.launch {


p5
로컬 디비에 담아야되서 viewmodelscope.launch 넣으신 거죠??

의도한 것은 아닌데 Copilot 쓰다가 저렇게 됐네요....
나중에 DB 연동 작업 시에 알맞게 수정할게요

MoonsuKang · 2025-01-19T09:33:11Z

feature/home/src/main/java/com/yapp/alarm/AlarmAddEditViewModel.kt

 import javax.inject.Inject

 @HiltViewModel
 class AlarmAddEditViewModel @Inject constructor() : BaseViewModel<AlarmAddEditContract.State, AlarmAddEditContract.SideEffect>(
    initialState = AlarmAddEditContract.State(),
 ) {
+    private var debounceJob: Job? = null // 디바운싱을 위한 Job


p4
Job을 사용하신 이유가 궁금합니다이
mutableStateFlow로 하면 아마 스트림 연산으로 dobounce를 처리할 수 있을거 같아서

요거 디바운싱 처리했다가 알라미에서는 디바운싱 안했길래 제거했어요

MoonsuKang · 2025-01-19T09:35:05Z

feature/home/src/main/java/com/yapp/alarm/AlarmAddEditViewModel.kt

+        }
+    }
+
+    private fun updateAlarmMessage() {


p3
시간계산 쪽은 뭔가 domain layer에서 처리해야될 것 같은 아이들이 있는거 같아요.
아니면 유틸함수로 빼도 될듯?

시간 계산이 위에 n시간 n분 남았습니다 를 말씀하시는건가요?

분리 하자면

// 남은 시간 계산 val duration = java.time.Duration.between(now, nextAlarmDateTime) val totalMinutes = duration.toMinutes() val days = totalMinutes / (24 * 60) val hours = (totalMinutes % (24 * 60)) / 60 val minutes = totalMinutes % 60 // 출력 문구 생성 val newMessage = when { days > 0 -> "${days}일 ${hours}시간 후에 울려요" hours > 0 -> "${hours}시간 ${minutes}분 후에 울려요" else -> "${minutes}분 후에 울려요" }

this area.
사용하는지는 잘 모르지만 유틸이나 도메인으로 분리하면 다른 뷰에서도 참조 가능하고 우리가 나중에 테스트 코드를 짜게 된다면..? 더 이점이 있을듯 ㅋㅋㅋ

이부분은 재사용하는 로직이 아니라서 따로 domain으로 안 빼둘게요

MoonsuKang · 2025-01-19T09:38:36Z

feature/home/src/main/java/com/yapp/alarm/AlarmAddEditViewModel.kt

+    private fun convertTo24HourFormat(amPm: String, hour: Int): Int {
+        return if (amPm == "오후" && hour != 12) {
+            hour + 12
+        } else if (amPm == "오전" && hour == 12) {
+            0
+        } else {
+            hour
+        }
+    }


p3

private fun convertTo24HourFormat(amPm: String, hour: Int): Int = when { amPm == "오후" && hour != 12 -> hour + 12 amPm == "오전" && hour == 12 -> 0 else -> hour }

반영했습니다!

MoonsuKang · 2025-01-19T09:52:28Z

feature/home/src/main/java/com/yapp/alarm/component/bottomsheet/AlarmSnoozeBottomSheet.kt

p5
어우 👍👍👍

MoonsuKang

호우

MoonsuKang · 2025-01-19T12:03:38Z

feature/home/src/main/java/com/yapp/alarm/AlarmAddEditViewModel.kt

+        }
+    }
+
+    private fun updateAlarmMessage() {


분리 하자면

// 남은 시간 계산 val duration = java.time.Duration.between(now, nextAlarmDateTime) val totalMinutes = duration.toMinutes() val days = totalMinutes / (24 * 60) val hours = (totalMinutes % (24 * 60)) / 60 val minutes = totalMinutes % 60 // 출력 문구 생성 val newMessage = when { days > 0 -> "${days}일 ${hours}시간 후에 울려요" hours > 0 -> "${hours}시간 ${minutes}분 후에 울려요" else -> "${minutes}분 후에 울려요" }

this area.
사용하는지는 잘 모르지만 유틸이나 도메인으로 분리하면 다른 뷰에서도 참조 가능하고 우리가 나중에 테스트 코드를 짜게 된다면..? 더 이점이 있을듯 ㅋㅋㅋ

DongChyeon added 9 commits January 18, 2025 00:13

[FEAT/#45] 알람 남은 시간 실시간 반영

9d9dd72

[UI/#45] 알람 미루기, 사운드 설정 아이템 UI 구현

b52dc41

[UI/#45] AlarmCheckItem 아이콘, 글자 색 수정

8b135d1

[UI/#45] Switch 컴포넌트 비활성화 상태 추가

c27b57f

[FEAT/#45] 요일 선택에 따른 공휴일 알람 끄기 활성화 여부 반영

27e7997

[UI/#45] AlarmSnoozeBottomSheet 컴포넌트 구현

750987c

[UI/#45] RadioButton 컴포넌트 비활성화 상태 추가

82da905

[REFACTOR/#45] AlarmAddEditContract.State를 여러 데이터 클래스로 분리

c14383d

[FEAT/#45] AlarmAddEditScreen에 AlarmSnoozeBottomSheet 연동

ff07fb2

DongChyeon added 📱 UI 사용자 인터페이스 관련 작업 💪 동현동현동현동현동현 labels Jan 19, 2025

DongChyeon self-assigned this Jan 19, 2025

MoonsuKang reviewed Jan 19, 2025

View reviewed changes

[REFACTOR/#45] convertTo24HourFormat 함수 리팩토링

c824f11

MoonsuKang approved these changes Jan 19, 2025

View reviewed changes

DongChyeon merged commit 0bfc6c8 into develop Jan 19, 2025
1 check passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[UI/YAF-74] 알람 설정 시 알람 미루기를 설정할 수 있다 #46

[UI/YAF-74] 알람 설정 시 알람 미루기를 설정할 수 있다 #46

DongChyeon commented Jan 19, 2025

MoonsuKang left a comment

MoonsuKang Jan 19, 2025

DongChyeon Jan 19, 2025

MoonsuKang Jan 19, 2025

DongChyeon Jan 19, 2025

MoonsuKang Jan 19, 2025

DongChyeon Jan 19, 2025

MoonsuKang Jan 19, 2025

DongChyeon Jan 19, 2025

MoonsuKang Jan 19, 2025

DongChyeon Jan 19, 2025

MoonsuKang Jan 19, 2025

MoonsuKang left a comment

MoonsuKang Jan 19, 2025

[UI/YAF-74] 알람 설정 시 알람 미루기를 설정할 수 있다 #46

[UI/YAF-74] 알람 설정 시 알람 미루기를 설정할 수 있다 #46

Conversation

DongChyeon commented Jan 19, 2025

Related issue 🛠

CheckPoint ✅

Work Description ✏️

Uncompleted Tasks 😅

To Reviewers 📢

MoonsuKang 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

MoonsuKang left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment