c++ - 从触摸回调访问 vector 元素的方法时出现 BAD ACCESS 错误

Question

我有一个类结构如下:

class A
{
 public:
     virtual void func() = 0;
};

class B : public A
{
 public:
     virtual void func();
};

和我在相应的.cpp 文件中提供了B 的函数的实现。

void B::func()
{
     cocos2d::log("Hello");
}

现在，当我尝试从另一个文件访问 B 的 func 时，如下所示:

class XYZ
{
 public:
    public void func2();
 protected:
    cocos2d::EventListenerTouchOneByOne * _Listener = nullptr;
    std::vector<A *> _List;

    bool touchBeginCallback(cocos2d::Touch *touch, cocos2d::Event *event);
    void touchEndCallback(cocos2d::Touch *touch, cocos2d::Event *event);
};

在 XYZ.cpp 中

void XYZ::func2()
{
    _List.push_back(new B());
    _List.push_back(new B());
    _List.push_back(new B());
    _List.push_back(new B());
    _List.push_back(new B());

    for(auto itr = _List.begin(); itr != _List.end(); itr++)
    {
        A * a = (*itr);
        if(a)
        {
            a->func(); // Here it works fine
        }
    }

    _Listener = cocos2d::EventListenerTouchOneByOne::create();
    _Listener->setSwallowTouches(true);
    _Listener->onTouchBegan = (cocos2d::EventListenerTouchOneByOne::ccTouchBeganCallback) CC_CALLBACK_2(XYZ::touchBeginCallback, this);
    _Listener->onTouchEnded = (cocos2d::EventListenerTouchOneByOne::ccTouchCallback) CC_CALLBACK_2(XYZ::touchEndCallback, this);

    cocos2d::Director::getInstance()->getEventDispatcher()->addEventListenerWithSceneGraphPriority(_Listener, this);
}

bool XYZ::touchBeginCallback(cocos2d::Touch *touch, cocos2d::Event *event)
{
    return true;
}

void XYZ::touchEndCallback(cocos2d::Touch *touch, cocos2d::Event *event)
{
    for(auto itr = _List.begin(); itr != _List.end(); itr++)
    {
        A *a = (*itr);
        if(a)
        {
            a->func(); // Throws bad access error
        }
    }
}

如果我从 XYZ 中的方法调用 func() 调用(或者更确切地说，访问 B 类型的对象)，它工作正常，但如果我从 touchEndCallback 方法调用它，则会因 BAD_ACCESS 错误而崩溃。

当我在 Debug模式下检查 _List 在 func2() 中时，它正确地显示 _List 有五个 B 类型 成员，但是当我检查列表时 在 touchEndCallback 方法中，它表明 _List 有五个类型 A 的成员。

为什么会发生这种崩溃，我该如何解决？

Answer 1

解引用迭代器以获取指针 A*。循环应如下所示:

for(auto itr = _List.begin(); itr != _List.end(); itr++)
{
    if (*itr)
        (*itr)->func();
}

或者您可以将范围用于:

  for(auto pa : _List)  // now you're gettting A* 's directly
    {
        if (pa)
           pa->func();    
    }

编辑:使用原始指针也是个坏主意。考虑 unique_ptr 或 shared_ptr。如果仍然有错误，可能是因为您正在删除其他地方的对象，所以 List 包含无效指针。但是没有完整的代码很难说发生了什么。如果对象在其他地方被删除，那么您需要明确地将 List 的 A* 设置为 nullptr，否则 if (a) 仍会计算为真。