Question

我正在用Java实现一个单链表。我不喜欢这段代码的是我每次添加元素时都需要检查if (head.next == null)。但是在添加第一个元素时，条件只满足一次。

有没有办法在没有这种条件的情况下实现单链接的非循环列表？

package sample;

import java.util.Iterator;
import java.util.NoSuchElementException;

public class SinglyLinkedList<T> implements Iterable<T> {

    private Node<T> head = new Node<T>(null);
    private Node<T> last = null;

    public SinglyLinkedList(T... elements) {
        addAll(elements);
    }

    public void add(T element) {
        if (head.next == null) {
            head.next = new Node<T>(element);
            last = head.next;
        } else {
            Node<T> newNode = new Node<T>(element);
            last.next = newNode;
            last = last.next;
        }
    }

    public void addAll(T... elements) {
        for (T element : elements) {
            add(element);
        }
    }

    @Override
    public String toString() {
        Iterator<T> iterator = iterator();
        if (!iterator.hasNext()) {
            return "[]";
        }
        StringBuilder builder = new StringBuilder();
        builder.append("[");
        while (iterator.hasNext()) {
            T element = iterator.next();
            builder.append(element);
            if (!iterator.hasNext()) {
                return builder.append("]").toString();
            }
            builder.append(", ");
        }
        return builder.toString();
    }

    @Override
    public Iterator<T> iterator() {
        return new Iterator<T>() {

            Node<T> current = head;

            @Override
            public boolean hasNext() {
                return current.next != null;
            }

            @Override
            public T next() {
                if (!hasNext()) {
                    throw new NoSuchElementException();
                }
                Node<T> temp = current;
                current = current.next;
                return temp.next.element;
            }

        };
    }

    private static class Node<T> {

        private Node<T> next;
        private T element;

        Node(T element) {
            this.element = element;
        }

        @Override
        public String toString() {
            return element.toString();
        }
    }
}

Answer 1

在很多情况下，“优秀的OO设计”允许您在没有 if / else 检查的情况下离开;最常使用某种形式的多态性。

含义：不是询问某些属性的某个对象，而是在客户端代码中对此做出决定，以某种方式确保您的客户端代码可以简单地在其他对象上调用方法。然后，“if”在最初生成“其他对象”并将其提供给客户端代码的代码中“隐藏”。（你会在这些videos中找到一些很好的例子。）

但是 - 我认为在这种情况下这将是明显的矫枉过正！

重点是：从可读性的角度来看，一次检查确实没有伤害（你可以将事情重构为更多的方法）。而且表现......也无关紧要。如果你的代码被频繁调用以至于它很重要，那么无论如何JIT都会启动，并且可能创建在大多数情况下直接采用正确分支的代码。

因此：这是一个很好的实现;而且我认为你不应该担心这个，如果那里检查！

Answer 2

你可以将last初始化为指向head，然后你的if是多余的：

private Node<T> head = new Node<T>(null);
private Node<T> last = head;

public void add(T element) {
        Node<T> newNode = new Node<T>(element);
        last.next = newNode;
        last = last.next;
}

在Java中将元素添加到单个链接列表中

2 个答案: