pop()函数有什么问题?为什么它没有正常工作?
class stack{
int *p, *Cursor;
int size ;
public:
stack(int sz) {Cursor = p = new int[size=sz+1];} //consider the stack empty when its size is 1
~stack() {delete[] p;} //Cursor and P will be destroyed when the program finishes
void push(int x) {Cursor+=1; *Cursor=x; size++;}
int pop() {if(Cursor == p) return -1; int temp = *Cursor; Cursor--; size--; return (temp);}
bool isEmpty(){return(Cursor == p);}
bool isFull(){return(Cursor == p+size);}
};
这是我的测试:
stack A(3);
std::cout<<"Empty: "<<A.isEmpty()<<std::endl;
std::cout<<"Full: "<<A.isFull()<<std::endl;
A.push(10);
A.push(20);
A.push(30);
std::cout<<std::endl;
std::cout<<" 1st pop: "<<A.pop()<<std::endl<<" 2nd pop: " <<A.pop()<<std::endl<<" 3rd pop: " <<A.pop()<<std::endl<<" 4th pop: " <<A.pop()<<std::endl;
我得到的输出是:
1st pop: -1
2nd pop: 10
3rd pop: 20
4th pop: 30
虽然我应该得到......:
1st pop: 30
2nd pop: 20
3rd pop: 10
4th pop: -1
问题是我哪里出错?
答案 0 :(得分:5)
没有出错,但如果你把它们放在一条pop
行中,std::cout
得到(在你的情况下)从右到左进行评估。一般来说。评估订单未指定。有关详细信息,请参阅here。
因此,您按照与插入相反的顺序正确获取元素,然后-1
,然后将它们反转打印。
答案 1 :(得分:1)
在代码的最后一行,您进行了几次函数调用(隐藏,但每个&lt;&lt;运算符实际上是一个函数调用)。未指定对这些函数调用的参数的评估。所以未指定首先调用“pop()函数。