Skip to content

Conversation

@snughnu
Copy link
Collaborator

@snughnu snughnu commented Feb 2, 2026

연관된 이슈

작업 내용 및 고민 내용

1. SkillSystem에 함수 2개 추가

  • calculateCurrentTotalGold(GameType): 게임의 현재 액션당 총 골드 획득량을 계산하는 함수입니다.
  • calculateTotalGoldAfterUpgrade(Skill): 특정 스킬 업그레이드 후 해당 게임의 총 골드 획득량을 계산하는 함수입니다.

2. SkillViewItemRow description을 수정

  • Equipment 처럼 "현재 -> 업그레이드 후" 형식으로 표시했습니다.
  • 같은 게임의 모든 티어(초급/중급/고급) 스킬이 합산되어 표시됩니다.

스크린샷

2026-02-03.00.36.46.mov

시뮬레이터에서 13mini가 고장나서 iPhoneSE로 화면을 확인해봤습니다.

멘트 최대 길이 예시 짧은 길이 예시
골드 획득: image image
골드 획득량: image image
레벨업시 액션당 골드 획득량 image image

리뷰 요구사항

  • "초급"을 레벨업할 때, "중급", "고급"의 description도 변하게 되는데, 적절한지 확인 부탁드립니다!!
  • description 멘트를 잘 못정하겠어요 😓
    현재 description을 "골드 획득: \(currentTotal.formatted()) -> \(afterTotal.formatted())"으로 해놨는데 더 적절한게 있다면 추천 부탁드립니다..!
    (의견 있으시다면 "이 멘트 확인해주세요"라고 부담없이 남겨주세요!)

@snughnu snughnu self-assigned this Feb 2, 2026
@snughnu snughnu added Fix 버그 수정 작업 UI 화면에 보이는 요소들을 개발합니다. labels Feb 2, 2026
Copy link
Collaborator

@tomchoi95 tomchoi95 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생 많으셨습니다 올리!
diff가 충분히 직관적이라서 너무 좋은 것 같습니다 👍 👍

현재 단위 포메팅이 적용이 되지 않는 것 같아, 함수가 아닌 extension의 formatted를 사용하면 좋을 것 같아요!

Copy link
Collaborator

@dongglehada dongglehada left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고 많으셨습니다 :>

설명 문구는 개인적으로도 잘 작성되었다고 느꼈어요.
제목에서 어떤 업무인지 명확하게 드러나 있어서, 현재 설명만으로도 충분히 잘 전달되는 것 같습니다!

@snughnu
Copy link
Collaborator Author

snughnu commented Feb 3, 2026

ScreenRecording_02-03-2026.13-57-53_1.mp4

단위 포맷 적용했습니다 🫡
(근데, 반올림 때문에 변화가 없는게 조금 이상하네요)

@tomchoi95
Copy link
Collaborator

밸런스 패치 진행하면서 다루는 골드 범위를 좀 넓힌다면 바뀔 수도 있을 것 같긴한데...
반올린 때문에 변화가 없는 건 좀 더 생각을 해봐야 할 수도 있겠네요...

Copy link
Collaborator

@sunjae0902 sunjae0902 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P3: 저는 합산하는 방식이 오히려 이전보다 헷갈리는 느낌이 들긴합니다..!
itemRow는 분리되어 있는데 하나를 눌렀을 때 다른 것도 같이 변하면 혼란스러울 것 같다는 생각이 들었어요ㅜㅜ

@snughnu
Copy link
Collaborator Author

snughnu commented Feb 3, 2026

image

이렇게 변경했습니다 🫡

@snughnu snughnu merged commit dcdcf09 into dev Feb 4, 2026
1 check passed
@snughnu snughnu deleted the fix/#245-skill-description branch February 4, 2026 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fix 버그 수정 작업 UI 화면에 보이는 요소들을 개발합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

스킬 설명을 변경합니다.

4 participants