c++ - Listing Observer 没有更新 "see"

Question

我必须创建一个类，它观察一个 vector 并在每次有变化时收到通知。我的问题是，我的程序不会对 vector 内容的更改使用react。有人可以告诉我，我在哪里犯了错误吗？

实现:

    class IObserver
{
public:
    virtual void notify() = 0;
    virtual ~IObserver(){};
};

class ObservableVector
{
    vector<int> ov;
    vector<IObserver*> registered;
public:
    void notifyObserver()
    {
        for (IObserver* el : registered)
        {
            el->notify();
        }
    }
    void add(int n)
    {
        ov.push_back(n);
        notifyObserver();
    }
    void registerObserver(IObserver* obs)
    {
        registered.push_back(obs);
    }
    const vector<int> & items() const
    {
        vector<int> it;
        for (int el : ov)
        {
            it.push_back(el);
        }
        return it;
    }
};

class ListingObserver : public IObserver
{
    ObservableVector & _ov;
public:
    ListingObserver(ObservableVector & ov) : _ov(ov)
    {
        _ov.registerObserver(this);
    }
    void notify()
    {
        vector<int> toNotify;
        toNotify = _ov.items();

        cout << "Elements: " << endl;
        for (int el : toNotify)
        {
            cout << el << endl;
        }
    }
};

int main()
{
    ObservableVector ov;
    ListingObserver lo(ov);
    //MeanObserver mo(ov);
    ov.add(1);
    ov.add(2);
    ov.add(3);
}

Answer 1

这个函数有问题

const vector<int> & items() const
{
    vector<int> it;
    for (int el : ov)
    {
        it.push_back(el);
    }
    return it;
}

这里你创建了一个局部 vector it，并返回对它的引用，但在函数结束时这个对象被销毁了。因此，您返回了对不存在对象的引用，因此您得到了未定义的行为。

要更正您的代码，您可以将此函数声明为

vector<int> items() const

并保留它现在的实现或者像这样简单地实现

const vector<int> & items() const
{
    return ov;
}