[📚정보] 유니티 C# 권장 코딩 컨벤션

[익명(125.132)]

코드 스타일에 대한 글이 종종 올라와서 써봄

제목은 좀 어그로고 그냥 내가 개인적으로 좋다고 생각하는 방법임

현재 어느 팀에서 리드 클라로 일하고있고 우리팀에 적용, 권장하고 있는 컨벤션임

https://vmsdurano.com/c-coding-guidelines-and-practices/?

별건 아니고 그냥 위 링크대로다

대충 간결하고 짧은 문법이 있으면 그걸 쓰자는 얘기임 대체로

대략 요약하면

1. ?: (삼항연산자)를 쓸수 있으면 if-else 대신 쓴다

2. 중괄호 없는 한줄짜리 블럭 안쓴다

3. switch expression 적극 사용

a = b switch

{

  1 => "1",

  2 => "2",

  _ => 대충 예외 던짐

}

4. using var는 IDispoable / IAsyncDisposable 구현한 애들 사용할땐 무조건 쓴다

5. 문자열 포맷팅이나 합성 등은 정말 다른 방법이 필요한 경우가 아니면 string interpolation 방식으로 통일

$"A: {a} / B: {b} 1+2 = {1+2}"

6. 한줄짜리 메소드나 프로퍼티는 식 본문 스타일로 씀

void Foo() => 1;

float GetPositionX => gameObject.transform.position.x;

7. 클래스 생성과 함께 초기값을 넣을 필요가 있을땐

new Foo() { x = 1, y = 2, z = 3 }

8. 값 두개, 세개 리턴할때는 튜플 사용

그거 외에도 두개, 세개씩 묶여서 행동해야 하는데 따로 클래스나 구조체로 만들만큼 여러곳에서 쓰지 않고 로컬에서 쓰는 경우에도 튜플 적극 사용

9. 

클래스 이름, public 메소드, 프로퍼티는 PascalCase로 

로컬 변수 및 매개변수 등은 camelCase로

private 필드는 _camelCase로

(2021년에 헝가리안 쓰는 사람 아직도 종종있던데 극혐)

10.

이름지을때 단어 사이사이에 _ 붙이지 않기

11. 

메소드, 필드, 프로퍼티 프라이빗끼리 퍼블릭끼리 모아둘것

그리고 링크에 없는것들인데 개인적으로 정한것들은

1. float 뒤에 f 꼭 붙이기

2. 이름지을때 동일한 문구가 반복되는걸 피하기

예)

class Monster

  enum MonsterType

    MonsterTypeUndead, MonsterTypeDemon

이러면 Monster.MonsterType.MonsterTypeUndead 이런식으로 접근해야하는데 대신 Monster.Type.Undead 이런식으로 이름짓기

위의 예는 일부러 만든 극단적인거지만 정도는 약해도 저런느낌으로 만드는경우 널림

3. interface 구현은 가급적 암시적대신 명시적으로 

interface를 public으로 구현해서 다른데서 맘대로 호출하는것 자체가 어딘가 설계에 문제가 있을확률이 높다고 봄

4.

자주 사용될 것 같은 메소드의 경우 XML태그 주석 꼭 붙이자 

https://docs.microsoft.com/ko-kr/dotnet/csharp/language-reference/xmldoc/recommended-tags 참조

인텔리센스에 코멘트 한줄 뜨는게 엄청 편함

링크에서 제시한것들중에 채택하지 않은것

* extension method 적극 활용하기

이거 라이브러리 만드는 입장에선 좋을지도 모르겠는데 클라 로직에 extension method 남용하면 전형적인 '나만아는코드' 될확률이 높다고 봐서 거름

* null propagation 적극 활용

이거 진짜 좋은데.. 이거는 아직 고민중임. 편하긴 한데 굳이 쓰지 말라는거 쓰기도 좀 그렇고

UnityObject 가 아닌애들에 한해서 허용한다고 하면 사람들이 100% 헷갈려하고

가급적 오브젝트들이 직접 레퍼런스를 들고 있지 않고 이벤트로 통신하는 식으로 시스템을 설계해서 널체크의 필요성 자체를 줄이려고 노력 중.

is null 도 마찬가지 문제가 있음. 사람마다 null = A랑 A = null 혼용하는거 통일하고 싶은데 부작용이 걱정됨

또 뭐가있더라.. 일단 생각나는건 이정도인듯

 

[잠둥이(fcrow)]

근데 이름지을때 단어 사이사이에 _ 붙이지 않기 <= 이건 왜 그런거야?? 갑자기 궁금...성능에 문제라도?

[익명(125.132)]

걍 닷넷 프레임웍에서 쓰는 이름규칙이랑 유사하게 맞춘거임 닷넷에서 _ 안쓰잖아

[리리(121.165)]

변수명은 성능에 전혀 영향을 끼치지 않음.

[지구정복_현재진행형(asdf060708)]

오 ㄳ

[아이리쨩다이쇼리(soos2075)]

10번 개많이쓰는데 쓰다보니 보기안좋긴하더라

[스시스왑(pengsuo)]

11번 너무 귀찮아 자동으로 포맷팅되는 기능 업나

[익명(125.132)]

라이더 플러그인중에 있다고는 들었음. vs나 vscode에서도 있는지는 모르겠네 우린 그냥 vs community로 통일해서..

[익명(usnned)]

와 나랑 거의 똑같네

[익명(usnned)]

마지막은 유니티 오브젝트를 파괴하는 타이밍만 잘 컨트롤하면 큰 문제 없다고 봐서 저는 쓰는 편입니다

[익명(175.223)]

나도 내 개인작업에선 마음껏 쓰는 편인데 난 사람을 못믿어서 ㅋㅋㅋ 아직도 고민중임.

[익명(usnned)]

인스펙터에 노출시킬 변수 public말고 [SerializeField] private로 하는 내용도 추가하면 좋을거 같아요

[익명(125.132)]

아 맞다 우리도 그부분은 그렇게 하고 있음 ㅋㅋ 다들 생각하는게 비슷한듯

[쭈구리야(zzu9riya)]

[익명(114.202)]

2번맨날 한줄짜리 해놓고 결국 추가하다보면 중괄호 쓰게됨

[익명(157.112)]

null도 null propagation도 제발좀 쓰지마 익스텐션 많이씀 나머지는 비슷함

[익명(223.39)]

2번은 뭔소리임? 한줄짜리도 중괄호 쓰란거임?

[익명(125.132)]

ㅇㅇ

[익명(27.126)]

조건에 && || 존나붙어도 삼항씀?

[로망바드(kong1659)]

난 95%는 안 지키고 있었네... 나중에 따로 공부 좀 해놔야겠다. ㅊㅊ

[익명(49.161)]

[익명(118.37)]

10번은 적으면 괜찮은데 많아지면 역효과더라

[익명(221.153)]

무조건 개추

[익명(14.10)]

좆디겜 병신 백수새끼들이 코딩컨벤션 신경쓰기나 하냐

[익명(14.10)]

평생 혼자 자기만족딸딸이나 할껀데 신경쓰지말고 좆대로 해 ㅋㅋㅋ

[르마스(118.39)]

이거 개병신 제안임... 코드 짧게 쓰는 방법에 대한 제안일 뿐이지 이걸 중소기업이상 팀프로젝트 협업 코딩 스탠다드로 박는곳은 피해라 ;;

[익명(182.213)]

이거 내얘끼네 방구석 코딩