Всем привет, подскажите пожалуйста, у меня есть метод, работает как

Question

Всем привет, подскажите пожалуйста, у меня есть метод, работает как

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

#dot #programming #russian

0

25.05.2023

11 ответов

11 просмотров

Dmitry

как минимум, перед циклом вытащи всех юзеров, что есть в наборе тикетов. и в цикле уже к листу этому обращайся. а то у тебя будет 100 тикетов с одним юзером и ты на каждом будешь ходить в базу ещё раз

0

25.05.2023

Ярослав Автор вопроса

Dmitry
как минимум, перед циклом вытащи всех юзеров, что ...

понял, спасибо

0

25.05.2023

Andrew

Вроде это можно написать 1 запросом к базе

0

25.05.2023

Ярослав Автор вопроса

Andrew
Вроде это можно написать 1 запросом к базе

Как, подскажи пожалуйста?

0

25.05.2023

Andrew

Ярослав
Как, подскажи пожалуйста?

Скинь код текстом

0

25.05.2023

Ярослав Автор вопроса

Andrew
Скинь код текстом

public async Task<IEnumerable<TicketDto>> GetAllTickets(int projectId) { var tickets = await _ticketRepository.GetAllTicketsAsync( projectId); foreach(var t in tickets) { var fullNameUser = await _userRepository.GetUserAsync(t.CreatedByUserId); t.CreatedByUserId = $"{fullNameUser.FirstName} {fullNameUser.LastName}"; } return _mapper.Map<IEnumerable<TicketDto>>(tickets); }

0

25.05.2023

Andrew

Ярослав
public async Task<IEnumerable<TicketDto>> GetAllTi...

public async Task<IEnumerable<TicketDto>> GetAllTickets(int projectId) { var tickets = await _context.Tickets .Where(t => t.ProjectId == projectId) .Select(t => new TicketDto { Id = t.Id, Title = t.Title, Description = t.Description, Status = t.Status, CreatedByUserId = t.CreatedByUser.FirstName + " " + t.CreatedByUser.LastName }) .ToListAsync(); return tickets; }

0

25.05.2023

Ярослав Автор вопроса

Andrew
public async Task<IEnumerable<TicketDto>> GetAllTi...

Спасибо большое👍

0

25.05.2023

Andrew

Ярослав
Спасибо большое👍