命令模式和OnPaint事件问题

本文关键字：事件 问题 OnPaint 模式 命令 | 更新日期: 2023-09-27 17:59:03

我正在尝试将命令模式调整为具有撤消功能的简单绘制应用程序。我一直在使用撤消操作上的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。但实际上没有显示"以前"的图像。

我已经阅读了许多网站，我也有一个代码项目网站的/文章中的应用程序示例。它提出了与TextBox和粗体/斜体操作相同的方法。它像地狱一样工作。唯一不同的是这种残酷的OnPaint方法。。

提前感谢您的建议！

[EDIT]匆忙中，我忘记了将一种引用类型分配给另一种并不是复制它（创建独立对象），而是更改它：

this.prevImage = simpleImage.Image;

很少有地方能解决这个问题。现在一切正常。。

命令模式和OnPaint事件问题

这里的重点不是直接在画布上绘制，而是要有一个表示绘画的数据结构。然后，您将向该绘制对象添加一条线，画布的主循环将从数据结构中绘制适当的图形。然后，您的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    
}

您的主应用程序只需保留一个List，并在命令更改绘画集合时重新绘制画布。

看起来有一个混叠问题。在您的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成为对图像的不必要的参考。但是，您的命令中仍然有另一个对Image的引用。垃圾收集器运行后，您将返回到对同一映像对象的引用。撤消然后执行以下操作：

simpleImage.Image = prevImage;

在这里，您没有更改Image，只是使Image引用了它已经实际引用的同一对象。

尽管我强烈同意m0sa的观点，但在这种情况下，修复方法是在创建命令时使prevImage成为原始图像的COPY。对于以下内容，我假设Image.Clone（）已经实现，尽管我自己从未尝试过：

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

注意：如果使用这种方法，使用大型图像或许多命令可能会很快耗尽内存。

相关文章：