[💬] 진짜 코드는 3일만 안봐도 새롭구나 + 금일의 작업

[地鬼(dczlatan)]

머리가 빠가인건가;;;

전투 시뮬레이터 몇일 안봤다고 무슨 오브젝트가 뭐했는지 찾아봐야됨 ㅠ.ㅠ

+ 금일의 작업.

미뤄놨던 news.cs (로그 찍는 클래스를 뉴스라고 표현했음) 의 census 메소드 삭제 및 옵저버 패턴 적용 완료했음.

기존에 Red팀, Blue팀 생존자를 엄청 이상한 방식으로 구했었는데, 그냥 onEventDead 메소드 콜 될 때, 어떤 팀인지 확인해서 팀원 수 줄이는 것으로 간단하게 계산함.

로그 찍을 때도 Blue팀인지 Red팀인지 확인하는걸 string news의 n번째 문자열이 R인지 B인지 확인하는 식으로 엉성하게 해놨는데

로그를 그낭 구조체로 만들어서 어떤 팀인지 linq로 쉽게 확인하는 방향으로 바꿨음. 전반적으로 코드가 깔끔해진 기분;;

https://github.com/emalron/simpleCombat

[라멛(175.214)]

나도 어제 만든 A* 다시보니까 새롭고 버그도 발견함 ㅋㅋ 원래 그런건가봄..

[地鬼(dczlatan)]

신비로움 ...

[귀미수탕(175.205)]

한줄한줄 전부 주석을 달면 새롭지 않게 됩니다!!

[地鬼(dczlatan)]

ㅎㄷㄷ

[익명(14.36)]

public필드들 private으로 바꾸고 getter, setter메서드 대신 프로퍼티를 써보는건 어떨까?

[익명(14.36)]

+ 만약에 clone메서드를 정의한 클래스를 상속해야 할 때, clone메서드도 오버라이드 해야하는데 이거 구현하기가 좀 귀찮을 때가 있을 거임

[익명(14.36)]

그럴때에는 복사 생성자를 써보자

[地鬼(dczlatan)]

복사생성자는 무엇임?

[익명(14.36)]

그 클래스 인스턴스를 인수로 받는 생성자임 그 생성자 안에서는 clone처럼 모든 값 복사해감

[익명(14.36)]

간단한 예시
https://blog.naver.com/alsehdgus13/221112208486

[익명(14.36)]

이게 장점이 뭐냐면 clone메서드는 객체생성, 대입 2가지 동작을 다 해야하는데 오버라이드 해야하면 이 작업이 매우 복잡해짐

[익명(14.36)]

근데 복사생성자는 외부에서 객채생성하고 그 생성자에서 대입 후 상위 클래스의 복사생성자 호출하면되니까 clone 메서드보다 코드 깔끔해짐

[地鬼(dczlatan)]

띠용? 헐 감사 ㅜㅜ

[익명(14.36)]

그냥 권장임 지금 코드에서는 clone메서드 오버라이드 할일이 없으니까 굳이 바로 적용할 필요 없고 해보면서 차이 알면 될듯

[地鬼(dczlatan)]

c# 레퍼런스를 넘기는건지 값을 넘기는건지 애매하게 느껴져서 항상 햇갈렸는데 복사생성자 개꿀띠