null에 대한 우아한 검사와 C#에서의 종료

Question

이것을 작성하는 우아한 방법은 무엇입니까?

changeColor()

는 공극 함수와 상기 코드를 실행하는 기능뿐만 아니라 보이드의 함수이다.

Answer 1

당신은 조건을 반대로하여 혼란을 줄일 수 있습니다.

if (lastSelection == null) 
{ 
    MessageBox.Show("No Selection Made"); 
    return; 
} 

lastSelection.changeColor(); 

Answer 2

나는 개인적으로 정말 트랩 것, 오류 조건 종료를 봤어요, 함수의 여러 반환 진술을 갖는 방어 프로그램에 따라 싫어하고를 통해 무엇을 할 수 있지만 :

if(lastSelection == null) 
{ 
    MessageBox.Show("No Selection Made"); 
    return; 
} 

lastSelection.changeColor(); 

그것은이에 속하는 전체 기능을 보지 않고 내 자신의 작품이 할 거라고 어떻게 말을하기 어렵다

Answer 3

을

나는 lastSelection이 결코 null이 아니 었는지 확인하고 현재 디자인에서 null이 될 경우 "빈"객체를 가리 키도록하고 싶습니다. 그런 다음 null 확인을 할 필요가 없습니다. 시스템에서 발생할 수있는 엔지니어링 오류는 항상 좋은 습관입니다.

IOW, 코드에서 부적절한 점이 null 체크를하는지, 아니면 에 null 체크가 전혀 필요하지 않은지 여부를 물어볼 것입니다..

Answer 4

나는 정상적인 경우 (lastSelection != null)을 먼저 확인한 다음 비정상적인 경우를 확인했습니다.

이

if (lastSelection == null) goto ERR; 
lastSelection.changeColor(); 
//... possibly more stuff... 
return; 
ERR: 
MessageBox.Show("No Selection Made"); 

Answer 5

보면 모두 다른 좋은 답변, 이을하지 마십시오있어 :

나는 단지 하나의 수정을 추가 할 수 있습니다 다음과 같이 먼저 null :

if (null == lastSelection) 
{ 
    MessageBox.Show("No Selection Made"); 
    return; 
} 

lastSelection.changeColor(); 
 

Answer 6

빨리 잡기 연습을 작성하는 것입니다 :

if (lastSelection != null) 
{ 
    lastSelection.changeColor(); 
    //your other normal code 
} 
else 
{ 
    MessageBox.Show("No Selection Made"); 
    //return; //no need for this now 
} 

Answer 7

이것은 아직 없습니다. 이것을 구현하는 다른 방법. lastSelection 변경할 수있는 기회가 있다면

 Action _action = (lastSelection != null 
           ? 
            new Action(lastSelection.changeColor) 
           : 
           () => Console.WriteLine("No Selection Made")); 
     _action.Invoke(); 

아마 조금 잔인한,하지만 당신은 확실히 사람들이 또한