Шаблон команд и проблема события OnPaint

Question

Я пытаюсь адаптировать Command Pattern к простому приложению рисования с отменой функциональности.И я застрял с OnPaint Событием при операциях отмены.Это код:

[решено] подробности в конце сообщения

interface ICommand {
    void Execute();
    void UnExecute();
}

class DrawLineCommand : ICommand {
    private SimpleImage simpleImage;
    private Image prevImage;
    public DrawLineCommand(SimpleImage simpleImage) {
        this.simpleImage = simpleImage;
        this.prevImage = simpleImage.Image;
    }
    public void Execute() {
        simpleImage.DrawLine();
    }
    public void UnExecute() {
        simpleImage.Image = prevImage;
    }
}

class CommandManager {
    private Stack undoStack = new Stack();
    public void ExecuteCommand(ICommand command) {
        command.Execute();
        undoStack.Push(command);
    }
    public void UnExecuteCommand() {
        if (undoStack.Count > 0) {
            ICommand command = (ICommand)undoStack.Pop();
            command.UnExecute();
        }
    }
}

class SimpleImage {
    private Point startPoint;
    private Point endPoint;
    private PictureBox pictureBox;
    public SimpleImage(PictureBox pictureBox) {
        this.pictureBox = pictureBox;
        pictureBox.Paint += new PaintEventHandler(pictureBox_Paint);
    }
    void pictureBox_Paint(object sender, PaintEventArgs e) {
        // this code shows the line during drawing
        // this code is under "if operation == drawLine" block
        Graphics graphics = e.Graphics;
        graphics.DrawLine(Pens.Red, startPoint, endPoint);

        // how can i refresh picturebox after undo operation?
        // "if operation == undo" then ??
    }
    public void DrawLine() {
        // this code actually saves finally drawn line
        Image img = Image;
        Graphics graphics = Graphics.FromImage(img);
        graphics.DrawLine(Pens.Red, startPoint, endPoint);
        Image = img;
    }

    public void Invalidate() {
        pictureBox.Invalidate();
    }
    public Image Image {
        get { return pictureBox.Image; }
        set { pictureBox.Image = value; }
    }
    public Point StartPoint {
        get { return startPoint; }
        set { startPoint = value; }
    }
    public Point EndPoint {
        get { return endPoint; }
        set { endPoint = value; }
    }
}

public partial class FormMain : Form {
    private PictureBox pictureBox;
    private SimpleImage simpleImage;
    private CommandManager commandManager;
    public FormMain() {
        InitializeComponent();
        simpleImage = new SimpleImage(this.pictureBox);
        commandManager = new CommandManager();
    }
    void pictureBox_MouseDown(object sender, MouseEventArgs e) {
        if (e.Button != MouseButtons.Left)
            return;

        simpleImage.StartPoint = e.Location;
    }
    void pictureBox_MouseMove(object sender, MouseEventArgs e) {
        if (e.Button != MouseButtons.Left)
            return;

        simpleImage.EndPoint = e.Location;
        simpleImage.Invalidate();
    }
    void pictureBox_MouseUp(object sender, MouseEventArgs e) {
        simpleImage.Invalidate();
        commandManager.ExecuteCommand(new DrawLineCommand(simpleImage));
    }
}

На самом деле рисует линию, выполняет команду и нажимает ее настек.Я не могу добиться работы UNDO.Я имею в виду.Пошаговая отладка Я вижу, как объект выскакивает из стека, а затем OnPaint выполняется.Но на самом деле никакого «предыдущего» изображения не отображается.

Я прочитал много сайтов, и у меня есть пример приложения из одной из статей / статей сайта codeproject.Он представляет тот же подход с TextBox и операциями Bold / Italicize.Это работает как в аду.Единственная разница в том, что этот жестокий OnPaint метод ..

Заранее благодарен за любые советы!

[EDIT] в спешке я забыл, что назначая один ссылочный типдругому не копировать его (создать независимый объект), изменив это:

this.prevImage = simpleImage.Image;

в нескольких местах решил проблему.Теперь все работает ..

Answer 1

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

this.prevImage = simpleImage.Image;

Теперь у вас есть две ссылки на один и тот же объект.Операция рисования линии происходит, когда вы работаете с тем же изображением:

Image img = Image; // Now a third reference to the same image object
Graphics graphics = Graphics.FromImage(img);
graphics.DrawLine(Pens.Red, startPoint, endPoint);
Image = img; // and you set the Image reference back to the same object

Вышеприведенное делает img ненужной ссылкой на изображение.Но у вас все еще есть другая ссылка на изображение в вашей команде.После запуска сборщика мусора вы возвращаетесь к ссылкам на тот же объект изображения.Затем Undo выполняет следующее:

simpleImage.Image = prevImage;

Здесь вы не изменили изображение, только сделали Image ссылку на тот же объект, на который он уже ссылался.

Хотя я полностью согласен с m0sa,в данном случае исправление состоит в том, чтобы сделать prevImage COPY исходного изображения во время создания команды.В следующем случае я предполагаю, что Image.Clone () реализован, хотя сам никогда не пробовал:

this.prevImage = simpleImage.Image.Clone();

ПРИМЕЧАНИЕ: вы можете быстро исчерпать память с большими изображениями или многими командами, если вы используетеэтот подход.

Answer 2

Смысл здесь не в том, чтобы рисовать непосредственно на холсте, а в том, чтобы иметь структуру данных, которая представляет вашу картину. Затем вы добавите линию к этому объекту рисования, и основной цикл холста будет рисовать соответствующую графику из структуры данных. Тогда ваши методы do / undo просто должны будут манипулировать структурой данных, а не рисовать.

Вам нужно что-то вроде этого:

interface IPaintable // intarface for Lines, Text, Circles, ...
{
    void OnPaint(Image i); // does the painting
}

interface IPaintableCommand // interface for commands 
{
    void Do(ICollection<IPaintable> painting); // adds line/text/circle to painting
    void Undo(ICollection<IPaintable> painting);  // removes line/text/circle from painting    
}

Ваше основное приложение просто сохранит список и перекрасит холст, когда команда изменит коллекцию рисунков.