DataRow를 처리하는 동안 순환 적 복잡성을 줄입니다.

Question

특정 DataRow에 특정 이름의 열이 있고 NULL이 아닌지 확인하는 다음 코드가 있습니다.

private static bool HasValue(DataColumn c, DataRow row) 
{ 
    if (c != null && row != null && row[c.ColumnName] != System.DBNull.Value) 
    { 
     return true; 
    } 
    return false; 
} 

또한 datarow의 열을 처리하고 해당 데이터 유형으로 파싱합니다.

foreach (DataColumn c in row.Table.Columns) 
{ 
    switch (c.ColumnName) 
    { 
    case Constants.Literals.ACTIVATIONDATETIME: 
           if (HasValue(c, row)) 
           { 
            bFound = true; credentialInfo.ActivationDateTime = DateTimeOffset.Parse(Convert.ToString(row[c.ColumnName])); 
           } 
           break; 
    } 
} 

Visual Studio는 이것을 순환 적으로 복잡하게 나타냅니다. 이 기능에 대한 순환 지수를 줄이는 방법이 있습니까?

Answer 1

을 대신 row.Table.Columns 통해 반복의 찾아 테이블에 열을 포함하는 경우, 당신은이 작업을 수행 할 수 있습니다

var column = row.Table.Columns[Constants.Literals.ACTIVATIONDATETIME]; 
if(HasValue(column, row)) 
{ 
    //column found. 
} 

이렇게하면 루프와 스위치가 제거되어 함수의 순환 복잡성이 크게 줄어 듭니다.

Answer 2

당신은 단순히 체인과 문을 반환하여 복잡성을에게 약간을 줄일 수 :

private static bool HasValue(DataColumn c, DataRow row) 
{ 
    return c != null && row != null && row[c.ColumnName] != DBNull.Value; 
}