I’m creating an application in C# 3.5 that uses the AutoCAD API to read a 2D AutoCAD drawing, make changes to the drawing using defined business logic, then adjust it back in AutoCAD. Due to the nature of the logic, the shape of the drawing has to be re-constructed – e.g. a rectangle is made up of 4 connecting straight lines.
I’m creating these shapes using the start and end co-ordinates of each line from AutoCAD, but some of the co-ordinates don’t exactly match up. For example, one point could at 0.69912839 (on one axis), but a line starting from the same point could be 0.69990821. These are in mm so the distance is minute (0.00078mm!)
I’ve created my own class (call it MyPoint, similar to PointF) because I’ve needed to add some additional logic to it. In that class I’ve created a method that takes two doubles and returns true or false depending on if the two points are within 0.001mm of each other. I’ve then overridden the Equals method, == and != operators so I can do (point1 == point2 or point1.Equals(point2)) which checks if all axis are within 0.001mm of each other – if they are, I class it as being the same point.
That’s fine and working brilliantly. Now, I need to check a collection of these point classes to get rid of all duplicates, so I’m using LINQ’s Distinct() method on my collection. However this method uses GetHashcode(), not Equals() to determine if the instances are equal. So, I’ve overriden GetHashcode() which uses the GetHashcode of the double class.
But, the above example fails because obviously they’re different values and therefore generate different hashcodes. Is there any way that two numbers that are within 0.001 of each other can generate the same hashcode? (Note the numbers don’t know about each other as GetHashcode is called separately on different class instances.) I’ve tried numerous ways which work for some examples but not for others.
One example is truncating the number to 3dp (multiply it by 10^3, then truncate it) and creating the hashcode on the result – which works for the above example (699 == 699.) But this doesn’t work for 0.69990821 and 0.70000120 (699 != 700.) I’ve tried rounding, which works for the second set of numbers (0.700 == 0.700) but not for the first (0.699 != 0.700.) I’ve even tried truncating the number to 3dp then adjusting it up to the next even number, which works for both the previous examples, but not for 12.9809 and 12.9818 (12980 != 12982.)
Is there another way, or should I scrap the Equals, ==, != and GetHashcode overrides, and create my own MyPoint.IsEqualTo() and MyPointCollection.Distinct() methods?
It would be easier just to remove the dependency on the Distinct method. Implement a System.Collections.IComparer (or the generic equivalent) and use a simple collection like a list. Then determine if the item is in the list with the comparer, and not add it if it already is contained.