1

一連のショップから売上を取得し、それらをリストとして返すこのクエリがあります。レポート要求画面で、ユーザーは多数の ID (ShopOwnerId、ShopRegionId、ShopTypeCode など) でフィルタリングできます。

私のクエリは、選択した日付間のすべての売上を取得し、選択に応じて絞り込むように構成されています。これは明らかに非常に非効率的です。:

private List<Sales> GetFilteredListOfSales(Request reportreq)
{
    ModelContainer ctn = new ModelContainer();
    List<ShopSale> shopsSales = new List<shopsale>();

    // If no filters are selected
    //
    if (reportreq.RegionalId == null && reportreq.OwnerId == null && reportreq.ShopTypeCode == null)
    {
        shopsSales = (from sale in ctn.ShopSales
                    where sale.DateSold >= reportreq.FromDate && sale.DateSold <= reportreq.ToDate
                    select sale).ToList();
    }


    // If the regional ID has a value...
    // 
    if (reportreq.RegionalId.HasValue)
    {
        shopsSales = (from sale in ctn.ShopSales
                  where sale.Shop.Owner.RegionalId == reportreq.RegionalId
                    && sale.DateSold >= reportreq.FromDate && sale.DateSold <= reportreq.ToDate
                  select sale).ToList();
    }

    // If the owner ID has a value...
    // 
    if (reportreq.OwnerId.HasValue)
    {
        shopsSales = (from sale in ctn.ShopSales
                  where sale.Shop.OwnerId == reportreq.OwnerId
                    && sale.DateSold >= reportreq.FromDate && sale.DateSold <= reportreq.ToDate
                  select sale).ToList();
    }

    if (!string.IsNullOrEmpty(reportreq.ShopTypeCode))
    {
        shopsSales = (from sale in ctn.ShopSales
                  where sale.Shop.ShopTypeCode.ToUpper().Contains(reportreq.ShopTypeCode.ToUpper())
                    && sale.DateSold >= reportreq.FromDate && sale.DateSold <= reportreq.ToDate
                  select sale).ToList();
    }



    return shopsSales;
}

このメソッドは、フィルターが選択されていない場合にのみ、特定の日付の間で sales テーブルをクエリすることがわかります。

私が立ち往生しているのは、ユーザーが複数のフィルターを選択できることです。たとえば、regionId と ShopTypeCode を選択する可能性があるため、if フィルターごとに ctn.ShopSales を照会したくありません。選択すると、以前に取得した値が消去されます。

この問題を回避する方法について誰かアドバイスはありますか? さらに情報が必要な場合は、お尋ねください。

ありがとう

4

1 に答える 1

4

確かに、毎回最初からクエリを作成する必要はありません。Where条件付きで追加の呼び出しを追加するだけです:

var query = ctn.ShopSales.Where(sale => sale.DateSold >= reportreq.FromDate 
                                     && sale.DateSold <= reportreq.ToDate);
if (reportreq.OwnerId.HasValue)
{
    query = query.Where(sale => sale.Shop.OwnerId == reportreq.OwnerId);
}
if (!string.IsNullOrEmpty(reportreq.ShopTypeCode))
{
    query = query.Where(sale.Shop.ShopTypeCode.ToUpper()
                            .Contains(reportreq.ShopTypeCode.ToUpper());
}

var shopSales = query.ToList();

最終行で具体化するまで、クエリは実行されないことに注意してください。

于 2012-10-10T10:20:11.933 に答える