Краткая запись метода LINQ

@Alex_trader · Регистрация: 11.11.2015

Author24 — интернет-сервис помощи студентам

Доброго времени суток, форумчане!
Подскажите пожалуйста более компактную запись метода

C#

 [HttpPost]
        [Route("SearchForm")]
        public IHttpActionResult SearchForm([FromBody] dynamic jsonData)
        {
            if (!this.ModelState.IsValid)
                return this.BadRequest(this.ModelState);
            List<ClientsGridModel> res = new List<ClientsGridModel>();
            SearchFormData data = JsonConvert.DeserializeObject<SearchFormData>(jsonData.ToString(), setJsonSettings());
            ClientInfoDTO[] bbsppClients = this._bbsppService.SearchAutoCompleteClients(data.SearchTerm, 30);
 
            foreach (ClientInfoDTO bbsppClient in bbsppClients)
            {
                res.Add(new ClientsGridModel
                {
                    ClientGuid = bbsppClient.ClientGuid,
                    Fio = bbsppClient.Fio,
                    Gender = bbsppClient.Gender,
                    DateOfBirth = bbsppClient.DateOfBirth,
                    Phones = bbsppClient.Phones.Count() != 0 ? bbsppClient.Phones[0] : "",
                    Contacts = bbsppClient.Contacts.Count() != 0 ? bbsppClient.Contacts[0] : "",
                    Comment = bbsppClient.Comment
                });
            }
 
            ClientsGridModel[] advs = GetAdvSourceAndLastContact(bbsppClients);
 
            if (advs.Length != 0)
                foreach (ClientsGridModel item in res)
                    foreach (ClientsGridModel adv in advs)
                        if (item.ClientGuid == adv.ClientGuid)
                        {
                            item.adv_InviteId = adv.adv_InviteId;
                            item.LastContact = adv.LastContact;
                        }
 
            List<ClientsGridModel> filterGuids = this._dataDirectoryQueries.GetSearchFormClients(data).ToList();
 
                if (filterGuids.Count() != 0)
                        foreach (ClientsGridModel guid in filterGuids)
                                res.RemoveAll(x => x.ClientGuid != guid.ClientGuid);
            return this.Ok(res);
        }

Добавлено через 1 час 14 минут
Как убрать всю эту кучу форычей и переписать более красиво и компактно этот метод?

@bax_tang · 10.01.2017, 01:05

Alex_trader, чем Вам так foreach не понравился? нормальный же цикл (да и лучше, когда вызывается он явно - нет неопределенности).

@Alex_trader · 10.01.2017, 16:17 **[ТС]**

bax_tang, слишком длинный метод, хотелось бы Linq использовать, для более компактной его записи

Добавлено через 10 часов 10 минут
Немного изменил метод, осталось три форыча, собираю результирующий список, выбирая элементы из двух других, неужели не существует более компактной записи этих форычей?

C#

 ClientInfoDTO[] bbsppClients = this._bbsppService.SearchAutoCompleteClients(data.SearchTerm, 30);
 ClientsGridModel[] clientsAdvertisings = this._dataDirectoryQueries.GetClientsAdvSource(bbsppClients);
 ClientsGridModel[] clientsLastContact = this._dataDirectoryQueries.GetClientsLastContact(bbsppClients);
 
 foreach (ClientInfoDTO bbsppClient in bbsppClients)
            {
                res.Add(new ClientsGridModel
                {
                    ClientGuid = bbsppClient.ClientGuid,
                    Fio = bbsppClient.Fio,
                    Gender = bbsppClient.Gender,
                    DateOfBirth = bbsppClient.DateOfBirth,
                    Phones = bbsppClient.Phones.Count() != 0 ? bbsppClient.Phones[0] : "",
                    Contacts = bbsppClient.Contacts.Count() != 0 ? bbsppClient.Contacts[0] : "",
                    Comment = bbsppClient.Comment
                });
            }
 
            if (clientsAdvertisings.Length != 0)
                foreach (ClientsGridModel item in res)
                    foreach (ClientsGridModel adv in clientsAdvertisings)
                        if (item.ClientGuid == adv.ClientGuid)
                        {
                            item.adv_InviteId = adv.adv_InviteId;
                            item.adv_Invite = adv.adv_Invite;
                        }
 
            if (clientsLastContact.Length != 0)
                foreach (ClientsGridModel item in res)
                    foreach (ClientsGridModel cnt in clientsLastContact)
                        if (item.ClientGuid == cnt.ClientGuid)
                            item.LastContact = cnt.LastContact;

@kol · 12.01.2017, 18:18

Alex_trader, Я понимаю что ты хочешь уменьшить по минимуму код, но ставь подалуйста скобочки, а то он совершенно нечитаемый. Я дальше первого цикла да же читать не стал. Во что могу подсказать.

C#

            res.AddRange(bbsppClients.Select(clientInfo => new ClientsGridModel
            {
                ClientGuid = clientInfo.ClientGuid,
                Comment = clientInfo.Comment
            }));

@Alex_trader · 12.01.2017, 19:39 **[ТС]**

Сообщение от kol

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

C#

 res.AddRange(bbsppClients.Select(bbsppClient => new ClientsGridModel
            {
                ClientGuid = bbsppClient.ClientGuid,
                Fio = bbsppClient.Fio,
                Gender = bbsppClient.Gender,
                DateOfBirth = bbsppClient.DateOfBirth,
                Phones = bbsppClient.Phones.Count() != 0 ? bbsppClient.Phones.Where(x => x != "0").FirstOrDefault() : "",
                Contacts = bbsppClient.Contacts.Count() != 0 ? bbsppClient.Contacts[0] : "",
                Comment = bbsppClient.Comment
            }));

Добавлено через 2 минуты
kol, спасибо, а с этим что можно придумать?

C#

 if (clientsAdvertisings.Length != 0)
            {
                foreach (ClientsGridModel item in res)
                {
                    foreach (ClientsGridModel adv in clientsAdvertisings)
                    {
                        if (item.ClientGuid == adv.ClientGuid)
                        {
                            item.adv_InviteId = adv.adv_InviteId;
                            item.adv_Invite = adv.adv_Invite;
                        }
                    }
                }
            }

@kol · 12.01.2017, 22:12

Alex_trader, c LINQ вы не можете изменять саму коллекцию. Можно отфильтровать и записать в новую коллекцию.

C#

1
2
3

            clientsAdvertisings = clientsAdvertisings
                .Where(clientAdv => res.Any(item => item.ClientGuid == clientAdv.ClientGuid))
                .ToList();

@bax_tang .NET senior 441 / 359 / 137 Регистрация: 23.09.2016 Сообщений: 980
	10.01.2017, 01:05	2
	Alex_trader, чем Вам так foreach не понравился? нормальный же цикл (да и лучше, когда вызывается он явно - нет неопределенности). 0

	12.01.2017, 22:12