Это плохой дизайн?

Question

У меня есть элемент управления и он обеспечивает механизм выбора. Однако элементы, которые он выбирает, не подходят для общего потребления и вместо этого должны проецироваться в другой тип, параметризованный по дате.

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

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

public class MySelectionControl 
{ 
    public Func<DateTime, IEnumerable<MyProjectedType>> CreateSelectionForDate 
    { 
    get 
    { 
     // Take a copy of the selection before creating the lambda so 
     // that the source items won't change if the selection does. 
     IEnumerable<MyRealType> copyOfSelection = this.realSelection.ToList(); 
     return weekEnding => 
      copyOfSelection.Select(x => new MyProjectedType(x, weekEnding)); 
    } 
    } 
} 

Это тогда можно назвать как метод, как в:

MySelectionControl control = new MySelectionControl(); 

// Gives the current selection for a given date. 
control.CreateSelectionForDate(DateTime.Today); 

// Takes a copy of the selection for later use, regardless of 
// whether the original selection changes. 
var selectedItemsProjection = control.CreateSelectionForDate; 

Итак, этот ужасный дизайн или умное использование делегатов?

Answer 1

Поскольку цель этого механизма заключается в создании контекста и способ для запуска из в этом контексте, казалось бы, лучше использовать его больше, как это:

Closure closure = selectionControl.GetClosure(DateTime.Today); 
closure.DateSelection ... 

Answer 2

Я думаю, что это довольно умно. Единственное, что я сделал бы, это превратить вашу собственность (CreateSelectionForDate) в метод, который (для меня в любом случае) сделает код более понятным.