私は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
クイックIF(condition ? if_true : if_false
、これらは何と呼ばれていますか?)を使用すると、それが悪い可能性があることがわかりますが、それでも5としてしか見ることができません。
更新:
循環的複雑度は現在33です...
コメントするentity.IncidentNumber = Row.get_Range("K1").get_Value();
と、複雑さは32になります。私は考えget_Range()
て、get_Value()
それぞれ1つでしたが、大丈夫です...
コメントアウトするentity.RepairedOnTime = Row.get_Range("W1").get_Value().ToString().ToLower() == "no" ? false : true;
と、複雑さは28になります...
get_Range()
、、get_Value()
クイック-3の場合、実行ToString()
してToLower()
カウントしますか?