发布模式错误,但不在调试模式下

时间:2012-05-10 14:34:52

标签: c++ multithreading release-mode

我的代码在调试模式下运行良好,但在发布模式下失败。

这是我的代码片段,它失败了:

LOADER->AllocBundle(&m_InitialContent);
while(!m_InitialContent.isReady())
{
    this->LoadingScreen();
}

AllocBundle()将加载m_InitialContent中包含的内容,并在完成后将其准备状态设置为true。这是使用多线程实现的。

this->LoadingScreen()应该呈现一个加载屏幕,但是目前还没有实现,所以该函数有一个空体。

显然这可能是错误的原因:如果我给函数LoadingScreen()一行代码:std::cout<<"Loading"<<std::endl;那么它将运行正常。

如果我不这样做,那么代码就会卡在while(!m_InitialContent.isReady())它甚至不会跳转到括号(this->LoadingScreen();)之间的代码。显然它也没有更新while语句中的表达式,因为它永远停留在那里。

有没有人有任何想法可能导致这个?如果是这样,问题可能是什么? 我完全不解。


编辑:请求附加代码

ContentLoader的成员:details::ContentBundleAllocator m_CBA;

    void ContentLoader::AllocBundle(ContentBundle* pBundle)
    {
        ASSERT(!(m_CBA.isRunning()), "ContentBundleAllocator is still busy");
        m_CBA.Alloc(pBundle, m_SystemInfo.dwNumberOfProcessors);
    }

void details::ContentBundleAllocator::Alloc(ContentBundle* pCB, UINT numThreads)
{
    m_bIsRunning = true;
    m_pCB = pCB;
    pCB->m_bIsReady = false;


    m_NumRunningThrds = numThreads;
    std::pair<UINT,HANDLE> p;
    for (UINT i = 0; i < numThreads; ++i)
    {
        p.second = (HANDLE)_beginthreadex(NULL,
                                          NULL,
                                          &details::ContentBundleAllocator::AllocBundle,
                                          this,
                                          NULL,&p.first);
        SetThreadPriority(p.second,THREAD_PRIORITY_HIGHEST);
        m_Threads.Insert(p);
    }
}

unsigned int __stdcall details::ContentBundleAllocator::AllocBundle(void* param)
{
//PREPARE
    ContentBundleAllocator* pCBA = (ContentBundleAllocator*)param;

//LOAD STUFF [collapsed for visibility+]

   //EXIT===========================================================================================================
        pCBA->m_NumRunningThrds -= 1;
        if (pCBA->m_NumRunningThrds == 0)
        {
            pCBA->m_bIsRunning = false;
            pCBA->m_pCB->m_bIsReady = true;
            pCBA->Clear();
    #ifdef DEBUG
            std::tcout << std::endl;
    #endif
            std::tcout<<_T("exiting allocation...")<<std::endl;
        }

    std::tcout<<_T("exiting thread...")<<std::endl;
    return 0;
}

bool isReady() const {return m_bIsReady;}

3 个答案:

答案 0 :(得分:6)

当您在调试模式下编译代码时,编译器会在幕后执行许多操作,以防止程序员因应用程序崩溃而导致的许多错误。当您在发布中运行时,所有投注均已关闭。如果您的代码不正确,那么您在Release中崩溃的可能性要大于Debug。

要检查的一些事项:

  1. 确保所有变量都已正确初始化
  2. 确保您没有任何死锁或竞争条件
  3. 确保您没有传递指向已解除分配的本地对象的指针
  4. 确保您的字符串正确地以NULL结尾
  5. 不要catch您不期待的例外情况,然后继续运行,就好像什么也没发生一样。

答案 1 :(得分:4)

您正在从不同线程访问变量m_bIsReady而没有内存障碍。这是错误的,因为它可能由优化器或处理器缓存缓存。您必须使用CriticalSection或互斥锁或库中可用的任何同步原语来保护此变量不会同时访问。

请注意,可能还有其他错误,但这个错误也绝对是错误的。根据经验:从不同线程访问的每个变量都必须使用互斥/临界区/其他方式进行保护。

答案 2 :(得分:0)

快速查看m_NumRunningThrds似乎无法防止同时访问,因此if (pCBA->m_NumRunningThrds == 0)可能永远不会得到满足。