C#에서 재정의/주석 처리하는 방법은 무엇입니까?

Question

때로는 기본 클래스의 메서드를 재정의합니다. 때로는 비어있는 메서드로 재정의하는 경우도 있습니다. 왜냐하면 내가 원하는 것은 비헤이비어를 막는 것입니다. 과거

나는 기본 방법 우회의 의도를 보여주기 위해 다음처럼 작성할 수

:

protected override void OnMouseUp(MouseEventArgs e) 
{ 
    // base.OnMouseUp(e); 
} 

(나는 코드의 주석 줄 알고 나쁜 일이 내가 가을 사용했다. 그것을 할)

그러나 나는 더 잘 할 싶어 : 나는 거라고 어떻게

• 을 오버라이드의 의도를 확인 하는가? 구체적으로 :
• 재정의 XML (? ) 문서에는 무엇을 적 으시겠습니까?

Answer 1

그것을 다시 복원

// This method is intentionally blank because 
// we do not want the base class functionality 

같은 의견은 당신이 무슨 짓을했는지 왜 그랬는지 첫 번째 댓글이 명확하게 명시

// base.SomeMethod(); 

보다 훨씬 더, 다음 개발자는 누가 원을 따라 온다 기본 메서드에 대한 호출이 실수로 주석 처리되었는지 궁금하지 않아도됩니다.

기본 클래스를 제어 할 수있는 경우 해당 메서드를 제거하고 클래스를 더 추상화하는 것이 좋습니다. 그런 다음 필요에 따라 하위 클래스에서만 해당 기능을 구현하도록 선택할 수 있습니다.

Answer 2

기본 클래스 호출에 대한 의견은 제 생각에는 의도를 분명히하는 것과 정반대입니다. 사람들은 주석 처리 된 행이 왜 여전히 존재하는지, 그리고 사용자가 삭제하지 않았기 때문에 여전히 사용 가능할지 궁금해 할 것입니다. 그래서 내가 주석으로 처리 된 라인을 제거합니다.

다른 방법과 마찬가지로 오버라이드를 문서화 할 수 있으며 설명서에서 비어있는 이유를 정확하게 지정하십시오. 메서드 본문에 이유를 주석으로 쓸 수도 있습니다. 그게 선호 사항의 문제입니다.

이 정보가 코드를 유지 관리하는 개발자 또는 코드 사용자 (예 : 사용자)에게만 중요하다고 생각합니다. 일반적으로 운영 체제에서만 호출되는 이벤트의 경우 (예에서와 같이) 요약 태그에 넣는 것은 실제로 필요하지 않습니다.

여전히 기본 클래스의 동작을 비활성화하는 메서드를 재정의해야하는 경우 디자인의 해당 부분을 다시 고려해야합니다. 그 행동은 나에게 조금 비현실적 인 것처럼 보입니다. 난 그냥

protected override void OnMouseUp(MouseEventArgs e) 
{ 
    // not calling the base implementation 
    ... 
} 

처럼

// base.OnMouseUp(e); 

가 만드는 라인을 코멘트를 넣어 것입니다 의도를 명확히 들어

/// <summary>Exiting drag mode on mouse up</summary> 
protected override void OnMouseUp(MouseEventArgs e) 
{ 
    ... 

:

Answer 3

문서에 대해서는, 난 그냥 built-in documentation tags을 사용 전화가 일시적으로 주석 처리되었다는 인상 (아마도 누군가 잊어 버렸습니다.)