Стек с использованием кучи

[Voronar]

Решаю задачки по C++ для общего развития. Столкнулся с проблемкой при проектировании класса стек.
Есть класс:

#ifndef DSTACK_H
#define DSTACK_ H
typedef unsigned long Item;

class DStack
{
private:
	Item * pitems;
	int size;
	int top;
public:
	DStack(int n=10);
	DStack(const DStack & st); //coping constructor
	~DStack();

	bool isempty() const; //methods
	bool isfull() const;
	bool push(const Item & item);
	bool pop(Item & item);
	DStack & operator=(const DStack & st);
};
#endif

А вот его реализация:

#include "DStack.h"

DStack::DStack(int n)
 {
	 size = n;
	 top = 0;
	 pitems = new Item[size];
 }
DStack::DStack(const DStack & st)
{
	this->top = st.top;
	this->size = st.size;
	pitems = new Item[st.size];
	for(int i = 0 ; i < st.size; i++)
	{
		this->pitems[i] = st.pitems[i];
	}
}
DStack &  DStack::operator=(const DStack & st)
{
	if(this == &st)
	{
		return *this;
	}
	delete[] this->pitems;

	size = st.size;
	pitems = new Item[st.size];
	for(int i = 0 ; i < st.size; i++)
	{
		this->pitems[i] = st.pitems[i];
	}

	return *this;
}
DStack::~DStack()
{
	delete[] pitems;
}
bool DStack::isempty() const
{
	return top==0;
}
bool DStack::isfull() const
{
	return top==size;
}
bool DStack::push(const Item & item)
{
	if(!isfull())
	{
		pitems[top] = item;
		top++;
	}
	else
	{
		return false;
	}

	return true;
}
bool DStack::pop(Item & item)
{
	if(!isempty())
	{
		item = pitems[top];
		top--;
	}
	else
	{
		return false;
	}

	return true;
}

Пример использования:

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

int main()
{
	DStack s1(5);
	DStack s2;
	Item i[5] = {1,2,3,4,5};
	s1.push(i[0]);
	s1.push(i[1]);
	s1.push(i[3]);

	s1.pop(i[5]);
	cout << i[5];
	cin.get();
	return 0;
}

Проблема в том, что я не могу создать массив элементов item. Хотя по идее делаю все правильно... Создаю указатель на нужный тип, выделяю память в куче.
Почему-то, при дебаге обнаруживаю, что мой динамический массив может вместить только один элемент.

[Abstraction]

Почему-то, при дебаге обнаруживаю, что мой динамический массив может вместить только один элемент.

Как именно обнаруживаете? И какого чёрта в программе Вы обращаетесь к i[5]?

[EUGY]

Это как всегда погрешность на единицу:

bool DStack::pop(Item & item)
{
	if(!isempty())
	{

                top--; // воткнуть сюда
		item = pitems[top];
		//top--; // отсюда убрать
	}
	else
	{
		return false;
	}

	return true;
}

ЗЫ.
Или нет?

[Voronar]

Abstraction
Ставлю точки прерывания и оператор за оператором отслеживаю состояние необходимых мне переменных.
А с индексом 5 я оплошал. Я хотел взять 5 элемент со значением 5. А так как индекс элемента равен значению в этом случае... Мой мозг захотел 5 и я случайно вместо i[4] написал ошибочный вариант.

[Voronar]

EUGY
Спасибо. Да, погрешность на 1.
Жалко, что в режиме отладки при расстановке точек прерывания я только вижу первый элемент динамического массива pitems, при наведении на него мышью. Скорее всего это из-за того, что это динамический объект данных, а items - указатель на его первый элемент. Или в VS можно просматривать содержимое динамических объектов данных?

[Abstraction]

Или в VS можно просматривать содержимое динамических объектов данных?

Можно, если немного схитрить. В окне Watch наберите pitems[2] - увидите содержимое третьего элемента. Ведь тип переменной pitems - "указатель на Item", про массив (а тем более его длину) нет ни слова - вот студия и не проявляет инициативы.

[Voronar]

Abstraction
Проверил - работает. Спасибо.