CA1868: 집합에 대한 'Contains'에 대한 불필요한 호출
속성 | 값 |
---|---|
규칙 ID | CA1868 |
제목 | 집합에 대한 'Contains'에 대한 불필요한 호출 |
범주 | 성능 |
수정 사항이 주요 변경인지 여부 | 주요 변경 아님 |
.NET 9에서 기본적으로 사용 | 제안 사항 |
원인
ISet<T>.Add 또는 ICollection<T>.Remove 호출은 호출에 의해 보호됩니다Contains. 또는 호출이 IImmutableSet<T>.Add IImmutableSet<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
자세한 내용은 방법: 코드 분석 경고 표시 안 함을 참조하세요.
.NET