使用递归和不使用额外空间对堆栈进行排序

Question

我正在尝试编写代码以在不使用额外空间的情况下对堆栈进行排序。这在逻辑上是正确的，但我的输出是完全随机的。有人能指出我的错误吗？

void insertatsortstack(int element, stack<int> s){

    if(s.empty()==1 || element > s.top())
    {
        s.push(element);
        return;
    }

    int temp=s.top();
    s.pop();
    insertatsortstack(element,s);
    s.push(temp);
}

void sortstack(stack<int> s){
    if(s.size()>0){
        int element=s.top();
        s.pop();
        sortstack(s);
        insertatsortstack(element,s);
    }
}

Answer 1

我认为您来自 Java 背景。您在函数中传递值，这将复制您的堆栈并且您的原始堆栈不会被更改。您需要通过引用传递值。

void insertatsortstack(int element, stack<int> & s);
void sortstack(stack<int> &s);

Working Code Implementation

（假设你使用的是std::stack）

Answer 2

#include 使用命名空间标准；

无效 insertatsortstack (int element, stack &s) {

if (s.empty () == 1 || 元素 > s.top ())

{
  

s.push(元素);

返回；

}

int temp = s.top ();

s.pop();

插入排序堆栈（元素，s）；

s.push (temp);

}

无效

sortstack（栈&s） {

if (s.size() > 0) {

int element = s.top();

s.pop();

排序栈；

插入排序堆栈（元素，s）；

}

}

int

主() {

栈s;

s.push(5);

s.push(1);

s.push(4);

s.push(3);

s.push(6);

s.push(9);

s.push(2);

s.push(3);

s.push(8);

排序栈；

while (s.empty () == false) {

cout

s.pop();

}

}