2

我对 C++ 有点陌生，我有一个很好奇的问题。有一段时间我遇到了分段错误，虽然我最终让它工作了，但我想知道为什么以前没有。这就是我所拥有的：

#include <string>
#include <sstream>
#include <iostream>
#include <vector>
using namespace std;
class A {
    private:
        int num;
    public:
        A(int i){this->num=i;}
        int getNum(){return this->num;}
};
class B {
    private:
        vector<A*> list;
    public:
        vector<A*> getList(){return this->list;}
        void addA(A* i){this->getList().push_back(i);}
        string as_a_string(){
            stringstream result;
            cout << "Flag1" <<endl; //only for debug, this prints
            for (vector<A*>::iterator x = this->getList().begin(); x != this->getList().end(); ++x) {
                cout << "Flag2" << endl; //only for debug, this prints
                A* w = *x;
                cout << "Flag3" << endl; //only for debug, this prints
                result << w->getNum() << " ";
                cout << "Flag4" << endl; //only for debug, this does not print
            }
            return result.str();
        }
};
int main() {
    A* a = new A(4);
    B* b = new B();
    b->addA(a);
    cout << b->as_a_string() << endl;
    return 0;
}

我通过替换 with 的每个实例解决了我的问题this->getList()（在 for 循环定义中this->list有 3 个；一个 in和两个）。为什么使用成员本身而不是通过方法访问它会影响该程序的工作？B::addA(A*)B::as_a_string()

4

3 回答 3

2

由于getList正在返回 的临时副本list，因此list当您尝试遍历它时 又会消失。

有几种不同的方法可以解决这个问题：

  1. 只需使用list（无需使用this->- 它不是 PHP、Python 或某些您总是必须通过this/self指针引用成员的语言）。
  2. 让我们getList返回对向量的引用。( vector<A*>& getList() { ... };)
  3. 使用局部变量作为副本

但是，选项 3 对 没有帮助addA，它要求您对 ORIGINAL 列表进行操作 - 就目前而言，您push_back对 的临时副本进行操作，该副本在返回list后立即被销毁push_back- 所以它实际上并没有在全部到实际的list成员变量）。

就个人而言，我会更改您的内部代码以list直接使用，并不断getList返回一个副本，以便如果某些外部函数想要获取列表，它可以。

编辑：您还通过不删除您在中创建的 A* 和 B* 对象来泄漏内存main。

于 2013-08-13T23:58:08.677 回答
2

问题是您B::getList()按值返回列表，因此实际上返回的是副本B::list而不是原始列表本身。

在你的循环中：

for (vector<A*>::iterator x = this->getList().begin();
     x != this->getList().end(); ++x)

您首先获取一个开始迭代器到这个临时列表。每次检查条件时，您都将其与结束迭代器与不同的临时列表对象进行比较。

解决此问题的简单方法是B::getList()通过引用返回列表：

vector<A*>& getList();
于 2013-08-13T23:58:16.143 回答
2

你的代码：

class B {
    private:
        vector<A*> list;
    public:
        vector<A*> getList(){return this->list;}
        void addA(A* i){this->getList().push_back(i);}

正在临时调用 push_back()，因为 getList() 按值返回。调用 add() 后，成员变量列表不变。您可以更改 getList() 函数签名以返回如下引用：

vector<A*>& getList(){return this->list;}

老实说，您可能最好使用您用来避免问题的解决方案，因为提供一个返回对您的实现细节的引用的公共访问器是您通常想要避免的事情。

于 2013-08-14T00:04:56.680 回答