Поделиться через

CA1828: не используйте CountAsync/LongCountAsync при использовании AnyAsync

  • Статья
Свойство Значение
Идентификатор правила CA1828
Заголовок Не используйте CountAsync/LongCountAsync, если можно использовать AnyAsync
Категория Производительность
Исправление является критическим или не критическим Не критическое
Включен по умолчанию в .NET 8 Как предложение

Причина

Метод CountAsync или LongCountAsync использовался, где AnyAsync метод будет более эффективным.

Описание правила

Это правило помечает вызовы метода LINQ CountAsync и LongCountAsync, используемые для проверки наличия хотя бы одного элемента в коллекции. Чтобы эти методы можно было использовать для вычисления количества, требуется перечисление всей коллекции. Та же самая проверка выполняется быстрее с помощью метода AnyAsync, так как не требует перечисления коллекции.

Устранение нарушений

Чтобы устранить нарушение, замените вызов метода CountAsync или LongCountAsync вызовом метода AnyAsync. В следующих двух фрагментах кода показано нарушение правила и способы его устранения:

using System.Linq;
using System.Threading.Tasks;
using static Microsoft.EntityFrameworkCore.EntityFrameworkQueryableExtensions;

class C
{
    public async Task<string> M1Async(IQueryable<string> list)
        => await list.CountAsync() != 0 ? "Not empty" : "Empty";

    public async Task<string> M2Async(IQueryable<string> list)
        => await list.LongCountAsync() > 0 ? "Not empty" : "Empty";
}

using System.Linq;
using System.Threading.Tasks;
using static Microsoft.EntityFrameworkCore.EntityFrameworkQueryableExtensions;

class C
{
    public async Task<string> M1Async(IQueryable<string> list)
        => await list.AnyAsync() ? "Not empty" : "Empty";

    public async Task<string> M2Async(IQueryable<string> list)
        => await list.AnyAsync() ? "Not empty" : "Empty";
}

Совет

Для этого правила в Visual Studio доступно исправление кода. Чтобы использовать его, поместите курсор на нарушение и нажмите клавиши CTRL+ (период). Выберите Не использовать CountAsync() или LongCountAsync(), если можно использовать AnyAsync() из списка.

Code fix for CA1828 - Do not use CountAsync() or LongCountAsync() when AnyAsync() can be used

Когда лучше отключить предупреждения

Вывод предупреждений для этого правила можно отключить, если вас не беспокоит влияние ненужного перечисления коллекции для подсчета количества.

Отключение предупреждений

Если вы просто хотите отключить одно нарушение, добавьте директивы препроцессора в исходный файл, чтобы отключить и повторно включить правило.

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

Чтобы отключить правило для файла, папки или проекта, задайте его серьезность none в файле конфигурации.

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

Дополнительные сведения см. в разделе Практическое руководство. Скрытие предупреждений анализа кода.

См. также