Объединить два метода в один

Question

Как я могу объединить методы ниже? и должен ли я на самом деле сделать это?

public IQueryable<ItemDTO> RepGetMonthItems(string inOut, string planFact, int month) 
{ 
    return GetItemsWithCategory(). 
      Where(i => i.InOut.Equals(inOut)). 
      Where(i => i.PlanFact.Equals(planFact)). 
      Where(i => i.DateTime.Month.Equals(month)); 
} 

public IQueryable<ItemDTO> RepGetYearItems(string inOut, string planFact, int year) 
{ 
    return GetItemsWithCategory(). 
      Where(i => i.InOut.Equals(inOut)). 
      Where(i => i.PlanFact.Equals(planFact)). 
      Where(i => i.DateTime.Year.Equals(year)); 
} 

Answer 1

Я согласен с принципом «сделай одну вещь». Но я также верю в СУХОЙ.

Итак, методы расширения на помощь!

Имейте основную функцию, чтобы сделать одну вещь (получить элементы отчета) и методы методов расширения на год или месяц.

Объявите класс метод расширения, как это:

public static class QueryExtensions 
    { 
     public static IQueryable<ItemDTO> ForYear(this IQueryable<ItemDTO> query, int year) 
     { 
      return query.Where(i => i.DateTime.Year.Equals(year)); 
     } 

     public static IQueryable<ItemDTO> ForMonth(this IQueryable<ItemDTO> query, int month) 
     { 
      return query.Where(i => i.DateTime.Month.Equals(month)); 
     } 
    } 

Создать урезанную RepGetItems метод, как это:

public IQueryable<ItemDTO> RepGetItems(string inOut, string planFact) 
    { 
     return GetItemsWithCategory(). 
      Where(i => i.InOut.Equals(inOut)). 
      Where(i => i.PlanFact.Equals(planFact)); 
    } 

Usage тогда выглядит следующим образом:

var yearResults = originalQuery.RepGetItems(input, fact).ForYear(2015); 
var monthResults = originalQuery.RepGetItems(input, fact).ForMonth(10); 

или даже:

var yearMonthResults = originalQuery.RepGetItems(input, fact).ForYear(2015).ForMonth(10); 

Общая гибкость, без потери принципа «одной цели».

Answer 2

В запрос можно легко добавить условия .Where.

public IQueryable<ItemDTO> RepGetYearItems(string inOut, string planFact, int monthyear, bool useMonth) 
{ 
    var query = GetItemsWithCategory(). 
     Where(i => i.InOut.Equals(inOut)). 
     Where(i => i.PlanFact.Equals(planFact)); 

    if (useMonth) 
    { 
     query = query.Where(i => i.DateTime.Month.Equals(monthyear)); 
    } 
    else 
    { 
     query = query.Where(i => i.DateTime.Year.Equals(monthyear)); 
    } 

    return query; 
} 

или с помощью int?

public IQueryable<ItemDTO> RepGetYearItems(string inOut, string planFact, int? year, int? month) 
{ 
    var query = GetItemsWithCategory(). 
     Where(i => i.InOut.Equals(inOut)). 
     Where(i => i.PlanFact.Equals(planFact)); 

    if (year != null) 
    { 
     query = query.Where(i => i.DateTime.Year.Equals(year.Value)); 
    } 

    if (month != null) 
    { 
     query = query.Where(i => i.DateTime.Month.Equals(month.Value)); 
    } 

    return query; 
} 

Обратите внимание, что эта версия отличается от одного из @TimSchmelter ... Это не фиксируется (это зависит от «типа» SQL), что происходит если у вас есть в запросе некоторый «мертвый код» (куски запроса, которые не являются «активными», потому что некоторые параметры имеют определенное значение, например, int его запрос). В его запросе явно содержится «мертвый код» (целое i => !year.HasValue || i.DateTime.Year.Equals(year.Value)). Если возможно, я предпочитаю его пропустить.

Answer 3

Может быть, вы могли бы использовать Nullable<int> за год и месяц:

public IQueryable<ItemDTO> RepGetItems(string inOut, string planFact, int? year, int? month) 
{ 
    return GetItemsWithCategory(). 
     Where(i => i.InOut.Equals(inOut)). 
     Where(i => i.PlanFact.Equals(planFact)). 
     Where(i => !year.HasValue || i.DateTime.Year.Equals(year.Value)). 
     Where(i => !month.HasValue || i.DateTime.Month.Equals(month.Value)); 
} 

Answer 4

Нет, вы не должны. Три ответа, которые я вижу, нарушают правило «сделать одно» Роберта «Дядя Боб» Мартин. Читайте об этом here.

Answer 5

Как об использовании необязательного параметра здесь:

public IQueryable<ItemDTO> RepGetMonthItems(string inOut, string planFact, int month, int year = -1) 
    { 
     return GetItemsWithCategory().Where(i => i.InOut.Equals(inOut) 
      && i.PlanFact.Equals(planFact)) && year == -1? 
      i.DateTime.Month.Equals(month) : i.DateTime.Year.Equals(year)); 
    } 

Взгляните на этот пример:

Как вы можете видеть каждый раз, когда вы вызываете метод Где() вы делаете много дополнительной работы.