클래스의 항목 순서 : 필드, 속성, 생성자, 메서드


637

클래스 구조 측면에서 항목 순서에 대한 공식적인 C # 지침이 있습니까?

가요 :

  • 공공 장소
  • 개인 분야
  • 속성
  • 생성자
  • 방법
    ?

품목 순서에 대해 단단하고 빠른 규칙이 있는지 궁금합니다. 나는 온통 종류입니다. 나는 어디에서나 할 수 있도록 특정 표준을 고수하고 싶다.

진짜 문제는 더 복잡한 속성이 메소드와 매우 비슷해 보이고 생성자보다 맨 앞에 위치하지 않는다는 것입니다.

팁 / 제안이 있습니까?


7
실제로, 실제 질문에 대답하기 위해 공식 지침은 없습니다. StyleCop은 Microsoft의 특정 그룹 내에서 사용하도록 개발 된 지침을 구현합니다. 이 지침은 공식 지침이 아니며 Microsoft 그룹간에 균일하지 않을 수도 있습니다.
John Saunders

2
한 가지 쉬운 트릭은 .net에서 일부 복잡한 클래스의 메타 데이터를 보는 것입니다 (V의 F12). 적어도 회원들 publicprotected회원들에게 주문 방법을 알게 될 것 입니다.
nawfal

19
이 질문은 공식 지침이 있는지 묻는 질문에 근거한 것이 아닙니다. 지침이 있거나 없습니다!
Simon Mᶜ 켄지

2
@nawfal 나는 이것이 오래된 의견이라는 것을 알고 있습니다. 나는 당신이 언급 한 트릭을 좋아하지만 그것이 보여주지 private않거나 internal회원 이 아니라는 것을 언급 할 가치가 있습니다 (믿습니다). 보는 좋은 방법 publicprotected그러나. 우리는 여기서, .NET 프레임 워크 클래스의 소스를 볼 수 있습니다 referencesource.microsoft.com
아담 Plocher

답변:


950

StyleCop 규칙 설명서 에 따르면 순서는 다음과 같습니다.

클래스, 구조체 또는 인터페이스 내에서 : (SA1201 및 SA1203)

  • 상수 필드
  • 필드
  • 생성자
  • 종료 자 (소멸자)
  • 대의원
  • 행사
  • 열거 형
  • 인터페이스 ( 인터페이스 구현 )
  • 속성
  • 인덱서
  • 행동 양식
  • 구조
  • 클래스

이러한 각 그룹 내에서 액세스별로 주문 : (SA1202)

  • 공공의
  • 내부의
  • 내부 보호
  • 보호
  • 은밀한

각 액세스 그룹 내에서 정적 순서로 정렬 한 후 비 정적 순서로 정렬 (SA1204)

  • 공전
  • 비 정적

각 정적 / 비 정적 필드 그룹 내에서 읽기 전용, 비 읽기 전용 순서 : (SA1214 및 SA1215)

  • 읽기 전용
  • 비 읽기 전용

풀린 목록의 길이는 130 줄이므로 여기서 풀지 않습니다. 풀린 메소드 부분은 다음과 같습니다.

  • 공개 정적 메소드
  • 공개 방법
  • 내부 정적 메소드
  • 내부 방법
  • 보호 된 내부 정적 메소드
  • 보호 된 내부 방법
  • 보호 된 정적 메소드
  • 보호 된 방법
  • 개인 정적 메소드
  • 개인적인 방법

이 문서에서는 규정 된 순서가 적합하지 않은 경우 (예 : 여러 인터페이스가 구현되고 인터페이스 메서드와 속성을 함께 그룹화해야 함) 부분 클래스를 사용하여 관련 메서드와 속성을 그룹화합니다.


31
이 게시물에 노력해 주셔서 감사합니다. StyleCop을 표준으로 만들려고 노력 중입니다 (일관되고 일을 쉽게 찾을 수 있더라도). 이것은 귀중합니다.
Kenny Mann

47
개인적으로 정적 메소드의 순서가 성가신 것으로 나타났습니다. 정적 공개 메소드에 대한 인수가 먼저 오는 것을 볼 수 있지만 일반적으로 멤버 뒤에 개인 정적 메소드를 원합니다. 그들은 결국 유틸리티입니다.
Jonathan Wright

18
나는 부분 수업 팁을 좋아했다
Keith Sirmons

10
부분 수업에 대한 참고 사항. 컴파일 시간 동안 모든 부분이 단일 유형으로 컴파일된다는 점을 감안할 때 항상 추가 오버 헤드를 생성 해야하는 합당한 이유를 확인하려고합니다. 부분 클래스의 주요 이유는 자동 생성 소스 코드를 확장하거나 대규모 프로젝트에서 작업 할 때 여러 개발자가 동일한 클래스에서 별도의 파일을 작업 할 수 있도록하기위한 것입니다.
Nope

