POCU 아카데미용 C# 코딩 표준

마지막 수정일: 2021-03-31

본 코딩 표준은 Pope Kim의 C# 코딩 표준POCU 아카데미에 적합하게 수정한 것입니다. 취소선(취소선) 표시가 된 항목들은 무시해주세요.

머리말

기본 원칙

  1. 가독성을 최우선으로 삼는다. (대부분의 경우 코드 그 자체가 문서의 역할을 해야 함)
  2. 정말 합당한 이유가 있지 않는 한, 통합개발환경(IDE)의 자동 서식을 따른다. (비주얼 스튜디오의 “Ctrl + K + D” 기능)
  3. 본 코딩표준을 따라 잘 짜여진 기존의 코드에서 배운다.

참조문서

이 코딩 표준은 아래의 코딩 표준들에서 영감을 얻었음.

IDE 도우미

비주얼 스튜디오에 import할 수 있는 세팅은 여기서 다운받을 수 있습니다.

POCU 아카데미에 비적용 사항

  • 있을 경우 추후 추가 계획

I. 메인 코딩 표준

  1. 클래스와 구조체의 이름은 파스칼 표기법을 따른다.

    class PlayerManager;
    struct PlayerData;
    
  2. 지역 변수 그리고 함수의 매개 변수의 이름은 카멜 표기법을 따른다.

    public void SomeMethod(int someParameter)
    {
        int someNumber;
        int id;
    }
    
  3. 메서드 이름은 기본적으로 동사(명령형)+명사(목적어)의 형태로 짓는다.

    public uint GetAge()
    {
        // 함수 구현부...
    }
    
  4. 단, 단순히 부울(boolean) 상태를 반환하는 메서드의 동사 부분은 최대한 Is, Can, Has, Should를 사용하되 그러는 것이 부자연스러울 경우에는 상태를 나타내는 다른 3인칭 단수형 동사를 사용한다.

    public bool IsAlive(Person person);
    public bool HasChild(Person person);
    public bool CanAccept(Person person);
    public bool ShouldDelete(Person person);
    public bool Exists(Person person);
    
  5. 아래에 제시된 예를 제외하곤 모든 메서드의 이름은 파스칼 표기법을 따른다.

    public uint GetAge()
    {
        // 함수 구현부...
    }
    
  6. public 메서드가 아닌 경우 카멜 표기법을 따른다. Visual Studio를 사용 시에는 별도의 스타일 규칙을 추가해야 할 수도 있음. (자세한 설명)

    private uint getAge()
    {
        // 함수 구현부...
    }
    
  7. 상수의 이름은 모두 대문자로 하되 밑줄로 각 단어를 분리한다.

    const int SOME_CONSTANT = 1;
    
  8. 상수로 사용하는 개체형 변수에는 static readonly를 사용한다.

    public static readonly MY_CONST_OBJECT = new MyConstClass();
    
  9. static readonly 변수는 모두 대문자로 하되 밑줄로 각 단어를 분리한다.

  10. 초기화 후 값이 변하지 않는 변수는 readonly로 선언한다.

    public class Account
    {
        private readonly string mPassword;
        
        public Account(string password)
        {
            mPassword = password;
        }
    }
    
  11. 네임스페이스의 이름은 파스칼 표기법을 따른다.

    namespace System.Graphics
    
  12. 부울(boolean) 변수는 앞에 b를 붙인다.

    bool bFired;			    // 지역변수
    private bool mbFired;		// private 멤버변수
    
  13. 부울 프로퍼티는 앞에 Is, Has, Can, Should 중에 하나를 붙인다.

    public bool IsFired { get; private set; }
    public bool HasChild { get; private set; }
    public bool CanModal { get; private set; }
    public bool ShouldRedirect { get; private set; }
    
  14. 인터페이스를 선언할 때는 앞에 I를 붙인다.

    interface ISomeInterface;
    
  15. 열거형을 선언할 때는 앞에 E를 붙인다

    public enum EDirection
    {
        North,
        South
    }
    
  16. 구조체를 선언할 때는 앞에 S를 붙인다

    public struct SUserID;
    
  17. private 멤버 변수명은 앞에 m을 붙이고 파스칼 표기법을 따른다

    public class Employee
    {
        public int DepartmentID { get; set; }
        private int mAge;
    }
    
  18. 값을 반환하는 함수의 이름은 무엇을 반환하는지 알 수 있게 짓는다.

    public uint GetAge();
    
  19. 단순히 반복문에 사용되는 변수가 아닌 경우엔 i, e 같은 변수명 대신 index, employee 처럼 변수에 저장되는 데이터를 한 눈에 알아볼 수 있는 변수명을 사용한다.

  20. 뒤에 추가적인 단어가 오지 않는 경우 줄임말은 모두 대문자로 표기한다.

    public int OrderID { get; private set; }
    public int HttpCode { get; private set; }
    
  21. getter와 setter 대신 프로퍼티를 사용한다.

    틀린 방식 :

    public class Employee
    {
        private string mName;
        public string GetName();
        public string SetName(string name);
    }
    

    올바른 방식 :

    public class Employee
    {
        public string Name { get; set; }
    }
    
  22. 지역 변수를 선언할 때는 그 지역 변수를 사용하는 코드와 동일한 줄에 선언하는 것을 원칙으로 한다.

  23. double이 반드시 필요한 경우가 아닌 이상 부동 소수점 값에 f를 붙여준다

    float f = 0.5F;
    
  24. switch 문에 언제나 default: 케이스를 넣는다.

    switch (number)
    {
    case 0:
        ... 
        break;
    default:
        break;
    
  25. switch 문에서 default: 케이스가 절대 실행될 일이 없는 경우, default: 안에 Debug.Fail() 또는 Debug.Assert(false) 란 코드를 추가한다.

    switch (type)
    {
    case 1:
        ... 
        break;
    default:
        Debug.Fail("unknown type");
        break;
    }
    
  26. 재귀 함수는 이름 뒤에 Recursive를 붙인다.

    public void FibonacciRecursive();
    
  27. 클래스 안에서 멤버 변수와 메서드의 등장 순서는 다음을 따른다.

    a. public 멤버변수/프로퍼티
    b. internal 멤버변수/프로퍼티
    c. protected 멤버변수/프로퍼티
    d. private 멤버변수
        단, 프로퍼티와 대응하는 private 멤버변수는 프로퍼티 바로 위에 적음
    e. 생성자
    f. public 메서드
    g. Internal 메서드
    h. protected 메서드
    i. private 메서드
    
  28. 대부분의 경우 함수 오버로딩을 피한다.

    올바른 방식:

    public Anim GetAnimByIndex(int index);
    public Anim GetAnimByName(string name);
    

    틀린 방식:

    public Anim GetAnim(int index);
    public Anim GetAnim(string name);
    
  29. 클래스는 각각 독립된 소스 파일에 있어야 한다. 단, 작은 클래스 몇 개를 한 파일 안에 같이 넣어두는 것이 상식적일 경우 예외를 허용한다.

  30. 파일 이름은 대소문자까지 포함해서 반드시 클래스 이름과 일치해야 한다.

    public class PlayerAnimation {}
    

    PlayerAnimation.cs

  31. 여러 파일이 하나의 클래스를 이룰 때(즉, partial 클래스), 파일 이름은 클래스 이름으로 시작하고, 그 뒤에 마침표와 세부 항목 이름을 붙인다.

    public partial class Human;
    

    Human.Head.cs Human.Body.cs Human.Arm.cs

  32. 특정 조건이 반드시 충족되어야 한다고 가정(assertion)하고 짠 코드 모든 곳에 assert를 사용한다. assert는 복구 불가능한 조건이다.(예: 대부분의 함수는 다음과 같은 assert를 가질 수도… Debug.Assert(매개변수의 null 값 검사) )

  33. 비트 플래그 열거형은 이름 뒤에 Flags를 붙인다.

    public enum EVisibilityFlags
    {
    }
    
  34. 디폴트 매개 변수 대신 함수 오버로딩을 선호한다.

  35. 디폴트 매개 변수를 사용하는 경우, null이나 false, 0 같이 비트 패턴이 0인 값을 사용한다.

  36. 변수 가리기(variable shadowing)는 허용되지 않는다. 외부 변수가 동일한 이름을 사용중이라면 내부 변수에는 다른 이름을 사용한다.

    public class SomeClass
    {
        public int Count { get; set; }
        public void Func(int count)
        {
            for (int count = 0; count != 10; ++count)
            {
                // count를 사용
            }
        }
    }
    
  37. 언제나 System.Collections에 들어있는 컨테이너 대신에 System.Collections.Generic에 들어있는 컨테이너를 사용한다. 순수 배열을 사용하는 것도 괜찮다.

  38. var 키워드를 사용하지 않으려 노력한다. 단, 대입문의 우항에서 데이터형이 명확하게 드러나는 경우, 또는 데이터형이 중요하지 않은 경우에는 예외를 허용한다. IEnumerablevar를 사용하거나 우항의 new 키워드를 통해 어떤 개체가 생성되는지 알 수 있는 등이 허용되는 경우의 좋은 예이다.

    var text = "string obviously";
    var age = 28;
    var employee = new Employee();
        
    string accountNumber = GetAccountNumber();
    
  39. 싱글턴 패턴 대신에 정적(static) 클래스를 사용한다.

  40. async void 대신에 async Task를 사용한다. async void가 허용되는 유일한 곳은 이벤트 핸들러이다.

  41. 외부로부터 들어오는 데이터의 유효성은 외부/내부 경계가 바뀌는 곳에서 검증(validate)하고 문제가 있을 경우 내부 함수로 전달하기 전에 반환해 버린다. 이는 경계를 넘어 내부로 들어온 모든 데이터는 유효하다고 가정한다는 뜻이다.

  42. 따라서 내부 함수에서 예외(익셉션)를 던지지 않으려 노력한다. 예외는 경계에서만 처리하는 것을 원칙으로 한다.

  43. 위 규칙의 예외: enum 형을 switch 문에서 처리할 때 실수로 처리 안 한 enum 값을 찾기 위해 default: 케이스에서 예외를 던지는 것은 허용.

    switch (accountType)
    {
    case AccountType.Personal:
        return something;
    case AccountType.Business:
        return somethingElse;
    default:
        throw new ArgumentOutOfRangeException(nameof(AccountType));
    }
    
  44. 함수의 매개변수로 null을 허용하지 않는 것을 추구한다. 특히 public 함수일 경우 더욱 그러하다.

  45. null 매개변수를 사용할 경우 변수명 뒤에 OrNull를 붙인다

    public Anim GetAnim(string nameOrNull)
    {
    }
    
  46. 함수에서 null을 반환하지 않는 것을 추구한다. 특히 public 함수일 경우 더욱 그러하다. 그러나 때로는 예외를 던지는 것을 방지하기 위해 그래야 할 경우도 있다.

  47. 함수에서 null을 반환할 때는 함수 이름 뒤에 OrNull을 붙인다.

    public string GetNameOrNull();
    
  48. 개체 초기자(object initializer)를 사용하지 않으려고 노력한다. 그 대신 생성자와 이름으로 지정한 매개변수(named parameter)를 사용하는 게 더 좋은 방법이다. 이 원칙에 대한 두가지 예는 다음과 같다.

    a. 개체 생성을 딱 한 군데서만 할 경우 (예: 한군데서만 사용하는 DTO)
    b. 개체 생성을 해당 클래스 안에 있는 정적 메서드에서 하는 경우 (예: Factory 패턴)

II. 소스 코드 포맷팅

  1. 탭(tab)은 비주얼 스튜디오 기본값을 사용하며, 비주얼 스튜디오를 사용하지 않을 시 띄어쓰기 4칸을 탭으로 사용한다.

  2. 중괄호( { )를 열 때는 언제나 새로운 줄에 연다.

  3. 중괄호 안( { } )에 코드가 한 줄만 있더라도 반드시 중괄호를 사용한다.

    if (bSomething)
    {
        return;
    }
    
  4. 한 줄에 변수 하나만 선언한다.

    틀린 방식:

    int counter = 0, index = 0;
    

    올바른 방식:

    int counter = 0;
    int index = 0;
    

III. 특정 프레임워크용 가이드라인

A. XAML 컨트롤

  1. 컨트롤의 이름(예: x:name)은 정말 필요한 경우에만 만든다..

  2. 컨트롤의 변수명 앞에는 x를 붙인다.

    xLabelName
    
  3. 컨트롤의 변수명 앞에는 컨트롤 종류도 붙인다.

    xLabelName
    xButtonAccept
    

B. ASP .NET Core

  1. REST API의 요청 바디(request body)용으로 DTO(Data Transfer Object, 데이터 전송 개체)를 만들 때, 각 값형 프로퍼티를 nullable로 만들어 모델 유효성 검증(model validation)이 자동으로 이뤄지게 한다.

    [Required]
    public Guid? ID { get; set; }
    
  2. 컨트롤러 메서드에서 요청에서 처음으로 할 일은 입력값 검증이다. 이 검증을 통과하면 모든 입력값은 유효한 것으로 간주한다. 따라서 [required] 애트리뷰트가 달린 nullable 속성은 그 후 부터는 모두 null이 아니다.

  3. 위 원칙과는 달리 [RouteParam] 애트리뷰트가 달린 변수는 ?를 가지지 않는다.

    public bool GetAsync([RouteParam]Guid userID)
    

C. 서비스/리포 패턴

  1. 내부적으로만 사용되는 DTO 클래스와 열거형(enum)들은(예: 내부 마이크로서비스용 DTO, 서비스/리포 사이의 DTO) 변수명 앞에 X를 붙인다. 이는 이 DTO들이 임시 클래스와 열거형임을 의미한다.

    public sealed class XNode
    {
    }
       
    public enum EXTransactionStatus
    {
    }
    

Buy Me A Coffee