单链表Shift方法疑问：是否需将移除节点的next设为null？

你的疑问非常合理，这事儿没有绝对的对错，得看你链表的使用场景：

两种实现的区别分析 #

原实现（不清next引用） #

原代码逻辑如下：

shift(){
    if(!this.head) return undefined;
    var currentHead = this.head;
    this.head = currentHead.next;
    this.length--;
    if(this.length === 0){
        this.tail = null;
    }
    return currentHead;
}

这种写法下，返回的节点还保留着指向原链表下一个节点的引用。如果调用方拿这个节点只是读取val值，完全没问题，还省了一步赋值操作。但如果调用方修改了这个返回节点的next属性，可能会意外篡改原链表的后续节点——毕竟这个节点还连着原链表的内部结构。

添加currentHead.next = null的实现 #

修改后的代码：

shift(){
    if(!this.head) return undefined;
    var currentHead = this.head;
    this.head = currentHead.next;
    currentHead.next = null; // 新增这一步
    this.length--;
    if(this.length === 0){
        this.tail = null;
    }
    return currentHead;
}

这相当于把返回的节点彻底从原链表“摘干净”，变成一个孤立的节点。这么做的核心优势是封装性更强，外部拿到节点后无论怎么操作，都不会影响原链表的内部状态。尤其适合把链表作为公共API对外提供的场景——你没法保证所有调用方都会规规矩矩使用返回的节点。

总结 #

• 如果你的链表是项目内部自用，调用方都是可控的自己人，能保证不会随意修改返回节点的next，那原实现完全够用；
• 但如果是做工具库、公共组件，或者不确定调用方会如何使用返回节点，那加上currentHead.next = null是更严谨的做法，能避免很多潜在的隐性BUG。

内容的提问来源于stack exchange，提问作者fBpv