Feat/440 선물 등록 시 사전 질문 작성 by HamBP · Pull Request #446 · Nexters/Boolti

HamBP · 2026-03-04T15:21:33Z

Issue

close 선물하기 등록 시 사전 질문 작성 폼 #440

작업 내용

선물 받기 시 사전 질문이 있을 경우 사전 질문을 작성한다.
홈 -> 선물 사전 질문 작성 화면으로 넘어가면서 동일한 API 두 개(선물 정보 조회, 사전 질문 목록 조회)를 중복 호출하게 되는데, 이를 방지하기 위해 간단한 캐싱 로직 작성.

BE 논의 내용

기존 선물 API에 공연 정보 추가
API 호출 순서 변경 요청 받음.
- 기존: 사전 질문 등록 -> 선물 받기
- 변경: 선물 받기 -> 사전 질문 등록 (권한 체크 위함)
- 선물 받기 API만 성공했을 경우 질문이 누락된 상태로 티켓이 존재. 약간의 대비로 실패 시 리트라이 및 파베 로그 전송 로직만 작성해 두었음.

github-actions · 2026-03-04T15:29:58Z

Test Results

12 files 12 suites 0s ⏱️
9 tests 9 ✅ 0 💤 0 ❌
20 runs 20 ✅ 0 💤 0 ❌

Results for commit 7a93107.

♻️ This comment has been updated with latest results.

...rc/main/java/com/nexters/boolti/presentation/screen/prequestionedit/PreQuestionEditScreen.kt

mangbaam

Recommended Action

[필수] putPreQuestionAnswer()의 unsafe cast → safe cast 수정 (크래시 방지)
[필수] receiveGift()에서 recordExceptionHandler 대신 explicit try-catch로 변경하여 실패 이벤트 전송
[필수] fetchGiftAndPreQuestions() 실패 시 Error 상태 전환 + Loading UI 추가
[중요] 선물 등록 성공 + 사전질문 실패를 구분하는 별도 이벤트/UX 검토
[중요] cachedUser null 시 에러 피드백 추가
[권장] receiveGift() 성공 후 캐시 무효화

mangbaam · 2026-03-15T12:30:55Z

...main/java/com/nexters/boolti/presentation/screen/giftprequestion/GiftPreQuestionViewModel.kt

