중복 Null 검사 피하기 (사용자 컨트롤을 매개 변수로 전달)

Question

Oracle 데이터베이스에서 여러 날짜를 읽고이를 DateTimePickers에 할당하는 응용 프로그램을 사용하고 있습니다. DB를의 값이 null 될 수 있기

dtpAdjSerDt.Value = dr.GetDateTime(iadj_service_date); 

그러나,이 할당은 NULL 체크가 필요합니다.

값이 null이면 기본값을 반환하는 확장 메서드를 만들 수 있기를 원합니다.

public static DateTime fSafeGetDateTime(OracleDataReader reader, int colIndex) 
{ 
    if (!reader.IsDBNull(colIndex)) 
     return reader.GetDateTime(colIndex); 
    else 
     return DateTime.Now; 
} 

이 접근 방식에는 한 가지 문제점이 있습니다. db 값이 null 인 경우 DateTimePicker.Checked는 false 여야합니다. 그러나 위에서 제안한 전략을 사용하면 사실로 설정됩니다. 이 이러한 목표를 달성하기 위해 (중복 코드를 방지) 가장 깨끗한 방법이 될 것입니다 무엇을 할 수없는 경우

public static DateTime fSafeGetDateTime(OracleDataReader reader, int colIndex, DateTimePicker control) 
{ 
    if (!reader.IsDBNull(colIndex)) 
     return reader.GetDateTime(colIndex); 
    else 
    { 
     control.Checked = false; 
     return DateTime.Now; 
    } 
} 

:

나에게 이런 일을 할 수있는 방법이 있습니까?

public static DateTime? SafeGetDateTime(this OracleDataReader reader, int colIndex) 
{ 
    return reader.IsDBNull(colIndex) ? null 
            : (DateTime?) reader.GetDateTime(colIndex); 
} 

는 이제 사용할 수 있습니다 :

Answer 1

난 당신이 단지 DateTime?을 반환하고 호출자가 불이행 할 수 있도록 제안

DateTime? fromReader = reader.SafeGetDateTime(1); 
if (fromReader == null) 
{ 
    control.Checked = false; 
} 
DateTime fromReaderOrNow = fromReader ?? DateTime.Now; 

:

DateTime fromReaderOrNow = reader.SafeGetDateTime(1) ?? DateTime.Now; 

을 그리고 당신은 제어 부분을 필요로하는 경우

이 실제로 인 경우 다중으로 수행해야합니다. 르 장소, 당신은 할 수있는 방법을 감쌀 수 있습니다 - 그리고 당신은 SafeGetDateTime 코드를 반복하지 않을 것입니다.

(나는 정직하게, 시작하는 당신의 UI 코드에서 더욱 우리의 데이터베이스 코드를 분리하는 것이 좋습니다 것입니다.)