MFC中CDialog::Create工作机制及OnInitDialog与ChangeFolder同步问题

先看你的代码：

myDlg = new Dlg(doc, myDlg , myDlgID);
myDlg -> Create(myDlgID, this);
myDlg -> ChangeFolder(directory);

你担心的核心问题是：ChangeFolder依赖OnInitDialog中初始化的资源，但CDialog::Create是通过Windows消息触发OnInitDialog的，无法确定OnInitDialog何时执行，怕ChangeFolder先运行导致资源未初始化。

先纠正一个关键误解 #

非模态对话框的CDialog::Create调用后，窗口会被创建，但OnInitDialog是由WM_INITDIALOG消息触发的——这个消息会进入当前线程的消息队列。如果调用Create的线程在Create返回后立即执行ChangeFolder，而此时消息队列还没轮到处理WM_INITDIALOG，那ChangeFolder确实会在OnInitDialog之前运行，触发错误。

你的互斥锁思路行不通 #

不管是单线程还是多线程场景，你说的“构造函数加锁、OnInitDialog末尾解锁”的逻辑都有问题：

• 单线程下：同一线程内代码串行执行，互斥锁不会阻塞自己，ChangeFolder依然会在OnInitDialog之前执行，锁起不到任何作用。
• 多线程下：构造函数加锁后，OnInitDialog在同一线程中执行时可以重复获取锁，但其他线程调用ChangeFolder会被无意义阻塞，而且逻辑上也没必要在构造阶段就加锁。

正确的解决方式 #

场景1：单线程（绝大多数情况）

这是最常见的场景，根本不需要同步锁，直接从调用时机或状态检查入手：

1. 最优方案：调整调用时机
   把ChangeFolder的调用移到OnInitDialog的末尾，确保资源初始化完成后再执行：

   BOOL Dlg::OnInitDialog()
   {
       CDialog::OnInitDialog();
       
       // 初始化你的资源
       // ...
       
       // 直接在这里调用ChangeFolder
       ChangeFolder(m_defaultDirectory); // 可以把目录参数存在成员变量里
       
       return TRUE;
   }
   

   这样从根源上避免了时机问题，代码最简洁。

2. 次选方案：加初始化状态标志
   如果必须从外部调用ChangeFolder，就在对话框类里加一个标志位，在OnInitDialog完成后标记初始化完成，ChangeFolder开头先检查：

   class Dlg : public CDialog
   {
   private:
       bool m_bInitialized = false;
       // 其他成员...
   public:
       virtual BOOL OnInitDialog()
       {
           BOOL bRet = CDialog::OnInitDialog();
           if (bRet)
           {
               // 初始化资源
               // ...
               
               m_bInitialized = true;
           }
           return bRet;
       }
       
       void ChangeFolder(CString directory)
       {
           if (!m_bInitialized)
           {
               ASSERT(FALSE); // 调试模式下触发断言，提示调用时机错误
               return; // 发布模式下可以返回错误码或静默处理
           }
           
           // 正常处理文件夹切换逻辑
           // ...
       }
   };
   

场景2：多线程（其他线程可能调用ChangeFolder）

如果有其他线程会触发ChangeFolder，就需要用轻量级的同步机制（优先用临界区，比互斥锁开销小）保护初始化状态和资源访问：

class Dlg : public CDialog
{
private:
    bool m_bInitialized = false;
    CCriticalSection m_csInitLock; // MFC的临界区类，用于线程同步
    // 其他成员...
public:
    virtual BOOL OnInitDialog()
    {
        BOOL bRet = CDialog::OnInitDialog();
        if (bRet)
        {
            // 初始化资源
            // ...
            
            m_csInitLock.Lock();
            m_bInitialized = true;
            m_csInitLock.Unlock();
        }
        return bRet;
    }
    
    void ChangeFolder(CString directory)
    {
        m_csInitLock.Lock();
        if (!m_bInitialized)
        {
            m_csInitLock.Unlock();
            ASSERT(FALSE);
            return;
        }
        
        // 这里访问初始化后的资源，全程持有锁，避免多线程冲突
        // ...
        
        m_csInitLock.Unlock();
    }
};

总结 #

• 单线程下完全不需要互斥锁，调整调用时机或加状态标志就够了。
• 多线程下用临界区保护初始化状态和资源访问，比互斥锁更高效。
• 你的原始互斥锁思路逻辑有问题，不要这么用。

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