4
@ FrançoisWahl 부분 클래스를 큰 단일 유형으로 결합하는 컴파일러와 관련된 오버 헤드가 있습니까?
dav_i

38

가시성 또는 항목 유형 (필드, 속성, 방법 등)별로 그룹화하는 대신 기능별 그룹화는 어떻습니까?


3
StyleCop 권장 사항을 사용하여 "정렬"하는 경우 일종의 기능입니다. 일부 방법은 공용이고 다른 방법은 개인용 인 이유가 있습니다. 코드가 실제로 더 잘 읽을 수 있습니다. 클래스의 .cs 파일을 열면 개인 메서드보다 "중요한"공용 메서드가 즉시 나타납니다 (해당 클래스를 사용하는 사람)
hfrmobile

75
클래스에 너무 많은 메소드, 속성 등이있어 섹션별로 그룹화 해야하는 경우 클래스가 너무 많이 수행하고 있다는 신호일 수 있습니다.
Ryan Lundy

10
클래스가 작더라도 공개 메소드를 호출하는 해당 개인 메소드로 공개 메소드를 그룹화하는 것이 이치에 맞지 않습니까?
Markus Meyer

11
공개 메소드 Foo ()가 protected / private InternalFoo ()를 호출하는 경우 +1이면 두 번째 메소드가 소스의 DoFoo () 바로 아래에 있고 다른 보호 / 개인 메소드 중 더 아래에 있지는 않습니다.
Anders Forsgren

60
기능별 그룹화를 클래스라고합니다.
MrDosu

26

이것은 오래되었지만 여전히 관련성이 높은 질문이므로 다음과 같이 추가하겠습니다. 이전에 읽었을 수도 있고 읽지 않았을 수도있는 클래스 파일을 열 때 가장 먼저 찾는 것은 무엇입니까? 필드? 속성? 나는 경험을 통해 거의 항상 생성자를 찾아 나간다는 것을 깨달았습니다. 가장 이해해야 할 것은이 객체가 어떻게 구성되어 있기 때문입니다.

따라서 클래스 파일에서 생성자를 먼저 배치하기 시작했으며 그 결과는 심리적으로 매우 긍정적이었습니다. 다른 많은 것들 후에 생성자를 넣는 표준 권장 사항은 불쾌감을 느낍니다.

C # 6의 다가오는 기본 생성자 기능은 생성자의 자연스러운 위치가 클래스의 최상위에 있다는 증거를 제공합니다. 실제로 기본 생성자는 열기 괄호 앞에도 지정됩니다.

이와 같이 재정렬이 얼마나 큰 차이가 있는지는 재밌습니다. using시스템 네임 스페이스를 사용하여 명령문을 정렬 하는 방법을 상기시킵니다 . Visual Studio의 "Organize Usings"명령이이 순서를 사용했습니다. 이제 usings는 알파벳순으로 정렬되며 시스템 네임 스페이스에 특별한 처리가 없습니다. 결과는 더 단순하고 깨끗하게 느껴집니다.


2
제 생각에 클래스 초기화 / 구성은 복잡합니다. 필드는 명시 적 생성자가 실행되기 전에 초기화되므로 기본적으로 멤버를 사용 / 생성 된 순서대로 배치한다는 주장에 따라 초기화 된 필드는 명시 적으로 선언 된 생성자보다 먼저 나타납니다. 초기화 된 정적 필드와 정적 생성자는 더욱 흥미롭게 만듭니다.
David Culp

1
실제로, 그들은 인간이 찾는 경향, 즉 인간이 코드를 먼저 읽을 수 있도록하는 문학적 프로그래밍의 개념이다.
밝은

1
C # 6에 대한 계획에서 기본 생성자가 제거되었습니다. stackoverflow.com/a/26915809/5085211
fuglede

4
10 중 9 번, 공용 인터페이스를 찾고 있는데, 모든 공용 멤버를 먼저 배치 한 다음 내부 멤버, 내부 보호, 마지막으로 개인 멤버를 배치하는 이유입니다.
매트 데이비스

15

언어 또는 업계 표준에 대해서는 잘 모르지만 각 지역을 #region으로 묶어 순서대로 정리하는 경향이 있습니다.

문장 사용

네임 스페이스

수업

개인 회원

공공 재산

생성자

공개 방법

개인 방법


이것이 정확히 내가하는 방법입니다. 클래스 멤버와 개인 멤버를 제외하고는 공개 상수와 열거 형 등이 있습니다.
deegee

