我正在开发一个从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()计数？