Question

我试图在Java中实现链表样式堆栈（不使用内置的明显方法）。推送操作很简单，但是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);
      }

 }

Answer 1

通常在像你这样的结构中，你有两个类，一个是ListElement，另一个是List。我认为名字StackElement和Stack会更好，但是，继续......

问题是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);

这里的问题是您希望使用head.getNext()方法中的修订值pop()更新myList。

这不会发生......当您致电myList并且pop()行未更改head = head.getNext()时，Java会为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);
....

Answer 2

通常使用像这样的链接列表，你需要一个Header或LinkedList类，它包含对栈顶的引用。

它会有一个字段ListElement top以及函数pop和push。

正如您所料，

push只是设置一个新的ListElement的开始，而该元素的next等于旧的start。

pop只返回start元素的值并设置start = start.getNext()，从而移除顶部。

Answer 3

当你说

时，你没有修改myList

head = head.getNext();

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

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

列表

head（元素类型）

元素

next（元素类型）
value（类型为Integer）

Answer 4

您可以找到正确的实施here。

您的代码有什么问题：

/* 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 ：将至少包含堆栈函数（push / pop），以及对堆栈的第一个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;                 
     }
}

java中的链表列表

4 个答案: