Question

所以我有这个函数来添加监听器，它会转换一个类的共享指针，以便我可以在收到通知后稍后调用它。

void registerListener(std::shared_ptr<T> listener)
{
    if (!listener) {
        qCWarning(OBSERVER_LOGGER) << "Attempted to register a null observer.";
        return;
    }
    // TODO make a foreach function that removes dead listeners to get rid of this code dupe
    for (auto iter=listeners.begin(); iter != listeners.end(); ) {
        if (auto shared = iter->lock()) {
            if (listener == shared) {
                return;
            }
            iter++;
        } else {
            iter = listeners.erase(iter);
        }
    }
    auto weak = std::weak_ptr<T>(listener);
    listeners.push_back(weak);
}

void notify(std::function<void(std::shared_ptr<T>)> onNotify)
{
    // TODO make a foreach function that removes dead listeners to get rid of this code dupe
    for (auto iter=listeners.begin(); iter != listeners.end(); ) {
        if (auto shared = iter->lock()) {
            onNotify(shared);
            iter++;
        } else {
            iter = listeners.erase(iter);
        }
    }
}

private:
std::vector<std::weak_ptr<T>> listeners;

由于某种原因，“iter-＆gt; lock（）”段错误。我会说这是一个Qt应用程序，但我故意没有创建任何线程（我知道）所以我只是非常困惑我做错了让这些weak_ptrs中断。所以，如果我在gdb中运行它，它的工作正常。但如果我设置，“设置禁用随机化关闭”然后我得到错误。因此，对于未初始化的变量，我觉得这是一个奇怪的问题。如果它有帮助，那么当我在gdb中崩溃时，这就是堆栈。

#0  0x00007f856bd8beec in std::_Sp_counted_base<(__gnu_cxx::_Lock_policy)2>::_M_get_use_count() const ()
#1  0x00007f856bd844a8 in std::_Sp_counted_base<(__gnu_cxx::_Lock_policy)2>::_M_add_ref_lock_nothrow() ()
#2  0x00007f856bd9cd7d in std::__shared_count<(__gnu_cxx::_Lock_policy)2>::__shared_count(std::__weak_count<(__gnu_cxx::_Lock_policy)2> const&, std::nothrow_t) ()
#3  0x00007f856bda9948 in std::__shared_ptr<IEntityListener<Assignment>, (__gnu_cxx::_Lock_policy)2>::__shared_ptr(std::__weak_ptr<IEntityListener<Assignment>, (__gnu_cxx::_Lock_policy)2> const&, std::nothrow_t) ()
#4  0x00007f856bda8a62 in std::shared_ptr<IEntityListener<Assignment> >::shared_ptr(std::weak_ptr<IEntityListener<Assignment> > const&, std::nothrow_t) ()
#5  0x00007f856bda701a in std::weak_ptr<IEntityListener<Assignment> >::lock() const ()
#6  0x00007f856bda5624 in Observer<IEntityListener<Assignment> >::notify(std::function<void (std::shared_ptr<IEntityListener<Assignment> >)>) ()
#7  0x00007f856bda3a1a in EntityObserver<Assignment>::notifyCreated(std::shared_ptr<Assignment>) ()

编辑：迈克尔伯尔发布了听众登记的可能性，而新的听众被添加，这可能完全发生。这会导致迭代器无效，并且当我在不是weak_ptr，BOOM的内存部分上调用weak_ptr.lock（）时。我认为这里有道德，我必须找到它。

Answer 1

当调用notify()时，通过onNotify()函数对象调用的函数可能导致间接调用registerListener()（或者可以添加或删除条目的其他代码） listeners集合）？

如果是这样，那么iter notify()循环中使用的for可能会失效。您可能希望将notify()更改为类似以下内容，将所有shared_ptr个对象排队通知，以便在任何期间修改listeners集合并不重要onNotify()回调：

#include <queue>

void notify(std::function<void(std::shared_ptr<T>)> onNotify)
{
    std::queue<std::shared_ptr<T>> notify_targets;

    for (auto iter=listeners.begin(); iter != listeners.end(); ) {
        if (auto shared = iter->lock()) {
            notify_targets.push(shared);
            iter++;
        } else {
            iter = listeners.erase(iter);
        }
    }

    while (!notify_targets.empty()) {
        onNotify(notify_targets.front());
        notify_targets.pop();
    }

}

Answer 2

我在您展示的代码中找不到明显的问题。所以我假设，问题在于你没有展示的代码。请记住，Q_OBJECT装饰对象也具有与qt相关的生命周期管理。也许有一些干扰...

如果您将代码与我在此处发布的代码进行比较，并且您很快就会找到关键的区别，这可能会有所帮助。

#include <vector>
#include <algorithm>
#include <memory>

template <typename _Observable>
class IObserver
{
public:
    virtual ~IObserver() {}
    virtual void OnChange(const _Observable* what) = 0;
};

template <class T>
class CObservable
    : public IObserver<T> // Make sure this class breaks once IObserver<> changes.
{
public:
    typedef IObserver<T> Observer_t;
    typedef std::weak_ptr<Observer_t> Observer_reference;
    typedef std::shared_ptr<Observer_t> Observer_strong_reference;
    typedef T Class_t;
    typedef std::vector<Observer_reference> ObserverRefCollection;
private:

    ObserverRefCollection m_observers;

    void CleanupZombies()
    {
        m_observers.erase(std::remove_if(m_observers.begin(), m_observers.end(),
            [this](Observer_reference iter) -> bool
        {
            Observer_strong_reference o = iter.lock();
            return !o;
        }
        ), m_observers.end());
    }
public:
    void RegisterObserver(Observer_strong_reference& observer)
    {
        if (!observer)
            return;
        for (auto& iter : m_observers)
        {
            if (observer == iter.lock())
                return;
        }
        m_observers.push_back(Observer_reference(observer));
    }

    /*virtual*/ void OnChange(const Class_t* what)
    {
        bool hasZombies = false;
        for (auto& iter : m_observers)
        {
            Observer_strong_reference o = iter.lock();
            if (o)
            {
                o->OnChange(what);
            }
            else
            {
                hasZombies = true;
            }
        }
        if (hasZombies)
            CleanupZombies();
    }
};

class CObservableUint32
    : public CObservable<CObservableUint32>
{
    uint32_t m_value;
public:
    void Set(uint32_t newValue)
    {
        bool changed = newValue != m_value;
        m_value = newValue;
        if (changed)
        {
            OnChange(this);
        }
    }
    uint32_t Get() const
    {
        return m_value;
    }
};

class CSomeObserver
    : public IObserver < CObservableUint32 >
{

public:
    CSomeObserver()
        : IObserver<CObservableUint32>()
    {

    }
    virtual ~CSomeObserver()
    {

    }
    virtual void OnChange(const CObservableUint32* what)
    {

    }
};

其他地方......

    CObservableUint32 observable;
    {
        std::shared_ptr<IObserver<CObservableUint32> > observer = std::make_shared<CSomeObserver>();
        observable.RegisterObserver(observer);
        observable.Set(42UL);
    }
    observable.Set(100);

C ++ weak_ptr.lock（）段错误

2 个答案: