Равные, но не взаимозаменяемы

Question

У меня есть два объекта, которые что-то вроде этого:

class Container { 
    public HashSet<Item> Items { get; } 
} 

class Item { 
    public Container Parent { get; set; } 
    public string Value1 { get; set; } 
    public int Value2 { get; set; } 
} 

Каждый Item экземпляр должен принадлежать к Container, например, и есть один-ко-многим между ними, что всегда синхронизированы с обоих концов.

Теперь я сталкиваюсь с реализацией метода, который сравнивает два экземпляра Item, чтобы узнать, соответствуют ли их значения Value1 и Value2. Этот метод будет не учитывать значение Parent, так как каждая пара экземпляров, которые я сравниваю, определенно отличается от этого значения, поэтому сделать этот метод будет бесполезным, так как тогда он будет иметь тот же результат (false), что и object.ReferenceEquals метод.

Мой вопрос заключается в следующем. Должен ли я реализовать этот метод как метод объекта public override bool Equals(object obj) (вместе с GetHashCode)? Или не исключает того, что он игнорирует его свойство Parent? Почему или почему нет? Альтернативная идея, которую я имею, - это просто реализовать ее как public bool EqualsIgnoreParent(Item other), которая не отменяет ничего; Тогда я мог бы позвонить из пользовательского компаратора.

Answer 1

Благодаря mikez's comment и Эрика Липперта blog post, я узнал, что я не должен переопределить Item «s Equals или GetHashCode методов, поскольку Item» s случаев изменчивы и добавляют к HashSet.

Следуя предложению here, я решил отделить две концепции равенства.

1. Идентификация. Я сделал не переопределить ItemEquals и GetHashCode методы. Я оставил эти методы, поскольку они находятся в object. Это то, что будет использовать HashSet.
2. Эквивалентность. Я создал публичный одноэлементный класс под названием ValueComparer, который вложен в Item и реализует IEqualityComparer<Item>. Вот где я помещаю логику сравнения, которую я описываю в своем вопросе.

Answer 2

Я думаю, вы хотели реализовать IEquatable. поэтому я предлагаю вам просто перегружать Euqals (object other) с Euqals (Item other), в то время как перезаписывает Equals (object other), вызывая Equals (Item other).

конечно, сделать GetHashCode вернуть -1 всегда, чтобы заставить сравнение равенства быть сделано с помощью Equals.here это объяснить, почему нужно перезаписать GetHashCode метод - Why is it important to override GetHashCode when Equals method is overridden?

Answer 3

Да, вы можете реализовать это с Равных и GetHashCode, до тех пор, пока Value1 и Value2 не изменяются, или ваш хэш не зависит от изменений. (В противном случае Collections, используя ваш Hashcode не будет работать)

Кроме того, вы можете уменьшить класс Item для основных членов

class Item { 
    public string Value1 { get; set; } 
    public int Value2 { get; set; } 
} 

и поддерживать связь в дополнительном словаре: (который был бы эффективным, если вы реализуете GetHashCode правильно)

Dictionary<Item, Container> ContainersOfItems; 

Answer 4

не делайте этого, если вы не 100% уверены, что член Родителя никогда не делает никакой разницы. Там может быть много случаев, о которых вы не думаете, (что, если вам нужно положить предметы из 2 контейнеров в одном словаре?), Поэтому я рекомендую придерживаться дизайна. Если по определению равенство определяется только другими членами, идите на это. В противном случае нет.
Вы всегда можете написать логику равенства конкретных случаев. Помните, что Dictionary, List, HashSet и другие коллекции позволяют определить пользовательский номер IEqualityComparer.
Есть еще один вариант, но будьте очень осторожны. Переопределите оба метода, но сделайте GetHashCode более толерантным, чем Equals, это означает, что неравные объекты имеют одинаковый хеш-код, а НЕ наоборот. Make GetHashCode игнорировать родителя, но не Equals. это ничего не повредит, а затем, если вы хотите игнорировать член Родитель в словарях, напишите IEqualityComparer, который сравнивает их так же, как GetHashCode.

Answer 5

Я бы предположил, что это может быть более значимым для разделенного Пункта так это выглядит примерно так:

class Container<TContent> { 
    public HashSet<Item<TContent>> Items { get; } 
} 

class Item<TContent> { 
    public Container Parent; 
    public TContent Content; 
} 

Если вы сделаете это, то вы можете определить тип для хранения вашего контента, который имеет Equals метод подходит для этого типа и инициализируйте свой HashSet с помощью IEqualityComparer<Item<TConcent>>, который устанавливает равенство для каждого поля Item<TContent>Content.