mmm, vandaag eens wat verder in gedoken en heb het probleem gevonden. Er zat een foutje in de .Equals van het basis object (deze vergelijkt op data niveau of objecten gelijk zijn en vloog de bocht uit als er een object met een lijst in ging (details zal ik even besparen)). Deze heb ik nu uitgebreid dat hij ook even kijkt naar ReferenceEquals (in dat geval is het object uiteraard gelijk).
Nadere inspectie levert mij op dat om de een of andere manier de ==/Equals in sommige gevallen niet gebruik maakten van de basis class. Hierdoor verschilde de resultaten tussen de verschillende vergelijkingen.
De te testen class
C#:
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
| public class TestClass : TmpBaseEntity
{
public string Value { get; set; }
}
public class TmpBaseEntity
{
public override bool Equals(object obj)
{
Trace.WriteLine("Hit equals");
return base.Equals(obj);
}
public static bool operator ==(TmpBaseEntity a, TmpBaseEntity b)
{
Trace.WriteLine("Hit ==");
if (a == (object)null && b == (object)null) { return true; }
if (a == (object)null || b == (object)null) { return false; }
return a.Equals(b);
}
public static bool operator !=(TmpBaseEntity a, TmpBaseEntity b)
{
Trace.WriteLine("Hit !=");
if ((object)a == null && (object)b == null) { return false; }
if ((object)a == null || (object)b == null) { return true; }
return !a.Equals(b);
}
} |
C#:
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
| [TestMethod]
[TestCategory(BaseEntityEqualityTestsCategory)]
public void EqualityWorksAsExpectedOnGenericTypes()
{
EnhancedList<TestClass> tc = new EnhancedList<TestClass>()
{
new TestClass() { Value = "v1" },
new TestClass() { Value = "v2" },
new TestClass() { Value = "v3" }
};
GetElementFromListAndComare(tc);
}
private void GetElementFromListAndComare<T>(EnhancedList<T> list)
where T : class
{
T var1 = GetFirst(list);
T var2 = GetFirst(list);
List<CompareRes> compares = new List<CompareRes>();
compares.Add(new CompareRes("var1 == var2", () => var1 == var2));
compares.Add(new CompareRes("var1 == var1", () => var1 == var1));
compares.Add(new CompareRes(" var1 == list[0]", () => var1 == list[0]));
compares.Add(new CompareRes("var2 == list[0]", () => var2 == list[0]));
compares.Add(new CompareRes("var1.Equals(var2)", () => var1.Equals(var2)));
compares.Add(new CompareRes("var1.Equals(var1)", () => var1.Equals(var1)));
compares.Add(new CompareRes("var2.Equals(var1)", () => var2.Equals(var1)));
compares.Add(new CompareRes("var1.Equals(list[0])", () => var1.Equals(list[0])));
compares.Add(new CompareRes("list[0].Equals(var1)", () => list[0].Equals(var1)));
compares.Add(new CompareRes("list.Contains(var1)", () => list.Contains(var1)));
compares.Add(new CompareRes("ReferenceEquals(var1, var2)", () => ReferenceEquals(var1, var2)));
compares.Add(new CompareRes("ReferenceEquals(var1, list[0])", () => ReferenceEquals(var1, list[0])));
List<CompareRes> errors = compares.Where(C => !C.Result).ToList();
Assert.AreEqual(0, errors.Count, "Failed" + string.Join("\r\n", errors.Select(E => E.Action)));
}
class CompareRes
{
public CompareRes(string action, Func<bool> result)
{
Trace.WriteLine("Running: " + action);
Action = action;
Result = result();
Trace.WriteLine("Succes: " + Result);
}
public string Action { get; set; }
public bool Result { get; set; }
}
private T GetFirst<T>(List<T> list)
where T : class
{
return list.First();
}
} |
Dit levert de volgende "resultaten" op:
Running: var1 == var2
Succes: True
Running: var1 == var1
Succes: True
Running: var1 == list[0]
Succes: True
Running: var2 == list[0]
Succes: True
Running: var1.Equals(var2)
Hit equals
Succes: True
Running: var1.Equals(var1)
Hit equals
Succes: True
Running: var2.Equals(var1)
Hit equals
Succes: True
Running: var1.Equals(list[0])
Hit equals
Succes: True
Running: list[0].Equals(var1)
Hit equals
Succes: True
Running: list.Contains(var1)
Hit equals
Succes: True
Running: ReferenceEquals(var1, var2)
Succes: True
Running: ReferenceEquals(var1, list[0])
Succes: True
De bug in de Equals zelf is hier niet van toepassing omdat ik gewoon de base aanroep en dit enkel is om te illustreren wat er gebeurt
Frapante is dat als je een breakpoint zet in de test en vervolgens via het Immediate window het volgende uitvoert de .Equals en == wel geraakt worden:
T x = GetFirst(list);
T x2 = GetFirst(list);
x2 == x
Hit ==
Hit equals
true
(Dit leverde voorheen dus een false op omdat de .Equals uit de bocht vloog)
Het lijkt er dus op dat binnen de Generic functie de "verkeerde" == wordt aangeroepen. Dit zorgde voor het vreemde gedrag. Snap nog steeds niet helemaal waarom hij de andere afslag neemt, maar heb iig de bug in mijn Equals kunnen tackelen door nog eens goed naar de code te kijken.
Dat de immediate window de == ook weer anders behandelt zorgde voor nog resultaten die nog verwarrender waren... Een deel van mijn gerapporteerde bevindingen was namelijk op basis van de immediate window tests.
[edit]
Het lijkt er trouwens op dat het aanpassen van de
where T : class naar
where T : TmpBaseEntity wel de goede functie aanroept...
:strip_icc():strip_exif()/u/190884/dug.jpg?f=community)
Whut.... Zal wel weer ergens iets verkloot hebben... 
/u/19474/avatar2015.png?f=community){kind=avatar}
:strip_icc():strip_exif()/u/6143/rincewind.jpg?f=community)
/u/249034/arrow.png?f=community){kind=icon}
:strip_icc():strip_exif()/u/43400/avatar.jpeg?f=community){kind=avatar}
/u/3813/100_8985_ICON2.JPG?f=community)