Добрый день,
По примеру из книги NHibernate CookBook делаю,
В некоторых местах уже используются == и !=, теперь они в классах-потомках Entity.
Ну и мысль возникает, не перенести ли это один раз в базовое Entity, ведь тогда все вроде как проще работать должно.
Есть подозрения, что == в базовом классе может вызывать какие-нибудь непредвиденные проблемы по сравнению с == в самом классе-потомке, с экземплярами и коллекцией которых мы работаем напрямую (с ними или с их прокси).
Плюсы (в этом же и минусы) - можно сравнивать любые два энтити (c одинаковым типом Id, т.к. там дженерик), хотя конечно если они не приводимы друг к другу, то пример из книжки CookBook считает их неравными.
Главный плюс, что если в новой Entity решим использовать операцию сравнения, то не надо будет заново все перегружать.
Хотя тут вроде минус, что сравнение напрямую (x == y) может привести к загрузке всех полей объекта, а сравнение по Id (x.id == y.id), если там были прокси, не приведет.
Вот что планируется сделать. Есть ли тут подводные камни? Принято ли перегружать == прямо у базового Entity?
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
18.
19.
20.
21.
22.
23.
24.
25.
26.
27.
28.
29.
30.
31.
32.
33.
34.
35.
36.
37.
38.
39.
40.
41.
42.
43.
44.
45.
46.
47.
48.
49.
50.
51.
52.
53.
54.
55.
56.
57.
58.
59.
60.
61.
62.
public abstract class Entity<TId>
{
public virtual TId Id { get; set; }
public override bool Equals(object obj)
{
return Equals(obj as Entity<TId>);
}
private static bool IsTransient(Entity<TId> obj)
{
return obj != null &&
Equals(obj.Id, default(TId));
}
public virtual Type GetUnproxiedType()
{
return GetType();
}
public virtual bool Equals(Entity<TId> other)
{
if (other == null)
return false;
if (ReferenceEquals(this, other))
return true;
if (!IsTransient(this) &&
!IsTransient(other) &&
Equals(Id, other.Id))
{
var otherType = other.GetUnproxiedType();
var thisType = GetUnproxiedType();
return thisType.IsAssignableFrom(otherType) ||
otherType.IsAssignableFrom(thisType);
}
return false;
}
public override int GetHashCode()
{
if (Equals(Id, default(TId)))
return base.GetHashCode();
return Id.GetHashCode();
}
public static bool operator ==(Entity<TId> object1, Entity<TId> object2)
{
if ((object)object1== null)
{
return ((object)object2== null);
}
else
{
return object1.Equals(object2);
}
}
public static bool operator !=(Entity<TId> object1, Entity<TId> object2)
{
return !(object1 == object2);
}
}
