DEG-172-일상UI-색상변경-던전UI-눌림표시 #58

Merged
heain0122 merged 3 commits from DEG-172-일상UI-색상변경-던전UI-눌림표시 into main 2025-05-14 07:43:40 +00:00
Owner

작업내용

  1. 일상씬 체력 바 빨간색으로 수정
    image.png

  2. 던전씬 키보드 조작시 터치한 것 같은 효과

    • 버튼 색상이 바뀐 것 처럼 보이도록,, 버튼 위에 0.2초간 이미지를 보여주고 있습니다.
  3. 버그 수정

    • 출근 -> 던전 -> 수면 -> 출근 시 발생하던 sudden event panel null 에러픽스
### 작업내용 1. 일상씬 체력 바 빨간색으로 수정 <img width="359" alt="image.png" src="attachments/cce14f90-e4d9-4383-b400-f149e9bb168a"> 2. 던전씬 키보드 조작시 터치한 것 같은 효과 - 버튼 색상이 바뀐 것 처럼 보이도록,, 버튼 위에 0.2초간 이미지를 보여주고 있습니다. 3. 버그 수정 - 출근 -> 던전 -> 수면 -> 출근 시 발생하던 sudden event panel null 에러픽스
heain0122 added 2 commits 2025-05-14 07:19:51 +00:00
heain0122 added 1 commit 2025-05-14 07:24:47 +00:00
Sehyeon approved these changes 2025-05-14 07:29:22 +00:00
Sehyeon left a comment
Owner

코드 확인했습니다 수고하셨습니다
아래로 질문 하나만 남기겠습니다!

코드 확인했습니다 수고하셨습니다 아래로 질문 하나만 남기겠습니다!
@ -16,6 +16,7 @@ public class InteractionController : MonoBehaviour
private void Start()
{
PlayerStats.Instance.OnWorked -= SuddenAfterWorkEventHappen;
Owner

여기서 SuddenAfterWorkEventHappen -= 하는 이유가 있을까요? 중복 이벤트 할당때문인가용?

여기서 SuddenAfterWorkEventHappen -= 하는 이유가 있을까요? 중복 이벤트 할당때문인가용?
Owner

아 이거 때문에 null 오류가 발생한건가요? 이전 이벤트를 참조해서..인가요?

아 이거 때문에 null 오류가 발생한건가요? 이전 이벤트를 참조해서..인가요?
Author
Owner

씬이 전환되면서 InteractionController 의 Start가 한번 더 실행되고 OnWorked 에 2개 이상의 이벤트가 구독 돼서 SuddenAfterWorkEventHappen를 두 번 호출하려고 했던 것 같습니다.
그리고 첫번째 Start에서 구독한 SuddenAfterWorkEventHappen는 씬 전환 전에 있던 canvas의 sudden event panel을 참조하고 있어서 null에러가 났던 것 같습니다.
19번째 줄로 OnWorked 에 남아있는 구독을 모두 해제하고 20번째 줄에서 새로 등록하고 있습니다.

씬이 전환되면서 InteractionController 의 Start가 한번 더 실행되고 OnWorked 에 2개 이상의 이벤트가 구독 돼서 SuddenAfterWorkEventHappen를 두 번 호출하려고 했던 것 같습니다. 그리고 첫번째 Start에서 구독한 SuddenAfterWorkEventHappen는 씬 전환 전에 있던 canvas의 sudden event panel을 참조하고 있어서 null에러가 났던 것 같습니다. 19번째 줄로 OnWorked 에 남아있는 구독을 모두 해제하고 20번째 줄에서 새로 등록하고 있습니다.
Owner

아하..! 설명 감사합니다 오류는 구독 문제였군요
고생하셨습니다~!

아하..! 설명 감사합니다 오류는 구독 문제였군요 고생하셨습니다~!
99jamin approved these changes 2025-05-14 07:39:18 +00:00
99jamin left a comment
Owner

고생하셨습니다 참고하겠습니다!

고생하셨습니다 참고하겠습니다!
heain0122 merged commit 94a25882ed into main 2025-05-14 07:43:40 +00:00
Sign in to join this conversation.
No Reviewers
No Label
No Milestone
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Degulleo/Degulleo3D#58
No description provided.