LINQ Использование Single with Try..Catch vs First with Count! = 1

Question

Каким будет оптимальный способ получить единственный элемент в списке? если он не равен единице, запишите его.

Лучше ли использовать блок try..catch против Single? или использовать граф?

try 
{ 
    var item = list.Single(); 
} 
catch(System.InvalidOperationException) 
{ 
    //log 
    Console.WriteLine("The collection does not contain exactly one element."); 
} 

или

if(list.Count!=1) 
{ 
    //log 
    Console.WriteLine("The collection does not contain exactly one element."); 
} 
var item = list.FirstOrDefault(); 

Answer 1

Ну try {...} catch {...} будет делать трассировки стека и это означает много накладных. Итак, вторая возможность - лучшая (Count - хороший выбор при работе с List<T>). На самом деле, вам не нужно Linq на все:

// list.Count is just a integer field, a very cheap comparison 
    if (list.Count != 1) { 
    // 0 or many items (not a single one) 
    ... 
    } 
    else { 
    // list contains exactly one item 
    var item = list[0]; 
    ... 
    } 

остерегайтесь ловушки как это:

List<Object> list = new List<Object>() { 
    null, 
    123 
    }; 

    // item == null even if list has TWO items 
    var item = list.FirstOrDefault(); 

Answer 2

if (list == null || list.Count != 1) 
{ 
    Console.WriteLine("The collection does not contain exactly one element."); 
} 

Как прокомментировал Iridium, Single() или SingleOrDefault() не может быть лучший выбор здесь, потому что он бросает исключение, если есть более чем один элемент или список равен нулю.

я заменил, что с null -check и сравнение с count

Answer 3

Этот способ лучше

if(list.Count!=1) 
{ 
    //log 
    Console.WriteLine("The collection does not contain exactly one element."); 
} 
var item = list.FirstOrDefault();