그렇습니다, 나는 개인 방법 후에 공공 재산을 유지하고 싶습니다. 다른 사람들은 생성자를 공개 속성보다 선호하지만 내 머리에는 값 / 생성자 / 동작을 순서대로 선호합니다. 그런 다음 "값"은 상수 / privateMembers / properties 등으로 나뉩니다. 일반적으로 큰 뷰 모델을 제외하고는 지역을 사용하지 않습니다 ... 글쎄, WPF 뷰 모델은 특별한 종류이며,이 경우 일반적으로 각 개인 재산 바로 앞에 백업 개인 필드를 배치합니다. 이 경우 개인 필드와 공개 멤버의 집합은 동일한 단위입니다.
zameb

15

IDesign 의 코딩 표준 또는 Brad Abram 웹 사이트 에 나열된 표준을 사용하는 것이 좋습니다 . 그것들은 내가 찾은 최고의 두 가지입니다.

브래드는 이렇게 말할 것입니다 ...

클래스 멤버는 알파벳순으로 정렬되고 섹션 (필드, 생성자, 속성, 이벤트, 메소드, 개인 인터페이스 구현, 중첩 유형)으로 그룹화되어야합니다.


3
이 링크는 요즘 IDesign 홈페이지로 연결되는 것으로 보입니다. 코딩 표준이 요즘 전자 메일로 다운로드 된 링크 뒤에 숨겨져있는 것 같습니다. #justsaying
Liam

지침은 이론적 근거가 있어야합니다. 이에 대한 이론적 근거는 다음과 같습니다. 1. 이해하기 2. 경계 경계, 미묘하거나 모호하거나 예측하지 못하거나 상충되는 사례에 대한 판단을 적용 할 수 있습니다. 3. 조건이 변경되고 일부 지침이 더 이상 적용되지 않는 시점을 조정할 수 있도록합니다.
Pablo H

6

앞에서 언급했듯이 C # 언어에는 레이아웃을 지시하는 것이 없으므로 개인적으로 지역을 사용하며 평균 수업에 이와 같은 작업을 수행합니다.

public class myClass
{
#region Private Members

#endregion
#region Public Properties

#endregion

#region Constructors

#endregion
#region Public Methods

#endregion
}

어쨌든 그것은 나에게 의미가 있습니다


19
여기 stylecop이 지역 (SA1124 DoNotUseRegions)를 사용하지 않는 것이 좋습니다 않는 (단지 정보) 말을하는 것입니다
Gerwald


1
@zwcloud 물론 5538 줄의 파일에는 영역이 필요하지만 이것이 일반 파일에서 영역을 사용해야한다는 의미는 아닙니다.
Ghost4Man

1
@ Gerwald : StyleCop은 StyleCop을 사용하는 사람들에게만 해당됩니다. 그것은 많은 표준 중 하나입니다
zameb

1
@zameb : StyleCop 규칙은 C #의 가장 일반적인 코딩 지침 중 하나입니다. 모든 언어로 코딩 할 때는 항상 가장 일반적인 코딩 지침을 찾아서 따르려고합니다.
Gerwald

5

StyleCop에서

개인 필드, 공용 필드, 생성자, 속성, 공용 메서드, 개인 메서드

StyleCop은 MS 빌드 프로세스의 일부이므로 사실상 표준으로 볼 수 있습니다.


흥미 롭군 StyleCop을 정기적으로 사용하십니까?
mmcdole

한 프로젝트의 경우 예, MS 계약 작업에 반복적으로 사용되기 때문에 가능합니다. 그것은 매우 성가신 미소입니다
blowdart

1
오랫동안 StyleCop을 사용하고 해당 권장 사항을 사용하면 코드를 더 읽기 쉽게 만들 수 있습니다. 클래스의 .cs 파일을 열면 개인 메서드보다 "중요한"공개 메서드가 즉시 표시됩니다. 대중은 클래스가 제공하는 것과 테스트 할 수있는 것 (TDD 및 Test-First를 선호 함)의 클래스의 "인터페이스"입니다
hfrmobile

1
StyleCop에 따르면 공개 필드는 개인 필드보다 먼저 가야합니다. stylecop.com/docs/SA1202.html
Michael Freidgeim

1
"StyleCop은 MS 빌드 프로세스의 일부입니다"는 무슨 뜻입니까? Microsoft는 모든 코드에 StyleCop을 사용하고 있습니까?
Rico Suter

5

일반적으로 다음 패턴을 따르려고합니다.

  • 정적 멤버 (일반적으로 다른 컨텍스트가 있고 스레드로부터 안전해야 함 등)
  • 인스턴스 멤버

각 파트 (정적 및 인스턴스)는 다음 멤버 유형으로 구성됩니다.

  • 연산자 (항상 정적 임)
  • 필드 (생성자 전에 초기화 됨)
  • 생성자
  • 소멸자 ( 생성자를 따르는 전통입니다 )
  • 속성
  • 행동 양식
  • 행사

