Question

我必须创建一个类，它会观察一个向量，并在每次发生变化时得到通知。我的问题是，我的程序对更改向量的内容没有反应。有人可以告诉我，我犯了哪个错误吗？

实施：

    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;
}

在这里创建一个本地向量it，并返回对它的引用，但在函数结束时，该对象被销毁。所以你返回对不存在的对象的引用，因此你有未定义的行为。

要更正您的代码，您可以将此功能解除为

vector<int> items() const

并保留其现在的实现，或者使其像这样

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

Answer 2

我尝试了你的代码，它确实做出了反应。但是在items函数中存在一个问题，因为您正在返回对函数的本地堆栈上定义的变量的引用。换句话说，在函数结束后该变量不存在，因此名为toNotify的变量被设置为空。

也许这就是导致错误的原因，您可以通过简单地返回ov而不是将其元素复制到另一个变量并返回此变量来解决它。

Listing Observer没有“看到”更新

2 个答案: