Qt中用Map管理QGraphicsScene动态内存是否为正确实践？

你的这种做法能实现内存释放的基本目的，但绝对算不上最佳实践，存在冗余和潜在风险：

• 用std::map存储单个指针完全没必要，map是为多键值场景设计的，这里只用到固定key=0，直接用类成员变量指针就能搞定，徒增代码复杂度。
• 手动new/delete容易出错：比如函数中途抛出异常（如图片加载失败），可能导致旧内存未释放或新内存泄漏；后续逻辑修改时，若遗漏map的清空操作，也会直接造成内存泄漏。
• 你已经在创建QGraphicsScene时传入了this作为父对象，但没利用Qt的父子对象自动内存管理机制，白浪费了Qt自带的安全回收能力。

下面是几种更合理的替代方案：

方案1：复用现有QGraphicsScene（性能最优） #

如果只是切换显示的图片，完全不需要每次创建新场景，直接清空旧内容再添加新图片即可：

// 类成员变量声明：QGraphicsScene* m_scene = nullptr;
void myFunction()
{
    QPixmap pixmap(fileName);
    if (pixmap.isNull()) {
        // 处理图片加载失败的情况，比如提示用户
        return;
    }
    QPixmap scaledPixmap = pixmap.scaled(ui.graphicsView_UnprocessedImage->size(),
                                        Qt::KeepAspectRatio, Qt::FastTransformation);
    
    if (!m_scene) {
        m_scene = new QGraphicsScene(this);
        ui.graphicsView_UnprocessedImage->setScene(m_scene);
    } else {
        m_scene->clear(); // 清空旧的图片内容
    }
    m_scene->addPixmap(scaledPixmap);
}

这种方式避免了频繁创建销毁对象的开销，性能最佳，尤其适合需要频繁切换图片的场景。

方案2：利用Qt父子对象自动回收（最简单） #

如果确实需要切换场景，利用Qt的父子对象机制，让父对象（比如当前窗口）自动管理子对象的生命周期，同时用deleteLater()保证安全销毁：

// 类成员变量声明：QGraphicsScene* m_scene = nullptr;
void myFunction()
{
    QPixmap pixmap(fileName);
    if (pixmap.isNull()) {
        return;
    }
    QPixmap scaledPixmap = pixmap.scaled(ui.graphicsView_UnprocessedImage->size(),
                                        Qt::KeepAspectRatio, Qt::FastTransformation);
    
    // 用deleteLater将销毁操作放到事件循环执行，避免当前上下文出现悬空指针
    if (m_scene) {
        m_scene->deleteLater();
    }
    
    m_scene = new QGraphicsScene(this);
    m_scene->addPixmap(scaledPixmap);
    ui.graphicsView_UnprocessedImage->setScene(m_scene);
}

Qt会在父对象销毁时自动销毁所有子对象，deleteLater()则避免了在当前函数中直接销毁可能引发的崩溃问题。

方案3：使用C++智能指针（标准内存管理方式） #

如果不想依赖Qt的父子机制，用std::unique_ptr来独占管理指针，彻底告别手动delete：

// 类成员变量声明：std::unique_ptr<QGraphicsScene> m_scene;
void myFunction()
{
    QPixmap pixmap(fileName);
    if (pixmap.isNull()) {
        return;
    }
    QPixmap scaledPixmap = pixmap.scaled(ui.graphicsView_UnprocessedImage->size(),
                                        Qt::KeepAspectRatio, Qt::FastTransformation);
    
    // 重置unique_ptr时，旧内存会自动释放
    m_scene = std::make_unique<QGraphicsScene>(this);
    m_scene->addPixmap(scaledPixmap);
    ui.graphicsView_UnprocessedImage->setScene(m_scene.get());
}

std::unique_ptr是C++标准的独占式智能指针，赋值或销毁时自动释放内存，完全避免手动管理的风险。

总结 #

你的初始做法能工作，但存在冗余和出错隐患，优先推荐方案1（复用场景），其次根据需求选择方案2或3，完全没必要用std::map来存储单个指针。

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