私はこれをしばらく見つめていましたが、なぜ失敗するのかわかりません。次のような2Dマトリックスを実装するCMatrixクラスがあります。
class CMatrix {
public:
CMatrix(int height, int width);
~CMatrix();
// more stuff here
private:
void deleteData();
void allocData(int height, int width);
int rows, cols;
double** data;
};
CMatrix::CMatrix(int height, int width) {
allocData(height, width);
}
CMatrix::~CMatrix() {
deleteData();
}
私がそれを破壊しようとするまで、それはうまくいきます。次のように定義された、割り当てと割り当て解除を担当する2つの方法があります。
void CMatrix::allocData(int height, int width) {
this->rows = height;
this->cols = width;
data = new double*[rows];
for (int i = 0; i < rows; i++) {
data[i] = new double[cols];
for (int j = 0; j < cols; j++)
data[i][j] = 0;
}
}
void CMatrix::deleteData() {
for (int i = 0; i < rows; i++) {
delete data[i];
}
delete [] data;
}
この単純なmain
コードにより、誤動作が発生します。
int main(int argc, char**
CMatrix a(2, 3);
a[0][0] = 1;
a[0][1] = 2;
a[0][2] = 3;
a[1][0] = 4;
a[1][1] = 5;
a[1][2] = 6;
return 0;
}
valgrindはこれを言います:
==21005== Mismatched free() / delete / delete []
==21005== at 0x4C2A44B: operator delete(void*) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==21005== by 0x402B5C: CMatrix::deleteData() (main.cpp:375)
==21005== by 0x401731: CMatrix::~CMatrix() (main.cpp:138)
==21005== by 0x402F5D: main (main.cpp:598)
==21005== Address 0x5a06090 is 0 bytes inside a block of size 24 alloc'd
==21005== at 0x4C2AAA4: operator new[](unsigned long) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==21005== by 0x402A84: CMatrix::allocData(int, int) (main.cpp:366)
==21005== by 0x4015FB: CMatrix::CMatrix(int, int) (main.cpp:123)
==21005== by 0x402D63: main (main.cpp:415)
このメッセージは、メソッドdeleteData()
および。にあるときに出力されますi=0
。i
他のsについても、delete[]についても何も言いません。
私は何が間違っているのですか?