Реализация стека в виде массива

[DOJ]

Огромное спасибо за объяснение, и пояснение моих ошибок.

[DOJ]

Внесла исправление, но к сожалению вылетает. Не могли бы подсказать мне, дорогие форумчане, в чем моя ошибка, чего я не учла?

#include<Stack.h>
#include<iostream.h>
#include<cstdlib.h>
using namespace std;

const int max_size=15;
class Stack
{
public:
int var;//значение элементов стека
int top;//вершина стека, счетчик
int max_stack[max_size];//массив
void push();//добавление элементов
int pop();//удаление
};

void Stack::push()
{
  if (top==0)
  {
   while(!top==max_size)
   max_stack[top++]=var;
   }
   }
   int Stack::pop()
   {
   if (!top==0){
   return max_stack[top--];
   }
   }
   void main()
   {
   Stack s; //создание объекта класса
   cout<<"Введите элементы стека"<<endl;
   cin>>s.max_stack[max_size];
   s.push();
   s.pop();
   }

[psycho-coder]

Мне вот этот кусок не понятен:

!top==max_size

Так вообще можно?
Если тут проверка пока топ не равно макс_сайз, я бы написал так:

top!=max_size

[DOJ]

Спасибо конечно за ваше замечание. Но меняется ли от этого смысл!? Может конечно я не права.

[psycho-coder]

Сообщение от DOJ

Спасибо конечно за ваше замечание. Но меняется ли от этого смысл!? Может конечно я не права.

Я точно не знаю, поэтому и спросил.
Вот, что у меня получилось:

//#include<Stack.h> // я хз зачем это
#include<iostream>
//#include<cstdlib.h> // я хз зачем это
using namespace std;

const int max_size = 15;
class Stack
{
public:
	int var;//значение элементов стека
	int top;//вершина стека, счетчик
	int max_stack[max_size];//массив
	Stack(); // конструктор класса
	void push();//добавление элементов
	int pop();//удаление
};
Stack::Stack()
{
	top = 0; // инициализируем переменные
	var = 0; // для корректной работы
}

void Stack::push()
{
	if (top <= max_size) 
		max_stack[top++] = var; // заносим в стек
}
int Stack::pop()
{
	if (top > 0)
		return max_stack[--top]; // вытаскиваем из стека
}
void main()
{
	Stack s; //создание объекта класса
	cout << "Введите элементы стека" << endl;
	cin >> s.var;
	s.push();
	cout << s.pop() << endl; // выводим последний элемент.
}

P.S. у меня VS 2005

[Гром]

Судя по всему, смысл и впрямь меняется. Приоритет оператора ! выше, нежели ==. Так что

!top==max_size

имеет смысл

(!top)==max_size

Все-таки это немного разные вещи. Можно, конечно, явно расставить скобки, но зачем использовать два оператора, когда можно все необходимое выразить одним? Во-первых, будет четкое и непосредственное выражение мысли, во-вторых, это эффективнее (вот на такую вот фитюлечку, но тем не менее!)

[DOJ]

Спасибо за некоторые уточнения моего кода и за комментарий конечно. Пользуюсь Builder 2003. Но откомпилила и все равно вылетает.
Попробовала добавить:

system("PAUSE");
return;
ну пока тоже не особо. выводит первый элемент и выходит из программы. Только не судите строго я только учусь.

[psycho-coder]

Сообщение от Гром

Судя по всему, смысл и впрямь меняется. Приоритет оператора ! выше, нежели ==. Так что

!top==max_size

имеет смысл

(!top)==max_size

Вот... Объясните мне пожалуйста, что происходит в этих двух случаях. Спс.

Спасибо за некоторые уточнения моего кода и за комментарий конечно. Пользуюсь Builder 2003. Но откомпилила и все равно вылетает.
Попробовала добавить:

system("PAUSE");
return;
ну пока тоже не особо. выводит первый элемент и выходит из программы.

Какую ошибку выдает и где?

[netrino]

Сообщение от psycho-coder

Вот... Объясните мне пожалуйста, что происходит в этих двух случаях. Спс.

!top == max_size
top будет приведён к логическому типу и над ним будет произведена операция НЕ. Если top 0, то он станет 1, если top не 0, то станет 0 и уже это значение будет сравниваться с max_size

[netrino]

Сообщение от DOJ

Спасибо за некоторые уточнения моего кода и за комментарий конечно. Пользуюсь Builder 2003. Но откомпилила и все равно вылетает.
Попробовала добавить:

system("PAUSE");
return;
ну пока тоже не особо. выводит первый элемент и выходит из программы. Только не судите строго я только учусь.

Так ведь он и должен только один элемент выводить, в коде именно это и записано )
Так можно будет ввести несколько элементов и все они потом будут выведены:

void main()
{
	Stack s; //создание объекта класса
	cout << "Введите элементы стека" << endl;
	cin >> s.var;
	s.push();
	cin >> s.var;
	s.push();
	cin >> s.var;
	s.push();
	while(s.top)
		cout << s.pop() << endl; // выводим последний элемент.
}