如果选择该项目，为啥 QGraphicsItem 子项不再单击鼠标？
Posted
技术标签:
【中文标题】如果选择该项目，为啥 QGraphicsItem 子项不再单击鼠标？【英文标题】：Why are the QGraphicsItem's children not getting mouse clicks anymore if the item is selected?如果选择该项目，为什么 QGraphicsItem 子项不再单击鼠标？ 【发布时间】：2018-07-30 14:45:38 【问题描述】：我在QGraphicsRectItem 的子类上使用来自here 的SizeGripItem（添加信号的修改最少），如下定义：
标题
//! RectItem that sends a boxChanged signal whenever it is moved or resized via the handles.
class QSignalingBoxItem : public QObject, public QGraphicsRectItem
Q_OBJECT
private:
void setSelected_(bool selected);
protected:
QVariant itemChange(GraphicsItemChange change, const QVariant & value);
public:
QSignalingBoxItem(QRectF rect, QGraphicsItem * parent = nullptr);
~QSignalingBoxItem();
signals:
void boxChanged(QRectF newBox);
;
.cpp
namespace
class BoxResizer : public SizeGripItem::Resizer
public:
virtual void operator()(QGraphicsItem* item, const QRectF& rect)
QSignalingBoxItem* rectItem =
dynamic_cast<QSignalingBoxItem*>(item);
if (rectItem)
rectItem->setRect(rect);
;
QSignalingBoxItem::QSignalingBoxItem(QRectF rect, QGraphicsItem * parent) : QGraphicsRectItem(parent)
setFlags(QGraphicsItem::ItemIsMovable |
QGraphicsItem::ItemIsSelectable |
QGraphicsItem::ItemSendsScenePositionChanges);
QVariant QSignalingBoxItem::itemChange(GraphicsItemChange change, const QVariant & value)
switch (change)
case QGraphicsItem::ItemSelectedHasChanged:
setSelected_(value.toBool());
break;
case QGraphicsItem::ItemScenePositionHasChanged:
emit boxChanged(rect());
break;
default:
break;
return value;
void QSignalingBoxItem::setSelected_(bool selected)
//TODO: Test that it works as expected
if (selected)
auto sz = new SizeGripItem(new BoxResizer, this);
connect(sz, &SizeGripItem::resized, [&]() emit boxChanged(rect()); );
else
// Get the child
// If it's a SizeGripItem, delete it.
if (childItems().length() > 0)
foreach(QGraphicsItem* item, childItems())
auto sz = dynamic_cast<SizeGripItem*>(item);
if (sz)
delete sz;
break;
如果我在构造函数中应用SizeGripItem，该类将按预期工作。但是，我需要仅在选择该项目时才可以看到手柄并使其工作。但是，当我运行上面的代码时，手柄会显示，但任何点击手柄都会移动整个框（就像我点击了框的中间一样）而不是调整它的大小。
使用调试器运行，我看到itemChange 甚至没有被调用。
为什么会这样？当项目被选中时，我需要如何修改类以使其工作？
编辑
这就是类的使用方式。我有一个QGraphicsView 的扩展名及其私有m_scene，我在其中覆盖mousePress,Move,ReleaseEvents 以通过“单击和拖动”生成一个框。
把课程的其余部分放在一边，因为它唯一涉及QSignalingBoxItems 的部分是这三个函数，我正在添加这些函数。如果需要更多，我会根据需要添加。
void QSampleEditor::mousePressEvent(QMouseEvent * mouseEvent)
QImageView::mousePressEvent(mouseEvent);
if (mouseEvent->buttons() == Qt::RightButton && m_currentSample && m_activePixmap && m_activePixmap->isUnderMouse())
m_dragStart = mapToScene(mouseEvent->pos());
m_currentBox = new QSignalingBoxItem( m_dragStart, QSize(0,0) );
m_scene.addItem(m_currentBox);
mouseEvent->accept();
void QSampleEditor::mouseMoveEvent(QMouseEvent * mouseEvent)
QImageView::mouseMoveEvent(mouseEvent);
if (m_currentBox)
m_currentBox->setRect(rectFromTwoPoints(m_dragStart, mapToScene(mouseEvent->pos())));
mouseEvent->accept();
void QSampleEditor::mouseReleaseEvent(QMouseEvent * mouseEvent)
//TODO: Add checks that the sample is still there.
QImageView::mouseReleaseEvent(mouseEvent);
if (m_currentBox)
// Add to the StringSample
auto new_char = new CharacterSample;
connect(m_currentBox, &QSignalingBoxItem::boxChanged, new_char, &CharacterSample::boxChanged);
new_char->boxChanged(m_currentBox->rect());
m_currentSample->addCharacter(new_char);
m_currentBox = nullptr;
mouseEvent->accept();
【问题讨论】：
提供minimal reproducible example 这里无法提供完整的 MCVE，这是尽可能少且完整的。这是一个使用 GraphicsView + GraphicsScene 的 Qt 小部件应用程序。这是以标准方式插入到标准 Qt 类中的类的扩展。样板代码会淹没我自己的代码，并使“MCVE”的可读性和可用性比这更差 我问一个原因：你指出 itemChange 没有被调用，应该被调用，所以我认为问题出在你当前的实现中。 :) 现在不同了 ;) 我将添加一个如何创建实例的示例 您可以随时提供minimal reproducible example，因为您拥有的代码是 MCVE + 不必要的东西，那么您的工作就是消除那些不必要的东西并且您已经拥有 MCVE，如果问题是扩展总有办法分享，你可以使用 github 或类似的。看看我的最后一个答案：***.com/a/51596332/6622587，几乎没有几行它实际上是minimal reproducible example 【参考方案1】：当给定项目的任何可移动祖先项目被选中时，最年长的选定祖先会接收该事件 - 即使该项目的位置与子项目重叠。这在QGraphicsItem::mouseMoveEvent 中处理 - 如果存在可移动祖先，则不处理移动事件。最年长的可移动选定祖先接收到该事件并使用它来移动自己，但后代项目忽略它（这里 - 句柄！）。
关于代码的一般说明（您的和您正在重用的）：
以Q 开头的类型名称是为Qt 保留的。除非您将 Qt 放在命名空间中，否则不应使用此类名称。
SizeGripItem 应该被标记为没有内容 - 因为它的 paint 方法是无操作的。
通过 const 引用传递非数字、非指针的方法参数除非该方法需要一个副本才能在内部进行修改。
SizeGripItem 需要有一个 Resizer 或发出一个信号，而不是两者兼有 - 这两个选项是互斥的。
事实上，Resizer 是 Qt 4 的遗留物，其中插槽很冗长，无法连接到 lambda。在 Qt 5 中完全没有必要，因为信号可以连接到任何函子，包括 Resizer 类型的函子 - 因此使 QObject::connect 隐式向后兼容 Resizer 的显式使用（！）。
可以提出以下解决方案：
-
简单的解决方案 - 这会抑制问题的主要原因：使托管项目不可移动。然后手柄将起作用。它与选择状态无关。导致问题的是可移动性。
使
SizeGripItem 自身可移动。 SignalingBoxItem 不能移动。
重新实现SizeGripItem::HandleItem 的mouseMoveEvent 以接受相关事件并对它们做出反应。 SignalingBoxItem 仍可移动。
让SizeGripItem 成为其HandleItems 的事件过滤器，并按照解决方案#2 处理相关事件。 SignalingBoxItem 仍可移动。
使SizeGripItem 成为SignalingBoxItem 的同级。然后SignalingBoxItem 可以移动，而无需使用SizeGripItem 的手柄。当然，SignalingBoxItem 的任何祖先都不能移动。
以下是大约 240 行的完整示例，实现解决方案 1-3。每个解决方案都在一个条件块中进行了描述，并且该示例在启用它们的任何子集的情况下进行编译。没有选择解决方案，原始问题仍然存在。可以在运行时选择启用的解决方案。
首先，让我们从SizeGripItem开始：
// https://github.com/KubaO/***n/tree/master/questions/graphicsscene-children-51596611
#include <QtWidgets>
#include <array>
#define SOLUTION(s) ((!!(s)) << (s))
#define HAS_SOLUTION(s) (!!(SOLUTIONS & SOLUTION(s)))
#define SOLUTIONS (SOLUTION(1) | SOLUTION(2) | SOLUTION(3))
class SizeGripItem : public QGraphicsObject
Q_OBJECT
enum kMoveInHandle, kInitialPos, kPressPos ;
struct HandleItem : QGraphicsRectItem
HandleItem() : QGraphicsRectItem(-4, -4, 8, 8)
setBrush(Qt::lightGray);
setFlags(ItemIsMovable | ItemSendsGeometryChanges);
SizeGripItem *parent() const return static_cast<SizeGripItem *>(parentItem());
QVariant itemChange(GraphicsItemChange change, const QVariant &value) override
if (change == ItemPositionHasChanged) parent()->handleMoved(this);
return value;
#if HAS_SOLUTION(2)
bool sceneEvent(QEvent *event) override
return (data(kMoveInHandle).toBool() && hasSelectedMovableAncestor(this) &&
processMove(this, event)) ||
QGraphicsRectItem::sceneEvent(event);
#endif
;
#if HAS_SOLUTION(2) || HAS_SOLUTION(3)
static bool processMove(QGraphicsItem *item, QEvent *ev)
auto mev = static_cast<QGraphicsSceneMouseEvent *>(ev);
if (ev->type() == QEvent::GraphicsSceneMousePress &&
mev->button() == Qt::LeftButton)
item->setData(kInitialPos, item->pos());
item->setData(kPressPos, item->mapToParent(mev->pos()));
return true;
else if (ev->type() == QEvent::GraphicsSceneMouseMove &&
mev->buttons() == Qt::LeftButton)
auto delta = item->mapToParent(mev->pos()) - item->data(kPressPos).toPointF();
item->setPos(item->data(kInitialPos).toPointF() + delta);
return true;
return false;
static bool hasSelectedMovableAncestor(const QGraphicsItem *item)
auto *p = item->parentItem();
return p && ((p->isSelected() && (p->flags() & QGraphicsItem::ItemIsMovable)) ||
hasSelectedMovableAncestor(p));
#endif
std::array<HandleItem, 4> handles_;
QRectF rect_;
void updateHandleItemPositions()
static auto get = &QRectF::topLeft, &QRectF::topRight, &QRectF::bottomLeft,
&QRectF::bottomRight;
for (auto &h : handles_) h.setPos((rect_.*get.begin()[index(&h)])());
int index(HandleItem *handle) const return handle - &handles_[0];
void handleMoved(HandleItem *handle)
static auto set = &QRectF::setTopLeft, &QRectF::setTopRight,
&QRectF::setBottomLeft, &QRectF::setBottomRight;
auto rect = rect_;
(rect.*set.begin()[index(handle)])(handle->pos());
setRect(mapRectToParent(rect.normalized()));
public:
SizeGripItem(QGraphicsItem *parent = ) : QGraphicsObject(parent)
for (auto &h : handles_) h.setParentItem(this);
setFlags(ItemHasNoContents);
QVariant itemChange(GraphicsItemChange change, const QVariant &value) override
if (change == QGraphicsItem::ItemPositionHasChanged) resize();
return value;
void paint(QPainter *, const QStyleOptionGraphicsItem *, QWidget *) override
QRectF boundingRect() const override return rect_;
void setRect(const QRectF &rect)
rect_ = mapRectFromParent(rect);
resize();
updateHandleItemPositions();
void resize() emit rectChanged(mapRectToParent(rect_), parentItem());
Q_SIGNAL void rectChanged(const QRectF &, QGraphicsItem *);
#if SOLUTIONS
void selectSolution(int i)
#if HAS_SOLUTION(1)
setFlag(ItemIsMovable, i == 1);
setFlag(ItemSendsGeometryChanges, i == 1);
if (i != 1)
auto rect = mapRectToParent(rect_);
setPos(); // reset position if we're leaving the movable mode
setRect(rect);
i--;
#endif
for (auto &h : handles_)
int ii = i;
#if HAS_SOLUTION(2)
h.setData(kMoveInHandle, ii-- == 1);
#endif
#if HAS_SOLUTION(3)
if (ii == 1)
h.installSceneEventFilter(this);
else
h.removeSceneEventFilter(this);
#endif
#endif
#if HAS_SOLUTION(3)
bool sceneEventFilter(QGraphicsItem *item, QEvent *ev) override
if (hasSelectedMovableAncestor(item)) return processMove(item, ev);
return false;
#endif
;
那么，SignalingBoxItem：
class SignalingBoxItem : public QObject, public QGraphicsRectItem
Q_OBJECT
SizeGripItem m_sizeGripthis;
QVariant itemChange(GraphicsItemChange change, const QVariant &value) override
if (change == QGraphicsItem::ItemSelectedHasChanged)
m_sizeGrip.setVisible(value.toBool());
else if (change == QGraphicsItem::ItemScenePositionHasChanged)
emitRectChanged();
return value;
void emitRectChanged() emit rectChanged(mapRectToScene(rect()));
void setRectImpl(const QRectF &rect)
QGraphicsRectItem::setRect(rect);
emitRectChanged();
public:
SignalingBoxItem(const QRectF &rect = , QGraphicsItem *parent = )
: QGraphicsRectItem(rect, parent)
setFlags(ItemIsMovable | ItemIsSelectable | ItemSendsScenePositionChanges);
m_sizeGrip.hide();
connect(&m_sizeGrip, &SizeGripItem::rectChanged, this,
&SignalingBoxItem::setRectImpl);
void setRect(const QRectF &rect)
setSelected(false);
m_sizeGrip.setRect(rect);
setRectImpl(rect);
Q_SIGNAL void rectChanged(const QRectF &); // Rectangle in scene coordinates
#if SOLUTIONS
void selectSolution(int index)
setFlag(ItemIsMovable, !HAS_SOLUTION(1) || index != 1);
m_sizeGrip.selectSolution(index);
#endif
;
SampleEditor：
class SampleEditor : public QGraphicsView
Q_OBJECT
bool m_activeDrag = false;
SignalingBoxItem m_box;
QPointF m_dragStart;
public:
SampleEditor(QGraphicsScene *scene) : QGraphicsView(scene)
scene->addItem(&m_box);
connect(&m_box, &SignalingBoxItem::rectChanged, this, &SampleEditor::rectChanged);
Q_SIGNAL void rectChanged(const QRectF &);
void mousePressEvent(QMouseEvent *event) override
QGraphicsView::mousePressEvent(event);
if (event->button() == Qt::RightButton)
m_dragStart = m_box.mapFromScene(mapToScene(event->pos()));
m_activeDrag = true;
m_box.show();
m_box.setRect(m_dragStart, m_dragStart);
event->accept();
void mouseMoveEvent(QMouseEvent *event) override
QGraphicsView::mouseMoveEvent(event);
if (m_activeDrag)
m_box.setRect(m_dragStart, m_box.mapFromScene(mapToScene(event->pos())));
event->accept();
void mouseReleaseEvent(QMouseEvent *event) override
QGraphicsView::mouseReleaseEvent(event);
if (m_activeDrag && event->button() == Qt::RightButton)
event->accept();
m_activeDrag = false;
void resizeEvent(QResizeEvent *event) override
QGraphicsView::resizeEvent(event);
scene()->setSceneRect(contentsRect());
#if SOLUTIONS
void selectSolution(int index) m_box.selectSolution(index);
#endif
;
最后是演示代码：
int main(int argc, char *argv[])
QApplication a(argc, argv);
QWidget ui;
QGridLayout layout&ui;
QGraphicsScene scene;
SampleEditor editor(&scene);
QComboBox sel;
QLabel status;
layout.addWidget(&editor, 0, 0, 1, 2);
layout.addWidget(&sel, 1, 0);
layout.addWidget(&status, 1, 1);
sel.addItems(
"Original (Movable SignalingBoxItem)",
#if HAS_SOLUTION(1)
"Movable SizeGripItem",
#endif
#if HAS_SOLUTION(2)
"Reimplemented HandleItem",
#endif
#if HAS_SOLUTION(3)
"Filtering SizeGripItem",
#endif
);
sel.setCurrentIndex(-1);
#if SOLUTIONS
QObject::connect(&sel, QOverload<int>::of(&QComboBox::currentIndexChanged),
[&](int index) editor.selectSolution(index); );
#endif
QObject::connect(&editor, &SampleEditor::rectChanged, &status,
[&](const QRectF &rect)
QString s;
QDebug(&s) << rect;
status.setText(s);
);
sel.setCurrentIndex((sel.count() > 1) ? 1 : 0);
ui.setMinimumSize(640, 480);
ui.show();
return a.exec();
#include "main.moc"
【讨论】：
您好，首先：非常感谢您的广泛回答。我有几个问题。首先，关于你的评论4），为什么会这样？Resizer 来自 SizeGripItem 实现，而我使用该信号与矩形已编辑的其他对象（对调整大小、框或视图一无所知）进行通信。为什么两者互斥？
根据您的解决方案 1)，如果我正确理解了解释和代码，您只需使手柄可移动并从父矩形禁用 Movable 标志。如果我拖动它，为什么整个盒子会移动？跟亲子关系有关系吗？如果是这样，我在哪里可以找到这种行为的文档？
作为最后的评论，再次重申：我最诚挚和衷心地感谢您提供的不是一个，而是三个解决方案。然而，恕我直言，将所有内容压缩到单行以适应 240 行的完整程序，虽然对允许可测试性很有用，但对接下来阅读答案的人没有帮助。我遇到了一个问题，现在有 3 个解决方案，但我觉得我没有完全理解 为什么 存在问题以及 每个解决方案如何解决它。我确实有一些我会剖析并最终完全理解的代码，但是解释每个解决方案的几句话将使这成为一个很好的答案。
@GPhilo 解决方案 1 使 SizeGripItem 本身可移动 - 回想一下，它是一个覆盖（在场景坐标中）托管对象的虚拟矩形。它管理的底层对象不能在该解决方案中移动。解决方案 2 和 3 允许您让您的项目保持可移动，并且它们不会使 SizeGripItem 可移动。
@GPhilo 本身没有压缩。代码使用 Google clang-format 格式化，缩进 3 个空格，而不是默认的两个。如果您检查大多数 Google 代码库，它会看起来非常像。引用的局部性对于人类对代码的理解很重要——在源代码文件中物理紧凑并不是一件坏事。没有采用任何技巧来最小化代码——我只是删除了所有不必要的功能，并在适用的情况下使用了更简洁的构造，因为原始代码非常冗长。以上是关于如果选择该项目，为啥 QGraphicsItem 子项不再单击鼠标？的主要内容，如果未能解决你的问题，请参考以下文章
在自定义 QGraphicsView 中移动 QGraphicsItem 问题
为啥所有选中的 QGraphicsItem 都没有收到 mouseMove 事件？
为啥 itemAt() 并不总能找到 QGraphicsItem
为啥自定义 QGraphicsItem 会导致场景或视图移动，除非 boundingRect 为空？