Alapszab\u00e1ly, hogy HashSet, Dictionary vagy b\u00e1rmi m\u00e1s asszociat\u00edv kollekci\u00f3kban kulcsk\u00e9nt csak immutable objektumokat lehet haszn\u00e1lni. Ennek az az oka, hogy amikor berakunk egy elemet, akkor a kollekci\u00f3 megh\u00edvja az objektumon a GetHashCode-ot, \u00e9s az azonos hashk\u00f3d\u00fa objektumokat list\u00e1ba szervezi. Ha k\u00e9t objektum egyenl\u0151nek tekintend\u0151 (az Equals alapj\u00e1n), akkor a hascodejuknak azonosnak kell lenni. Ha ez nem teljes\u00fcl, vagy id\u0151vari\u00e1ns, azaz k\u00e9s\u0151bb m\u00e1r nem igaz, akkor ezek a kollekci\u00f3k megh\u00fcly\u00fclnek.
\nA p\u00e9ld\u00e1m \u00edgy n\u00e9zett ki:<\/p>\n
public virtual bool Equals(EntityBase other)
\n\t\t{
\n\t\t\tif (other == null)
\n\t\t\t{
\n\t\t\t\treturn false;
\n\t\t\t}
\n\t\t\tif (Id == 0)
\n\t\t\t{
\n\t\t\t\t\/\/Transient object
\n\t\t\t\treturn ReferenceEquals(this, other);
\n\t\t\t}
\n\t\t\treturn other.Id == Id;
\n\t\t}<\/p>\n
\t\tpublic override bool Equals(object obj)
\n\t\t{
\n if (!(obj is EntityBase))
\n {
\n return false;
\n }
\n\t\t\treturn Equals((EntityBase)obj);
\n\t\t}<\/p>\n
\t\tpublic override int GetHashCode()
\n\t\t{
\n\t\t\treturn Id;
\n\t\t}<\/p>\n
Eleve l\u00e1tszik, a GetHashCode \u00e9s az Equals nem azonos logika szerint m\u0171k\u00f6dik. A GetHashCode akkor ad vissza azonos hascodeot, ha az idk azonosak. Az Equals viszont az idt\u0151l f\u00fcgg\u0151en vagy az idkat hasonl\u00edtja \u00f6ssze, vagy referenci\u00e1lis kompar\u00e1l\u00e1st csin\u00e1l. Ez m\u00e1r kap\u00e1sb\u00f3l bug.<\/p>\n
A m\u00e1sik bug, hogy ez az objektum nem immutable, az Id v\u00e1ltozik, amikor adatb\u00e1zisba besz\u00farjuk az entit\u00e1st, \u00e9s kap egy idt. Emiatt az eg\u00e9sz koncepci\u00f3 beteg.<\/p>\n
A HashSet.Add megh\u00edvja a HashSet.AddIfNotPresetet, amiben van egy ciklus:<\/p>\n
for (int i = this.m_buckets[num % this.m_buckets.Length] – 1; i >= 0; i = this.m_slots[i].next)
\n\t{
\n\t\tif (this.m_slots[i].hashCode == num && this.m_comparer.Equals(this.m_slots[i].value, value))
\n\t\t{
\n\t\t\treturn false;
\n\t\t}
\n\t\tnum3++;
\n\t}<\/p>\n
Ez az ciklus akadt be. Az if felt\u00e9tele sose teljes\u00fclt, \u00edgy sose l\u00e9pett kis a ciklusb\u00f3l, mivel idt kapott az entit\u00e1s, \u00edgy megv\u00e1ltozott az Equals logik\u00e1ja.<\/p>\n","protected":false},"excerpt":{"rendered":"
Alapszab\u00e1ly, hogy HashSet, Dictionary vagy b\u00e1rmi m\u00e1s asszociat\u00edv kollekci\u00f3kban kulcsk\u00e9nt csak immutable objektumokat lehet haszn\u00e1lni. Ennek az az oka, hogy amikor berakunk…<\/p>\n","protected":false},"author":1,"featured_media":0,"comment_status":"closed","ping_status":"closed","sticky":false,"template":"","format":"standard","meta":{"footnotes":""},"categories":[8,4],"tags":[],"class_list":["post-1380","post","type-post","status-publish","format-standard","hentry","category-net","category-szakmai-elet"],"_links":{"self":[{"href":"https:\/\/soci.hu\/blog\/index.php\/wp-json\/wp\/v2\/posts\/1380","targetHints":{"allow":["GET"]}}],"collection":[{"href":"https:\/\/soci.hu\/blog\/index.php\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/soci.hu\/blog\/index.php\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/soci.hu\/blog\/index.php\/wp-json\/wp\/v2\/users\/1"}],"replies":[{"embeddable":true,"href":"https:\/\/soci.hu\/blog\/index.php\/wp-json\/wp\/v2\/comments?post=1380"}],"version-history":[{"count":1,"href":"https:\/\/soci.hu\/blog\/index.php\/wp-json\/wp\/v2\/posts\/1380\/revisions"}],"predecessor-version":[{"id":1381,"href":"https:\/\/soci.hu\/blog\/index.php\/wp-json\/wp\/v2\/posts\/1380\/revisions\/1381"}],"wp:attachment":[{"href":"https:\/\/soci.hu\/blog\/index.php\/wp-json\/wp\/v2\/media?parent=1380"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/soci.hu\/blog\/index.php\/wp-json\/wp\/v2\/categories?post=1380"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/soci.hu\/blog\/index.php\/wp-json\/wp\/v2\/tags?post=1380"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}