将MainWindow指针传递给其他类是否合理？Qt实现方案求评估

你的整体实现思路是正确的，利用Qt的信号槽机制拆分GUI层和业务逻辑层，符合Qt框架的设计哲学，现有运行逻辑没有功能问题。

你关心的内存风险分析 #

• 你当前示例中MainWindow和calcModule都是main函数内的栈变量，calcModule定义晚于MainWindow，销毁时会先析构calcModule再析构MainWindow，因此当前场景下不会出现野指针问题。
• 但如果后续代码调整变量生命周期、改为动态分配对象、或者调整定义顺序，calcModule持有的MainWindow裸指针就有可能变成野指针，触发内存访问异常。

优化建议 #

1. 彻底移除calcModule对MainWindow的依赖（最优方案） #

信号槽机制本身支持发送方和接收方完全解耦，你完全不需要在calcModule中持有MainWindow的指针，也不需要在calcModule内部执行连接操作：

• 删除calcModule类中持有的MainWindow *Mw成员变量
• 修改calcModule构造函数，移除MainWindow指针参数
• 将信号槽连接逻辑移到main函数中，调整后main代码如下：

int main(int argc, char *argv[])
{
    QApplication a(argc, argv);
    MainWindow w;
    calcModule cM;
    // 直接在main中完成信号槽绑定，两个类互相不需要知道对方的内部实现
    QObject::connect(&w, &MainWindow::sigMyButton, &cM, &calcModule::changeValState);
    w.show();
    return a.exec();
}

修改后calcModule的头文件不需要再include mainwindow.h，仅需要保留QObject相关的头文件即可，耦合度降到最低，也完全消除了指针相关的内存隐患。

2. 若确实需要持有GUI对象指针，改用Qt弱指针 #

如果后续业务逻辑确实需要calcModule访问MainWindow的成员，不要用裸指针，改用QPointer<MainWindow> Mw：

• QPointer是Qt提供的智能指针，当指向的QObject派生类对象销毁时，QPointer会自动置空，访问前判空即可避免野指针崩溃
• 参考使用方式：

// calcModule.h中修改成员定义
QPointer<MainWindow> Mw;

// 访问前先判空
if (!Mw.isNull()) {
    // 执行相关操作
}

3. 其他规范优化 #

• 类名遵循驼峰命名规范，calcModule改为CalcModule，符合C++/Qt开发惯例
• changeValState槽函数如果不需要外部调用，可以改为private slots，减少对外暴露的接口

内容的提问来源于stack exchange，提问作者Paweł Kwaśny