Qt中如何识别被鼠标点击的自定义QGraphicsItem并获取其ID

嘿，我看到你在获取自定义QGraphicsItem的ID时遇到了麻烦，咱们一步步来解决这个问题：

首先得指出你代码里的两个关键问题，这正是你拿不到正确点击项ID的核心原因：

1. 鼠标点击事件里的逻辑错误 #

你在mousePressEvent里已经用itemAt(event->pos(), QTransform())拿到了鼠标点击位置的项，但后面又遍历所有可选择的项，把最后一个可选择项赋值给item——这直接覆盖了之前拿到的点击项，完全忽略了鼠标点击的位置！这一步完全是多余且错误的。

2. 变量名冲突导致ID赋值错误 #

在paintLights函数里，你定义了两个同名的i变量：外面的int i是用来做ID计数的，但内部又定义了QGraphicsLayoutItem *i，这导致你调用mLight->setId(i)时，传入的是一个指针（被强制转成int），而不是你想要的计数ID！这肯定会让你拿到奇怪的ID值，完全不符合预期。

现在咱们来修正代码： #

首先修复paintLights里的变量冲突

把内部的QGraphicsLayoutItem变量改名，避免覆盖计数用的ID变量：

void EffectPreviewScene::paintLights(int x,int y) {
    int offsetX = 10;
    int offsetY = 10;
    int lightId = 0; // 改用独立变量做ID计数，避免冲突
    for (int xcnt=0;xcnt<x;xcnt++){
        for (int ycnt=0;ycnt<y;ycnt++){
            QGraphicsLayoutItem *layoutItem = new QGraphicsLayoutItem(0); // 变量名修改
            LightObject *mLight = new LightObject(offsetX,offsetY,QColor( 255, 0, 0));
            mLight->setId(lightId); // 传入正确的计数ID
            mLight->setX(xcnt);
            mLight->setY(ycnt);
            qDebug() << "xcnt=" << xcnt;
            qDebug() << "ycnt=" << ycnt;
            addItem(mLight);
            qDebug() << "offsetX=" << offsetX;
            offsetY += 40;
            qDebug() << "offsetY=" << offsetY;
            lightId++; // 递增ID计数
        }
    }
}

然后简化并修正mousePressEvent

直接使用itemAt获取点击的项，去掉多余的遍历逻辑：

void EffectPreviewScene::mousePressEvent(QGraphicsSceneMouseEvent *event) {
    if(!event->isAccepted()) {
        testUpdate();
        qDebug() << "graphics scene event not accepted";
        
        // 直接获取鼠标点击位置的目标项
        QGraphicsItem *clickedItem = itemAt(event->pos(), QTransform());
        
        if (clickedItem) {
            qDebug() << "found item";
            qDebug() << "lights.size()"<<lights.size();
            
            // 转换为自定义LightObject类型
            LightObject *selectedLight = dynamic_cast<LightObject*>(clickedItem);
            if (selectedLight){
                // 注意：不要每次点击都执行connect！会导致重复连接，信号触发多次槽函数
                // 建议把connect移到LightObject创建时，或者用Qt::UniqueConnection
                // connect(selectedLight,&LightObject::setColor,mColorManager,ColorManager::colorChanged);
                
                qDebug() << "color="<< selectedLight->color();
                qDebug() << "bounding rect=" << selectedLight->boundingRect();
                qDebug() << "id=" << selectedLight->id(); // 现在可以拿到正确的ID了！
            }
        }
    }
    // 务必调用父类的事件处理，避免影响场景其他功能
    QGraphicsScene::mousePressEvent(event);
}

额外的优化建议 #

• 信号槽连接优化：不要在每次点击时连接信号，否则同一个信号会触发多次槽函数。建议在创建LightObject时完成连接，或者添加Qt::UniqueConnection参数确保只连接一次。
• 高效类型转换：如果你想更高效地识别自定义类型，可以完善LightObject的type()函数：

int LightObject::type() const {
    // 返回一个大于QGraphicsItem::UserType的自定义值
    return QGraphicsItem::UserType + 1;
}

之后就可以用qgraphicsitem_cast<LightObject*>(clickedItem)来转换类型，比dynamic_cast更高效，且专门针对QGraphicsItem体系。

• 检查boundingRect：itemAt依赖于boundingRect的准确性，如果你的椭圆范围计算错误，可能导致点击识别失效，确保boundingRect返回的范围和你的绘制区域一致。

这样调整后，你应该就能正确获取到点击项的ID了！

内容的提问来源于stack exchange，提问作者Pavel Melnikov