[📜일지] 거대한 클래스 뿌시는 중

혼자 개발하는데 너무 길어지고 어디 글쓸데도 없고 해서 새벽에 한번 똥한번 싸고감.

3D 3인칭겜 만드는데, 플레이어 컨트롤러 잘 만들기 너무 어렵다.

지난 2년간 3번정도의 대격변과 약 10차례의 리팩토링을 거쳐도 아직도 너무 어렵다.

고작 2000줄이 다른이들에게도 거대한지는 모르겠지만

누더기처럼 만들다보니 어느새 2000줄이 되었고,

내 머리로는 이제 더이상 기능추가나 수정이 불가능해짐.

더불어 가독성도 나락간지 오래.

unirx와 unitask를 공부하고난 직후에 새로 만든 코드여서 그런지

무분별하게 사용된 신기술(?)들로 인해 더욱더 좆같은 코드가 되었다.

아무튼 필요에 의한 리팩토링을 시작한지 어언 일주일...

사실 이전에도 리팩토링을 시도해보았으나 엄두가 안나서 포기했는데, 차라리 그 때 해결할걸 그랬다.

아무튼 지난 일주일동안에도 여러가지 시행착오가 있었다.

1. 여러개의 클래스로 역할을 분담해서 쪼개기.

이방법의 단점은 한줄요약하자면 코드의 양이 오히려 너무 많이 늘어나, 일이 많아지고 실수의 여지가 많아진다.

기존 클래스가 참조변수가 많고, 대부분의 코드에서 그 참조변수들을 사용하고 있다.

예를들어

void HandleAttack(), void HandleBackstep(), void HandleDie() 등 전부 플레이어 캐릭터의 동작을 위한 함수이고

대부분 animator, characterController와 기타등등 커스텀 클래스, 파라미터스크립터블오브젝트를 참조시켜야 하고,

참조시키기 위해 생성자라던지, 아니면 참조변수들만 묶어놓은 추가 클래스를 만든다던지 해야했는데, 다른방법은 아직 알지 못하겠다.

위의 문제는 사실 좀만 부지런하면 할 수 있는 방법인데 더욱 치명적인 문제가 있었다.

원래 클래스가 IPlayerController라는 인터페이스를 상속중이고, 이 인터페이스를 이용해서 유저인풋을 플레이어컨트롤러에 전달중인데

그렇다면 IPlayerController를 상속받은 클래스를 하나 더 만들고 그리고 새로만들 클래스들을 연결해줄 생각을 하니

이게 최선인가 싶었다. 그래서 하루정도 코딩하고 하루정도 고민하다가 결국 기각.

2. 그냥 기존 거대클래스 다시 잘 써보기.

1번의 삽질을 하다가 다시 잘 생각해보니

현재 코드에서 unirx를 이상하게 사용한 부분만 걷어내면 쓸만할 것 같았다.

unirx의 함수형 프로그래밍, 구독방식의 패턴은 매우 매력적이었다.

그러나. 나처럼 잘못쓰면 독이된다.

예를 들어. 캐릭터컨트롤러의 지면판정 이벤트를

subject가 있다. 이것은 기존 캐릭터컨트롤러의 isGrounded판정의 단점을 개선한 것이다.

GroundObserver.OnFeetOff, GroundObserver.OnLand 두개의 subject를 구독해서 캐릭터가 지면에서 떨어졋을 때,

지면에 닿았을 때 원하는 함수를 작동시킬 수 있다.

private void Awake(){

_groundObserver.OnFeetOff.Subscribe(_ => { ... });

_groundObserver.OnLand.Subscribe(_ => { ... });

}

음 꽤 깔끔하고 좋은것같지만, 잘 쓰면 좋겠지만 나는 이것의 가장 큰 문제점은

다른 로직들과 겹쳐졌을 때, 이게 단독으로 실행되는 경우 디버깅이 어려워지고,

코드가 2000줄이 될 때, 실행 로직이 순차적으로 읽히지 않는 것이었다.

그렇다면 이렇게 불필요하게 사용된 unirx를 제거하면 어떨까로 접근하기 시작했다.

그러나 문제는 어디서부터 제거할지 참 난감했다.

기존 코드에서 제거가 아니고, 기존 코드에서 핵심적인 부분만 살살 들어내서

새로운 코드를 다시 세워보기로 했다.

3. partial 클래스 도입

하지만 아무리 새로 쓴다고 해도 결국은 몇천줄짜리 코드가 되어버릴것이 분명하다.

그래서 방법을 찾아보니 partial 클래스로 하나의 클래스를 여러개의 파일로 분할이 가능하다고 한다.

그래서 하나의 클래스를 주제에 맞게 여러개의 파일로 분할했다.

예를들어,

PlayerController.cs - 단순히 클래스만 선언, monobehaviour와 각종 인터페이스만 상속

PlayerController.SerializedFields.cs - 인스펙터에서 보이고 싶은 변수들만 정의

PlayerController.Commands.cs - IPlayerController에 정의된 함수들만 정의

PlayerController.HandleAttacks.cs - 공격관련 동작들

PlayerController.HandleJump.cs - 점프관련 동작들..

..등

그리고 unirx로 구현햇던 기능들중 대다수는 비동기 unitask 함수들로 교체했다.

아까 예시로 들었던 지면 판정같은 경우는 대부분 로직 중간에

await UniTask.WaitUntil(() => _groundObserver.IsGrounded);

같이 유니태스크로 함수로 지면판정을 기다리게 하여

함수를 순서대로 읽으면 완성된 로직을 알 수 있도록 했다.

이 과정에는

https://qiita.com/toRisouP/items/af7d32846ab99f493d92

이 글을 많이 참조했다. 국내 번역글도 있었던거 같은데 요즘 브라우저는 번역 다 잘되니 원글을 첨부함.

코루틴쓰다가 유니태스크 쓰니까 너무 좋다.

참고로 dotween도 유니태스크 지원이 되니 아직 안써본 인붕이들은 써보도록 하자.

아무튼 이제 리팩토링의 70프로정도 진행된 것 같다.

근데 게임은 도대체 언제 완성할 수 있는걸까.

[📜일지] 거대한 클래스 뿌시는 중

댓글 6

다른 게시글