矢量擦除迭代器外范围

Jav*_*wag 0 c++ vector msvc12

使用C++将我的游戏引擎中的一些代码从Mac移植到Windows,我得到了这个运行时错误:"矢量擦除外部范围".它适用于Mac!

void Entity::runDeferreds() {
    for (auto it = deferreds.begin(); it != deferreds.end(); /* nothing */ ) {
        if (it->Condition()) {
            it->Execution();

            it = deferreds.erase(it);
        } else {
            ++it;
        }
    }
}
Run Code Online (Sandbox Code Playgroud)

这将通过"延迟"任务列表进行迭代,这些任务存储在std::vector<DeferredCall>被调用的任务中deferreds.如果DeferredCall's' Condition()已经完成,那么它Execution()就会被运行,它应该被删除vector.但是,相反,我得到上述错误!

DeferredCall看起来像这样,而不是它太重要了:

struct DeferredCall {
    std::function<bool()> Condition;
    std::function<void()> Execution;
};
Run Code Online (Sandbox Code Playgroud)

救命?!

编辑: - 替代方法

我也试过这个,再次在Mac上工作:

deferreds.erase(std::remove_if(deferreds.begin(), deferreds.end(),
    [](DeferredCall &call) {
            if (call.Condition()) {
                call.Execution();
                return true;
            }

            return false;
        }
    ), deferreds.end());
Run Code Online (Sandbox Code Playgroud)

但是,在这种情况下,我得到"矢量迭代器不兼容".

小智 5

虽然它没有回答您的错误来自哪里,但您可以尝试按如下方式重新编写代码:

const auto pred = [](Deferred& d){ return !d.Condition(); };
auto itMatch = std::partition( defs.begin(), defs.end(), pred);

const auto action = [](Deferred& d){ d.Execution(); };
std::for_each(itMatch, defs.end(), action);

defs.erase(itMatch, defs.end());
Run Code Online (Sandbox Code Playgroud)

此外,std :: vector :: erase保证返回完全有效的迭代器.可能是vector::end().

链接:std :: partition,std :: for_each,std :: vector :: erase