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()计数?
解决方法
原文地址:https://www.jb51.cc/csharp/99662.html
版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。