MFC中CWinThread在OnTimer事件的内存泄漏与性能问题咨询

问题背景 #

作为MFC开发新手，当前场景为：通过0.5秒触发一次的OnTimer事件检查11个轴的位置是否为0，每个轴的位置校验函数最长耗时50秒。当前在对话框构造函数中循环为每个轴创建CWinThread线程，校验完成后销毁线程。疑问点包括：

1. 若将创建线程的循环放到OnTimer中多次执行，是否会引发内存泄漏或性能问题？
2. 如何安全更新详情对话框，避免其或应用冻结、挂起？

现有代码的核心问题 #

1. 线程销毁逻辑错误：CheckHomeDone中设置pThread->m_bAutoDelete = FALSE，但线程函数CheckHomeDoneThreadTemporary中直接delete pThread（通过AfxGetThread()获取当前线程指针），这种在线程内部销毁自身线程对象的操作极不安全，可能导致内存访问错误或泄漏。
2. 全局变量竞态条件：使用全局变量t_nMotor传递轴ID，多线程环境下会被覆盖，导致校验逻辑混乱。
3. 线程无限制创建风险：若OnTimer每0.5秒就创建12个线程，而每个线程最长运行50秒，短时间内会堆积大量线程，耗尽系统CPU和内存资源，引发严重性能问题。

问题解答 #

一、OnTimer中多次执行线程创建循环的影响 #

• 内存泄漏：线程销毁逻辑错误，加上线程堆积未正确清理，必然导致内存泄漏。
• 性能崩溃：大量线程同时运行会引发频繁的上下文切换，CPU占用率飙升，应用甚至系统会出现卡顿、无响应。

二、解决方案 #

1. 规范线程管理，避免泄漏与性能问题

• 控制线程数量：为每个轴维护线程指针数组（如CWinThread* m_pAxisThreads[12] = {nullptr};），仅当对应轴无运行线程时才创建新线程。
• 正确管理线程生命周期：使用m_bAutoDelete = TRUE（默认值），线程结束后自动销毁，无需手动delete；若需手动管理，需在主线程中等待线程结束后再释放线程对象。
• 消除全局变量竞态：通过结构体传递线程参数，将轴ID与对象指针一并传入线程函数。

修正后的核心代码示例：

// 定义线程参数结构体
struct ThreadParam {
    MachineSequence* pMachineSeq;
    int motorID;
};

UINT CMotionMovement::CheckHomeDoneThreadTemporary(LPVOID pParam)
{
    ThreadParam* pParamObj = reinterpret_cast<ThreadParam*>(pParam);
    if (pParamObj) {
        // 执行轴位置校验
        pParamObj->pMachineSeq->motionParameters.HomeDone(pParamObj->motorID);
        // 发送消息通知主线程更新UI
        ::PostMessage(AfxGetMainWnd()->m_hWnd, WM_UPDATE_AXIS_STATUS, 
                      (WPARAM)pParamObj->motorID, (LPARAM)/*校验状态*/);
        delete pParamObj; // 释放参数内存
    }
    return 0;
}

void CMotionMovement::CheckHomeDone(int motorID)
{
    // 检查该轴是否已有线程在运行
    if (m_pAxisThreads[motorID] != nullptr) {
        return;
    }

    ThreadParam* pParam = new ThreadParam();
    pParam->pMachineSeq = this;
    pParam->motorID = motorID;

    CWinThread* pThread = ::AfxBeginThread(CheckHomeDoneThreadTemporary, pParam, THREAD_PRIORITY_ABOVE_NORMAL);
    if (pThread != nullptr) {
        pThread->m_bAutoDelete = TRUE; // 线程结束自动销毁
        m_pAxisThreads[motorID] = pThread; // 记录线程指针
    } else {
        delete pParam; // 创建失败，释放参数
        AfxMessageBox(_T("Failed to create thread"), MB_ICONERROR | MB_OK);
    }
}

// 在OnTimer中检查线程状态，更新指针数组
void CMotionMovement::OnTimer(UINT_PTR nIDEvent)
{
    if(nIDEvent == MOTION_TIMER && bRunThread) {
        if(!bActiveTimer) {
            bActiveTimer = true;

            // 检查各轴线程是否结束
            for (int i = 0; i < 12; ++i) {
                if (m_pAxisThreads[i] != nullptr && 
                    WaitForSingleObject(m_pAxisThreads[i]->m_hThread, 0) == WAIT_OBJECT_0) {
                    m_pAxisThreads[i] = nullptr; // 线程结束，清空指针
                }
            }

            RedrawMotorPosStatus();

            if( moveMotorToNextVerificationPosition ) {
                moveVerificationMotor();
                linearEncoderPositionCounter = 0;
                verifyLinearEncoderPosition = true;
                displayEncoderVariation = false;
                moveMotorToNextVerificationPosition = false;
            }
            bActiveTimer = false;
        }
    }
    CDialog::OnTimer(nIDEvent);
}

2. 安全更新UI，避免冻结

MFC UI操作必须在主线程执行，线程需通过消息机制通知主线程更新UI：

• 定义自定义消息：

  #define WM_UPDATE_AXIS_STATUS (WM_USER + 100)
  

• 添加消息映射与处理函数：

  BEGIN_MESSAGE_MAP(CMotionMovement, CDialog)
      ON_MESSAGE(WM_UPDATE_AXIS_STATUS, OnUpdateAxisStatus)
      // 其他已有消息映射
  END_MESSAGE_MAP()
  
  LRESULT CMotionMovement::OnUpdateAxisStatus(WPARAM wParam, LPARAM lParam)
  {
      int motorID = static_cast<int>(wParam);
      int status = static_cast<int>(lParam);
      // 执行UI更新操作，比如更新轴状态文本、进度条等
      RedrawMotorPosStatus();
      return 0;
  }
  

3. 优化OnTimer逻辑

• 仅在OnTimer中检查线程状态、更新UI，避免批量创建线程；
• 按需触发轴校验，而非每次OnTimer都创建新线程。

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