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

概述我正在开发一个从Excel文件中提取数据的应用程序(我无法访问实际的数据库),并且我编写了一个方法,它具有从Excel电子表格中提取数据的唯一功能,如下所示. private IEnumerable<SMEntity> ExtractSMData(List<MSExcel.Range> SMData){ List<SMEntity> SMEntities = new List<SMEnt 我正在开发一个从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,这是从哪里来的？全部内容，希望文章能够帮你解决c# – 圈复杂度计数为31,这是从哪里来的？所遇到的程序开发问题。

如果觉得内存溢出网站内容还不错，欢迎将内存溢出网站推荐给程序员好友。