Применение шаблона Decorator к формам - PullRequest
7 голосов
/ 31 марта 2010

Я пытаюсь применить шаблон оформления декоратора к следующей ситуации:

У меня есть 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. Я понимаю, почему это происходит, но мне трудно понять, как заставить это работать с этим шаблоном.

Я знаю, вероятно, есть лучшие способы достижения того, чего я хочу. Я сделал этот пример как попытку применить шаблон декоратора к чему-либо. Возможно, это был не лучший шаблон, который я мог бы использовать (если вообще использовал) для такого сценария. Есть ли другой шаблон, более подходящий для этого, чем декоратор? Я что-то не так делаю, когда пытаюсь реализовать Pattern Decorator?

Спасибо

Ответы [ 3 ]

6 голосов
/ 31 марта 2010

Проблема в том, что вы на самом деле не реализуете шаблон декоратора. Для правильной реализации шаблона вам нужно создать подкласс Form для создания вашего декоратора, а затем перехватить все операции, выполненные на вашем декораторе, и перенаправить их в ваш личный экземпляр Form. Вы вроде как делаете это, за исключением того, что кроме присвоения ссылки в конструкторе FormDecorator, вы никогда больше не будете использовать этот частный экземпляр Form. Конечным результатом является то, что вы создаете GreenForm, затем оборачиваете его в NoMaximizeDecorator, а затем вы заключаете его в NoMinimizeDecorator. Но поскольку вы никогда не перенаправляете операции, выполненные с NoMinimizeDecorator, на завернутый экземпляр Form, только экземпляр NoMinimizeDecorator фактически применяет любое поведение к используемому экземпляру. Это соответствует тому, что вы наблюдаете при запуске кода: стандартное окно с отключенной кнопкой «Свернуть».

Form является действительно плохим примером для создания декораторов в C #, потому что большинство его свойств и методов не виртуальны, то есть если вы обращаетесь к декорированной форме через ссылку Form, у вас нет возможности перехватывать свойства базового класса - вы не можете эффективно "обернуть" Form.

EDIT

Мне приходит в голову, что утверждение «Форма - это действительно плохой пример для создания декораторов в C #», действительно ставит вопрос о том, что является хорошим примером. Как правило, вы будете использовать шаблон декоратора для обеспечения реализации пользовательского интерфейса без реализации всей реализации с нуля. очень общий пример - это общие коллекции. Большинство всего, что требует функциональности списка, зависит не от, например, List<String>, а от IList<String>. Так, если вы, например, хотите, чтобы пользовательская коллекция не принимала строки короче 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");
    }
}
2 голосов
/ 31 марта 2010

Это неправильное применение шаблона декоратора. Шаблон декоратора связан с поведением объектов. Вы строите объекты, которые попадают под творческий зонт. В то время как вы могли бы быть в состоянии обернуть голову «не имея кнопки максимизации», это звучит немного не в духе.

Я не думаю, что есть реальный способ исправить ваш дизайн. Шаблон декоратора просто не подходит. Любая попытка исправить это будет просто невероятно хитрой, когда вы можете просто использовать Builder.

То, что я мог бы увидеть, это украшать 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();
}

Но даже это немного уродливо. Возможно, вы сможете преобразовать это в свободный интерфейс для создания форм.

0 голосов
/ 31 марта 2010

попробуйте заставить ваш шаблон применять свойства декоратора к тому же объекту, не создавая новые формы:

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;
    }
}

А у тебя в основном:

static Form CreateForm() {
    Form form = new GreenForm();
    new NoMaximizeDecorator(form);
    new NoMinimizeDecorator(form);

    return form;
}
...