[🐣질문] 다른 코드도 봐주실수있나요?

[Lucete(syshim0408)]

우선 이 코드를 짜게 된 스토리는 이렇습니다.

액션게임을 만들어 보고 싶은데 처음에는 그냥 if문을 막 써서 짜다가 예외상황이 너무 많이 발생하고

구조가 이상해지는게 느껴져서 방법을 아예 바꾸고 이런저런것들을 참고해서 지금의 코드를 만들었습니다.

우선 ActorController 스크립트의 목적은 모든 입력을 받고 처리하는 곳으로 지정했습니다.

그리고 그 입력값들을 Actor로 넘겨주었습니다.

그래서 Actor 스크립트는 오직 ActorController에서 입력값을 받는 역할만 합니다.

그리고 CharacterManager에서는 Actor를 상속받고 Actor에서 받은 값들을 다시 BaseAction으로 넘겨주는 역할을 합니다.

그러면 BaseAction에서 그 값들을 받고 실제 동작을 할 Idle 스크립트나 Run 스크립트 아니면 Attack 스크립트에 최종적으로 넘겨줍니다.

그리고 아래는 BaseAction 코드와 CharacterManager 코드입니다.

public class CharacterManager : Actor

{

BaseAction _nowAction; (추상클래스라서 new로도 안되고 null을 집어넣어도 왜 초록색이 안사라질까요 ㅠ)

public override void MovingCheck(Vector3 moveDir) =>

_nowAction.CheckMoveDir(moveDir);

public override void OnNormalAtk(ActorController.InputKey key, ActorController.InputState state) =>

_nowAction.CheckInputKeys(key, state);

public override void OnDodge(ActorController.InputKey key, ActorController.InputState state) =>

_nowAction.CheckInputKeys(key, state);

public void ChangeAction(string name) => _nowAction.ChangeAction(name);

void Update()

{

}

}

밑에는 BaseAction 코드입니다

public abstract class BaseAction : MonoBehaviour

{

[Header("Managers")]

[SerializeField] protected CharacterManager _owner;

[SerializeField] protected Animator _animator;

public virtual void StartAction() { }

public abstract void UpdateAction();

public virtual void EndAction() { }

public abstract void ChangeAction(string name);

public virtual void CheckInputKeys(ActorController.InputKey key, ActorController.InputState state) { }

public virtual void CheckMoveDir(Vector3 dir) { }

void Start()

{

}

void Update()

{

}

}

그리고 최종적으로 값을 받아서 실행되는 스크립트 입니다 (예를들어 Idle)

public class Idle : BaseAction

{

public override void CheckInputKeys(ActorController.InputKey key, ActorController.InputState state)

{

if (key == ActorController.InputKey.NORMALATK && state == ActorController.InputState.PRESSED)

_owner.ChangeAction("Atk");

if (key == ActorController.InputKey.DODGE && state == ActorController.InputState.PRESSED)

_owner.ChangeAction("Dodge");

}

public override void CheckMoveDir(Vector3 dir)

{

if (dir != Vector3.zero)

_owner.ChangeAction("Moving");

}

public override void UpdateAction()

{

}

public override void ChangeAction(string name)

{

_animator.SetBool(name, true);

}

void Start()

{

}

void Update()

{

}

}

코드가 수정해야 할 것이 있거나 하면 지적좀 해주세요. 하다가 복잡해서 멘탈이 많이 나가기도 했고 이상한 오류도 많을것 같아요 ㅠㅠ

[Kaonuri(hon454)]

질문글에 답변자를 특정짓지는 말아줘

[Lucete(syshim0408)]

아 네 죄송합니다 수정할게요 ㅠㅠ

[Kaonuri(hon454)]

1. CharacterManager라는 이름은 부적절.

[Lucete(syshim0408)]

넵 수정할게요

[Kaonuri(hon454)]

2. BaseAction은 추상클래스이기 때문에 인스턴스 생성이 불가능. [serializeField]를 통해 에디터에서 해당 클래스를 상속받은 컴포넌트를 할당시켜줘야함.

[Lucete(syshim0408)]

네 그렇게도 해봤는데요. 근데 지금 제가 만든 스크립트에서는 실제로 BaseAction을 상속받은 애들이 다 Idle,Run,Attack 이런애들이라서요. 그냥 BaseAction 상속받은 스크립트 아무거나 만들어서 그거 넣을까요? 아니면 Idle을 넣을까요??

[Kaonuri(hon454)]

3. 구조 자체가 Command패턴을 활용하고 있어서 아예 코드 단에서 객체를 생성하거나 ScriptableObject를 활용하는게 더 유리한 방식인데 이상하게 만들고 있음

[Lucete(syshim0408)]

한번 다시 알아보겠습니다 감사합니다

[Kaonuri(hon454)]

더 알려주기에는 졸리니 자세한 설명은 생략한다

[Kaonuri(hon454)]

내가 보기에는 전형적인 디자인패턴충(추상화충)의 모습을 보이는 것 같다. 일단 추상화가 덜 되더라도 돌아가는 코드를 작성하고 나중에 단계적으로 추상화하셈

[Kaonuri(hon454)]

일단 어떻게든 이런방식을 쓰고싶다면 FSM을 먼저 알아봐.

[Lucete(syshim0408)]

네 알겠습니다!

[한글닉이최고(hslol00)]

BaseAction이 추상클래스라서 인스턴스 생성 불가능...인게 문제가 아니라 BaseAction은 모노비헤이비어를 상속받는애니까 new로 인스턴스 생성이 안되는게 문제임.. GetComponent해서 넣어주시면 됨

[한글닉이최고(hslol00)]

아니면 님 보니까 BaseAction하고 Idle하고 다 start update 안쓰는데 모노비헤이비어 상속 안받아도 되지않음? 그러면 new로 됨