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

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

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

1 个答案: