python - 为什么QUndoStack.push()会执行QUndoCommand.redo()？

Question

我创建了一个自定义 QPushButton，允许从菜单或 QColorDialog 中选择颜色。由于它是“主题”编辑器的一部分，我还添加了对 QUndoStack 的支持:每次更改自定义按钮的颜色时，它都会创建 QUndoCommand 的子类，并将其推送到 QUndoStack。
然后我意识到(根据 this )每次我执行 QUndoStack.push(cmd) 时，都会执行该命令(这显然创建了一个递归，被 PyQt 自动“忽略”，但仍然在标准输出中报告)。

我通过在调用 redo() 时阻止目标小部件上的信号来解决了问题，但问题仍然存在:为什么首先执行推送的命令？

从我的角度来看，如果我将一个命令压入撤销栈，它就已经被执行了。
必须“再次”执行(理论上已经)执行的 [子类] QUndoCommand 的情况是什么？
这种情况是否很常见，需要将这种实现作为默认行为？

以下是一个最小且不完整(但足以说明问题)的示例；它不支持撤消/重做操作的信号处理，但这不是重点(我认为？)。据我所知，当调用 QUndoCommand 创建的信号与创建信号本身的插槽重合时，问题就来了:

#!/usr/bin/env python2

import sys
from PyQt5 import QtCore, QtGui, QtWidgets

class UndoCmd(QtWidgets.QUndoCommand):
    def __init__(self, widget, newColor, oldColor):
        QtWidgets.QUndoCommand.__init__(self)
        self.widget = widget
        self.newColor = newColor
        self.oldColor = oldColor

    def redo(self):
        self.widget.color = self.newColor


class ColorButton(QtWidgets.QPushButton):
    colorChanged = QtCore.pyqtSignal(QtGui.QColor)
    def __init__(self, parent=None):
        QtWidgets.QPushButton.__init__(self, 'colorButton', parent)
        self.oldColor = self._color = self.palette().color(self.palette().Button)
        self.clicked.connect(self.changeColor)

    @QtCore.pyqtProperty(QtGui.QColor)
    def color(self):
        return self._color

    @color.setter
    def color(self, color):
        self._color = color
        palette = self.palette()
        palette.setColor(palette.Button, color)
        self.setPalette(palette)
        self.colorChanged.emit(color)

    def changeColor(self):
        dialog = QtWidgets.QColorDialog()
        if dialog.exec_():
            self.color = dialog.selectedColor()

class Window(QtWidgets.QWidget):
    def __init__(self):
        QtWidgets.QWidget.__init__(self)
        layout = QtWidgets.QHBoxLayout()
        self.setLayout(layout)
        self.colorButton = ColorButton()
        layout.addWidget(self.colorButton)
        self.undoStack = QtWidgets.QUndoStack()
        self.colorButton.colorChanged.connect(lambda: self.colorChanged(self.colorButton.oldColor))

    def colorChanged(self, oldColor):
        self.undoStack.push(UndoCmd(self.colorButton, oldColor, self.colorButton._color))


app = QtWidgets.QApplication(sys.argv)
w = Window()
w.show()
sys.exit(app.exec_())

Answer 1

这在 overview documentation 中有描述:

Qt's Undo Framework is an implementation of the Command pattern, for implementing undo/redo functionality in applications.

The Command pattern is based on the idea that all editing in an application is done by creating instances of command objects. Command objects apply changes to the document and are stored on a command stack. Furthermore, each command knows how to undo its changes to bring the document back to its previous state. As long as the application only uses command objects to change the state of the document, it is possible to undo a sequence of commands by traversing the stack downwards and calling undo on each command in turn. It is also possible to redo a sequence of commands by traversing the stack upwards and calling redo on each command.

要使用撤消框架，您应该确保任何应该可撤消的操作仅由 QUndoCommand 子类执行。

所以我假设您要执行两次操作:一次是直接执行，然后是执行该操作的结果，一次是通过撤消框架执行。相反，您应该只通过撤消框架执行操作。

推送的操作立即重做的原因可能是为了简单:redo() 函数已经方便地执行操作，那么为什么要添加另一个函数来做同样的事情？