Warum wird Single() nicht direkt zurückgegeben, wenn mehr als ein Element gefunden wird?

Question

fand ich (grob) diesen Code in das Enumerable.Single Verfahren, während es mit einigen Decompiler Inspektion:

foreach (TSource current in source) 
{ 
    if (predicate(current)) 
    { 
     result = current; 
     num += 1L; 
    } 
} 

if (num > 1L) 
{ 
    throw Error.MoreThanOneMatch(); 
} 

Wie Sie sehen können, ist es über alle Elemente Schleifen vor dem Wurf. Warum bricht es nicht wenn num > 1?

Answer 1

Agree, dass es von Sachen Leistung besser sein wird (EDIT: Wenn wir unser Prädikat mehr als ein Element passend zu erwarten, was wir nicht tun sollten):

foreach (TSource current in source) 
{ 
    if (predicate(current)) 
    { 
     result = current; 
     num += 1L; 

     if (num > 1L) 
      throw Error.MoreThanOneMatch(); 
    } 
} 

if (num == 0L) 
    throw Error.NoMatch(); 

return local; 

Sieht aus wie sie beschlossen zu machen Ergebnisse analysieren klarer und trennte es von der Aufzählung der Quelle. Aber dann frage ich mich, warum einfache Schalter nicht verwendet wurden:

switch((int)num) 
{ 
    case 0: throw Error.NoMatch(); 
    case 1: return local; 
    default: 
     throw Error.MoreThanOneMatch();  
} 

In Bezug auf Performance-Probleme - ich denke, es ist angenommen, dass Single sollte aufgerufen werden, wenn Sie sind wirklich einzelnes Ergebnis erwarten. Null oder mehr Ergebnisse ist ein Ausnahmepfad, der nicht häufig vorkommen sollte (wie jede Ausnahme). Es ist also mehr der logische Fehler Ihres Programms, wenn die Quelle viele Elemente enthält, die mit dem Prädikat übereinstimmen.

Answer 2

Von Single ist gemeint, genau eine, nicht keine und auch nicht mehr als eine.
Es zählt alle Artikel, um sicherzustellen, dass es nur eine ist.
Es wird eine Ausnahme ausgelöst, wenn es keine oder mehr als eine gibt.
SingleOrDefault stattdessen wirft, wenn es mehr gibt, aber default(T)/null zurückgibt, wenn es keine gibt.

Was Sie suchen, ist FirstOrDefault, die die Aufzählung bricht, wenn es die erste passend zum Prädikat fand. First wirft stattdessen, wenn es keine gibt, und bricht auch (direkt zurück) es ist foreach, wenn es den ersten gefunden hat.

FirstOrDefault der Quelle

foreach (TSource current in source) 
{ 
    if (predicate(current)) 
    { 
     return current; 
    } 
} 
return default(TSource); 

Während Erste der Quelle statt Rückkehr Standard ist

throw Error.NoMatch();