Как этот код может быть оптимизирован?

Question

Я только что получил код, переданный мне. Код написан на C#, и он вставляет данные в реальном времени в базу данных каждую секунду. Данные накапливаются во времени, что делает их большими.

Данные обновляются в течение второго много раз, затем в конце второго результата берется и вставляется.

Мы использовали адреса строк данных непосредственно во второй части через свойства. Например, многие операции, такие как этот datavaluerow.meanvalue + = mean; может иметь место. , мы выяснили, что это ухудшает производительность после запуска профилировщика из-за внутреннего выполнения, поэтому мы создали 2-й массив десятичных знаков, на которых выполняются обновления, тогда значения присваиваются датарам только в конце второго. Я запустил профайлер и узнал, что он все еще занимает много времени (хотя время, затраченное на то, что время, когда оно было добавлено, часто обращалось к datarows).

код, который exectued в конце второго заключается в следующем

public void UpdateDataRows(int tick) 
{ 
    //ord 
    //_table1Values is of type decimal[][] 
    for (int i = 0; i < _table1Values.Length; i++) 
    { 
    _table1Values[i][(int)table1Enum.barDateTime] = tick; 
    table1Row[i].ItemArray = _table1Values[i].Cast<object>().ToArray(); 

    } 
    // this process is done for other 10 tables    
} 

Есть ли способ улучшить этот подход.

Answer 1

Один очевидный вопрос: почему у вас есть 2D массив десятичных знаков, когда вы только обновляете их целыми числами? Не могли бы вы уйти с int[][]?

Далее, почему вы получаете доступ к (int)table1Enum.barDateTime на каждой итерации? Учитывая, что там есть конверсия, вы можете найти may найти, если вы извлечете это из цикла.

Однако, я подозреваю, что большая часть времени будет потрачена на _table1Values[i].Cast<object>().ToArray(). Вам действительно нужно это делать? Получение копии decimal[] (или int[]) будет быстрее, чем бокс, каждое значение на каждой итерации при каждом вызове, а затем создание другого массива.