Dynamic array of pointers declaration problem?

[johnnydiamond08]

Hello, I am attempting to declare an array of pointers dynamically based on user input. I am not sure if A) I'm implementing the syntax of declaring a dynamic array correcntly and B) if my code is set up correctly to print otherwise. Thank you all for taking the time to read this.

 int array_size;

        cout << "Please enter the size of the array: " << endl;
        cin >> array_size;



        if(array_size >= 8)
        {
            cout << "Invalid array size, please enter a valid integer size";
            cin  >> array_size;
        }

        Supply DynamicArray;
        DynamicArray = new int[array_size];




        //
        //initialize all elements in the array to the null pointer
        //

        for (int i = 0; i <= array_size; i++)
            {
               DynamicArray[i];
            }




    string des2;
        //
        //store pointer to    Pen object  p1 in [0]
        cout << endl;
        cin.ignore();
        cout << "Enter  Name of pen: "; getline(cin, description);              cout << description;
        cin.clear();
        cout << "Enter stock number: "; cin >> stockNumber;
        cout << "Enter quantity: "; cin >> quantity;
        cout << "Enter price: "; cin >> price;
        cout << "Enter pen color (1,2,3): "; cin >> nPenColor;
        cout << "Enter ink color (1,2,3): "; cin >> nInkColor;
        DynamicArray[3] = new Pen(description, stockNumber, quantity, price, nPenColor, nInkColor  );

        //store pointer to MemPen obect mp1 in [2]
        //
        cout << description;
        cout << endl;
        description = " ";
        cout << "Test" << endl;
        cout << des2;
        cin.ignore();
        cout << "Enter  Name of pen: "; getline(cin,description);               cout << description;


        cin.ignore(1000,'\n');

//        description.erase(std::remove(description.begin(),description.end(), '\n'), description.end());
        cout << "Enter stock number: "; cin >> stockNumber;
        cout << "Enter quantity: "; cin >> quantity;
        cout << "Enter price: "; cin >> price;
        cout << "Enter pen color (1,2,3): "; cin >> nPenColor;
        cout << "Enter ink color (1,2,3): "; cin >> nInkColor;
        cout << "Enter memory size in Mb: "; cin >> memory;
        //
        //loop to print out ALL elements in the array that point to an object,
        //that is, there is a valid address in the element of the array.
        DynamicArray[7] = new MemPen(  description,  stockNumber, quantity, price,  nPenColor, nInkColor, memory);


      cout << "Supply List" <<endl;
      cout << "===========" <<endl;
        for (int x = 0; x < 10; x++)
        {
            if (DynamicArray[x] != NULL)
             (*DynamicArray[x]).print();

        }

[ajh32]

THis is what you need to do. Although it would hep if you did some checking of what the user was enterring before attempting to allocate it to the variables etc.

#include <iostream>
#include <string>

using namespace std;


class Supply
{
public:
	Supply()
	{
	}

	virtual ~Supply()
	{
	}

	virtual void Print() = 0;
};

class Pen : public Supply
{
public:
	Pen() 
		: Supply()
	{
	}

	Pen(const string description, const int stockNumber, const int quantity, const int price, const int penColor, const int inkColor)
		: m_Description(description)
		, m_StockNumber(stockNumber)
		, m_Quantity(quantity)
		, m_Price(price)
		, m_PenColor(penColor)
		, m_InkColor(inkColor)
	{
	}

	virtual ~Pen()
	{
	}

	void Print()
	{
		cout << endl << m_Description << endl;
	}

protected:
	string m_Description;
	int m_StockNumber;
	int m_Quantity;
	int m_Price;
	int m_PenColor;
	int m_InkColor;
};

class MemPen : public Pen
{
public:
	MemPen()
		: Pen()
	{
	}

	MemPen(const string description, const int stockNumber, const int quantity, const int price, const int penColor, const int inkColor, const int memory)
		: Pen(description, stockNumber, quantity, price, penColor, inkColor)
		, m_Memory(memory)
	{
	}

	virtual ~MemPen()
	{
	}

protected:
	int m_Memory;
};

int main()
{
	int array_size;

	cout << "Please enter the size of the array: " << endl;
	cin >> array_size;

	if(array_size >= 8)
	{
		cout << "Invalid array size, please enter a valid integer size";
		cin  >> array_size;
	}

	Supply** DynamicArray = new Supply*[array_size];

	//
	//initialize all elements in the array to the null pointer
	//

	for (int i = 0; i <= array_size; i++)
	{
		DynamicArray[i] = NULL;
	}

	string description, des2;
	int stockNumber, price, quantity, nPenColor, nInkColor, memory;

	//
	//store pointer to    Pen object  p1 in [0]
	cout << endl;
	cin.ignore();
	cout << "Enter  Name of pen: "; 
	getline(cin, description);              
	cout << description;
	cin.clear();
	cout << "Enter stock number: "; 
	cin >> stockNumber;
	cout << "Enter quantity: "; 
	cin >> quantity;
	cout << "Enter price: ";
	cin >> price;
	cout << "Enter pen color (1,2,3): "; 
	cin >> nPenColor;
	cout << "Enter ink color (1,2,3): "; 
	cin >> nInkColor;
	DynamicArray[3] = new Pen(description, stockNumber, quantity, price, nPenColor, nInkColor);

	//store pointer to MemPen obect mp1 in [2]
	//
	cout << description;
	cout << endl;
	description = " ";
	cout << "Test" << endl;
	cout << des2;
	cin.ignore();
	cout << "Enter  Name of pen: "; 
	getline(cin,description);               
	cout << description;

	cin.ignore(1000,'\n');

	//        description.erase(std::remove(description.begin(),description.end(), '\n'), description.end());
	cout << "Enter stock number: "; 
	cin >> stockNumber;
	cout << "Enter quantity: "; 
	cin >> quantity;
	cout << "Enter price: "; cin >> price;
	cout << "Enter pen color (1,2,3): "; 
	cin >> nPenColor;
	cout << "Enter ink color (1,2,3): "; 
	cin >> nInkColor;
	cout << "Enter memory size in Mb: "; 
	cin >> memory;
	//
	//loop to print out ALL elements in the array that point to an object,
	//that is, there is a valid address in the element of the array.
	DynamicArray[array_size-1] = new MemPen(description,  stockNumber, quantity, price,  nPenColor, nInkColor, memory);

	cout << "Supply List" <<endl;
	cout << "===========" <<endl;

	for (int x = 0; x < array_size; x++)
	{
		if (DynamicArray[x])
		{
			(DynamicArray[x])->Print();
		}
	}


	// don't forget to release memory:
	for (int x = 0; x < array_size; x++)
	{
		if (DynamicArray[x])
		{
			delete DynamicArray[x];
			DynamicArray[x] = NULL;
		}
	}

	delete *DynamicArray;
	*DynamicArray = NULL;
}