CA1827: не используйте Count()/LongCount() при использовании Any()
Свойство | Значение |
---|---|
Идентификатор правила | CA1827 |
Заголовок | Не используйте Count()/LongCount() при использовании Any() |
Категория | Производительность |
Исправление является критическим или не критическим | Не критическое |
Включен по умолчанию в .NET 8 | Как предложение |
Причина
Метод Count() или LongCount() использовался, где метод Any() будет более эффективным.
Описание правила
Это правило помечает вызовы метода Count() и LongCount() LINQ, которые используются для проверки наличия хотя бы одного элемента коллекции. Эти методы перечисляют всю коллекцию для вычисления счетчика. Эта же проверка выполняется быстрее с помощью метода Any(), так как она избегает перечисления коллекции.
Примечание.
Это правило похоже на CA1860: избегайте использования метода расширения Enumerable.Any(). Однако это правило предполагает использование Count
свойства, в то время как это правило применяется к методу расширения Linq Count()
.
Устранение нарушений
Чтобы устранить нарушение, замените вызов метода Count или LongCount вызовом метода Any. В следующих двух фрагментах кода показано нарушение правила и способы его устранения:
using System.Collections.Generic;
using System.Linq;
class C
{
public string M1(IEnumerable<string> list)
=> list.Count() != 0 ? "Not empty" : "Empty";
public string M2(IEnumerable<string> list)
=> list.LongCount() > 0 ? "Not empty" : "Empty";
}
using System.Collections.Generic;
using System.Linq;
class C
{
public string M1(IEnumerable<string> list)
=> list.Any() ? "Not empty" : "Empty";
public string M2(IEnumerable<string> list)
=> list.Any() ? "Not empty" : "Empty";
}
Совет
Для этого правила в Visual Studio доступно исправление кода. Чтобы использовать его, поместите курсор на нарушение и нажмите клавиши CTRL+ (период). В списке выберите Не используйте методы Count() и LongCount(), если можно использовать метод Any().
Когда лучше отключить предупреждения
Вывод предупреждений для этого правила можно отключить, если вас не беспокоит влияние ненужного перечисления коллекции для подсчета количества.
Отключение предупреждений
Если вы просто хотите отключить одно нарушение, добавьте директивы препроцессора в исходный файл, чтобы отключить и повторно включить правило.
#pragma warning disable CA1827
// The code that's violating the rule is on this line.
#pragma warning restore CA1827
Чтобы отключить правило для файла, папки или проекта, задайте его серьезность none
в файле конфигурации.
[*.{cs,vb}]
dotnet_diagnostic.CA1827.severity = none
Дополнительные сведения см. в разделе Практическое руководство. Скрытие предупреждений анализа кода.
Связанные правила
- CA1826: используйте свойство вместо метода Linq Enumerable
- CA1828: не используйте CountAsync/LongCountAsync при использовании AnyAsync
- CA1829: используйте свойство Length/Count вместо метода Enumerable.Count()
- CA1860: избегайте использования метода расширения "Enumerable.Any()"