CA1868: 집합에 대해 'Contains'를 불필요하게 호출함

속성	값
규칙 ID	CA1868
제목	'Contains' 메서드의 불필요한 호출 (집합 관련)
범주	성능
수정 사항이 호환성을 깨뜨리는지 여부 또는 무중단인지 여부	주요 변경 아님
.NET 10에서 기본적으로 사용하도록 설정	제안 사항
적용 가능한 언어	C# 및 Visual Basic

원인

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

규칙 설명

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

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

위반 문제를 해결하는 방법

ICollection<T>.Contains(T) (또는 IImmutableSet<T>.Contains(T)) 호출을 ISet<T>.Add(T) 또는 ICollection<T>.Remove(T) (또는 IImmutableSet<T>.Add(T)나 IImmutableSet<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

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