c# - 如何解决 CA1502：避免过于复杂？

Question

根据代码分析结果，以下是警告信息，

CA1502 避免过度复杂 'METHOD()' 的圈复杂度为 27。重写或重构方法以将复杂度降低到 25。BusinessServices
ReportService.cs 310

什么是圈复杂度以及如何解决这个问题？

Answer 1

来自维基百科：

一段源代码的圈复杂度是通过源代码的线性独立路径的数量的计数。例如，如果源代码不包含诸如 IF 语句或 FOR 循环之类的决策点，则复杂度将为 1，因为通过代码只有一条路径。如果代码具有包含单个条件的单个 IF 语句，则将有两条路径通过代码：一条 IF 语句被评估为 TRUE 的路径和一个 IF 语句被评估为 FALSE 的路径。

解决此问题的最简单方法是将方法分解为两个或更多个较小的方法。Visual Studio 的内置重构工具（例如Refactor -> Extract Method）可用于将选定的代码部分提取到另一个方法中。

Answer 2

CA1502 是二进制分析仪。在某些情况下，您的代码并不是那么复杂或无法分解。以下是导致 CA1502 警告的示例：

public class Mapping : Profile
{
    [SuppressMessage("Microsoft.Maintainability", "CA1502"]
    public Mapping()
    {
        CreateMap<ItemInstance, ManifestRecords>()
            .ForMember(dest => dest.SomeProperty1, opt => opt.MapFrom(src => src.SomeProperty1))
            .ForMember(dest => dest.SomeProperty2, opt => opt.MapFrom(src => src.SomeProperty2))
            .ForMember(dest => dest.SomeProperty3, opt => opt.MapFrom(src => src.SomeProperty3))
            .ForMember(dest => dest.SomeProperty4, opt => opt.MapFrom(src => src.SomeProperty4))
            .ForMember(dest => dest.SomeProperty5, opt => opt.MapFrom(src => src.SomeProperty5))
            .ForMember(dest => dest.SomeProperty6, opt => opt.MapFrom(src => src.SomeProperty6))
            .ForMember(dest => dest.SomeProperty7, opt => opt.MapFrom(src => src.SomeProperty7))
            .ForMember(dest => dest.SomeProperty8, opt => opt.MapFrom(src => src.SomeProperty8))
            .ForMember(dest => dest.SomeProperty9, opt => opt.MapFrom(src => src.SomeProperty9))
            .ForMember(dest => dest.SomeProperty10, opt => opt.MapFrom(src => src.SomeProperty10))
            .ForMember(dest => dest.SomeProperty11, opt => opt.MapFrom(src => src.SomeProperty11))
            .ForMember(dest => dest.SomeProperty12, opt => opt.MapFrom(src => src.SomeProperty12))
            .ForMember(dest => dest.SomeProperty13, opt => opt.MapFrom(src => src.SomeProperty13))
            .ForMember(dest => dest.SomeProperty14, opt => opt.MapFrom(src => src.SomeProperty14))
            .ForMember(dest => dest.SomeProperty15, opt => opt.MapFrom(src => src.SomeProperty15))
            .ForMember(dest => dest.SomeProperty16, opt => opt.MapFrom(src => src.SomeProperty16));
    }
}

您可以看到我正在使用 AutoMapper，并且源代码中实际上并没有任何圈复杂度，尽管编译后的二进制表示中可能存在。

您可以使用以下属性抑制警告：

[SuppressMessage("Microsoft.Maintainability", "CA1502"]