Pourquoi un élément d'une liste générique ne sera-t-il pas supprimé à l'aide de la méthode Supprimé?

Question

J'ai cette classe.

public class Foo 
{ 
    public Guid Id { get; set; } 

    public override bool Equals(object obj) 
    { 
     Foo otherObj = obj as Foo; 

     return otherObj == null && otherObj.Id == this.Id; 
    } 

    public override int GetHashCode() 
    { 
     return this.Id.GetHashCode(); 
    } 
} 

Vous pouvez voir que j'ai outrepassé les valeurs Equals et GetHashCode pour cet objet.

Maintenant, je lance l'extrait de code suivant

// Create Foo List 
List<Foo> fooList = new List<Foo>(); 

fooList.Add(new Foo { Id = Guid.NewGuid()}); 
fooList.Add(new Foo { Id = Guid.NewGuid()}); 
fooList.Add(new Foo { Id = Guid.NewGuid()}); 
fooList.Add(new Foo { Id = Guid.NewGuid()}); 
fooList.Add(new Foo { Id = Guid.NewGuid()}); 

// Keep Id of last Create 
Guid id = Guid.NewGuid(); 

fooList.Add(new Foo { Id = id }); 

Console.WriteLine("List Count {0}", fooList.Count); 

// Find Foo in List 
Foo findFoo = fooList 
    .Where<Foo>(item => item.Id == id) 
    .FirstOrDefault<Foo>(); 

if (findFoo != null) 
{ 
    Console.WriteLine("Found Foo"); 

    // Found Foo now I want to delete it from list 
    fooList.Remove(findFoo); 
} 

Console.WriteLine("List Count {0}", fooList.Count); 

Lorsque cela est exécuté foo se trouve, mais la liste ne supprime pas l'élément trouvé.

Pourquoi est-ce? Je pensais que remplacer les Equals et GetHashCode devrait résoudre ce problème?

Answer 1

return otherObj == null && otherObj.Id == this.Id; 

Est-ce correct? Ne devrait-il pas être

return otherObj != null && otherObj.Id == this.Id;