Перечисление Linq Call занимает 5 минут - PullRequest
Новая
       14

Перечисление Linq Call занимает 5 минут

0 голосов
/ 11 июня 2018

У меня проблема с производительностью этого кода, я читаю из базы данных, а затем преобразовываю DataTable в список CampaignRecipientLib, и при вызове этого метода требуется минимум 5 минут: 

private static List<CampaignRecipientLib> GetGroupRcipientsToSchedule(int GroupId, int CustomerId, bool Delete = false, bool NewlyAddedCampaignGroupRecipient = false)
        {
            List<SqlParameter> param = new List<SqlParameter>();
            param.Add(new SqlParameter("@GroupID", GroupId));
            param.Add(new SqlParameter("@CustomerId", CustomerId));
            param.Add(new SqlParameter("@Delete", Delete));
            param.Add(new SqlParameter("@NewlyAddedCampaignGroupRecipient", NewlyAddedCampaignGroupRecipient));

            Stopwatch st = new Stopwatch();
            st.Start();
            var rec_names = SqlHelper.GetDataTable("[dbo].[p_GetRecipientListByGroupCode]", param, CommandType.StoredProcedure);
            st.Stop();
            return rec_names.AsEnumerable().Select(row =>
                 new CampaignRecipientLib
                 {
                     ID = row.Field<int>("cgr_RecipientId"),
                     CMPRID = 0,//row.Field<int>("cr_CampaignRecipientId"),
                     CMPRCode = codePrefix + Guid.NewGuid().ToString("N"),//row.Field<string>("cr_CampaignRecipientCode"),
                     CampaignId = 0,
                     SentCampaigns = string.IsNullOrEmpty(row.Field<string>("GroupScenarioList")) ? new List<int>() : row.Field<string>("GroupScenarioList").Split(',').Select(int.Parse).ToList(),
                     EmailTo = row.Field<string>("r_Email"),
                     Email = row.Field<string>("r_Email"),
                     FirstName = row.Field<string>("r_FirstName"),
                     LastName = row.Field<string>("r_LastName"),
                     Language = string.IsNullOrEmpty(row.Field<string>("r_LangCode")) ? null : row.Field<string>("r_LangCode"),
                     Scheduled = DateTime.Now,//row.Field<DateTime?>("cr_Scheduled").HasValue ? row.Field<DateTime?>("cr_Scheduled") : DateTime.Now,
                     IsdoubleBarrle = false,//row.Field<bool>("cr_IsDoubleBarrel"),
                     Offset = string.IsNullOrEmpty(row.Field<string>("r_Offset")) ? null : row.Field<string>("r_Offset"),
                     Toffset = row.Field<string>("r_Offset").StartsWith("-") ? -TimeSpan.Parse(row.Field<string>("r_Offset").Remove(0, 1)) : TimeSpan.Parse(row.Field<string>("r_Offset")),
                     ReadyTobeSent = false,//row.Field<bool>("cr_ReadyTobeSent"),
                     PickupReady = false,//row.Field<bool>("cr_PickupReady"),
                     DefaultLanguage = string.Empty//string.IsNullOrEmpty(row.Field<string>("cmp_LangCode")) ? null : row.Field<string>("cmp_LangCode")
                 }
                 ).ToList();
        }

Тот же звонок, в другом проекте супер быстрый, меньше секунды!Я имею в виду под другим проектом: служба Windows, и текущий проект, где это занимает больше времени: Приложение веб-формы Asp

Любая подсказка или совет?

Примечание: вызов базы данных занимает1,5 секунды, чтобы вернуть DataTable с 4000 строками, но часть перечисления занимает большую часть времени 

return rec_names.AsEnumerable().Select(row =>
                 new CampaignRecipientLib
                 { .... }).ToList();

=========================

Обновление:

Вот скриншот системы A, где она работает медленно:

system A where it's slow ЭтоВозможно, что отмеченная линия является причиной?потому что я конвертирую содержимое этого столбца в значение int, разделенное запятыми

, вот скриншот системы B, где он работает очень быстро, но мне не нужно связывать столбец GroupScenarioList в этом вызове

system B

1 Ответ

0 голосов
/ 14 июня 2018

Я нашел причину

codePrefix , являющуюся свойством, и он вызывает хранимую процедуру каждый раз для каждой строки в dataTable, 

 return rec_names.AsEnumerable().Select(row =>
                 new CampaignRecipientLib
                 {
                     ID = row.Field<int>("cgr_RecipientId"),
                     CMPRID = 0,//row.Field<int>("cr_CampaignRecipientId"),
                     CMPRCode = codePrefix + Guid.NewGuid().ToString("N"),//row.Field<string>("cr_CampaignRecipientCode"),
                     CampaignId = 0,
                     ... ....

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

...