[💬] (클린코드) Enemy 스크립트가 필요한가 ? 궁금점

[익명(14.36)]

구성 패턴을 사용 할시 Enemy 객체에

- CharacterMovementm.cs
- Health.cs
- CharacterRotate.cs

등등만 있으면 될것 같은데 Enemy 스크립트를 만들어야 하는 이유가 있을까요 ?

저는 만들지 않아도 된다고 생각하는데 유튜브 강의 영상이나 유데미 영상을 보면 Enemy 스크립트를 만드는 영상이 많더라구요 이해가 안가는데 여기서 또 궁금한 것은

만약 Health 컴포넌트를 재사용하기 위해 Health 스크립트안에 특정 상황 발생시 특정 이벤트를 Invoke 해주는데, 이제 이벤트 리스너는 어떤 스크립트를 만들어 체인을 연결해줘야 하는지 의문입니다.

즉 공격을 받았을시 특정 행동을 한다라고 가정할시
Health.cs 안에서 Invoke 후 특정 행동을 하는 이벤트 리스너 스크립트를 만들어서 체인을 연결해주면 되는걸까요 ?

위가 틀리다면 이래서 Enemy 스크립트가 필요한가 ? 라는 생각이 듭니다 도움 주시면 감사하겠습니다.

[익명(218.147)]

Enemy와 Health가 독립적이어야 재사용되지. Health는 Enemy를 몰라도 됨 - dc App

[익명(14.36)]

Enemy와 Health 서로 이해 관계를 분리 시켰습니다. 그래도 Enemy 스크립트가 필요한 경우가 있는게 타당할까요 ?

[익명(14.36)]

중복 되는 코드는 캡슐화 하여 코드를 분리하고 SOLID를 지키면서 코딩을 하고 있는데 유튜브를 보면 굳이 Enemy 스크립트를 사용하는 이유가 궁금해서 이렇게 질문글을 남깁니다 ㅠㅠ

[익명(218.147)]

Enemy를 안쓰면 Health 에서 일어나는 이벤트를 어디서 처리할거임? Health파일에선 이벤트 invoke만 해라 - dc App

[익명(14.36)]

만약 적이 공격을 받을시 회전하라는 이벤트가 필요하다고 가정했을때 Health.cs 안 DealDamage 함수 안에 이벤트OnDeal?.Invoke 해주고회전하라는 이벤트 리스너는 Enemy 스크립트가 아닌 CharacterRotate.cs 스크립트를 만든 후 회전하라는 특정 로직을 작성하고 Health.cs 안 이벤트 OnDeal에 체인을 연결해주면 되지 않을까요 ?

[익명(14.36)]

만약 그렇게 하지 않고 Enemy 스크립트 만들고 특정 코드 로직(회전하라)을 작성하면 여러가지 특수한 상횡이 필요하거나 발생할시 Enemy 스클비트 안에 적어야 한다는 단점이 생기는거 같은데 예를 들어 이벤트를 받는 이벤트 리스너 라던지 또는 함수 매개변수에 In/Out 하는 코드를 Enemy 스크립트에 작성함으로써 중앙 집중화 현상이 발생하여 SRP에 어긋나고 스파게티 코드가 발생하는 것 같아 이렇게 또 댓글을 남깁니다

[익명(218.147)]

Controller개념이 필요하면 중간에 만들어도됨. 거기서 이벤트 처리하고 Controller는 Enemy, Health둘 다 알게 하고 - dc App

[익명(14.36)]

연달아서 좋은 답글 감사합니다. 혹시 Player에서도 Controller을 재사용 할수 있다고 가정했을때 말씀하신 것처럼 Controller 스크립트 안에서 Enemy 종속성 부여 되어 있을때 특정에 의존하고 있어 캡슐화 되지 않고 PlayerController 스크립트도 만들어야 해서 Dry 되지 않음으로써 문제가 발생하는 것 같은데

[익명(14.36)]

이럴때는 어떻게 해결하는게 좋을까요

[익명(218.147)]

그럴땐 Enemy랑 Player를 같은 인터페이스로 묶고 Controller안에서는 인터페이스로 받으면 됨

[익명(218.147)]

public class Player : IUnit {} public class Enemy : IUnit {} public class Controller { private IUnit _unit; }

[익명(218.147)]

이렇게 하면 _unit 필드에 Player랑 Enemy다 담을 수 있음.

[익명(14.36)]

적만의 특수한 기능이 있을때는 Enemy 스크립트에 작성해도 무방하고 만약 작성한 것중 이게 다른 곳에(ex Player) 재사용될 필요가 있다면 컴포넌트로 분리해서 사용해도 된다는 말씀으로 이해했습니다 ! 또한 필요한 경우 인터페이스를 사용하여 특정에 의존하지 못하게 하나의 인터페이스로 묶어 사용하면 된다라는 것을 배웠습니다

[익명(14.36)]

시간 내서 좋은 답글 남겨주서서 감사합니다 덕분에 또 많이 배웠습니다. 오늘도 좋은 하루 되세요 !!!

[익명(218.147)]

예아

[익명(183.107)]

적과 아군이 있고 각각 필요한 코드가 따로 있으면 만드는게 좋지

[익명(14.36)]

만약 특정 행동이 발생했을시 회전하라가 필요할시 왜 CharacterRotate 스크립트로 만들지 않고왜 Enemy 스크립크를 만든 후 스크립트 안에서 특정 행동을 하는 로직을 작성하는 이유가 뭘까요

[익명(211.219)]

적 캐릭터가 회전할때는 다른 추가의 처리가 필요할수 있으니 미리 나눠두는거잖어

[익명(14.36)]

CharacterRotate는 다른 것에 신경쓰지 않고 그저 회전만한다. Enemy 안에는 적 캐릭터가 회전할때 적만의 추가 처리 로직을 작성한 후 CharactetRotate 참조를 얻어 Enemy에서 사용하면 된다는 뜻인가요 ?

[익명(211.219)]

아군과 적군을 하나의 파일로 만들면 뭔가 다른 처리가 필요할때마다. 이게 적군캐릭인지 아군캐릭인지 확인하는 코드가 필요하고 분기 처리를 해야하고 그런부분이 많이 발생할껄 아니까 미리 나눠둔 거라고 보면될듯?

[익명(14.36)]

오 그렇게도 생각할수 있겠네요 또 하나 배워갑니다 !! 좋은 댓글 감사합니다 !!

[익명(121.136)]

걍 상황에 맞춰서 만드는 거지 적한테 한번만 쓰일 코드들 굳이 캡슐화 해서 분리해야하나 네가 질문하는게 "왜 게으르 게 코딩하세요?" 인데 게으른데 이유가 있나? 네 의도나 다른 뜻이 그렇다는 게 아니고 걍 질문 자체가 파고들면 저럼 작은 프로젝트 진행할 때 생산성 문제만 없으면 그냥 마음대로 하셈

[익명(121.136)]

여기서 아무리 더 질문해봤자 네가 원하는 답도 안나올 것이고 나와봤자 아무 의미 없을 거임