ArrayList removeAll() 不删除对象

Question

我有成员类的简单ArrayLists：

ArrayList<Member> mGroupMembers = new ArrayList<>();
ArrayList<Member> mFriends = new ArrayList<>();

会员等级：

public class Member {
    private String userUID;
    private String userName;

    public String getUserUID() {
        return userUID;
    }

    public String getUserName() {
        return userName;
    }

    public void setUserName(String userName) {
        this.userName = userName;
    }

    public void setUserUID(String userUID) {
        this.userUID = userUID;
    }


}

朋友的 ArrayList 包含所有用户的朋友。我只是想做的就是从朋友列表中删除组成员，如果存在的话：

mFriends.removeAll(mGroupMembers);

但它对mFriends 列表没有任何作用...

查看日志语句，该朋友确实出现在mGroupMember 列表中。

为什么这不起作用？

Answer 1

如何确定两个成员相等？我猜它们是否具有相同的 ID，您认为它们相等，但是 java 希望它们在内存中是完全相同的引用，但情况可能并非如此。要更正此问题，您可以覆盖 equals 函数以使其在 id 相等时返回：

public class Member {
    //..

    @Override
    public boolean equals(Object anObject) {
        if (!(anObject instanceof Member)) {
            return false;
        }
        Member otherMember = (Member)anObject;
        return otherMember.getUserUID().equals(getUserUID());
    }
}

此外，当您覆盖.equals 时，建议您也覆盖hashCode，以便对象在Set 或Map 等散列函数中也能正常工作。

Answer 2

如 cmets 中所述，ArrayList 中的元素只有在其 equals() 方法返回 true 时才会被删除。非覆盖方法根据引用检查相等性（即它们必须是内存中的同一个对象）。

您可能想要基于Member 的属性覆盖equals，如下例所示：

@Override
public void equals(Object o) {
    if(o == null) {
        return false;
    } else if (!(o instanceof Member)) {
        return false;
    } else {
        return ((Member) o).getUserUID().equals(this.userUID) && ((Member) o).getUserName().equals(this.userName);
    }
}

另外，你应该在覆盖equals()的时候覆盖hashCode()，这样当两个对象相等时，它们就有相同的哈希码。 hashCode 的非覆盖实现也是基于引用相等。

Answer 3

你必须知道

ArrayList#removeAll(Collection)

打电话给

ArrayList#contains(Object)

调用

ArrayList#indexOf(Object)

最终调用

Object#equals

---

因此，如果 equals 未被正确覆盖 (following the equals contract rules)，您将无法获得正确的行为。