在wxWidgets中创建全局wxCriticalSection变量是不是很糟糕?

时间:2018-01-14 13:03:56

标签: c++ multithreading wxwidgets

我正在使用wxTreeListCtrl来显示树。对于树节点的每次扩展,我创建一个调用后端API并返回响应的线程。执行每个线程后,执行HandleExpansionRequest(),将响应元素添加到树中。这些操作运行正常。

问题是: 情况1:当线程正在执行后端函数时,如果用户关闭主框架,则wxQueueEventEntry()的执行会产生异常。 要解决这个问题: 我没有在窗口关闭时删除线程,而是使用变量isFrameClosed并在线程为主代码创建wxQueueEvent之前检查它。我怀疑的是,这是一个很好的方法来阻止程序关闭后的程序崩溃但线程仍在执行(因为我正在创建全局wxCriticalSection)?

BEGIN_EVENT_TABLE(MyFrame, wxFrame)
EVT_TREELIST_ITEM_EXPANDED(wxID_ANY, MyFrame::OnItemExpand)   
EVT_THREAD(ITEM_BROWSE_WORKER_EVENT,MyFrame::OnExpandThreadCompletion)
END_EVENT_TABLE()

//using below 2 variables to avoid crash when item expansion request is sent and user closes the frame.
 wxCriticalSection cs_forisFrameClosed;
 static bool isFrameClosed;

MyFrame::MyFrame()
{
   {
    wxCriticalSectionLocker lock(cs_forisFrameClosed);
    isFrameClosed = false;
   }
   //remaining code ...
}

MyFrame::~MyFrame()
{
   {
    wxCriticalSectionLocker lock(cs_forisFrameClosed);
    isFrameClosed = true;
   }
}

void MyFrame::OnItemExpand(wxTreeListEvent& event)
{
     MyThread *thread = new MyThread(this);
    if (thread->Create() != wxTHREAD_NO_ERROR)
    {
        PrintError(__FUNCTION__, "Can't create thread!");
    }
    thread->Run();
}

MyThread::MyThread(MyFrame *frame) : wxThread()
{
    m_frame = frame;
}

wxThread::ExitCode MyThread::Entry()
{
    string resp;
    GetResponseFromBackEnd(resp);
    {
        wxCriticalSectionLocker lock(cs_forisFrameClosed);
        if (!isFrameClosed)
        {
            wxThreadEvent event(wxEVT_THREAD, ITEM_BROWSE_WORKER_EVENT);
            event.SetString(resp);
            wxQueueEvent(m_frame, event.Clone());
        }
    }
}


void MyFrame::OnExpandThreadCompletion(wxThreadEvent &event)
{
    // this function is executed after every thread returns.
    // take response from event and update tree with children.      
}

2 个答案:

答案 0 :(得分:1)

正如the docs所说,分离的线程会自行删除,因此您无法删除它们,因为它会导致执行两次。相反,您必须实现某种与线程通信的方式,并要求它在需要时退出。 wxMessageQueue提供了最简单的方法之一。

答案 1 :(得分:0)

尝试在close方法中加入()所有生成的线程。