c# – 圈复杂度计数为31,这是从哪里来的？

我正在开发一个从Excel文件中提取数据的应用程序(我无法访问实际的数据库),并且我编写了一个方法,它具有从Excel电子表格中提取数据的唯一功能,如下所示.

private IEnumerable<SMEntity> ExtractSMData(List<MSExcel.Range> SMData)
{
    List<SMEntity> SMEntities = new List<SMEntity>();

    foreach (MSExcel.Range Row in SMData)
    {
        SMEntity entity = new SMEntity();
        entity.IncidentNumber = Row.get_Range("K1").get_Value();
        entity.SRNumber = Row.get_Range("L1").get_Value();
        entity.SRCategory = Row.get_Range("M1").get_Value();
        entity.SiebelClientCall = EntityConversions.DateTimeConversion(Row.get_Range("N1").get_Value());
        entity.SiebelOpenedDate = EntityConversions.DateTimeConversion(Row.get_Range("O1").get_Value());
        entity.IncidentOpenDate = EntityConversions.DateTimeConversion(Row.get_Range("P1").get_Value());
        entity.PickedUpBeforeClient = Row.get_Range("Q1").get_Value().ToString().ToLowerCase() == "no" ? false : true;
        entity.OutageStartTime = EntityConversions.DateTimeConversion(Row.get_Range("R1").get_Value());
        entity.DetectionPoint = EntityConversions.DateTimeConversion(Row.get_Range("S1").get_Value());
        entity.SecondsToDetection = EntityConversions.ConvertDetectionTimetoInt(Row.get_Range("T1").get_Value());
        entity.OutageEndTime = EntityConversions.DateTimeConversion(Row.get_Range("U1").get_Value());
        entity.MTTR = EntityConversions.ConvertMTTRStringToInt(Row.get_Range("V1").get_Value());
        entity.RepairedOnTime = Row.get_Range("W1").get_Value().ToString().ToLowerCase() == "no" ? false : true;
        SMEntities.Add(entity);
    }

    return SMEntities;
}

我运行代码分析(我正在使用Visual Studio 2012并在.NET 4.5中开发)并且我有一个CA1502：避免过多的复杂性(复制如下).作为一名初级开发人员(我17岁),我尝试使用MSDN了解更多信息,但是我有点难以理解为什么我的圈复杂度为33.

CA1502

Avoid excessive complexity

'Extraction.ExtractSMData(List<Range>)' has a cyclomatic
complexity of 33. Rewrite or refactor the method to reduce complexity
to 25.

Core.Extraction.cs:104

我可以看到我的quick-ifs(条件？if_true：if_false,这些叫什么？)它可能是坏的,但我仍然只能看到它为5.

更新：

圈速复杂度现在为33 ……

如果我注释掉entity.IncidentNumber = Row.get_Range(“K1”).get_Value();复杂性变为32.我认为get_Range()和get_Value()各占一个但是没关系……

如果我注释掉entity.RepairedOnTime = Row.get_Range(“W1”).get_Value().ToString().ToLower()==“no”？ false：是的;复杂性变成28 ……

get_Range(),get_Value(),quick-if是3,执行ToString()和ToLower()计数？

解决方法

我计算方法本身,foreach和两个条件运算符的复杂度为4.如果对get_Range的13个调用中的每一个都值1复杂度,并且对get_Value的13个调用中的每一个都值1复杂度,那么总复杂度将加起来为30(仍然是1短,但接近).我不确定为什么这两个功能可能会增加复杂性,但似乎有道理.

尝试删除一个调用get_Range和get_Value的行,看看圈复杂度是否降至29.

c# – 圈复杂度计数为31,这是从哪里来的？

解决方法

相关文章