C++ 多项式复制构造函数和 ostream 覆盖导致模因泄漏?

Question

嗨，我正在研究 c++ 中的多项式类。到目前为止，一切正常。但是现在我遇到了一个我根本无法发现的错误：/

多项式.cpp

// copy-constructor
    Polynomial::Polynomial(const Polynomial &p){
        size = p.size;
        buffer = p.buffer;
        poly = new double(buffer);
        for (int i = 0; i < size; ++i) poly[i] = p[i];
        for (int i = size; i < buffer; ++i) poly[i] = 0;
    }
// output stream override | it's a non-member function
ostream& operator<<(ostream& os, const v1::Polynomial& p){      
    int degree = p.degree();
    stringstream ss;
    if (degree == 0) ss << '0';
    else if (degree > 0){
        ss << '(';
        for (int i = degree; i >= 0; --i){
            ss << p[i];
            ss << "x^";
            ss << i;
            if (i > 0)
                ss << " + ";
        }
        ss << ')' << endl;
    }
    os << ss.str();
    return os;
}

所以我就是这样称呼复制构造函数的：

// note: printing 'a' itself does not cause problems...
v1::Polynomial b(a);    
cout << "Polynomial b: " << b << " degree: " << b.degree() << endl;;

Visual Studio 的堆栈日志说它在第 23 行（这里：上面的一行，我实际上想打印"b"），然后它继续调用一些堆函数等。在没有调试的情况下运行程序（通过cmd）会导致APPCRASH，"多项式b："是最后显示的内容。

不幸的是，我不知道如何在Visual Studio中调试，我习惯了linux中的valgrind，我目前还没有设置：/

有人知道吗？或者您需要其他信息？

无论如何，提前非常感谢=）

Answer 1

poly = new double(buffer);

在这里，您可以分配单个double并将其设置为 buffer 。你可能的意思是

poly = new double[buffer];

或任何您想要的尺寸。

更好的解决方案是使用 std::vector 而不是原始数组。您可以使用=复制它，使用std::vector::resize调整其大小并使用std::vector::reserve保留更多空间。