C# .net Декоратор шаблона преобразования/установки коллекции

Question

У вас есть вопрос, как лучше оптимизировать или ускорить процесс. В настоящее время мой код, кажется, работает немного медленно ...

У меня есть следующие классы

public class DataFoo : IFoo { } 

public class Foo 
{ 
    internal IFoo UnderlyingDataObject{get;set;} 

    public Foo(IFoo f) 
    { 
      UnderlyingDataObject = f; 
    } 
} 

Теперь, во многих случаях, я в конечном итоге нужно или вызов метода, который будет предоставлять обратно список. Этот метод первоначально будет получить массив объектов DataFoo и перебрать все возвращаемые объекты инстанцирования нового объекта Foo, проходящий в DataFoo ... Вот пример ...

public List<Foo> GetListOfFoo(Guid id) 
{ 
    DataFoo[] q = GetArrayOfDataFoo(id); 
    List<Foo> rv = new List<Foo>(); 

    for(var i = 0; i < q.Length; i++) 
    { 
      rv.Add(new Foo(q[i])); 
    } 
    return rv; 
} 

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

Answer 1

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

Ваше определение GetListOfFoo можно было бы улучшить немного, чтобы избежать ненужного изменения размеров List<Foo>, указав начальную емкость:

DataFoo[] q = GetArrayOfDataFoo(id); 
List<Foo> rv = new List<Foo>(q.Length); 

Но если не вы имеете дело с очень большими массивами и относительно себя с очень малыми периодами времени, то это не будет иметь большого значения.

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

Хотелось посмотреть, что GetArrayOfDataFoo делает. Я предполагаю, что ваша проблема происходит за пределами того, что вы нам показали.