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

Question

我正在开发一个从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: Avoid excessive complexity(在下面复制).作为一名初级开发人员(我17岁),我尝试使用MSDN了解更多信息,但是我有点难以理解为什么我的圈复杂度为33.

CA1502

避免过度复杂

'Extraction.ExtractSMData(List<Range>)' 具有33的圈复杂度.重写或重构该方法以将复杂度降低到25.

Core.Extraction.cs:104

我可以看到我的快速ifs(condition ? 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 : true;复杂性变成了28 ......

get_Range(),get_Value()快速 - 如果是3,做ToString()和ToLower()计数？

Answer 1

我计算出方法本身、foreach和两个条件运算符的复杂度总计为 4。如果 13 个调用中的每一个都get_Range值得 +1 复杂性，并且 13 个调用中的每一个都get_Value值得 +1 复杂性，那么总复杂性将加起来为 30（仍然短 1，但接近）。我不确定为什么这两个函数会增加复杂性，但这似乎是合理的。

get_Range尝试删除调用and的行之一get_Value，看看圈复杂度是否下降到 29。