CA1868: 집합에 대한 'Contains'에 대한 불필요한 호출

속성	값
규칙 ID	CA1868
타이틀	집합에 대한 'Contains'에 대한 불필요한 호출
범주	성능
수정 사항이 주요 변경인지 여부	주요 변경 아님
.NET 8에서 기본적으로 사용	제안 사항

원인

ISet<T>.Add 또는 ICollection<T>.Remove 호출은 호출에 의해 보호됩니다Contains. 또는 호출이 IImmutableSet<T>.AddIImmutableSet<T>.Remove 호출에 의해 IImmutableSet<T>.Contains보호됩니다.

규칙 설명

ICollection<T>.Remove(T) 둘 다 ISet<T>.Add(T) 조회를 수행하므로 미리 호출 ICollection<T>.Contains(T) 하는 것이 중복됩니다. 항목이 추가 또는 제거되었는지 여부를 나타내는 부울 값을 반환하는 호출 또는 Remove(T) 직접 호출 Add(T) 이 더 효율적입니다.

항목이 추가되거나 제거된 경우 새 집합을 반환하거나, 그렇지 않은 경우 원래 집합을 반환한다는 점을 제외하고 이 논리도 적용됩니다 IImmutableSet<T>.Add(T)IImmutableSet<T>.Remove(T).

위반 문제를 해결하는 방법

다음에 오거나(또는 IImmutableSet<T>.Contains(T)IImmutableSet<T>.Add(T)IImmutableSet<T>.Remove(T)) 호출 ICollection<T>.Contains(T)ISet<T>.Add(T)ICollection<T>.Remove(T) 한 후의 호출을 후자 메서드에 대한 단일 호출로 바꿉니다.

예시

다음 코드 조각은 CA1868 위반을 보여줍니다.

void Run(ISet<string> set)
{
    if (!set.Contains("Hello World"))
    {
        set.Add("Hello World");
    }
}

Sub Run(set As ISet(Of String))
    If Not set.Contains("Hello World") Then
        set.Add("Hello World")
    End If
End Sub

다음 코드 조각은 이 위반을 해결합니다.

void Run(ISet<string> set)
{
    set.Add("Hello World");
}

Sub Run(set As ISet(Of String))
    set.Add("Hello World")
End Sub

경고를 표시하지 않는 경우

성능이 중요하지 않은 경우 이 경고를 표시하지 않는 것이 안전합니다.

경고 표시 안 함

단일 위반만 표시하지 않으려면 원본 파일에 전처리기 지시문을 추가하여 규칙을 사용하지 않도록 설정한 후 다시 사용하도록 설정합니다.

#pragma warning disable CA1868
// The code that's violating the rule is on this line.
#pragma warning restore CA1868

파일, 폴더 또는 프로젝트에 대한 규칙을 사용하지 않도록 설정하려면 구성 파일에서 심각도를 none으로 설정합니다.

[*.{cs,vb}]
dotnet_diagnostic.CA1868.severity = none

자세한 내용은 방법: 코드 분석 경고 표시 안 함을 참조하세요.