DEG-179 [Fix] 스테이지 제한 #70

Sehyeon · 2025-05-15T02:38:09Z

Sehyeon commented

2025-05-15 02:38:09 +00:00

작업 내용

스테이지를 전부 클리어 한 이후에도 던전 진입 가능 -> 입장 제한 처리

기타 오류 사항

게임 시작 시 로딩창 추가로 인해 씬 로드에 시간이 걸려 채팅창 null 오류 발생

현재 발견 오류 상황
출근 -> 야근 이벤트 -> 취침 -> 집안일(결근) -> 취침시 결근 이벤트 발동하는 오류 발생
아직 해결못했습니다..

작업 내용 - 스테이지를 전부 클리어 한 이후에도 던전 진입 가능 -> 입장 제한 처리 기타 오류 사항 - 게임 시작 시 로딩창 추가로 인해 씬 로드에 시간이 걸려 채팅창 null 오류 발생 현재 발견 오류 상황 출근 -> 야근 이벤트 -> 취침 -> 집안일(결근) -> 취침시 결근 이벤트 발동하는 오류 발생 아직 해결못했습니다..

Sehyeon added 1 commit 2025-05-15 02:38:10 +00:00

DEG-179 [Fix] 스테이지 제한 545df9f6d7

99jamin approved these changes 2025-05-15 02:42:10 +00:00

99jamin left a comment

고생하셨습니다. 업그레이드 매니저도 확인했습니다

👍 1

jay approved these changes 2025-05-15 02:46:16 +00:00

jay left a comment

고생 많으셨습니다!

👍 1

Assets/KSH/PlayerStats.cs

						
				@ -384,6 +384,8 @@ public class PlayerStats : MonoBehaviour,ISaveable

				    public void ModifyReputation(float reputation)

				    {

				        if (reputation == 0f) return;

jay commented

2025-05-15 02:45:27 +00:00

여기서 0 들어가면 밑에 else문이랑 zero~부분 실행 안되는데 이슈 없을까요?

else
        {
            ReputationStat = 0f;
        }
        

if (ReputationStat <= 0)
        {
            ZeroReputation?.Invoke();
            ReputationStat = 0.0f;
        }

여기서 0 들어가면 밑에 else문이랑 zero~부분 실행 안되는데 이슈 없을까요? ```c# else { ReputationStat = 0f; } if (ReputationStat <= 0) { ZeroReputation?.Invoke(); ReputationStat = 0.0f; } ```

Sehyeon commented

2025-05-15 02:48:55 +00:00

0체크는 새로 들어오는 평판 값(reputation) 기준이고 아래의 조건문은 기존의 평판값(ReputationStat)이라 문제가 없을거라 생각이 듭니다.
평판 값에 변경이 없을 때 빠르게 빠져나가는 용으로 추가했습니다

0체크는 새로 들어오는 평판 값(reputation) 기준이고 아래의 조건문은 기존의 평판값(ReputationStat)이라 문제가 없을거라 생각이 듭니다. 평판 값에 변경이 없을 때 빠르게 빠져나가는 용으로 추가했습니다

Sehyeon commented

2025-05-15 02:49:42 +00:00

없어도 괜찮지만.. Mathf.Round를 더 호출하다가 오류가 발생할까봐 미리 방지한 것이기도 합니다

jay commented

2025-05-15 02:53:45 +00:00

아 궁금한 부분은 기존의 평판값 변경 로직이 저 부분인 것 같은데 0으로 들어오면 Reputation 값도 0으로 변경조차 안되니까 밑에 zeroReputation 로직 등도 실행 안되는게 아닌가 생각했습니다. 이상 없으면 넘겨주세요!

Sehyeon commented

2025-05-15 02:57:50 +00:00

아하~ 평판 값은 +, -로 제어하고 있습니다!
reputation 값이 0이면 +0 처리이지 Reputation 값이 0으로 변경되진 않습니다
만약 현재 평판(ReputationStat)이 2인 상황에서 reputation 값이 -3으로 들어온다면
2-3 = -1이 되어 아래의 코드,
if (ReputationStat <= 0)
{
ZeroReputation?.Invoke();
ReputationStat = 0.0f;
}
가 실행됩니다.
따라서 ZeroReputation 로직 등은 정상적으로 작동합니다!

아하~ 평판 값은 +, -로 제어하고 있습니다! reputation 값이 0이면 +0 처리이지 Reputation 값이 0으로 변경되진 않습니다 만약 현재 평판(ReputationStat)이 2인 상황에서 reputation 값이 -3으로 들어온다면 2-3 = -1이 되어 아래의 코드, if (ReputationStat <= 0) { ZeroReputation?.Invoke(); ReputationStat = 0.0f; } 가 실행됩니다. 따라서 ZeroReputation 로직 등은 정상적으로 작동합니다!

Sehyeon commented

2025-05-15 02:59:40 +00:00

이거랑 별개로 ModifyReputation 코드 자체가 조금 이상(코드 중복)한 건 맞습니다^^.. UI에서 소수점이나 마이너스가 안뜨도록 하다보니 이렇게 되었습니다 허허

👍 1

Sehyeon closed this pull request

2025-05-15 04:04:26 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.

Sign in to join this conversation.

3 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Degulleo/Degulleo3D#70