Degulleo/Degulleo3D
7
0
Fork 0
Code Issues Pull Requests Actions Packages Releases Wiki Activity

DEG-143-세이브-강화-병합 #46

Closed
99jamin wants to merge 12 commits from DEG-143-세이브-강화-병합 into main
pull from: DEG-143-세이브-강화-병합
merge into: Degulleo:main
Conversation 2 Commits 12 Files Changed 14 +165 -26
99jamin commented 2025-05-13 06:14:37 +00:00
Owner
Copy Link

식사, 집안일 횟수 제한을 제외한 세이브 요소들을 연동했습니다.
playerStat.cs
GameManager.cs
에 ExtractSaveData(),ApplySaveData(Save save) 함수로 작성했으니 확인 부탁드립니다.

강화 수치와 플레이어 스탯도 연동했습니다.
playerController.cs
playerActionAttack.cs
DungeonPanelController.cs
에서 각각 수정했습니다.

공격 속도는 comboDuration만 줄여도 체감이돼서 해당 변수만 조정했습니다.

하트는 2개를 미리 추가하고 투명화 처리한 후에 강화 수치에 따라 보이도록 조정했습니다.

다른 수치는 playerController에서 조절하도록 했습니다.

지금은 단순한 계산식으로 레벨을 적용했습니다.
밸런스가 안맞거나 조금 이상한 부분이 있으면 말씀해주세요.

식사, 집안일 횟수 제한을 제외한 세이브 요소들을 연동했습니다. playerStat.cs GameManager.cs 에 ExtractSaveData(),ApplySaveData(Save save) 함수로 작성했으니 확인 부탁드립니다. 강화 수치와 플레이어 스탯도 연동했습니다. playerController.cs playerActionAttack.cs DungeonPanelController.cs 에서 각각 수정했습니다. 공격 속도는 comboDuration만 줄여도 체감이돼서 해당 변수만 조정했습니다. 하트는 2개를 미리 추가하고 투명화 처리한 후에 강화 수치에 따라 보이도록 조정했습니다. 다른 수치는 playerController에서 조절하도록 했습니다. 지금은 단순한 계산식으로 레벨을 적용했습니다. 밸런스가 안맞거나 조금 이상한 부분이 있으면 말씀해주세요.
99jamin added 7 commits 2025-05-13 06:14:38 +00:00 
씬 통합 전까지 오류가 뜰 예정이라 일단 주석처리 했습니다.
99jamin added 1 commit 2025-05-13 08:19:46 +00:00
jay approved these changes 2025-05-13 08:42:48 +00:00
jay left a comment
Owner
Copy Link

고생하셨습니다! 확인하면서 몇가지 질문이 있어 남깁니다.

  1. 최대 체력 강화 시 게임을 아예 끝내고 다시 시작해도 그대로 유지되던데(4부터 시작하는 둥) 만약을 위해 최대 갯수 초과 시 처리 or 새 게임 시작 시 하트 초기화 있으면 좋을 것 같습니다(데모니까 초기화해도 상관 없을 것 같네요) 이미 있다면 무시해주세요!
  2. 공격 속도 강화 시 애니메이션 스피드도 같이 올라가고 있을까요? 전 사실 체감 잘 모르겠긴 하네요...
고생하셨습니다! 확인하면서 몇가지 질문이 있어 남깁니다. 1. 최대 체력 강화 시 게임을 아예 끝내고 다시 시작해도 그대로 유지되던데(4부터 시작하는 둥) 만약을 위해 최대 갯수 초과 시 처리 or 새 게임 시작 시 하트 초기화 있으면 좋을 것 같습니다(데모니까 초기화해도 상관 없을 것 같네요) 이미 있다면 무시해주세요! 2. 공격 속도 강화 시 애니메이션 스피드도 같이 올라가고 있을까요? 전 사실 체감 잘 모르겠긴 하네요...
Assets/KSH/DungeonPanelController.cs Outdated
@ -13,0 +14,4 @@
void Start()
{
//int level = UpgradeManager.Instance.upgradeStat.CurrentUpgradeLevel(StatType.Heart); // 1~3
jay commented 2025-05-13 08:35:05 +00:00
Owner
Copy Link

전체적으로 주석들은 남긴 이유 같이 위에 적어주시면 좋을 것 같습니다

전체적으로 주석들은 남긴 이유 같이 위에 적어주시면 좋을 것 같습니다
Sehyeon approved these changes 2025-05-13 08:50:14 +00:00
Sehyeon left a comment
Owner
Copy Link

수고하셨습니다. 브랜치 이동이 불가하여 코드만 보고 피드백 남깁니다!

수고하셨습니다. 브랜치 이동이 불가하여 코드만 보고 피드백 남깁니다!
Assets/KJM/KJM_Test/Save/TestScript.cs
@ -108,2 +106,4 @@
//health = this.health,
//reputation = this.reputation,
mealCount = this.mealCount,
houseworkCount = this.houseworkCount
Sehyeon commented 2025-05-13 08:45:56 +00:00
Owner
Copy Link

저희 집안일 제한은 현재 없는 상태라(figma 참고) 집안일 횟수 제한 관련 변수는 삭제하셔도 괜찮을 것 같습니다!

저희 집안일 제한은 현재 없는 상태라(figma 참고) 집안일 횟수 제한 관련 변수는 삭제하셔도 괜찮을 것 같습니다!
Sehyeon commented 2025-05-13 08:51:25 +00:00
Owner
Copy Link

또 궁금한게 저희가 엔딩이 나면 세이브는 어떻게 처리되나요? 모든 값 초기화인가요 아니면 저장 값이 유지되나요?
흠 물론 프로토타입이라 가볍게 생각해도 괜찮을 듯 합니다

또 궁금한게 저희가 엔딩이 나면 세이브는 어떻게 처리되나요? 모든 값 초기화인가요 아니면 저장 값이 유지되나요? 흠 물론 프로토타입이라 가볍게 생각해도 괜찮을 듯 합니다
99jamin added 1 commit 2025-05-13 12:14:47 +00:00
99jamin added 1 commit 2025-05-13 12:19:46 +00:00
99jamin added 1 commit 2025-05-13 12:29:23 +00:00
99jamin added 1 commit 2025-05-13 13:21:31 +00:00
99jamin closed this pull request 2025-05-13 16:39:00 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
jay
Sehyeon
Labels
No items
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Degulleo/Degulleo3D#46
No description provided.
Delete Branch "DEG-143-세이브-강화-병합"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?