+                .retry(2)
+                .catch { throwable ->
+                    FirebaseCrashlytics.getInstance().recordException(throwable)
+                    _events.send(GiftPreQuestionEvent.GiftRegistrationFailed)


선물 받기 API 성공 → 사전질문 제출 실패 시 GiftRegistrationFailed 이벤트가 발생합니다. 사용자는 "등록 실패"를 보지만, 실제로 서버에는 선물이 이미 등록된 상태입니다. 사전질문 답변만 누락된 상태로 티켓이 존재하게 됩니다.

또한 .catch가 CancellationException도 잡아서 structured concurrency를 위반합니다.

권장:

PreQuestionSubmitFailed 같은 별도 이벤트를 만들어 사전질문 수정 화면으로 안내

.retry(2) { it !is CancellationException } 조건 추가

.catch에서 CancellationException은 rethrow

mangbaam · 2026-03-15T12:31:54Z

...main/java/com/nexters/boolti/presentation/screen/giftprequestion/GiftPreQuestionViewModel.kt

+
+    private fun fetchGiftAndPreQuestions() {
+        viewModelScope.launch(recordExceptionHandler) {
+            _uiState.update { GiftPreQuestionUiState.Loading }


fetchGiftAndPreQuestions() - 실패 시 무한 로딩

GiftPreQuestionViewModel.kt

recordExceptionHandler가 예외를 잡으면 UI 상태가 Loading에 머물러 사용자는 빈 화면만 보게 됩니다. 에러 상태도 없고 재시도 버튼도 없습니다.

권장:

GiftPreQuestionUiState.Error 상태 추가

명시적 try-catch로 에러 상태 전환 및 재시도 UI 제공

mangbaam · 2026-03-15T12:33:20Z

presentation/src/main/java/com/nexters/boolti/presentation/screen/home/HomeViewModel.kt

-
+            val senderId = gift.senderUserId
+            val hasPreQuestion = ticketingRepository.getPreQuestions(gift.showId).first().isNotEmpty()
            val myUserId = authRepository.cachedUser.first()?.id ?: return@launch


authRepository.cachedUser.first()?.id ?: return@launch로 cachedUser가 null이면 아무 피드백 없이 종료됩니다. 딥링크로 진입한 사용자가 아무 반응 없이 막히게 됩니다.

권장: GiftStatus.FAILED 이벤트를 보내서 에러 다이얼로그 표시

mangbaam · 2026-03-15T12:33:56Z

...main/java/com/nexters/boolti/presentation/screen/giftprequestion/GiftPreQuestionViewModel.kt

+
+    fun putPreQuestionAnswer(questionId: Long, answer: String) {
+        _uiState.update {
+            val state = it as GiftPreQuestionUiState.Success


val state = it as GiftPreQuestionUiState.Success // unsafe cast!
receiveGift()에서는 as?를 사용하는데 여기만 as를 사용합니다. Loading 상태에서 호출되면 ClassCastException으로 앱이 크래시합니다.

수정: as? ... ?: return@update it 패턴으로 변경

mangbaam · 2026-03-15T12:35:58Z

presentation/src/main/java/com/nexters/boolti/presentation/screen/home/HomeViewModel.kt

+
+        pendingGift = null

        giftRepository.receiveGift(giftUuid)


giftRepository.receiveGift() 자체가 예외를 던지면 recordExceptionHandler가 잡아서 Crashlytics에만 기록합니다. GiftRegistrationFailed 이벤트가 전송되지 않아 사용자에게 아무 피드백이 없습니다.

권장: recordExceptionHandler 대신 명시적 try-catch로 실패 이벤트 전송

mangbaam · 2026-03-15T12:36:37Z

...rc/main/java/com/nexters/boolti/presentation/screen/giftprequestion/GiftPreQuestionScreen.kt

+        ) {
+            val state = uiState
+
+            if (state is GiftPreQuestionUiState.Success) {


Success 상태만 렌더링하고, Loading 시 빈 화면이 표시됩니다. 로딩 인디케이터도 에러 UI도 없습니다.

mangbaam · 2026-03-15T12:37:53Z

data/src/main/java/com/nexters/boolti/data/repository/GiftRepositoryImpl.kt

 ) : GiftRepository {
+    private data class CacheEntry<T>(val value: T, val cachedAt: Long)
+
+    private val giftCache = ConcurrentHashMap<String, CacheEntry<Gift>>()


receiveGift() 후 캐시 무효화 없음

선물 등록 후에도 60초간 이전 상태의 캐시를 반환합니다. receiveGift() 성공 시 giftCache.remove(giftUuid) 추가를 권장합니다

mangbaam · 2026-03-15T12:38:32Z

presentation/src/main/java/com/nexters/boolti/presentation/screen/home/HomeViewModel.kt

    }

    private fun processGiftWhenLoggedIn(giftUuid: String) {
        viewModelScope.launch(recordExceptionHandler) {


recordExceptionHandler로 모든 에러 무시

선물 정보/사전질문 조회 실패 시 다이얼로그가 안 뜹니다. 사용자는 딥링크 후 아무 반응이 없습니다.

mangbaam · 2026-03-15T12:38:49Z

presentation/src/main/java/com/nexters/boolti/presentation/screen/home/HomeScreen.kt

+            val deepLink = intent.data.toString()
+            intent.data = null
+            val regex = "^boolti://gift/([\\w-])+$".toRegex()
+            if (regex.matches(deepLink)) {


딥링크 regex 미매칭 시 로깅 없음

HomeScreen.kt에서 boolti://gift/ 패턴 미매칭 시 사일런트 드롭됩니다. 디버깅이 어렵습니다.

HamBP added 11 commits January 21, 2026 02:07

chore: 버전 업

52f8d58

Merge branch 'develop' of github.com:Nexters/Boolti into develop

253df9d

fix: response의 answer와 description을 nullable 타입으로 변경했어요.

d6a2d6b

feat: 선물 dto 필드 추가

00a5a24

fix: 점심 모드를 변경했어요.

b965c3e

refactor: 같은 브랜치에 여러 조건을 묶었어요.

a42e040

feat: 사전 질문을 작성하는 화면을 위한 파일을 만들었어요.

ab3812d

feat: 등록하기 시 만약 사전 질문이 있다면 질문 작성 화면으로 내비게이션 해요.

251ab9e

feat: git에 누락된 파일을 추가했어요.

b3c78b9

fix: cold start 시 로그인 정보를 가져오지 못 하는 문제를 수정했어요.

29aca98

feat: 공연 정보와 사전 질문 목록을 가져와요

6c4397b

HamBP self-assigned this Mar 4, 2026

HamBP added the feat 새로운 기능 label Mar 4, 2026

HamBP added 5 commits March 5, 2026 01:06

feat: 공연 정보를 표시해요

a7c013d

feat: 사전 질문 작성 폼을 만들었어요.

74cdac9

feat: 등록하기 버튼을 추가했어요.

85375f9

refactor: 텍스트들을 리소스화 했어요.

a68e790

feat: 페이지 이탈 전 다이얼로그를 띄워줘요.

cf5897f

mangbaam reviewed Mar 4, 2026

View reviewed changes

...rc/main/java/com/nexters/boolti/presentation/screen/prequestionedit/PreQuestionEditScreen.kt Show resolved Hide resolved

HamBP added 10 commits March 5, 2026 04:40

feat: 등록하기 버튼 클릭 시 필요한 API를 호출해요.

027432f

feat: 첫 번째 API 실패 시 오류 이벤트를 내보내요.

85f11b7

feat: API 호출 성공 여부를 체크하여 메시지를 띄워요

d6cd666

feat: 선물 받기 API 호출 순서를 변경했어요.

f2227a6

feat: 캐시된 값이 있다면 캐시된 값을 사용해요.

f0f1c5d

Merge branch 'develop' of github.com:Nexters/Boolti into develop

93d908c

merge

72a9961

feat: 등록 버튼 enabled 처리했어요.

8acfb7c

chore: 버전 업

bf8d778

feat: retry 로직을 넣었어요.

7a93107

mangbaam reviewed Mar 15, 2026

View reviewed changes

Conversation

HamBP commented Mar 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Issue

작업 내용

BE 논의 내용

Uh oh!

github-actions bot commented Mar 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results

Uh oh!

Uh oh!

mangbaam 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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

HamBP commented Mar 4, 2026 •

edited

Loading

github-actions bot commented Mar 4, 2026 •

edited

Loading