1

我正在尝试在 Java 中实现一个链表样式堆栈(不使用内置的明显方式)。push 操作很简单,但是 pop 操作很麻烦。我很清楚,将“弹出”值返回给调用者并同时调整链表会很有用,但这似乎不起作用。

public class ListElement {

/* constructor */
public ListElement(Integer data) {
    value = data;
}

/* next pointer and the data */
private ListElement next;
private Integer value;

/* gets the next element */
public ListElement getNext() {
    return next;
}

/* gets the data */
public Integer getValue() {
    return value;
}

/* sets the next element */
public void setNext(ListElement elem) {
    next = elem;
}

/* sets the data */
public void setValue(Integer data) {
    value = data;
}

/* prints out the list */
public void printList(ListElement head) {
    String listString = "";
    ListElement elem = head;
    while (elem != null) {
        listString = listString + elem.getValue().toString() + " ";
        elem = elem.getNext();
    }
    System.out.println(listString);
}

/* inserting at the front */
public ListElement push(ListElement head, Integer data) {
    ListElement elem = new ListElement(data);
    elem.setNext(head);
    return elem;
}

/* pop the first element */
public Integer pop (ListElement head){
    Integer popped = head.getValue();
    head = head.getNext();

    return popped;
}

 public static void main(String[] args) {



    System.out.println("Constructing List with 2 ...");
    ListElement myList = new ListElement(2);
    myList.printList(myList);

    System.out.println("Adding 1 to beginning ...");
    myList = myList.push(myList, 1);
    myList.printList(myList);

    System.out.println("Adding 0 to beginning ...");
    myList = myList.push(myList, 0);
    myList.printList(myList);

    System.out.println("Pop ...");
    Integer popped =  myList.pop(myList);
    System.out.println("Value is " + popped);
    myList.printList(myList);
      }

 }
4

4 回答 4

2

通常在像你这样的结构中,你有两个类,一个是ListElement,另一个是List. 我认为名称StackElementStack会更好,但是,继续前进....

问题是 Java 在您的 pop 方法中,正如您所怀疑的那样:

/* pop the first element */
public Integer pop (ListElement head){
    Integer popped = head.getValue();
    head = head.getNext();
    return popped;
}

而且在主要方法中:

System.out.println("Pop ...");
Integer popped =  myList.pop(myList);

这里的问题是您期望 myList 使用方法head.getNext()中的修改后的值进行更新pop()

这不会发生......myList当您调用时,Java 会创建一个不同的“引用”,pop()并且该行head = head.getNext()不会改变myList我们的pop().

这种情况通常通过第二类来解决,称之为“列表”(或者正如我所说,应该是Stack)......

public class Stack() {
    private StackElement head = null;
    public push(Integer value) {
        StackElement topush = new StackElement(value);
        topush.setNext(head);
        head = topush;
    }
    public Integer pop() {
        StackElement topop = head;
        if (head != null) {
            head = topop.getNext();
            return topop.getValue();
        }
        return null;
    }
}

然后,在您的 main 方法中,您应该使用这个新的 Stack 类:

Stack stack = new Stack();
stack.push(2);
....
于 2013-11-11T05:03:05.267 回答
1

通常对于这样的链接列表,您需要一个包含对堆栈顶部的引用的 Header 或 LinkedList 类。

它将有一个字段ListElement top和函数poppush

push正如您所料,只需将 start 设置为新的 ListElement 并且该元素next等于 old start

pop只是返回start元素的值和设置,start = start.getNext()从而删除顶部。

于 2013-11-11T04:56:49.653 回答
1

当您说时,您没有修改 myList

head = head.getNext();

您只是在 pop 方法中修改局部变量。

我最喜欢的解决方案是将“列表”和“元素”完全分开,就像两个单独的文件一样。然后事情应该开始变得更有意义。

列表

  • 头部(元素类型)

元素

  • 下一个(元素类型)
  • 值(整数类型)
于 2013-11-11T04:57:53.297 回答
1

您可以在这里找到正确的实现。

您的代码有什么问题是:

/* pop the first element */
public Integer pop (ListElement head){
    Integer popped = head.getValue();
    head = head.getNext();
    return popped;
}

您修改head仅对您的函数本地的元素。

您需要将代码分成两个类:

  • ListElement:将仅包含值/下一个信息,

    public class ListElement {
         ListElement next;
         Integer value;
    }
    
  • LinkedStack:将仅包含至少堆栈函数(推送/弹出),以及对堆栈的第一个 ListElement 的引用。调用 pop 函数时需要更新此引用。

    public class LinkedStack {
    
         private ListElement first;
    
         public void push(Integer item) {
             ListElement oldfirst = first;
             first = new ListElement();
             first.value= item;
             first.next = oldfirst;
         }
    
    
         public Integer pop() {
             Integer item = first.value;      
             first = first.next;           
             return item;                 
         }
    }
    
于 2013-11-11T05:03:12.910 回答