Загрузить список более эффективно

Question

Мне было поручено обновить некоторые устаревшие коды и действительно нужно уменьшить время выполнения на части. Этот список загружается довольно часто. Мне удалось сократить время от примерно минуты до пятнадцати секунд, но на самом деле нужно идти дальше. Следующие строки кода работают достаточно хорошо, но я пытаюсь сжать все, что в моих силах.

List<MyObject> _moList = new List<MyObject>(DB.GetAll(queryString, parameters, MyObject.Extract)); 
_moList.AsParallel().ForAll(s => s.RelativeCost = GetRelativeCost(s)); 

У меня есть несколько вопросов. Во-первых, можно ли объединить эти две строки в одну строку, и если да, то как? Во-вторых, будет ли это улучшать производительность вообще?

Некоторые элементы не в том, что MyObject имеет около сорока свойств (не уверен, что это уместно или нет), и GetRelativeCost несколько дороже, когда дело доходит до циклов времени/процессора (отсюда параллельный запуск).

Любая помощь была бы принята с благодарностью!

P.S. Я также работаю с другими углами, в частности, пытаясь уменьшить «стоимость» GetRelativeCost, но мне нужно получить каждый цикл, который я могу извлечь из этого, чтобы сделать его приемлемым для пользователя.

Answer 1

Вы можете объединить их в одну строку, удалив переменную и вызывая AsParalel по результату нового.

(new List<MyObject>(DB.GetAll(queryString, parameters, MyObject.Extract))).AsParallel().ForAll(s => s.RelativeCost = GetRelativeCost(s)); 

Но, не ожидайте, что это улучшит производительность вашего кода.

Answer 2

1. У вас есть профайлер? Вы уверены, что оптимизируете узкое место?
2. Почему вы пытаетесь оптимизировать обработку списка, а не GetRelativeCost?
3. Что такое DB.GetAll? Вы уверены, что он работает оптимально?
4. Предполагаю ли я, что правильный MyObject.Extract - это какая-то функция гидратации объекта? Вы уверены, что это оптимально?
5. Почему GetRelativeCost Выполняется по списку loading? Не имеет смысла откладывать его исполнение?

Подробнее о последнем пункте. Если у вашего GetRelativeCost есть некоторые побочные эффекты - вы все равно должны их удалить, так как это нарушение принципа СУХОЙ. Если нет, то вы можете переписать RelativeCost для обеспечения отложенной инициализации, как

private int? _relativeCost 
public int RelativeCost { 
    get { 
     if (!_relativeCost.HasValue) 
      _relativeCost = GetRelativeCost(); 
     return _relativeCost; 
    } 
} 

Таким образом, вы задержать GetRelativeCost выполнения, пока это действительно не нужно. таким образом, вы оптимизируете, не вычисляя GetRelativeCost для объектов, которые действительно не нуждаются в RelativeCost, а также ускоряют этот конкретный кусок кода, задерживая дорогостоящие вычисления в GetRelativeCost по стоимости увеличенных вычислений позже.

Подводя итог: проложить профайлер, определить узкое место, оптимизировать узкое место. Если узкое место, похоже, находится в GetRelativeCost - попробуйте сделать его ленивым, как я описал выше.