c# - Incluso las comprobaciones de "IsNullOrEmpty" dan advertencias de "Posible enumeración múltiple de IEnumerable"

resharper

Question

Ya hay una pregunta en SO sobre "posibles enumeraciones múltiples" , pero esta pregunta es más específica.

Considere el siguiente método, que toma una IEnumerable<string> como entrada y ejecuta un método dado contra cada uno de sus elementos:

public static bool SomeMethod(IEnumerable<string> enumerable)
{
    if (enumerable.IsNullOrEmpty())
    {
        // throw exception.
    }
    else
    {
        return (enumerable.All(SomeBooleanMethod));
    }
}

En el código anterior, IsNullOrEmpty es solo un método de extensión que se ejecuta

return (!ReferenceEquals(enumerable, null) || enumerable.Any());

El problema es que ReSharper me advierte acerca de "Posibles enumeraciones múltiples de IEnumerable", y realmente no sé si esto realmente puede ser un problema o no.

Entiendo el significado de la advertencia, pero ¿qué podría hacer realmente en esta situación si realmente necesita verificar y lanzar una excepción en caso de nulidad o vacío?

Answer 1

Como @tdammers identifica, las "enumeraciones múltiples" a las que se hace referencia son las dos enumeraciones requeridas por Any y All . Ya que quieres rechazar una secuencia vacía, lo mejor que puedo hacer es:

public static bool SomeMethod(IEnumerable<string> enumerable)
{
    if (enumerable == null)
        throw new ArgumentNullException();

    // Manually perform an All, keeping track of if there are any elements
    bool anyElements = false;

    bool result = true;

    foreach (string item in enumerable)
    {
        anyElements = true;
        result = result && SomeBooleanMethod(item);

        // Can short-circuit here
        if (!result)
            break;
    }

    if (!anyElements)
        throw new ArgumentException();    // Empty sequence is invalid argument

    return result;
}

Answer 2

Significa que estás iterando (parcialmente) sobre IEnumerable más de una vez: primero en tu llamada a Any() (que necesita al menos inicializar una iteración para ver si el enumerable devuelve algún elemento), y una segunda vez en All ( que se repite desde el principio).

La razón por la que ReSharper le advierte acerca de esto es que la enumeración sobre un número enumerable puede causar efectos secundarios, y una iteración involuntaria dos veces puede desencadenar los efectos secundarios dos veces, lo que puede o no ser deseable.