그런 다음 멤버를 가시성별로 정렬합니다 (더 적게에서 더 잘 보이도록).

  • 은밀한
  • 내부의
  • 내부 보호
  • 보호
  • 공공의

순서는 교리가 아닙니다. 간단한 클래스는 읽기가 쉽지만,보다 복잡한 클래스는 상황 별 그룹화가 필요합니다.


5

선호하는 것은 종류별로 주문한 다음 다음과 같이 가시성을 낮추는 것입니다.

public methods
public events
public properties

protected methods
protected events
protected properties

private methods
private events
private properties
private fields

public delegates
public interfaces
public classes
public structs

protected delegates
protected interfaces
protected classes
protected structs

private delegates
private interfaces
private classes
private structs

나는 이것이 스타일 경찰에 위배된다는 것을 알고 있으며 누군가 내가 인터페이스의 앞에 유형의 구현 세부 정보를 배치 해야하는 이유를 제시 할 수 있다면 나는 기꺼이 바꿀 것입니다. 현재 개인 멤버를 마지막으로 배치하는 것이 좋습니다.

참고 : 공개 또는 보호 된 필드를 사용하지 않습니다.


3
동의했다. 개인 멤버를 먼저 두는 개념이 변수가 먼저 선언되어야하는 C 시대의 이월이 아닌지 정말 궁금합니다. 거의 항상 클래스 내부가 아닌 공용 인터페이스를 먼저보고 싶습니다.
매트 데이비스

1
그것은 실제로 많은 의미가 있습니다. 나는 그것이 C.에서
인수

가장 큰 문제 중 하나는 속성 IMO 일 수 있습니다. getter / setter에 당신이 모르는 논리가있을 때, 그것은 메소드의 부작용을 물을 가능성이 훨씬 높습니다 (자연스럽게 기대할 것입니다). 그래서, 나는 그 필드와 함께 속성을 선호합니다 , 그래서 처음 수업을 볼 때, 나는 문제가있는 것을 본다. 내가 방법을 읽을 때, 나는 보통 방법으로 즉시 탐색 / 점프 합니다
Ryan The Leach


3

내가 제안한 유일한 코딩 지침은 클래스 정의의 맨 위에 필드를 두는 것입니다.

나는 생성자를 다음에 두는 경향이 있습니다.

내 일반적인 의견은 파일 당 하나의 클래스를 고수해야하며 클래스가 속성 대 메소드의 구성이 큰 관심사 일 정도로 충분히 클 경우 클래스는 얼마나 크며 어쨌든 리팩토링해야합니까? 여러 가지 우려를 나타내는가?


3
일단 지역이 필요하면 ...
Hamish Smith

3

개인 필드를 생성자와 함께 맨 위에 놓고 공개 인터페이스 비트를 넣은 다음 개인 인터페이스 비트를 선호합니다.

또한 클래스 정의가 항목 순서를 크게 중요하게 정의 할 수있을 정도로 길면 클래스가 너무 크고 복잡하여 리팩터링해야한다는 코드 냄새 일 수 있습니다.


3

나는 그것을 가능한 한 단순하게 유지합니다 (적어도 나를 위해)

열거 형
선언
생성자 메서드를
재정의 함 이벤트 속성




2

어떤 식 으로든 언어를 강요하는 언어는 없습니다. 나는 가시성 (공개, 보호, 비공개)으로 사물을 그룹화하는 경향이 있으며 #regions를 사용하여 관련 사물을 속성, 방법 또는 기타에 관계없이 기능적으로 그룹화합니다. 건설 방법 (실제 ctor 또는 정적 팩토리 기능)은 고객이 알아야 할 첫 번째 항목이므로 일반적으로 맨 위에 있습니다.


지역을 사용하여 가시성으로 분리하고 지역 코드 레이아웃을 사용하면 정직합니다. rauchy.net/regionerate
잊혀진 세미콜론

#regions를 사용하는 데 문제가 없지만 지역에 넣고 싶은 유혹이 생기면 수업을 나누는 것이 좋습니다.
mikecamimo

2

나는 이것이 오래된 것을 알고 있지만 내 순서는 다음과 같습니다.

공개, 보호, 개인, 내부, 추상 순서

  • 상수
  • 정적 변수
  • 필드
  • 행사
  • 생성자
  • 행동 양식
  • 속성
  • 대의원

또한 (속기 접근 방식 대신) 이와 같은 속성을 작성하고 싶습니다.

// Some where in the fields section
private int someVariable;

// I also refrain from
// declaring variables outside of the constructor

// and some where in the properties section I do
public int SomeVariable
{
    get { return someVariable; }
    set { someVariable = value; }
}
당사 사이트를 사용함과 동시에 당사의 쿠키 정책개인정보 보호정책을 읽고 이해하였음을 인정하는 것으로 간주합니다.
Licensed under cc by-sa 3.0 with attribution required.