[💬] 유니티 초보 코드짠거 코드리뷰좀 부탁함

[Indie(116.40)]

RPG게임의 플레이 가능한 캐릭터의 현재스텟 변동, 장비 착용후 스텟변동을 구현한건데

코드가 많이 지저분해보이는데 좀더 개선할 방법이 생각이 안남

좀더 간략화할수있는 방법 피드백좀 부탁함..

캐릭터의 능력치를 구현한 데이터는 스크립트가능오브젝트(SO)를 받아와 적용하는데 공용능력치라 상속의 부모스크립트가 담당하고

여기는 플레이어블 고유 캐릭터의 능력치만 담당하고있어

장비를 교체하거나 버프등 능력치가 변동되는 행동을 하게되면 현재스텟계산을 호출해서  현재의 스텟을 재계산하게 했음

장비가 변경되면 호출하여 능력치와 프리펩을 설정하게 함

손에드는 무기와 방패는 실제로 표현되기때문에 프리팹과 애니메이션을 적용하는 함수임

장비를 착용가능한지 스텟의 요구량을 체크하는 함수임

설계할때는 간단해보였는데 능력치가 많다보니까 코드가 늘어지게되었는데 딕셔너리를 활용하면 어떻게든 반복문으로 처리할수있을거같은데

머리속에서 생각이 안남...

좀더 간략화할수있도록 인붕이들의 피드백이 필요함....

[Indie 1(210.217)]

함수안에서 함수선언해서 쓰네. 멤버변수를 배열로 변환해서 사용할수도 있구나 형변환은 as 로 하는구나.. 오..

[익명(211.202)]

데이터와 Modifier를 분리해봐 스텟이라는 데이터는 단순히 데이터일 뿐임 모든것을 하나의 객체에 몰아넣으면 가독성도 떨어지고 확장성도 떨어질 수 밖에 없음

[익명(211.202)]

너는 하나의 클래스에서 모든 스텟을 정의하고 재계산하는데 당연히 힘들지. 스텟이 추가될 때, 계산 방식이 바뀌어야 할 때, 계산식이 추가될 때마다 이 클래스를 수정해야되고 하나하나 예외처리 해줘야함 장비로 인한 스텟 증가, 버프로 인한 스텟 증가도 단순히 Modifier일뿐이야 스텟 데이터의 값은 변경될 필요가 없어. 니가 원할 때 Modifier를 컨트롤해주는 클래스를 호출해서 값을 계산해주기만 하면 돼. 코드 자체도 비효율적인 부분이 많지만 지금은 설계 자체가 클래스가 복잡해지고 가독성이 떨어질 수 밖에 없음

[익명(211.202)]

데이터와 Modifier객체를 분리하면 어떻게 될까? 지금처럼 이 클래스에서 모든 계산 방식에 대해 재계산 해줄 필요가 없어. 스텟이라는 클래스가 EquipmentItem을 참조할 필요도 없음 어떤 버프같은게 들어와도 이 버프 타입을 알 필요도 없음. Item은 단순히 아이템의 능력치에 맞는 Modifier만을 가지고 있고, Modifier적용은 ModifierSystem을 만들어서 적용하면 되는거임 ModifierSystem을 잘 설계해서 계산 그룹을 적절히 나누고 순서를 잘 지정해주면 어떤 상황에도 대부분 대응 가능한 확장성있는 클래스로 재사용할수 있어

[Kh0717(i0792374)]

@ㅇㅇ(211.202) 그러면 나중에 데미지계산같은 현재스텟이 필요한 항목도 초기값 + 모디파이값 묶어서 주면 된다는거네 한번 분리해서 설계해볼께 ㄱㅅㄱㅅ - dc App

[익명(211.202)]

@Kh0717 맞아 핵심은 원본 데이터와 로직을 분리하는거야. 유튜브에 git-amend라는 사람 Stat Modifier 영상이 있는데 그거 보면 감좀 잡힐걸? 물론 그 코드도 함정이 있긴 하지만 왜 이렇게 구성하는지, 이렇게 구성해서 어떤 식으로 사용하는지 이해갈거야