Проверки нескольких условий в запросе?

Question

У меня есть запрос ниже, где я передаю 4 параметра из пользовательского интерфейса. BU и Region имеют параметры переключателя (All, Option1, Option2, Option3 и т. Д.) В пользовательском интерфейсе. Параметры 1 - 3 - это значения в данных, тогда как «Все» - нет.

Есть ли способ написать ниже более эффективно? Например, если значения BU или Region являются «Все» из пользовательского интерфейса »(которые не являются значениями в наборе данных), есть ли символ, который я могу присвоить bu или region, если все выбрано так, что запрос знает, чтобы выбрать все значения?

Я не знаю, есть ли такая вещь, но я хочу избежать необходимости делать что-то вроде ниже.

 public string GetAvgSeatPrice(string bu, string region, DateTime? startDate, DateTime? endDate) 
    { 

     if (bu.Equals("All")) 
     { 
      var averageSeatPrice = (from r in db.Registrations 
            where 
             //r.BusinessUnit.Equals(bu) && 
             r.Region.Equals(region) && 
             r.StartDate >= startDate && 
             r.EndDate <= endDate && 
             r.ActualPrice > 0 
            select r.ActualPrice).Average(); 

      var AvgSeatPrice = "$" + string.Format("{0:0.00}", averageSeatPrice); 

      return AvgSeatPrice; 
     } 
     else if (region.Equals("All")) 
     { 
      var averageSeatPrice = (from r in db.Registrations 
            where 
             r.BusinessUnit.Equals(bu) && 
             // r.Region.Equals(region) && 
             r.StartDate >= startDate && 
             r.EndDate <= endDate && 
             r.ActualPrice > 0 
            select r.ActualPrice).Average(); 

      var AvgSeatPrice = "$" + string.Format("{0:0.00}", averageSeatPrice); 

      return AvgSeatPrice; 
     } 
     else if (bu.Equals("All") && region.Equals("All")) 
     { 
      var averageSeatPrice = (from r in db.Registrations 
            where 
             //r.BusinessUnit.Equals(bu) && 
             // r.Region.Equals(region) && 
             r.StartDate >= startDate && 
             r.EndDate <= endDate && 
             r.ActualPrice > 0 
            select r.ActualPrice).Average(); 

      var AvgSeatPrice = "$" + string.Format("{0:0.00}", averageSeatPrice); 

      return AvgSeatPrice; 
     } 
     else 
     { 
      var averageSeatPrice = (from r in db.Registrations 
            where 
             r.BusinessUnit.Equals(bu) && 
             r.Region.Equals(region) && 
             r.StartDate >= startDate && 
             r.EndDate <= endDate && 
             r.ActualPrice > 0 
            select r.ActualPrice).Average(); 

      var AvgSeatPrice = "$" + string.Format("{0:0.00}", averageSeatPrice); 

      return AvgSeatPrice; 
     } 

    } 

} 

}

Answer 1

Просто сделать это:

public string GetAvgSeatPrice(string bu, string region, DateTime? startDate, DateTime? endDate) 
    { 
     var averageSeatPrice = (
      from r in db.Registrations 
      where (bu == "ALL" || r.BusinessUnit.Equals(bu)) 
      && (region == "ALL" || r.Region.Equals(region)) 
      && r.StartDate >= startDate 
      && r.EndDate <= endDate 
      && r.ActualPrice > 0 
      select r.ActualPrice).Average(); 

     var AvgSeatPrice = "$" + string.Format("{0:0.00}", averageSeatPrice); 

     return AvgSeatPrice; 
    } 

Из-ИЛИ (||), если выбрано значение «ALL» второе условие никогда не проверяется и не сузить выбор вниз по данному свойству. Он проверяется только в том случае, если выбор НЕ «ВСЕ», и в этом случае выбор сужается.

Answer 2

Я надеюсь, что это не совсем то, что вы просите, чтобы не предоставили (я мог бы истолковать мультипликатор если/другое дело/и т.д. два отдельных пути от вашего вопроса). Но вы могли бы сделать что-то вроде этого:

var averageSeatPrice = 
    from r in db.Registrations 
    where r.StartDate >= startDate && 
      r.EndDate <= endDate && 
      r.ActualPrice > 0 

// note I'm assuming your "bu" "all" option is represented as the string "All" 
if (bu.ToUpper() != "ALL") 
    averageSeatPrice = averageSeatPrice 
     .Where(w => w.BusinessUnit == bu); 

if (region.ToUpper() !+ "ALL") 
    averageSeatPrice = averageSeatPrice 
     .Where(w => w.Region == region); 

averageSeatPrice = averageSeatPrice select r.ActualPrice).Average(); 

К сожалению, я менее ясно с синтаксисом LinQ вы используете (или я сделал бы мой ответ с этим), надеюсь, что это помогает, хотя.

В принципе, если заявление государства, если эта опция не все, то лавировать на той части, где положение в противном случае он никогда не пристегивается, и никогда не становится частью запроса.

Kritner - можете ли вы подробно рассказать о том, что вы здесь делаете? кажется, интересный подход слишком

Учитывая BusinessUnit может иметь следующие значения:

option1 
option2 
option3 

если bu предоставляется в option1 то мы должны наш ИНЕКЕ содержать where BusinessUnit = 'option1'

так:

if (bu.ToUpper() != "ALL") // bu is option1, evaluates true 
    averageSeatPrice = averageSeatPrice // add the where clause to the IQueryable 
     .Where(w => w.BusinessUnit == bu); // deferred execution, so have not yet executed query, and can continue to modify and only do a single trip to db 

Теперь о ther hand, если bu - это «ВСЕ» или string.empty, или что бы вы ни хотели, чтобы это не было допустимым значением, тогда мы не хотим, чтобы BusinessUnit был частью предложения where вообще ... опубликовал код выполняет этот сценарий, а также:

if (bu.ToUpper() != "ALL") // this evaluates to false (bu is "ALL"), don't add the where clause information at all. 
    averageSeatPrice = averageSeatPrice 
     .Where(w => w.BusinessUnit == bu); 

Каждый averageSeatPrice = averageSeatPrice.Where(...) не просто прикрепил на запрос до точки, в которой вы пронумеровать результаты.