1

我填写了一些ObservableCollection<Employe>收藏：

// Program.Data.Employees - it is ObservableCollection<Employe>.
Program.Data.Employees.Add(new Employe() { Name="Roman", Patronymic="Petrovich", Surname="Ivanov" });
Program.Data.Employees.Add(new Employe() { Name = "Oleg", Patronymic = "Vladimirovich", Surname = "Trofimov" });
Program.Data.Employees.Add(new Employe() { Name = "Anton", Patronymic = "Igorevich", Surname = "Kuznetcov" });

在我的代码的其他地方，我尝试从这个集合中删除一些项目：

// Program.Data.Employees - it is ObservableCollection<Employe>.
Employe x = Program.Data.Employees.First(n => n.Guid == emp.Guid); // x is not null.
Int32 index = Program.Data.Employees.IndexOf(x); // I got -1. Why?
Boolean result = Program.Data.Employees.Remove(x); // I got 'false', and item is not removed. Why?
// But this works fine:
Program.Data.Employees.Clear();

我可以清除收藏，但我不能删除必要的项目。为什么会发生？

UPD：Equals我Employe班级的方法

public bool Equals(Employe other) {
    return
    other.Guid == this.Guid &&
    String.Equals(other.Name, this.Name, StringComparison.CurrentCultureIgnoreCase) &&
    String.Equals(other.Patronymic == this.Patronymic, StringComparison.CurrentCultureIgnoreCase) &&
    String.Equals(other.Surname == this.Surname, StringComparison.CurrentCultureIgnoreCase) &&
    other.Sex == this.Sex &&
    String.Equals(other.Post == this.Post, StringComparison.CurrentCultureIgnoreCase);
}
4

1 回答 1

3

我尝试了以下代码来重现您的错误：

class Employee
{
  public string Name { get; set; }
  public Guid Guid { get; set; }
}

// ...
ObservableCollection<Employee> employees = new ObservableCollection<Employee>();
var guid1 = Guid.NewGuid();
employees.Add(new Employee { Name = "Roman", Guid = guid1 });
employees.Add(new Employee { Name = "Oleg", Guid = Guid.NewGuid() });

var x = employees.First(e => e.Guid == guid1);
var index = employees.IndexOf(x); // index = 0, as expected
var result = employees.Remove(x); // result = true, as expected

它按预期工作。我建议，你设置一个断点var x = ...并检查，如果

  • 该集合确实包含您正在查找的项目
  • 如果First()真的返回该项目

然后转到下一行并检查是否index正确返回。最后再次检查，如果result真的是假的。

我看到您的代码失败的几个可能原因：

  • 您没有发布完整的代码，并且在x=Program.Data.Employees.First()和之间发生了一些事情Program.Data.Employees.IndexOf()
  • 您使用多线程代码（这也会导致两个语句之间发生“某些事情”）。这种情况下，需要同步对集合的访问
  • 您不ObservableCollection直接使用由数据层构造的派生类，而是使用一些派生类（例如DataServiceCollection，但这个也应该可以正常工作）。在这种情况下，请在调试器中检查您的集合的实际类型

另一个导致集合错误的典型原因是，如果您尝试在迭代集合时删除项目（即在foreach循环内）：但在这种情况下，应该抛出异常（并且IndexOf应该可以正常工作），所以这只适用于您使用一些实现非标准行为的派生类。

编辑 （作为回报您发布您的Equal方法）

您的Equal方法存在严重错误：

String.Equals(other.Patronymic == this.Patronymic, StringComparison.CurrentCultureIgnoreCase)
... // applies also for following comparisons

应该

String.Equals(other.Patronymic, this.Patronymic, StringComparison.CurrentCultureIgnoreCase)
...

此外，如果您使用的是 Guid，请考虑仅比较 GUID，因为这通常意味着“唯一标识符”，因此它应该足以识别某些实体。

于 2012-11-17T11:48:46.350 回答