0

私は今かなり迷っています。ベクトルクラスを作成しました。最後まで、すべてが私が望むように機能します。デストラクタが呼び出されると、エラーメッセージが表示されます。デバッグアサーションが失敗しましたBLOCK_TYPE_IS_VALID（pHead-> nblockuse）。SOでこのような質問をたくさん見ましたが、試したことがうまくいきませんでした。

.hの一部

private:
    int* _myArray;
    int _size;
    int _capacity;

#include "File.h"

 const string RETURN_CARRIAGE_STR = "\n";
 const string SIZE_STR = "Size ";
 const string CAPACITY_STR = "Capacity ";
 const int INITIAL_CAPACITY = 2;

 int main(void)
{
cout << "\nCreating a vector Sam of size 4.";
MyVector sam( 4 );

cout << "\nPush 12 values into the vector.";
for (int i = 0; i < 12; i++)
    sam.push_back(i);

cout << "\nHere is sam: ";
cout << sam;
cout << "\n---------------\n";

    cout << "\nCreating a vector Joe of size 4.";
MyVector joe( 4 );
cout << "\nPush 6 values into the vector.";
for (int i = 0; i < 6; i++)
    joe.push_back(i * 3);

cout << "\nHere is joe: ";
cout << joe;
cout << "\n---------------\n";

cout << "\nTest the overloaded assignment operator \"joe = sam\": ";
joe = sam;


cout << "\nHere is sam: ";
cout << sam;
cout << "\n---------------\n";

cout << "\nHere is joe: ";
cout << joe;
cout << "\n---------------\n";

// pass a copy of sam by value
PrintV(sam);

cout << endl;
system("PAUSE");
 }

 void PrintV(MyVector v)
 {
cout << "\n--------------------\n";
cout << "Printing a copy of a vector\n";
cout << v;
 }

 // Default Constructor
 MyVector::MyVector()
 {
_myArray = new int[INITIAL_CAPACITY];
_size = 0;
_capacity = INITIAL_CAPACITY;
//cout << "Default Constructor" << endl;
 }

 MyVector::MyVector(int aSize)
 {
_myArray = new int[aSize];
_size = 0;
_capacity = aSize;

//cout << "Parameterized Constructor" << endl;
 }

 MyVector::~MyVector()
 {
if(_myArray != NULL)
{
    delete[] this->_myArray; //  --------------This is where I get an error
    this->_myArray = NULL;
}

//cout << "Destructor" << endl;
 }


 int MyVector::GetSize()
 {
return _size;
//cout << " size";
 }

 int MyVector::GetCapacity()
 {
return _capacity;
//cout << _capacity << " capacity" << endl;
 }

 void MyVector::Clear()
 {
int* resize_arr = new int[INITIAL_CAPACITY];
delete[] _myArray;
_myArray = resize_arr;
_capacity = INITIAL_CAPACITY - 1;
_size = 0;
 }

 void MyVector::push_back(int newValue)
 {
if(_size < _capacity)
{
    _myArray[_size] = newValue;
}
else  {
    int* resize_arr = new int[_capacity*2];
    for(int i = 0; i <= _size; i++)
        resize_arr[i] = _myArray[i];

    resize_arr[_size] = newValue;
    delete[] _myArray;
    _myArray = resize_arr;
    _capacity = _capacity * 2;
}
_size++;
//cout << _size << " size" << endl;
 }

 int MyVector::at(int idx)
 {
return _myArray[idx];
//cout << _myArray[idx] << " value at index" << endl;
 }

 ostream& operator<<(ostream& os, MyVector& vec)
 {
ostringstream convert;
ostringstream convertCap;
convert << vec.GetSize();
string sizeConverted = convert.str();
convertCap << vec.GetCapacity();

string capConverted = convertCap.str();
string firstTemp = "";
firstTemp = SIZE_STR + sizeConverted  + RETURN_CARRIAGE_STR + CAPACITY_STR + capConverted + RETURN_CARRIAGE_STR;

for(int idx = 0; idx < vec.GetSize(); idx++)
{
    string secondTemp = "";
    ostringstream convertSize;
    convertSize << vec.at(idx);
    string vecAtIdx = convertSize.str();
    secondTemp = vecAtIdx + RETURN_CARRIAGE_STR;
    cout << secondTemp;
}

os <<  firstTemp;
return os;
 }

 MyVector& MyVector::operator=(MyVector& setterVect)
 {
delete [] _myArray;

//MyVector* mPtr = new MyVector();

_myArray = setterVect._myArray;
_capacity = setterVect._capacity;
_size = setterVect._size;

return *this;
 }
4

2 に答える 2

5 
_myArray = setterVect._myArray;

コピー代入演算子が壊れています。その演算子の後、両方のインスタンスの の値は同じです_myArray。そのため、一方が破棄されるとすぐに、もう一方には存在しない何かへのポインターが残ります。

そして、物語の教訓 - を使用しますstd::vector。

于 2012-12-11T02:41:53.323 に答える
1

エラーはヒープの破損を意味します。ヒープを破壊する方法はたくさんあります。Davidが上で説明したように、メモリのチャンクを解放してから書き込むのは1つの方法です。

ほとんどのヒープは、メモリのチャンクの前後に数バイトの簿記情報を格納します。コードが誤動作してヒープデータを変更すると、このタイプのエラーが発生します。

于 2012-12-11T03:11:34.893 に答える