如何重建包含向量的非指针类成员？

Question

我遇到了一个奇怪的问题：我无法正确重置（破坏和构造）包含向量的属性。尝试访问向量时会导致分段错误。

这是我的代码（用 C++11 编写）。我认为我尽可能简化了它以强调这个问题，但我可能错了，对此感到抱歉。 目标是打印两次不同的（随机）向量。第一个向量运行良好，第二个因未知原因完全失败。

#include <iostream>
#include <ctime>
#include <cstdlib>
#include <vector>

class A
{
        std::vector<int> n;

    public :
        A();
        std::string toString() const;
};

A::A()
{
    for (int i = 0; i < 10; i++)
        n.push_back(std::rand()%10);
}

std::string A::toString() const
{
    for (auto i : n)
        std::cout << i << ' ';
    std::cout << std::endl;
}

class B
{
        A a;

    public :
        void resetA();
        A getA() const;
};

void B::resetA()
{
    a = A();
}

A B::getA() const
{
    return a;
}

int main()
{
    srand(time(NULL));

    B b;
    std::cout << b.getA().toString();
    b.resetA();
    std::cout << b.getA().toString();


    return EXIT_SUCCESS;
}

出于某种原因，我想尽可能避免使用指针和动态分配。它不太符合我的 UML 概念。

此外，此代码在使用简单 int（无向量）时运行良好。

谢谢。

Answer 1

您的 toString() 不返回任何内容，因此您的程序具有未定义的行为（实际上，返回的随机垃圾肯定不是有效的 std::string 对象）。

也许您想改用字符串流？

#include <sstream>

// ...

std::string A::toString() const
{
    std::ostringstream s;
    for (auto i : n)
        s << i << ' ';
    s << '\n';
    return s.str();
}

Live example.

一般来说，编译时打开尽可能多的警告是个好主意。这肯定会被报告为警告。对于这个特定的警告（no-void 函数没有返回任何内容），我强烈建议将其视为错误。