C++内存分配错误

Question

我是一名 Java 程序员，但现在我必须用 C++ 编写一些代码。几年前我学习了 C++ 基础知识，所以我不太适合。

我写了一个描述多项式的小类。它是这样的：

#include "Polynom.h"
#include <iostream>

using namespace std;

Polynom::Polynom()
{
    this->degree = 0;
    this->coeff = new int[0];
}

Polynom::Polynom(int degree)
{
    this->degree = degree;
    this->coeff = new int[degree + 1];
}

Polynom::~Polynom()
{
    delete coeff;
}

void Polynom::setDegree(int degree)
{
    this->degree = degree;
}

void Polynom::setCoeffs(int* coeff)
{
    this->coeff = &*coeff;
}

void Polynom::print()
{
    int i;
    for(i = degree; i >= 0; i --)
    {
        cout<<this->coeff[i];
        if(i != 0)
            cout<<"x^"<<i;
        if(i > 0)
        {
            if(coeff[i - 1] < 0)
                cout<<" - ";
            else
                cout<<" + ";
        }
    }    
}

好的，现在我尝试读取多项式的次数和系数并将其打印在控制台中。这是代码：

#include <iostream>
#include "Polynom.h"
using namespace std;

int main()
{
    int degree;

    cout<<"degree = ";
    cin>>degree;
    int* coeff = new int[degree];
    int i;
    for(i = 0; i <= degree; i++)
    {
        cout<<"coeff[x^"<<i<<"] = ";
        cin>>coeff[i];
    }
    Polynom *poly = new Polynom(degree);
    //poly->setDegree(degree);
    poly->setCoeffs(coeff);
    cout<<"The input polynome is: ";
    poly->print();
    return 0;
}

编译代码时，一切正常。运行时，如果我给出一个偶数度数，然后给出一些系数，程序就可以正常运行。 但是：如果我定义一个奇数度数（例如 3 或 5），然后给出系数，程序不会打印多项式并返回以下错误：

malloc.c:3096: sYSMALLOc: Assertion `(old_top == (((mbinptr) (((char *) &((av)->bins[((1) - 1) * 2])) - __builtin_offsetof (struct malloc_chunk, fd)))) && old_size == 0) || ((unsigned long) (old_size) >= (unsigned long)((((__builtin_offsetof (struct malloc_chunk, fd_nextsize))+((2 * (sizeof(size_t))) - 1)) & ~((2 * (sizeof(size_t))) - 1))) && ((old_top)->size & 0x1) && ((unsigned long)old_end & pagemask) == 0)' failed.

为什么这会发生吗？我哪里没有为数组分配足够的内存？我在 Google 上搜索了此错误，并偶然发现了此页面 ，但是那里提到的解决方案对我没有多大帮助。

也许您可以在我的代码中看到另一个问题？我非常感谢你的帮助。

提前致谢。

Answer 1

您的代码存在大量错误。 C++ 与 Java 完全不同，看起来您使用指针就好像它们就像 Java 中的引用一样，但它们显然不是。

Polynom::Polynom()
{
    this->degree = 0;
    this->coeff = new int[0];
}

这会创建一个大小为零的数组，这在 C++ 中是合法的，但几乎不是您想要的。

Polynom::~Polynom()
{
    delete coeff;
}

C++ 中的数组必须使用 delete[] 删除：

Polynom::~Polynom()
{
    delete [] coeff;
}

这：

void Polynom::setDegree(int degree)
{
    this->degree = degree;
}

没有意义 - 您更改了度数，但没有更改与其关联的数组。

void Polynom::setCoeffs(int* coeff)
{
    this->coeff = &*coeff;
}

我不知道你认为这是在做什么，我怀疑你也不知道。而且你有内存泄漏。

这只是一个开始——我怀疑还有更多糟糕的事情。您需要做两件事：

• 阅读一本有关 C++ 的书。由于您有编程经验，我建议您Accelerated C++。

• 忘记您的 Java 知识。正如我所说，这两种语言几乎没有任何共同点。

Answer 2

int* coeff = new int[degree];
int i;
for(i = 0; i <= degree; i++)

您正在为 level 元素分配空间并放置 level+1 元素...该行为未定义。

Answer 3

在 main() 函数中，int *coeff = new int[level] 为您提供一个长度 level 数组，其元素索引范围为0 到 level-1（含）。在循环中，您将访问元素 0 到 level（含）。这是未定义的行为，可能会也可能不会导致运行时错误等。

Answer 4

for 循环中的语句 i <= Degree 导致了这种情况。由于数组索引从 0 开始，因此有效范围为 0-> Degree-1。由于您正在写入无效的内存位置，因此您的程序的行为不可预测。