Anche i controlli "IsNullOrEmpty" danno "possibili enumerazioni multiple di avvisi IEnumerable"

Question

C'è già uno question on SO about "possible multiple enumerations", ma questa domanda è più specifica.

perche il seguente metodo, che prende un IEnumerable<string> come input ed esegue un dato metodo contro ciascuno dei suoi elementi:

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

Nel codice precedente, IsNullOrEmpty è solo un metodo di estensione che corre

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

Il problema è che ReSharper mi avvisa di "Possibili enumerazioni multiple di IEnumerable" e davvero non so se questo può effettivamente essere un problema o meno.

Capisco il significato dell'avviso, ma cosa potresti fare davvero in questa situazione se davvero hai bisogno di controllare e lanciare un'eccezione in caso di nullità o vuoto?

Answer 1

Significa che si sta effettuando (parzialmente) iterando su IEnumerable più di una volta: prima nella propria chiamata a Any() (che deve almeno inizializzare un'iterazione per verificare se l'enumerable restituisce qualsiasi elemento) e una seconda volta in All (che itera dall'inizio).

La ragione per cui ReSharper ti avverte è che elencare su un numero enumerabile può causare effetti collaterali e due volte l'iterazione involontaria può attivare due volte gli effetti collaterali, il che può essere o meno desiderabile.

Answer 2

Come identifica @tdammers, le "enumerazioni multiple" a cui si fa riferimento sono le due enumerazioni richieste da Any e All. Dal momento che si desidera rifiutare una sequenza vuota, il meglio che riesco a trovare è:

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; 
}