CrtIsValidHeapPointer(pUserData) Error

[mlholder]

Hi all,

I am currently trying to teach myself about classes, constructors and destructors. I am writing a program that uses a dynamic array as a vector and trying to implement some of the predefined vector functions. However, when the program ends in main, the error I get is CrtIsValidHeapPointer(pUserData)and it always occurs with my destructor. I have not seen this error before, nor do I know where or how to start looking to correct it. Any and all help is greatly appreciated.

#include <cstdlib>
#include <iostream>

using namespace std;

class VectorDouble
{
	public:
		//Constructors
		VectorDouble();
		VectorDouble(int);

		//Copy Constructor
		VectorDouble(const VectorDouble &sourceObject);

		//Destructor
		~VectorDouble();

		//Overloaded Operators
		void operator = (const VectorDouble &sourceObject);
		friend bool operator == (const VectorDouble &lhs, const VectorDouble &rhs);

		//Member Functions
		void push_back(double);
		int capacity() const;
		int size() const;
		void reserve(int);
		void resize(int);
		void output();

	private:
		double *list;
		int maxCount; //Capacity of vector
		int count; //Size of vector
};

int main()
{
	char ch;
	VectorDouble v1;
	
	v1.push_back(1);
	v1.push_back(2);
	v1.push_back(3);
	v1.push_back(4);
	v1.output();

	cin.get(ch);
	return 0;
}

//Constructors
//===============================================================
VectorDouble::VectorDouble()
{
	list = new double[3];
	maxCount = 3;
	count = 0;
}

VectorDouble::VectorDouble(int initialCapacity)
{
	if (initialCapacity < 0)
	{
		char ch;

		cout << "Illegal value for initial vector capacity. Aborting program.\n";
		cin.get(ch);
		exit(1);
	}

	list = new double[initialCapacity];
	maxCount = initialCapacity;
	count = 0;
}

//Copy Constructor
//===============================================================
VectorDouble::VectorDouble(const VectorDouble &sourceObject)
{
	list = new double[sourceObject.capacity()];
	maxCount = sourceObject.maxCount;
	count = sourceObject.count;
}

//Destructor
//===============================================================
VectorDouble::~VectorDouble()
{
	delete[] list;
	list = NULL;
}

//Overloaded Operators
//===============================================================
void VectorDouble::operator = (const VectorDouble &sourceObject)
{
	list = sourceObject.list;
	maxCount = sourceObject.maxCount;
	count = sourceObject.count;
}

bool operator == (const VectorDouble &lhs, const VectorDouble &rhs)
{
	if (lhs.count == rhs.count)
	{
		for (unsigned int i = 0; i < lhs.count; i++)
		{
			if (lhs.list[i] != rhs.list[i])
				return false;
		}

		return true;
	}

	return false;
}

//Member Functions
//===============================================================
void VectorDouble::push_back(double num)
{
	if (count == maxCount)
	{
		VectorDouble temp(capacity() * 2);

		for (unsigned int i = 0; i < size(); i++)
		{
			temp.list[i] = list[i];
			temp.count++;
		}

		temp.list[count] = num;
		temp.count++;

		maxCount = capacity() * 2;
		count = temp.count;

		for (int i = 0; i < size(); i++)
			list[i] = temp.list[i];
		
	}
	else
	{
		list[count] = num;
		count++;
	}
}

int VectorDouble::capacity() const
{
	return maxCount;
}

int VectorDouble::size() const
{
	return count;
}

void VectorDouble::reserve(int newCapacity)
{
	//TO DO
}

void VectorDouble::resize(int newSize)
{
	//TO DO
}

void VectorDouble::output()
{
	cout << "Vector capacity: " << capacity() << endl;
	cout << "Vector size: " << size() << endl;
	cout << "Vector elements:\n";

	for (unsigned int i = 0; i < count; i++)
		cout << list[i] << endl;
	cout << endl;
}

[mobotus]

The code works for me but, you don't need to assign the pointer to NULL in the destructor.

VectorDouble::~VectorDouble()
{
	delete[] list;

}

[mlholder]

Thank you for the speedy response. It also worked the first few times as well, so I started "pushing back" numbers to test it more and started getting the error. Im not sure if it makes any difference but Im using Microsoft Visual Studio Express 2013 for Windows Desktop. I made another test of it and once it told me about a breakpoint, I clicked on the "step over" button and a new window pops up saying that no symbols loaded? Here is some of what I am getting:

C:\Users\Michael\Documents\Programming\C++\Problem Solving with C++ Eighth Edition\Programming Projects\Chapter 11 Friends- Classes\Programming Project 3\Debug\wntdll.pdb: Cannot find or open the PDB file.
C:\Windows\SysWOW64\wntdll.pdb: Cannot find or open the PDB file.
C:\Windows\wntdll.pdb: Cannot find or open the PDB file.
C:\Windows\symbols\dll\wntdll.pdb: Cannot find or open the PDB file.
C:\Windows\dll\wntdll.pdb: Cannot find or open the PDB file.

wntdll.pdb not loaded.

Again, thank you for your help.

[mobotus]

http://stackoverflow.com/questions/15937707/error-message-cannot-find-or-open-the-pdb-file