Комплекс GROUP BY на DataTable

Question

Комплекс GROUP BY на DataTable

1

У меня есть сложный объект CostPageDTO, как показано ниже:

public class CostPageDTO
{
    public string CostPageNumber { get; set; }
    public string Description { get; set; }
    public char OrderType { get; set; }
    public string VendorName { get; set; }
    public List<ItemDTO> Items { get; set; }
}

public class ItemDTO
{
    public string BrandCode { get; set; }
    public string ItemDescription { get; set; }
    public string ItemID { get; set; }
}

Нам нужно создать List<CostPageDTO> из datatable. Я начал, как показано ниже: но я не уверен, как применить предложение GROUP BY здесь, чтобы создать List<ItemDTO> внутри CostPageDTO.

DataTable table = new DataTable();
SqlDataReader reader = command.ExecuteReader();
table.Load(reader);
reader.Close();

List<CostPageDTO> costPages = new List<CostPageDTO>();
Parallel.ForEach(table.AsEnumerable(), (dr) =>
{
    costPages.Add(new CostPageDTO()
    {
        CostPageNumber  = dr[0].ToString(),
        Description = dr[1].ToString(),
        OrderType = Convert.ToChar(dr[2].ToString()),
        VendorName = dr[3].ToString()
    });
});

Как мы можем создать необходимый список из DataTable

ЛИТЕРАТУРЫ

Как преобразовать DataTable в общий список?

Lijo 09 дек. 2013, в 15:06

Источник

0

вы используете ef или linq to sql?
Daniel A. White 09 дек. 2013, в 13:49
1

Вы также должны рассмотреть возможность использования ConcurrentBag<T> . List<T> не является потокобезопасным.
Daniel A. White 09 дек. 2013, в 13:50
0

Почему бы не изменить функцию DAL, чтобы использовать Linq2SQL или платформу сущностей, и она могла бы возвращать список напрямую вместо DataTable. Также List <T> не является потокобезопасным, поэтому вы можете использовать его таким образом.
Magnus 09 дек. 2013, в 13:51
0

@ DanielA.White Я получаю данные из SqlDataReader. SqlDataReader reader = command.ExecuteReader(); table.Load(reader);
Lijo 09 дек. 2013, в 13:51
0

зачем тебе параллель?
Daniel A. White 09 дек. 2013, в 14:52
0

@ DanielA.White Чтобы ускорить обработку ... Если это неправильная мысль, у меня тоже есть непараллельный подход.
Lijo 09 дек. 2013, в 14:55
1

@Lijo Такая операция будет узким местом при вводе-выводе, а не при обработке. Вероятно, нет причин распараллеливать что-то подобное. Шансы довольно высоки, что вы не только делаете свою программу намного более сложной, но на самом деле делаете ее медленнее. Кроме того, другие уже упоминали, что вы не синхронизируете должным образом, так что это даже не (обязательно) приведет к правильному результату.
Servy 09 дек. 2013, в 15:15
0

@Lijo Просто используйте foreach а не parallel.ForEach ForEach ...
Servy 09 дек. 2013, в 15:34
0

@Servy. Поскольку у меня нет ясности в утверждении, которое является bottlenecked at the IO, not the processing могли бы опубликовать ответ, уточняющий это.
Lijo 09 дек. 2013, в 16:18
1

@Lijo Сетевой запрос, который вы отправляете в БД для получения данных, будет на много порядков медленнее, чем время, потраченное на обработку результата. Когда эта обработка составляет 0,001% времени выполнения, сокращение этого времени вдвое или более фактически не влияет на время выполнения вашего приложения.
Servy 09 дек. 2013, в 16:20

Показать ещё 8 комментариев

Теги:

c#

linq

1 ответ

Ещё вопросы

Вы также должны рассмотреть возможность использования ConcurrentBag<T> . List<T> не является потокобезопасным.
Почему бы не изменить функцию DAL, чтобы использовать Linq2SQL или платформу сущностей, и она могла бы возвращать список напрямую вместо DataTable. Также List <T> не является потокобезопасным, поэтому вы можете использовать его таким образом.
@ DanielA.White Я получаю данные из SqlDataReader. SqlDataReader reader = command.ExecuteReader(); table.Load(reader);
@ DanielA.White Чтобы ускорить обработку ... Если это неправильная мысль, у меня тоже есть непараллельный подход.
@Lijo Такая операция будет узким местом при вводе-выводе, а не при обработке. Вероятно, нет причин распараллеливать что-то подобное. Шансы довольно высоки, что вы не только делаете свою программу намного более сложной, но на самом деле делаете ее медленнее. Кроме того, другие уже упоминали, что вы не синхронизируете должным образом, так что это даже не (обязательно) приведет к правильному результату.
@Lijo Просто используйте foreach а не parallel.ForEach ForEach ...
@Servy. Поскольку у меня нет ясности в утверждении, которое является bottlenecked at the IO, not the processing могли бы опубликовать ответ, уточняющий это.
@Lijo Сетевой запрос, который вы отправляете в БД для получения данных, будет на много порядков медленнее, чем время, потраченное на обработку результата. Когда эта обработка составляет 0,001% времени выполнения, сокращение этого времени вдвое или более фактически не влияет на время выполнения вашего приложения.

Petko Petkov · Accepted Answer · 2013-12-09T14-28-00.000Z

Я предполагаю, что ваш запрос базы данных возвращает соединение между CostPage и Item. Если это так, сначала вам нужно сгруппировать свои строки, чтобы получить значения для CostPage, после этого проекта для вашего типа DTO. Я действительно сомневаюсь, что вы увидите много преимуществ при распараллеливании кода.

Ваш код должен выглядеть примерно так:

costPages = table.AsEnumerable().GroupBy(dr=> new 
            {
                CostPageNumber  = dr[0].ToString(),
                Description = dr[1].ToString(),
                OrderType = Convert.ToChar(dr[2].ToString()),
                VendorName = dr[3].ToString()
            })
            .Select(x => new CostPageDTO(){
                CostPageNumber = x.Key.CostPageNumber,
                Description = x.Key.Description,
                OrderType = x.Key.OrderType,
                VendorName = x.Key.VendorName,
                Items = x.Select(dr=> new ItemDTO{
                    //ItemDTO mapping goes here
                    ItemID=dr[Constants.SearchPage.ITMID].ToString()
                }).ToList()
            }).ToList();

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