从java列表中删除相等的项目

Question

我有一个项目列表，其中每个项目都是一个包含 2 个公共字符串的简单类。 我有一个 equals 方法，它对两个字符串都使用 String 的 equalsIgnoreCase 方法。

public class data
{
    public String a;
    public String b;

    public boolean equals(data d)
    {
        if(a.equalsIgnoreCase(d.a) && b.equalsIgnoreCase(d.b))
        {
            return true;
        }
        else
        {
            return false;
        }
    }
}

我希望能够删除一个元素，即使它与列表中的元素不同但等于它。

现在我正在这样做：

public void remove(data dataToRemove)
{
    for(data i : _list)
    {
        if(i.equals(dataToRemove))
        {
            _list.remove(i);
            break;
        }
    }
}

有没有更好的方法来做到这一点？

Answer 1

几个cmets：

• 您的equals 方法不会覆盖Object 的equals 方法（参数应该是Object 类型，而不是data 类型）。
• 您应该改进您的 equals 方法以解决空值等问题。
• 最后，当你覆盖equals() 时，你也应该覆盖hashcode() - 否则你可能会在使用Sets 或Maps 时遇到一些奇怪的行为。

如果您正确地覆盖了equals 方法，那么您可以只使用remove 方法。 下面看Netbeans自动生成的equals和hashcode，修改为使用equalsIgnoreCase方法。

public static void main(String[] args) {
    List<Data> list = new ArrayList<Data>();
    list.add(new Data("a", "b"));
    list.add(new Data("a", "c"));
    System.out.println(list.size()); //2
    list.remove(new Data("A", "b"));
    System.out.println(list.size()); //1
}

public static class Data {

    public String a;
    public String b;

    public Data(String a, String b) {
        this.a = a;
        this.b = b;
    }

    @Override
    public boolean equals(Object obj) {
        if (obj == null) return false;
        if (getClass() != obj.getClass()) return false;
        final Data other = (Data) obj;
        boolean sameA = (this.a == other.a) || (this.a != null && this.a.equalsIgnoreCase(other.a));
        if (!sameA) return false;
        boolean sameB = (this.b == other.b) || (this.b != null && this.b.equalsIgnoreCase(other.b));
        if (!sameB) return false;
        return true;
    }

    @Override
    public int hashCode() {
        int hash = 3;
        hash = 89 * hash + (this.a == null ? 0 :this.a.toUpperCase().hashCode());
        hash = 89 * hash + (this.b == null ? 0 : this.b.toUpperCase().hashCode());
        return hash;
    }

}

Answer 2

最清晰的方法是只调用列表的Remove 方法而不使用任何循环并将参数传递给您的对象。它使用您在对象上定义的 equals 方法来查找并删除它（如果它存在于列表中）。

_list.remove(data);

您也不必指定 hashcode 方法，但您应该养成在覆盖 equals 方法时始终创建一个的习惯，以防在 Set 或 Map 等集合中使用它。