删除 List<object> 的重复对

Question

问题

我正在使用 3D WPF 库中的简单线条显示几何图形。在下一张图片中可以看到它的一个例子：

您可以在其中看到一组三角形和四边形。我绘制这个的方式是我提供了一个List<Point3D>，我在其中放置了代表每个段的点对。

问题是有很多重复的边，我想避免这种情况，因为这种类型的表示似乎非常需要资源。

点列表是在每个包含 N 个顶点的Element 上迭代生成的。它不知道是否共享特定的边缘。

<p0, p1, p1, p2, p2, p333, p333, p89, p89, p2, p2, p1 ...>

这个想法是删除重复的对（注意顺序可能不一样）。在上面的示例中，删除的对应该是最后一个 (p2, p1)，因为它表示与第二对点 (p1, p2) 相同的边。可能有一对、两对或更多对重复的点。

我需要尽快执行此操作，性能是这里的首要任务。

想法

在列表中添加点时，我可以暂时存储其中两个并检查列表是否已包含它们，但这意味着每次添加点时都要查看列表，这对我来说似乎不是一个好主意（列表将包含数千个点 5000-50000)。

我生成点列表的元素具有多个具有唯一 ID 的节点，因此我认为可以通过创建有序 Tuple<Point3D, Point3D> 的 Dictionary 然后删除重复项来以某种方式使用它。

我还没有尝试过最后一个想法，因为我还不确定如何实现它，我想知道是否还有其他可以做的事情。

Answer 1

您可以使用HashSet 来存储所有边。检查速度很快，边缘是否已经设置好。但是你应该覆盖GetHashCode 和Equals。我做了一个简单的例子。

class MyLine
{
    public MyPoint P1 { get; private set; }
    public MyPoint P2 { get; private set; }
    public MyLine(MyPoint p1, MyPoint p2)
    {
        P1 = p1;
        P2 = p2;
    }
    protected bool Equals(MyLine other)
    {
        return (Equals(P1, other.P1) && Equals(P2, other.P2)) || Equals(P1, other.P2) && Equals(P2, other.P1);
    }
    public override bool Equals(object obj)
    {
        if (ReferenceEquals(null, obj)) return false;
        if (ReferenceEquals(this, obj)) return true;
        if (obj.GetType() != this.GetType()) return false;
        return Equals((MyLine)obj);
    }
    public override int GetHashCode()
    {
        unchecked
        {
            return P1.GetHashCode() + P2.GetHashCode();
        }
    }
}
class MyPoint
{
    public string Id { get; private set; }
    public MyPoint(string id)
    {
        Id = id;
    }
    protected bool Equals(MyPoint other)
    {
        return string.Equals(Id, other.Id);
    }
    public override bool Equals(object obj)
    {
        if (ReferenceEquals(null, obj)) return false;
        if (ReferenceEquals(this, obj)) return true;
        if (obj.GetType() != this.GetType()) return false;
        return Equals((MyPoint)obj);
    }
    public override int GetHashCode()
    {
        return (Id != null ? Id.GetHashCode() : 0);
    }
}

那么你应该可以像这样添加每一行：

public static void Main(string[] args)
{
    HashSet<MyLine> lines = new HashSet<MyLine>();
    var line = new MyLine(new MyPoint("a"), new MyPoint("b"));
    lines.Add(line);
    line = new MyLine(new MyPoint("b"), new MyPoint("a"));
    lines.Add(line);
}

还可以使用GetHashCode 和Equals 将所有行存储在List 中，然后使用Distinct 方法。

public static void Main(string[] args)
{
    List<MyLine> lines = new List<MyLine>();
    var line = new MyLine(new MyPoint("a"), new MyPoint("b"));
    lines.Add(line);
    line = new MyLine(new MyPoint("b"), new MyPoint("a"));
    lines.Add(line);
    lines = lines.Distinct().ToList();
}

Answer 2

使用 HashSet<Tuple<Point3D, Point3D>> 。每当你得到一条新边 - p1,p2，检查 (p1,p2) 在集合中的存在。还要检查 (p2,p1) 的存在。如果都不存在，则将 (p1,p2) 和 (p2,p1) 添加到集合中并使用边。

您可以通过制作自己的哈希和相等函数来进一步加快这一速度，这些函数将看到 (p1,p2) 等于 (p2,p1)。除非需要，否则不要这样做，Set 操作非常快，我怀疑改进会很大。