C ++ MFC将文本添加到编辑控件会导致程序崩溃

Question

test.h中的类：

class CHelixV3Dlg : public CDialogEx
{
   DECLARE_DYNAMIC(CHelixV3Dlg);
friend class CHelixV3DlgAutoProxy;

// Construction
public:
   CHelixV3Dlg(CWnd* pParent = NULL);   // standard constructor

// Dialog Data
   enum { IDD = IDD_HELIXV3_DIALOG };


// Implementation
protected:
   CHelixV3DlgAutoProxy* m_pAutoProxy;
   HICON m_hIcon;
public:
   DWORD WINAPI loop(LPVOID); //thread identifier
   void AppendText(CEdit &edit, LPCTSTR pszText);
};

我有一个像这样定义的方法，它应该打印一个字符串到editbox，它通常有效（在test.cpp中）：

void TestDlg::AppendText(CEdit &edit, LPCWSTR pszText)
{
   // get the initial text length
   int nLength = edit.GetWindowTextLength();
   // put the selection at the end of text
   edit.SetSel(nLength, nLength);
   // replace the selection
   edit.ReplaceSel(pszText);
 }

然后这是我的线程，它监听按键（test.cpp）

 DWORD WINAPI TestDlg::KeyThread(LPVOID PARAMS)
 {
    TestDlg* Testdlg;
    while (1)
    {
       Sleep(1);
       if (GetAsyncKeyState(VK_F1) & 1)
       {
           Enabled = !Enabled;
           if (Enabled) {
            CEdit* log = (CEdit*)GetDlgItem(IDC_EDIT1);
            log->AppendText(*log, "test"); //causes a crash
       }
    }
   return 0;
  }

对话框初始化，其中＆＃34;循环＆＃34;线程已创建：

BOOL CHelixV3Dlg::OnInitDialog()
{
TestDlg* Testdlg;
CDialogEx::OnInitDialog();

//creates the thread->
CreateThread(NULL, NULL, &Testdlg->KeyThread, NULL, NULL, NULL);

/* IGNORE THIS */
CMenu* pSysMenu = GetSystemMenu(FALSE);
if (pSysMenu != NULL)
{
    BOOL bNameValid;
    CString strAboutMenu;
    bNameValid = strAboutMenu.LoadString(IDS_ABOUTBOX);
    ASSERT(bNameValid);
    if (!strAboutMenu.IsEmpty())
    {
        pSysMenu->AppendMenu(MF_SEPARATOR);
        pSysMenu->AppendMenu(MF_STRING, IDM_ABOUTBOX, strAboutMenu);
    }
}

// Set the icon for this dialog.  The framework does this automatically
//  when the application's main window is not a dialog
SetIcon(m_hIcon, TRUE);         // Set big icon
SetIcon(m_hIcon, FALSE);        // Set small icon

// TODO: Add extra initialization here

return TRUE;  // return TRUE  unless you set the focus to a control
}

所以，程序会编译，但是当它运行并按F1时，它会崩溃。

Answer 1

您的代码中存在多个问题。其他人已经指出了未初始化的Testdlg指针。

这是另一个：

log->AppendText(*log, "test");

我确定你想说：

Testdlg->AppendText(*log, "test");

但是，这不会起作用，因为这是在工作线程上。您需要做的是从工作线程发送/发送消息到主线程的队列（创建测试对话框窗口）。在邮件的处理程序中，您实际上可以调用AppendText方法。

以下是您的操作方式（请注意，我在浏览器中输入了所有内容，因此可能包含错误）：

在标题（TestDlg.h）中声明一个处理程序：

LRESULT OnMessageFromWorker(WPARAM wParam, LPARAM lPARAM);

在源文件（TestDlg.cpp）中：

定义您的信息：

#define MY_WORKER_THREAD_MSG   WM_APP + 123

在消息地图中添加消息条目

ON_MESSAGE(MY_WORKER_THREAD_MSG, &TestDlg::OnMessageFromWorker)

从工作线程发送消息：

 DWORD WINAPI TestDlg::KeyThread(LPVOID PARAMS)
 {
    HWND dlgWnd = (HWND)PARAMS;
    while (1)
    {
       Sleep(1);
       if (GetAsyncKeyState(VK_F1) & 1)
       {
           Enabled = !Enabled;
           if (Enabled) 
           {
               CString* msg = new CString("test");
               PostMessage(dlgWnd, 0, reinterpret_cast<LPARAM>(msg));
           }
       }
    }
   return 0;
  }

请注意，在此示例中，您需要传递对话框窗口的HWND，而不是this指向线程函数的指针。换句话说：

CreateThread(nullptr, nullptr, &Testdlg->KeyThread, this->GetSafeHwnd(), nullptr, nullptr);

处理消息：

LRESULT TestDlg::OnMessageFromWorker(WPARAM wParam, LPARAM lPARAM)
{
   CString* msg = reinterpret_cast<CString*>(lPARAM);
   CEdit* log = (CEdit*)TestDlg->GetDlgItem(IDC_EDIT1);
   AppendText(*log, *msg);
   delete msg; // you must delete the object

   return 0;
}

Answer 2

首先，您应该将TestDlg :: Thread定义为静态成员函数。当你调用CreateThread时，第四个参数应该是'this'。

在头文件（TestDlg.h或其他）中：

static DWORD WINAPI TestDlg::KeyThread(LPVOID PARAMS);

在OnInitDialog中：

//创建主题 - ＆gt;

CreateThread(NULL, NULL, &Testdlg->KeyThread, this, NULL, NULL);


DWORD WINAPI TestDlg::KeyThread(LPVOID PARAMS)
 {
    TestDlg* Testdlg = (TestDlg*) PARAMS;
    while (1)
    {
       Sleep(1);
       if (GetAsyncKeyState(VK_F1) & 1)
       {
           Enabled = !Enabled;
           if (Enabled) {
            CEdit* log = (CEdit*)TestDlg->GetDlgItem(IDC_EDIT1);
            log->AppendText(*log, "test"); //causes a crash
       }
    }
   return 0;
  }

它仍然可能无效，因为CWnd指针可能只在主线程中有效。 YMMV