Feature/tgyuu/#116

[tgyuuAn]

1. 📄 관련된 이슈 및 소개

closed #122
colsed #116

2. 🔥 변경된 점

• 갤럭시 S22와 같은 하단 Navigation Bar가 있는 휴대폰에서 마진을 해결했지만,

Pixel폰과 같은 기본적으로 gestureNavigation Bar로 이루어진 폰에서는 아래와 같은 흰색 여백이 생겼다..

이를 해결해주었다.

(이제 모든 폰 에서 UI 완벽)

- 출석 일정을 가져오는 과정에서 여러 가지 버그가 있었다.

기존 코드는 아래와 같이 데이터가 없을 경우 checkNotNull에서 illegatStateException을 뱉으며 Result.Failure을 반환하여

뒤에 있는 함수들이 모두 줄줄이 제대로 동작하지 않았다.

val attendanceResponse = task.toObject<AttendanceResponse>()
        checkNotNull(attendanceResponse)

이를 막기 위해,

val attendanceResponse = task.toObject<AttendanceResponse>()
        attendanceResponse ?: AttendanceResponse(eventId = eventId, "", defaultDeadLine)
    }
}

private val defaultDeadLine = "2000-01-01T00:00:00.000000"

로 코드로 바꾸어, null일 경우 deadline으로 아예 있을 수가 없을 정도의 과거 출석 이벤트로 만들어

UseCase filter에서 자동으로 걸러지게 만들었다.

                deferredList.awaitAll()
                    .map { it.getOrThrow() }
                    .filter { it.isBeforeEndTime() }

viewModel에서 zip을 이용해서 event 정보를 얻어올 때,

이미 deadline이 지난 출석에 대해서는 위 usecase에서 filter가 되지만,

아래 원래 코드에서 EventAttendanceStatus에서 들어가는 eventId와 title, content는 Usecase에서 반환되는 eventList가 아닌

함수를 호출할 때 들어오는 파라미터 eventList (필터링이 되지 않은 것) 에서 데이터를 뽑아오기 때문에

데이터가 어긋나는 경우가 생겼다.

(eventList는 필터링되지 않은 데이터, attendanceList는 해당 eventList를 바탕으로 출석이 정보를 얻어오는 데, deadline이 지났는 지 안 지났는 지 필터링 되는 eventList)

private suspend fun getEventListAttendance(eventList: List<Event>, userId: String) =
        getEventListAttendanceUseCase(eventList.map { it.eventId })
            .onSuccess { attendanceList ->
                val eventAttendanceList = eventList.zip(attendanceList)
                    .map { (event, attendance) ->
                        EventAttendanceStatus(
                            eventId = event.eventId,
                            title = event.title,
                            content = event.content,
                            remainAttendanceDateTime = attendance.calculateDeadline(),
                        )
                    }

그래서 UseCase에서 필터링 되는 attendanceList와 함수의 파라미터로 들어가는 eventList의 동기를 맞춰주기 위해

 val filteredEventList = eventList.filter {
                    it.eventId in attendanceList.map { it.eventId }
                }

                val eventAttendanceList = filteredEventList.zip(attendanceList)
                    .map { (event, attendance) ->
                        EventAttendanceStatus(
                            eventId = event.eventId,
                            title = event.title,
                            content = event.content,
                            remainAttendanceDateTime = attendance.calculateDeadline(),
                        )
                    }

위와 같이 eventList에 대해서도 filter를 거쳤다.

3. ✅ 꼭 확인해줬으면 하는 부분

4. 📸 스크린샷(선택)

5. 💡알게된 혹은 궁금한 사항들

System에서 제공하는 statusBar와 하단 NavigationBar때문에 짜증나서 WindowInset에 대해 공부해보려고 하던 도중,

Inset 공식 문서 에서 제공하는 동영상에서

아래와 같이 Modifier.windowInsetsPadding 이라는 엄청난 Modifier를 발견했다.

심지어 WindowInsets.StatusBar 혹은 WindowInsets.navigationBar와 같이 손쉽게 기본 설정 바의 정보를 얻어 올 수도 있었다.

아주 도그 허니

[jeongjaino]

고생하셨어요 태규상 !

몇가지 질문만 슥 남겨볼게요 이상없으시면 그대로 머지 해주셔도 돼용

1. 현재 커스텀 달력에서 '<' 버튼 누르면 이전 년도의 행사도 볼 수 있나요 ?

2. defaultDateTime이 현재 2000년대로 설정되어 있는데, LocalDateTime의 MAX, MIN 상수를 사용하시는건 어때요 ?

3. 기존에 네비게이션 바 측정하는 로직이 있었지 않나요??? 그럼 windowInsect을 사용하면, 간편하게 사용가능한건가요?

[jeongjaino]

LGTM 😎

[tgyuuAn]

고생하셨어요 태규상 !

몇가지 질문만 슥 남겨볼게요 이상없으시면 그대로 머지 해주셔도 돼용

1. 현재 커스텀 달력에서 '<' 버튼 누르면 이전 년도의 행사도 볼 수 있나요 ?

2. defaultDateTime이 현재 2000년대로 설정되어 있는데, LocalDateTime의 MAX, MIN 상수를 사용하시는건 어때요 ?

3. 기존에 네비게이션 바 측정하는 로직이 있었지 않나요??? 그럼 windowInsect을 사용하면, 간편하게 사용가능한건가요?

1. 물론입니다...!

2. 헉 그런 꿀팁이 반영하고 머지허겠습니다...!

3. 기존에는 사라진 코드에 네비게이션 바 높이를 측정하는 함수가 있었습니다..!

(안드로이드 폰 기본에 맨 밑에 깔려있는게 네비게이션 바라고 하드라구요)