[🐣질문] 초보) 구현은되는데 이렇게하는게 맞나요?

[익명(qqqosc)]

C#처음만져보고 유니티도 처음해봅니다. 클래스? 객체? 인스펙터? 솔직히 아직도 저걸 설명하라하면 못할거같은데 느낌만 아는?

일주일정도 아무생각없이 유튜브 코드 따라치다가 너무 이해가 안되고 잘 안느는것 같아서 안보고 최대한 아는 지식 활용해서 만들어봤습니다.

UI활성화/비활성화를 만들었는데 코드를 짜면서 뭔가 이상하다고 느껴져서 한번 질문합니다

원하는 기능은

1. 활성화되어 있는 창이 있을때, Esc를 누르면 현재 띄워져 있는 창(가장 마지막으로 띄운)을 닫는다.

2. 활성화되어 있는 창이 없다면, 해당하는 키의 창을 띄운다.(현재 E, Esc)

그래서 활성화 비활성화만 담당하는 클래스, 해당하는 ui기능을 구현할 클래스를 만들었습니다.

현재 설정창클래스, 인벤토리창클래스, ui컨트롤러클래스

세개를 만들었습니다.

먼저 설정창 클래스입니다. 원래는 SetActive(false,true)로 했었는데 슬라이드되면서 들어오게 하고 싶어서 좀 찾아보니 Dotween이란게 있어서 사용했습니다.

활성화 비활성화 형식이 아니라 일부로 캔버스 바깥에 설치한뒤 DOAnchorPos로 화면내부로 들어왔다 나갔다할 수 있도록 했습니다.

인벤토리 클래스도 벡터의 값하고 timescale이 없다는 것만 뺴면 동일합니다.

시리얼라이즈필드로 선언해서 인스펙터에서 직접 해당하는 오브젝트를 할당했습니다.

컨트롤러는 스택을 이용해 가장 맨 위에 있는 UI를 찾아서 Esc가 눌리면 해당하는 창을 닫도록 구현을 했습니다.

같은 키를 누를때도 현재 창이 스택에 있는지 확인하고 껏다 켰다 할 수 있게 했습니다.

원하는대로 작동하긴 하는데 이런식으로 코드를 짜는게 맞나요?

C언어는 정말 조금 해봤는데 너무 헷갈립니다.

요약

내 나름대로 생각을 이어가면서 코드를 작성했는데,

지금 코드를 짜는 방식이 C#에 적절한가요? 혹시 어떤 생각을 가지고 접근해야할지 팁좀 주실 수 있나요?

나중에 UISetting하고 Inventory에는 이름에 맞는 기능을 추가로 붙일 예정입니다.

[익명(14.32)]

나라면 각 UI에 해당하는 클래스를 공통 부모 클래스로 묶은 담에 openUIStack을 애초에 그 부모 클래스로 컨테이닝해서 거기서 Hide를 했을듯

[익명(14.32)]

class MyUIBase : public MonoBehaviour{ public void Hide() {}}class UISetting : pubilc MyUIBase{ public void Hide() { ~~~~ }}Stack openUIStack;이런식?

[익명(qqqosc)]

그럼 컨트롤러에서 스택을 관리하는게 아니라 MyUIBase에서만 스택을 관리하고 컨트롤러에서는 키입력받고 함수실행만 담당하는건가요?

[익명(14.32)]

그런거는 나도 모르겠고 그냥 GameObject로 관리하는 것보다는 보다 구체적인 형상으로 관리하는게 편할거 같아서 .. 뭣보다 이렇게 하면 currentUI가 inventory인지 setting인지 신경 안써도 되잖음.. 구분없이 그냥 그걸로 Hide하면 되니까

[익명(qqqosc)]

이상하다고 느낀 부분을 어떻게 바꿔야할지 뭔가 알거같네요 감사합니다

[태보미(taebomi)]

윗댓처럼 해주고 pop 지우고 그냥 peek을 pop으로 해줘도 무방할것 같은거에요.

[익명(qqqosc)]

저렇게 수정하면 확인할 필요가 없어져서겠죠? 감사합니다.