[🐣질문] 유니티 이런건 변수 만들어서 관리해야해요?

[익명(222.234)]

코드는 이런식인데:

if (newContainer.GetComponent<UnitContainer>().IsContainUnit())

    newContainer.GetComponent<UnitContainer>().GetContainingUnit().gameObject.GetComponent<CombatUnitMovement>().AssignUnitTo(_container);

newContainer.GetComponent<UnitContainer>().SetContainingUnit(gameObject);

목적지(newContainer)에 이미 들어있는 유닛을 내가 원래 있던 자리(_container)로 보내고 내가 newContainer로 이동하는 코든디
나중에 보면 어지러울 것 같은데, 이런건 어떻게 관리하면 돼요?

딱히 재활용을 하는 건 아니라 따로 변수 만들어서 관리하자니 배보다 배꼽이 커지는 것 같기도 하고요.

[익명(125.182)]

저라면 3번 반복되는 UnitContainer container = newContainer.GetComponent(); 이정도? 지역변수로 잠깐 쓸듯하네요

[익명(222.234)]

뉴비라 나중에 코드가 어캐될지 모르니 정리 해둬야겠네요. 감사합니다.

[익명(119.194)]

위에 사람 말대로 하는데, 궁금한게 if문 안에 저건 구조가 왜 저렇게 되어버린거임.. GetComponent.Get머시기.GetComponent 구조가 에바임..

[익명(222.234)]

유닛 컨테이너는 유닛을 하나씩 보관하는 오브젝트고, 그 유닛 컨테이너안에 보관된 유닛 오브젝트의 CombatUnitMovement 클래스에 접근해서 클래스 안의 메서드로 지정한 유닛 컨테이너로 보내는게 목적이라 UnitContainer오브젝트 -> UnitContainer클래스 -> CombatUnit오브젝트 -> CombatUnitMovement클래스 순으로 접근해서 AssignUnitTo 메서드를 호출하려고 했어요. 멤버 변수를 퍼블릭으로 바꾸면 클래스 캡슐화가 깨지기 때문에 나중에 유지보수가 힘들 것 같다고 생각해서 길어졌네요. 그냥 캡슐화를 깨고 Public 변수로 활용하는게 일반적인가요?

[익명(119.194)]

그렇게할거면 오브젝트->클래스->오브젝트->클래스 순으로 접근하지 말고 차라리 클래스->클래스 순으로 접근하던가 하는게 나아보이는데.. 근데 나는 처음부터 이론상 최대한 좋은 구조로만 만드는 것보다 어떻게든 완성하는 것이 중요하다고 생각하는 사람이라 알아서 생각하셈