Loading

Get List consume time

Get List consume time

5y655Learn iOS Programming

I have get data from two table using join in linq c# and show a data list in front end .But its very consume time

I want to not time consume

it's me code

var Result = (from UM in _context.UserP
join C in _context.ContactUs on UM.UserId equals C.UserId
where UM.UserId == C.UserId
select new
{
UM.UserImage,
C.Name,
C.UpdatedDate,
C.Email,
C.Message,
C.ContactUsId
}).ToList();
if (Result.Count > 0)
{
foreach (var item in Result.OrderByDescending(t => t.Name).ToList())
{
ReportManagementViewModel obj = new ReportManagementViewModel();
obj.UserName = item.Name;
obj.Email = item.Email;
DateTime aDate = Convert.ToDateTime(item.UpdatedDate);
obj.Date = aDate.ToString("MMMM dd, yyyy");
obj.Message = item.Message;
obj.UserImage = item.UserImage;
obj.ContactUsId = item.ContactUsId;
List.Add(obj);

1 Reply

Know the answer? Post it — somebody with the same question will find it here.

Sachin Singh5y ago
Accepted answer
Remove ToList and perform OrderBy in the query it self, filtering on database is always faster than in memory
var Result = from UM in _context.UserP
join C in _context.ContactUs on UM.UserId equals C.UserId
where UM.UserId == C.UserId
order by c.Name desc
select new
{
UM.UserImage,
C.Name,
C.UpdatedDate,
C.Email,
C.Message,
C.ContactUsId
})
if (Result.Count > 0)
{
foreach (var item in Result)
{
ReportManagementViewModel obj = new ReportManagementViewModel();
obj.UserName = item.Name;
obj.Email = item.Email;
DateTime aDate = Convert.ToDateTime(item.UpdatedDate);
obj.Date = aDate.ToString("MMMM dd, yyyy");
obj.Message = item.Message;
obj.UserImage = item.UserImage;
obj.ContactUsId = item.ContactUsId;
List.Add(obj);

}