7

私は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()カウントしますか?

4

2 に答える 2

1

メソッド自体、foreach、および 2 つの条件演算子の複雑さを合計 4 つとして計算します。への 13 回の呼び出しのget_Rangeそれぞれが複雑さ +1 の価値があり、 への 13 回の呼び出しのそれぞれが複雑さ +1 のget_Value価値がある場合、複雑さの合計は 30 になります (まだ 1 つ不足していますが、近いです)。これらの 2 つの関数が複雑さを増している理由はわかりませんが、もっともらしいと思われます。

呼び出している行の 1 つを削除しget_Rangeget_Value、循環的複雑度が 29 に下がるかどうかを確認してください。

于 2012-12-06T13:28:00.593 に答える
-1

戻り値の型は IEnumerable であるため、リストを使用しないでください。それは IEnumerable を役に立たなくします。それ以外の場合は、Lazy-Evaluation を使用しないでください: http://blogs.msdn.com/b/pedram/archive/2007/06/02/lazy-evaluation-in-c.aspx

次の場合は、yield return を使用することをお勧めします。

private IEnumerable<SMEntity> ExtractSMData(List<MSExcel.Range> SMData)
{
    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.PickedUpBeforeClient = !Row.get_Range("Q1").get_Value().ToString().ToLowerCase() == "no"
        entity.RepairedOnTime = !Row.get_Range("W1").get_Value().ToString().ToLowerCase() == "no"

        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.OutageStartTime = EntityConversions.DateTimeConversion(Row.get_Range("R1").get_Value());
        entity.DetectionPoint = EntityConversions.DateTimeConversion(Row.get_Range("S1").get_Value());
        entity.OutageEndTime = EntityConversions.DateTimeConversion(Row.get_Range("U1").get_Value());


        entity.MTTR = EntityConversions.ConvertMTTRStringToInt(Row.get_Range("V1").get_Value());
        entity.SecondsToDetection = EntityConversions.ConvertDetectionTimeToInt(Row.get_Range("T1").get_Value());


        yield return entity;
    }

}

次のように書くこともできます。

private IEnumerable<SMEntity> ExtractSMData(List<MSExcel.Range> SMData)
{
    foreach (MSExcel.Range Row in SMData)
    {
        yield return new SMEntity 
        {

            IncidentNumber = Row.get_Range("K1").get_Value(),
            SRNumber = Row.get_Range("L1").get_Value(),
            SRCategory = Row.get_Range("M1").get_Value(),
            PickedUpBeforeClient = !Row.get_Range("Q1").get_Value().ToString().ToLowerCase() == "no"
            RepairedOnTime = !Row.get_Range("W1").get_Value().ToString().ToLowerCase() == "no"

            SiebelClientCall = EntityConversions.DateTimeConversion(Row.get_Range("N1").get_Value()),
            SiebelOpenedDate = EntityConversions.DateTimeConversion(Row.get_Range("O1").get_Value()),
            IncidentOpenDate = EntityConversions.DateTimeConversion(Row.get_Range("P1").get_Value()),
            OutageStartTime = EntityConversions.DateTimeConversion(Row.get_Range("R1").get_Value()),
            DetectionPoint = EntityConversions.DateTimeConversion(Row.get_Range("S1").get_Value()),
            OutageEndTime = EntityConversions.DateTimeConversion(Row.get_Range("U1").get_Value()),


            MTTR = EntityConversions.ConvertMTTRStringToInt(Row.get_Range("V1").get_Value()),
            SecondsToDetection = EntityConversions.ConvertDetectionTimeToInt(Row.get_Range("T1").get_Value())
        };
    }
}
于 2012-12-06T13:42:09.397 に答える