Я пытаюсь применить Шаблон разработки Декоратора к следующей ситуации:
У меня есть 3 других вида форм: Зеленый, Желтый, Красный.
Теперь, каждая из тех форм может иметь различный набор атрибутов. У них может быть минимизировать отключенное поле, максимизируемое отключенное поле, и они могут быть поверх остальных окон.
Я пытался смоделировать это следующий путь:
Form <---------------------------------------FormDecorator
/\ /\
|---------|-----------| |----------------------|-----------------|
GreenForm YellowForm RedForm MinimizeButtonDisabled MaximizedButtonDisabled AlwaysOnTop
Вот мой код GreenForm:
public class GreenForm : Form {
public GreenForm() {
this.BackColor = Color.GreenYellow;
}
public override sealed Color BackColor {
get { return base.BackColor; }
set { base.BackColor = value; }
}
}
FormDecorator:
public abstract class FormDecorator : Form {
private Form _decoratorForm;
protected FormDecorator(Form decoratorForm) {
this._decoratorForm = decoratorForm;
}
}
и наконец NoMaximizeDecorator:
public class NoMaximizeDecorator : FormDecorator
{
public NoMaximizeDecorator(Form decoratorForm) : base(decoratorForm) {
this.MaximizeBox = false;
}
}
Таким образом, вот под управлением код:
static void Main()
{
Application.EnableVisualStyles();
Application.SetCompatibleTextRenderingDefault(false);
Application.Run(CreateForm());
}
static Form CreateForm() {
Form form = new GreenForm();
form = new NoMaximizeDecorator(form);
form = new NoMinimizeDecorator(form);
return form;
}
Проблема состоит в том, что я получаю форму, которая не является зеленой, и это все еще позволяет мне максимизировать ее. Это только принимает в соображении форму NoMinimizeDecorator. Я действительно постигаю, почему это происходит, но я испытываю затруднения при понимании, как сделать эту работу с этим Шаблоном.
Я знаю, вероятно, существуют лучшие способы достигнуть того, что я хочу. Я сделал этот пример как попытку применить Шаблон "декоратор" к чему-то. Возможно, это не было лучшим шаблоном, который я, возможно, использовал (если один, вообще) к этому виду сценария. Действительно ли там какой-либо другой шаблон более подходит, чем Декоратор для выполнения этого? Я делаю что-то не так при попытке реализовать Шаблон "декоратор"?
Спасибо
Проблема здесь в том, что вы на самом деле не реализуете паттерн декоратора. Для правильной реализации паттерна вам нужно подкласс Form
, чтобы создать свой декоратор, а затем перехватывать все операции, выполняемые над декоратором, и передавать их частному экземпляру Form
. Вы как бы делаете это, за исключением того, что кроме присвоения ссылки в конструкторе FormDecorator
, вы больше никогда не используете этот частный экземпляр Form
. В итоге вы создаете GreenForm
, затем оборачиваете ее в NoMaximizeDecorator
, а затем оборачиваете ее в NoMinimizeDecorator
. Но поскольку вы никогда не пересылаете операции, выполненные против NoMinimizeDecorator
, в обернутый экземпляр Form
, только экземпляр NoMinimizeDecorator
фактически применяет какое-либо поведение к используемому экземпляру. Это соответствует тому, что вы наблюдаете при выполнении вашего кода: стандартное окно с отключенной кнопкой Minimize.
Form
- действительно плохой пример для создания декораторов в C#, потому что большинство его свойств и методов невиртуальны, что означает, что если вы обращаетесь к декорированной форме через ссылку Form
, у вас нет возможности перехватить свойства базового класса - вы не можете эффективно "обернуть" Form
.
EDIT
Мне пришло в голову, что утверждение "Form - действительно плохой пример для создания декораторов в C#" заставляет задуматься о том, что является хорошим примером. Как правило, вы используете паттерн декоратора для обеспечения реализации пользовательского интерфейса без реализации всей реализации с нуля. очень распространенным примером являются общие коллекции. Почти все, что хочет получить функциональность списка, зависит не от List
, а от IList
. Поэтому, если вы, например, хотите создать пользовательскую коллекцию, которая не будет принимать строки короче 5 символов, вы будете использовать что-то вроде следующего:
public class MinLengthList : IList<String>
{
private IList<string> _list;
private int _minLength;
public MinLengthList(int min_length, IList<String> inner_list)
{
_list = inner_list;
_minLength = min_length;
}
protected virtual void ValidateLength(String item)
{
if (item.Length < _minLength)
throw new ArgumentException("Item is too short");
}
#region IList<string> Members
public int IndexOf(string item)
{
return _list.IndexOf(item);
}
public void Insert(int index, string item)
{
ValidateLength(item);
_list.Insert(index, item);
}
public void RemoveAt(int index)
{
_list.RemoveAt(index);
}
public string this[int index]
{
get
{
return _list[index];
}
set
{
ValidateLength(value);
_list[index] = value;
}
}
#endregion
#region ICollection<string> Members
public void Add(string item)
{
ValidateLength(item);
_list.Add(item);
}
public void Clear()
{
_list.Clear();
}
public bool Contains(string item)
{
return _list.Contains(item);
}
public void CopyTo(string[] array, int arrayIndex)
{
_list.CopyTo(array, arrayIndex);
}
public int Count
{
get { return _list.Count; }
}
public bool IsReadOnly
{
get { return _list.IsReadOnly; }
}
public bool Remove(string item)
{
return _list.Remove(item);
}
#endregion
#region IEnumerable<string> Members
public IEnumerator<string> GetEnumerator()
{
return _list.GetEnumerator();
}
#endregion
#region IEnumerable Members
System.Collections.IEnumerator System.Collections.IEnumerable.GetEnumerator()
{
return ((IEnumerable)_list).GetEnumerator();
}
#endregion
}
public class Program
{
static void Main()
{
IList<String> custom_list = new MinLengthList(5, new List<String>());
custom_list.Add("hi");
}
}
попробуйте заставить ваш шаблон применять свойства декоратора к тому же объекту, не создавая новые формы:
public abstract class FormDecorator {
protected Form _decoratorForm;
protected FormDecorator(Form decoratorForm) {
this._decoratorForm = decoratorForm;
}
public abstract void Decorate();
}
public class NoMaximizeDecorator : FormDecorator
{
public NoMaximizeDecorator(Form decoratorForm) : base(decoratorForm) {
Decorate();
}
public override void Decorate() {
_decoratorForm.MaximizeBox = false;
}
}
И в вашем Main:
static Form CreateForm() {
Form form = new GreenForm();
new NoMaximizeDecorator(form);
new NoMinimizeDecorator(form);
return form;
}
Это неправильное применение узора декоратора. Узор декоратора связан с поведением предметов. Вы строите объекты, которые попадают под зонтик творения. В то время как вы можете обернуть голову вокруг «не имея кнопки максимизации», будучи поведением, это звучит немного не так.
Я не думаю, что есть реальный способ исправить ваш дизайн. Узор декоратора просто не подходит. Любая попытка исправить это будет невероятно грубой, когда вы можете просто использовать Builder.
То, что я мог бы видеть, это украшать Строителя формы для выполнения этих действий во время строительства. Это выглядело бы примерно так...
public interface IFormBuilder {
public Form BuildForm();
}
public class FormBuilder : IFormBuilder {
public Form BuildForm(){
return new Form();
}
}
public class NoMaximizeFormBuilder : IFormBuilder {
private IFormBuilder _builder;
public NoMaximizeFormBuilder (IFormBuilder builder){
_builder = builder;
}
public Form BuildForm(){
f = _builder.BuildForm();
f.MaximizeBox = false;
return f;
}
}
И вы могли бы использовать его так...
static void Main()
{
Application.EnableVisualStyles();
Application.SetCompatibleTextRenderingDefault(false);
Application.Run(CreateForm());
}
static Form CreateForm() {
var b = new FormBuilder();
var b = new NoMaximizeFormBuilder(b);
return b.Build();
}
Но даже это немного уродливо.Возможно, вы сможете преобразовать это в свободный интерфейс для создания форм.