변수가 값과 같은지 확인한 다음 해당 값을 중복해서 반환합니까?

Question

우리 회사는 VB.net으로 작성된 소프트웨어를 가지고 있습니다.

If x Is Nothing Then Return Nothing ' x's type should be an instance of a class 
    Return x 

이에 해당하는 일을 할 것 같다 :

이

if(x == null) //Object.ReferenceEquals() if you want to be more accurate/verbose? 
    return null; 
return x; 

이 매우 중복 보인다 나는 다음과 같은 발생했습니다. VB에서이 작업을 수행 할만한 이유가 있습니까?

Answer 1

은 코드 누적이 될 수 있습니다. 코드를 수정하는 것이 최소한의 변화 만있을 때 일어나는 일입니다. 적절하게 다시 엔지니어링하는 것이 아닙니다. 예를 들어

, 그것은 과거에되었을 수도 있습니다 :

If x is Nothing Then 
    Return 0 
End If 
Return x 

아마도 발신자가 nothing를 처리 할 수 ​​있기 때문이다.

이렇게 변경하면 은 일 수 있으므로 기능이 최소한으로 변경되었습니다.

이것은 때때로 변경을 최소화하기위한 필드 버그 수정 중에 취해진 접근법으로 더 많은 피해를 입힐 수 있지만 앞으로 "전체"변경을 제안하는 의견은 적어도 기대할 것입니다.

어쨌든 '이유'는 대체로 부적절합니다. if 문을 안전하게 제거 할 수 있습니다.