Question

在我的表单中，我试图让我的撤消按钮工作。我使用pictureBox1绘制正方形，pictureBox2显示绘制的历史记录。

Functionaility

在Picturebox上鼠标事件我将所有绘制的对象添加到我的列表数组中，所以稍后当我点击撤消按钮时，它应该显示我绘制的矩形的历史记录。所以，如果我在picturebox1上有4个矩形，我点击撤消picturebox2应该显示3个矩形。如果我再次点击撤消按钮，picturebox2应该显示2个矩形。

问题是当我点击撤消按钮它没有像我希望的那样工作。它不断向我展示我画的所有矩形，并没有显示我绘制的物体的历史。

代码

    Point startArea;   // mouse-down position
    Point currentArea; // current mouse position
    bool drawingBox;     // busy drawing
    List<Rectangle> rectangleList = new List<Rectangle>(); //previous rectangles
    List<Image> ImageCollection = new List<Image>();
    int ImageHistory;

    private void Test_Load(object sender, EventArgs e)
    {
        pictureBox1.Image = Properties.Resources.background;

    }

    private Rectangle getRectangle()
    {
        return new Rectangle(
            Math.Min(startArea.X, currentArea.X),
            Math.Min(startArea.Y, currentArea.Y),
            Math.Abs(startArea.X - currentArea.X),
            Math.Abs(startArea.Y - currentArea.Y));
    }

    private void pictureBox1_MouseDown(object sender, MouseEventArgs e)
    {
        currentArea = startArea = e.Location;
        drawingBox = true;

    }

    private void pictureBox1_MouseMove(object sender, MouseEventArgs e)
    {
        currentArea = e.Location;
        if (drawingBox)
        {
            pictureBox1.Invalidate();
        }
    }



    private void pictureBox1_MouseUp(object sender, MouseEventArgs e)
    {
        if(drawingBox)
        {
            drawingBox = false;
            var rc = getRectangle();
            if(rc.Width > 0 && rc.Height > 0)
            {
                rectangleList.Add(rc);
                pictureBox1.Invalidate();
            }
        }
        ImageHistory++;
        ImageCollection.Add(pictureBox1.Image);

    }

    private void pictureBox1_Paint(object sender, PaintEventArgs e)
    {
        if (rectangleList.Count > 0)
        {
            using (var g = Graphics.FromImage(pictureBox1.Image))
            {
                g.DrawRectangles(Pens.Black, rectangleList.ToArray());
                pictureBox1.Invalidate();
            }
        }

        if(drawingBox)
        {                                                
            e.Graphics.DrawRectangle(Pens.Red, getRectangle());

            pictureBox1.Invalidate();                
        }

    }

    private void button1_Click(object sender, EventArgs e)
    {
        ImageHistory--;
        pictureBox2.Image = ImageCollection[ImageHistory];
    }
}

Answer 1

更改以下行

ImageCollection.Add(pictureBox1.Image);

到

ImageCollection.Add((Image)pictureBox1.Image.Clone())

Answer 2

虽然接受的答案可以解决问题，但在尝试解决问题时应该考虑一些重要的注意事项：

您不需要使用List<Image>作为撤消缓冲区，它会占用大量内存。目前，由于您没有处理未使用的图像，因此您的应用程序很快就会进入内存泄漏状态。
使用List<Rectangle>作为撤消缓冲区就足够了。它显着减少了内存使用量。您无需跟踪要处置的任何内容，也无需处置。
为了实现这个想法，最好将List<Rectangle>保留为撤消缓冲区，每次绘制矩形时，将其添加到缓冲区中，每次撤消时，只需删除最后一个矩形并使绘图无效表面。这样您就不需要更改图片框的图像，只需在绘图表面上绘制矩形，而不是在图像上绘制它们。最后，如果您需要导出包含所有矩形的图像，只需在位图上绘制它们并导出图像。
您可以将Undo方法和Redo方法以及CanUndo属性封装在DrawingSurface控件中，并在您的应用程序中使用这些方法，而不是直接放置这些方法按钮背后的逻辑。

示例

您可以看到有关Draw multiple freehand Poly-line or Curve drawing - Adding Undo Feature的示例。

从数组

2 个答案: