绘图程序撤销功能故障排除：如何解决连续操作撤销问题？

windows

2024-03-21 05:06:27

绘图程序中的重做和撤销功能故障排除

引言

在构建绘图程序时，重做和撤销功能至关重要，它们使用户能够纠正错误并探索不同的绘画可能性。然而，实现这些功能可能会遇到挑战。本文将深入探讨一个常见的撤销功能问题及其解决方案，帮助程序员解决故障排除问题。

问题

许多绘图程序允许用户连续撤销多个操作。然而，在某些情况下，撤销功能可能仅恢复最近撤销的操作，即使连续执行了多个撤销操作。

根源分析

撤销功能的工作原理：

检索最后一个绘制方块的坐标。
检查该方块是否之前被绘制过。
如果方块之前被绘制过，则将撤销操作压入撤销操作栈。
如果方块没有被绘制过，则直接从屏幕上移除该方块。
从已绘制方块和颜色列表中删除最后一个绘制的方块。

问题根源：

仔细检查撤销函数后，发现问题在于它仅在方块之前被绘制过的情况下才将撤销操作压入撤销操作栈。这意味着连续撤销操作不会被正确存储在栈中。

解决方案

要解决这个问题，撤销函数需要始终将撤销操作压入栈中，无论方块是否之前被绘制过。

改进后的撤销函数：

检索最后一个绘制方块的坐标。
将撤销操作（坐标和颜色）压入撤销操作栈。
如果方块之前被绘制过，则恢复其之前的颜色。
否则，从屏幕上移除该方块。
从已绘制方块和颜色列表中删除最后一个绘制的方块。

改进后的功能

经过这些修改，撤销功能现在可以正确地将所有撤销操作存储在栈中，即使这些操作是连续执行的。这意味着重做功能能够重新应用所有撤销的操作，恢复正确的画布状态。

相关代码片段

void UndoLastPaint(HWND hWnd)
{
    // 检索最后一个绘制方块的坐标
    int x = paintedCoordinates.back().first;
    int y = paintedCoordinates.back().second;

    // 将撤销操作压入栈中
    PaintAction undoneAction;
    undoneAction.x = x;
    undoneAction.y = y;
    undoneAction.color = paintedColors.back();
    undoneActions.push(undoneAction);

    // 如果方块之前被绘制过，则恢复其之前的颜色
    if (wasPainted)
    {
        // 计算方块占据的矩形区域
        RECT squareRect = { x * squareSize, y * squareSize, (x + 1) * squareSize, (y + 1) * squareSize };

        // 重置方块颜色
        HDC hdc = GetDC(hWnd);
        HBRUSH brush = CreateSolidBrush(color);
        FillRect(hdc, &squareRect, brush);
        ReleaseDC(hWnd, hdc);
    }
    else
    {
        // 从屏幕上移除方块
        RECT squareRect = { x * squareSize, y * squareSize, (x + 1) * squareSize, (y + 1) * squareSize };
        InvalidateRect(hWnd, &squareRect, TRUE);
        UpdateWindow(hWnd);
    }

    // 从数据集中移除最后一个绘制的方块
    paintedCoordinates.pop_back();
    paintedColors.pop_back();
}