Fetch Несколько элементов из linq

Question

У меня есть код устаревшего кода в моем приложении и вы хотите его оптимизировать. arrayOfAttrValue имеет уникальные атрибуты. Можно ли использовать LINQ для оптимизации цикла? Если да, тогда вы можете показать мне, как?

foreach (AttrValue attr in arrayOfAttrValue) 
{ 
    switch(attr.Attribute) 
    { 
     case Constants.Gender 
      mymodel.Gender = attr.Value; 
      break; 
     case Constants.Identifier 
      mymodel.AppIdentifier = attr.Value; 
      break; 
    } 
} 

Мое намерение не обязательно использовать только LINQ. Любой другой способ свести к минимуму цикл также поможет.

Спасибо.

Answer 1

Нет, вы не можете сделать это в «истинном» LINQ, потому что LINQ занимается производством новых объектов из старых объектов. Здесь mymodel является ранее существовавшим объектом, который вы хотите изменить.

Вы можете использовать Array.ForEach или List.ForEach но

1. Они не «истинный» LINQ и

2. Полученный код будет эквивалентен (немного медленнее, потому что там будет делегатом)

Тем не менее, вероятно, хотел downvoter некоторые LINQ, так что я дам некоторые LINQ:

arrayOfAttrValue.All(attr => { 
    mymodel.Gender = attr.Attribute == Constants.Gender ? attr.Value : mymodel.Gender; 
    mymodel.AppIdentifier = attr.Attribute == Constants.Identifier ? attr.Value : mymodel.AppIdentifier; 
    return true; 
}); 

Еще одна линия, игнорирующая линии {}.

Answer 2

У вас есть список атрибутов, представляющих пары ключ-значение. Естественным способом сохранения таких структур данных является словарь. Таким образом, преобразовать формат ввода данных в словарь:

var attributes = arrrayOfAttrValue.ToDictionary(a => a.Attribute, a => a.Value); 

Или, если каждый атрибут не является уникальным в массиве, словарь создание будет сложнее, но это формат данных у вас есть. Для того, чтобы работать с вашими данными проще, вы должны преобразовать их в удобный формат:

var attributes = arrayOfAttrValue 
        .GroupBy(a => a.Attribute)     
        .ToDictionary(g => g.Key, g => g.Select(a => a.Value).Last()); 

После создания словаря атрибутов, вы можете просто проверить, если у вас есть значение атрибута и присвоить это значение для моделирования свойства. Атрибуты извлечения Теперь простым и понятным для любого читателя:

if (attributes.ContainsKey(Constants.Gender)) 
    model.Gender = attributes[Constants.Gender]; 

if (attributes.ContainsKey(Constants.Identifier)) 
    model.AppIdentifier = attributes[Constants.Identifier]; 

Answer 3

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

bool gotGender, gotAppIdentifier; 
foreach (AttrValue attr in arrayOfAttrValue.Reverse()) 
{ 
    switch(attr.Attribute) 
    { 
    case Constants.Gender: 
     if(!gotGender){ 
     mymodel.Gender = attr.Value 
     gotGender = true; 
     } 
     break; 
    case Constants.Identifier: 
     if(!gotAppIdentifier){ 
     mymodel.AppIdentifier = attr.Value 
     gotAppItentifier = true; 
     } 
     break; 
    } 
    if(gotGender&&gotAppIdentifier) break; 
} 

Answer 4

Нет необходимости сделать петлю вручную в коде, вы можете сделать это просто с .LastOrDefault():

mymodel.Gender = arrayOfAttrValue 
    .Where(attr => attr.Attribute == Constants.Gender) 
    .Select(attr => attr.Value).LastOrDefault() ?? mymodel.Gender; 
mymodel.AppIdentifier = arrayOfAttrValue 
    .Where(attr => attr.Attribute == Constants.Identifier) 
    .Select(attr => attr.Value).LastOrDefault() ?? mymodel.AppIdentifier; 

?? mymodel.Gender убеждается мы не ставим его Default(T) (т.е. null) в ситуации, когда в противном случае оно было установлено ранее. Затем это соответствует функциональной логике вашего первоначального вопроса.

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

Если у вас есть проблемы с производительностью это, то вы, вероятно, захотите рассмотреть вопрос об использовании лучшей структуры данных, чем arrayOfAttrValue (что-то, что является индексным доступом к которой Attribute таким как Dictionary<,>).