為什么不能從鏈接列表中刪除重復項？

Question

     /*
DEFINITIONS

 public class ListNode {
        int val;
        ListNode next;
        public ListNode(int x){
            val = x;
            next = null;
        }

    }


    public class LinkedList {
        ListNode head;
    }
    */


    public void removeDups(){
        ListNode head = this.head;
        if(head == null) return;

        HashSet<ListNode> set = new HashSet<>();
        set.add(head);
        while(head.next != null){
            if(set.contains(head.next)){
                head.next = head.next.next;
            } else{
                set.add(head.next);
                head = head.next;
            }
        }
    }

我應該從一個未排序的列表中刪除重復項，但是它在Java中不起作用。

1->2->4->9->9->4應該返回1->2->4->9但仍返回1->2->4->9->9->4

有什么問題 我清楚地將所有節點插入到哈希集中，並檢查它是否包含，但是不確定發生了什么？

Answer 1

您將在發布的代碼中刪除重復的節點，而不是重復的有價值的節點。

 /*
DEFINITIONS

public class ListNode {
    int val;
    ListNode next;
    public ListNode(int x){
        val = x;
        next = null;
    }

}


public class LinkedList {
    ListNode head;
}
*/


public void removeDups(){
    ListNode head = this.head;
    if(head == null) return;

    HashSet<Integer> set = new HashSet<Integer>();
    set.add(head.val);
    while(head.next != null){
        if(set.contains(head.next.val)){
            head.next = head.next.next;
        } else{
            set.add(head.next.val);
            head = head.next;
        }
    }
}

Answer 2

您正在嘗試檢查set包含ListNode ，但沒有在ListNode類中重寫equals方法。 嘗試覆蓋它，或添加要設置的值而不是整個節點，因為值是簡單的整數，所以它將起作用。

您還應該覆蓋hashCode方法，就像在覆蓋equals時應該做的一樣-謝謝Andy Turner

Answer 3

只是添加到v78的答案中 ，該答案正確描述了當前無法正常工作的原因：

另外，您可以在ListNode類中實現equals和hashCode ，如果節點具有相同的val則可以將它們視為相等：

int hashCode() { return val; }

boolean equals(Object other) {
  if (other == this) return true;
  return (other instanceof ListNode)
      && val == ((ListNode) other).val;
}