3

我正在为堆栈创建一个链表实现。我已经完成了 pop 和 push 方法,但我似乎无法正确使用 peek 方法。我现在在那里的代码返回我认为的内存地址。

这是我的代码:

public class LinkedStack<T> implements StackADT<T> {

private int count;
private LinearNode<T> contents;

public LinkedStack() {
    count = 0;
    contents = null;
}

@Override
public void push(T element) {
    LinearNode<T> top = new LinearNode<T>(element);

    if (contents == null) {
        contents = top;
    } else {
        LinearNode<T> current = contents;

        while (current.getNext() != null) {
            current = current.getNext();
        }
        current.setNext(top);
    }
    count++;
}

@SuppressWarnings("unchecked")
@Override
public T pop() {
    T item = (T) contents;
    contents = contents.getNext();
    count--;
    return item;
}

@SuppressWarnings("unchecked")
@Override
public T peek() throws NoSuchOperationException {

    T top = (T) contents;
    if(top == null){
        throw new NoSuchOperationException();
    }
    return top;
}

@Override
public boolean isEmpty() {
    // TODO Auto-generated method stub
    return false;
}

@Override
public int size() {
    return count;
}
}

这是我调用 peek 方法时它输出的内容。我使用我的 push 方法添加了一个对象,并使用 size 方法对其进行了测试。它表明我添加了一个元素。然后我调用了我的 pop 方法并再次显示大小以确保它有效。

这是我 peek 方法的输出:

线性节点@33f42b49

这是我的 LinearNode 类:

public class LinearNode<T> {
private T element;
private LinearNode<T> next;

public LinearNode() {
    this.element = null;
    this.next = null;
}
public LinearNode(T element) {
    this.element = element;
    this.next = null;
}
public T getElement() {
    return element;
}
public void setElement(T _element) {
    this.element = _element;
}
public LinearNode<T> getNext() {
    return next;
}
public void setNext(LinearNode<T> next) {
    this.next = next;
}

}

4

3 回答 3

3

听起来 LinearNode 类需要一个 toString() 方法。

于 2011-10-06T19:26:46.190 回答
1

pop() 和 peek() 应该返回 LinearNode 引用的内容,而不是实际的 LinkedNode。

@SupressWarnings 表明您没有正确使用泛型。删除它们,然后查看您返回的内容 - 看起来您返回的不是“T”,而是直接返回 LinkedNode。您不应该在此代码中需要任何 SupressWarnings。

如果你想存储字符串,那么你应该像这样定义堆栈:

LinkedStack<String> stack = new LinkedStack<String>();

您可以认为泛型类型T将在您的堆栈类中表示一个字符串。

在这种情况下,peek() 方法签名将返回一个字符串。

另外:检查您的 pop 和 push 方法,两者都应该能够在 O(1) 中完成,但是,您正在遍历列表。你把它弄反了。

于 2011-10-06T20:25:46.363 回答
0

您可以使用以下命令进行简单查看: System.out.println("顶部的元素是:" + top.getElement); 并返回 top.getElement。请记住,您实际上是从堆栈顶部及其元素中删除一个节点。由于 peek 实际上只看到堆栈的顶部元素,因此您不需要使用 top = top.getNext 或 count--;。希望能帮助到你!

于 2013-04-22T04:55:57.817 回答