private static class Node<E> {
    E item;
    Node<E> next;
    Node<E> prev;

    Node(Node<E> prev, E element, Node<E> next) {
        this.item = element;
        this.next = next;
        this.prev = prev;
    }
}

final Node<E> newNode = new Node<>(null, e, f); // 构造函数的第3个参数不就是 next 元素了。

为什么会断链呢？
f指向原来的first，将新的newNode设置为first。
这个时候就判断，如果原来的链表就是空的，那么last也就是first咯。
如果不为空，那么就是将f<原来的首节点>的prev就指向first。

我猜你之所以觉得会断链是因为，你没看到first的next指向f。额next这个在Node中已经传进去了
final Node<E> newNode = new Node<>(null, e, f);
private static class Node<E> {

    E item;
    Node<E> next;
    Node<E> prev;

    Node(Node<E> prev, E element, Node<E> next) {
        this.item = element;
        this.next = next;
        this.prev = prev;
    }
}