我有一个视图模型类,用于获取具有其商品列表的客户列表。
餐厅Viewmodel
public class RestaurantsListVM
{
public Client client { get; set; }
public List<Offer> offers { get; set};
}
客户端模型
public class Client
{
public Guid Id { get; set; }
public String RestaurantName { get; set; }
}
优惠模式
public class Offer
{
public Guid Id { get; set; }
public String OfferName { get; set; }
public decimal OfferPercentage { get; set; }
在我的数据库中,我有一个 ClientOffer表,它还会向客户提供以下内容:
***"ClientOfferId,ClientId,OfferId"***
所以我创建了这个函数来从数据库中检索数据。
public List<RestaurantsListVM> GetRestaurants()
{
List<RestaurantsListVM> restaurantlist = new List<RestaurantsListVM>();
var clients = new Client().GetClients();
foreach (Client c in clients)
{
RestaurantsListVM restaurantdetails = new RestaurantsListVM();
restaurantdetails.client = c;
restaurantdetails.offers = new Offer().GetOffers(c.Id);
restaurantlist.Add(restaurantdetails);
}
return restaurantlist;
}
工作正常。但问题是它在sql server中一次又一次地执行查询,同时检索每个客户端的商品,性能下降。
我应该如何提高代码效率以获得更好的性能?
答案 0 :(得分:1)
为什么不为ClientOffer创建模型?
public class ClientOffer
{
public Guid client_id { get; set; }
[ForeignKey("client_id")]
public Client client { get; set; }
public Guid offer_id { get; set; }
[ForeignKey("offer_id")]
public Offer offer { get; set; }
}
在您的客户端模型中,您可以添加优惠集
public class Client
{
public Guid Id { get; set; }
public String RestaurantName { get; set; }
public ICollection<ClientOffer> offers { get; set; }
}
因此,您可以从模型客户端
循环提供属性答案 1 :(得分:1)
您需要一个LINQ查询,它将在一个SQL查询中连接表。目前,您获得所有客户,然后为每个客户获得他们的报价。类似的东西:
var clientOffers = (from cli in dbContext.Clients
join cli_ofr in dbContext.ClientOffer
on cli.ClientId
equals cli_ofr.ClientId
join ofr in dbContext.Offers
on cli_ofr.OfferId
equals ofr.OfferId
select new {
Client = new Client { Guid = cli.Guid...},
Offer = new Offer { Guid = ofr.Guid... }
}).toList();
这将生成一个SQL查询,它将返回创建视图模型所需的所有数据。