_ASSERTE(_BLOCK_TYPE_IS_VALID(pHead-> nBlockUse))错误

时间:2013-12-17 22:08:46

标签: c++

我正在试图找出我跑步时程序失败的原因。到目前为止,当我运行我的程序时,它失败了。我调试了错误,它将我带到了dbgdel.cpp。第32行“_ASSERTE(_BLOCK_TYPE_IS_VALID(pHead-> nBlockUse));”。我找到了答案无济于事,但它与我的内存泄漏有关?

这是我的标题:

#include <iostream>
using namespace std;
namespace project
{
#ifndef MATRIX_H
#define MATRIX_H

typedef int* IntArrayPtr;
class Matrix
{
public:
    friend ostream& operator<<(ostream& out, Matrix& object);
    //friend istream& operator>>(istream& in, Matrix& theArray);
    //Default Constructor
    Matrix();
    Matrix(int max_number_rows, int max_number_cols, int intial_value);
    //Destructor
    ~Matrix();
    //Copy Constructor
    Matrix(const Matrix& right_side);
    //Assignment Operator
    Matrix& operator=(const Matrix& right_side);

    void Clear();
    int Rows();
    int Columns();
    bool GetCell(int x,int y, int& val);
    bool SetCell(int x,int y, int val);
    int getNumber(int r, int c);
    //void Debug(ostream& out);
private:
    int initialVal;
    int rows;
    int cols;
    IntArrayPtr *m;
};
#endif
}

我的实施档案:

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

namespace project
{
Matrix::Matrix()
{
typedef int* IntArrayPtr;
IntArrayPtr *m = new IntArrayPtr[1];
for(int i = 0; i < 1; i++)
    m[i] = new int[1];

for(int r = 0; r < 1; r++)
    for(int c = 0; c < 1;c++)
        m[r][c] = 0;
}
Matrix::Matrix(int max_number_rows, int max_number_cols, int initial_value)
{
initialVal = initial_value;
rows = max_number_rows;
cols = max_number_cols;
IntArrayPtr *m = new IntArrayPtr[rows];
for(int i = 0; i < rows; i++)
    m[i] = new int[cols];

for(int r = 0; r < max_number_rows; r++)
    for(int c = 0; c < max_number_cols;c++)
        m[r][c] = initial_value;

}
Matrix::~Matrix()
{
for(int i = 0; i <= rows; i++)
    delete [] m[i];
delete [] m;
}
void Matrix::Clear()
{
for(int r = 0; r < rows; r++)
    for(int c = 0; c < cols;c++)
        m[r][c] = initialVal;
}

int Matrix::Rows()
{
return rows;
}

int Matrix::Columns()
{
return cols;
}

bool Matrix::GetCell(int x,int y, int& val)
{
if(x < rows || y < cols)
    return false;
else
{
    val =   m[x - 1][y - 1];
    return true;
}
}

bool Matrix::SetCell(int x, int y, int val)
 {
if(x < rows || y < cols)
    return false;
else
{
    m[x - 1][y - 1] = val;
    return true;
}
}
int Matrix::getNumber(int r, int c)
{
return m[r][c];
}

ostream& operator<<(ostream& out, Matrix& object)
{

for(int r = 0; r < object.rows; r++)
{
    for(int c = 0; c < object.cols; c++)
    {
        out << " " << object.m[r][c];
    }
out << endl;
}
return out;
}

Matrix& Matrix::operator=(const Matrix& right_side)
{
if (this != &right_side)
{
    rows = right_side.rows;
    cols = right_side.cols;
    delete[] m;
    IntArrayPtr *m = new IntArrayPtr[rows];
    for(int i = 0; i < rows; i++)
        m[i] = new int[cols];

    for(int r = 0; r < rows; r++)
        for(int c = 0; c < cols;c++)
            m[r][c] = right_side.initialVal;
}
return *this;
}

/*
void Matrix::Debug(ostream& out)
{
out << "Number of rows = " << rows << endl;
out << "Number of columns = " << cols << endl;
out << "Initializer = " << initialVal << endl;
out << "Current contents of the matrix: " << endl;
out << m << endl;
}
*/
/*
istream& operator >>(istream& in, Matrix& theArray)
{
in >> 


}

void interfaceMatrix()
{
int userChoice, rows, columns, value;

cout << "Default matrix or custom(1 for default, 0 for custom): ";
cin >> userChoice;
    if (userChoice == 1)
    {
        Matrix object;
        return object;
    }
    else if(userChoice == 0)
    {
        cout << "Enter number of rows: ";
        cin >> rows;
        cout << "Enter number of columns: ";
        cin >> columns;
        cout << "Enter initial value of each element: ";
        cin >> value;
        if(rows <= 0 || columns <= 0)
        {
            cout << "Invalid input." << endl;
            exit(1);
        }
        else
        {
            Matrix object(rows, columns, value);
            return object;
        }
    }
    else
    {
        cout << "Invalid choice." << endl;
        exit(1);
    }

}
*/
}

在我的驱动程序中,我只使用了Matrix测试(2,2,2),因此我可以为每个元素创建一个初始值为2的2 x 2数组。我得到了上述错误。

2 个答案:

答案 0 :(得分:2)

您正在分配rows个行数,但是要取消分配rows+1个行数。 检查析构函数。 &lt; =必须&lt;。

除此之外,您的代码中还存在许多其他[潜在]错误:

  • 您正在设置本地m变量,而不是设置类的m数据成员(这就是为什么我的约定总是在数据成员之前用m_来防止这种混淆)。此错误同时出现在构造函数和赋值运算符中。
  • 您使用rows分配矩阵,但max_number_rows初始化矩阵。虽然它现在可以正常工作,但如果row稍后以不同方式进行初始化(例如,如果使用row初始化std::max(max_number_rows,1000),则可能会导致错误。您在赋值运算符中的代码更好。
  • 您在GetCell和SetCell中的if-test不正确。它应该是>=而不是<
  • 赋值运算符复制矩阵的大小,但为所有单元格分配initialValue。这不是一项任务。这种实现可能会/将会混淆代码的其余部分。
  • IntArrayPtr的typedef是不必要的

答案 1 :(得分:1)

两个问题:

  1. 您没有在&#34; m&#34;中分配任何值。您的对象的成员,您正在分配名为&#34; m&#34;
  2. 的局部变量
  3. 你通过从i = 0循环到i&lt; =行来解除分配,你想要i = 0到i&lt;行