Changeset 11263 for trunk/sources/HeuristicLab.Core/3.3/Collections/DirectedGraph/DirectedGraph.cs

Timestamp:

08/04/14 14:58:47 (8 years ago)

Author:

bburlacu

Message:

#2223:

• Vertex: remove cloning of arcs in the cloning constructor
• DirectedGraph; Adjust cloning constructor to add arcs to cloned vertices

File:

• trunk/sources/HeuristicLab.Core/3.3/Collections/DirectedGraph/DirectedGraph.cs (modified) (2 diffs)

trunk/sources/HeuristicLab.Core/3.3/Collections/DirectedGraph/DirectedGraph.cs

@@ -57 +57 @@
       vertices = new HashSet<IVertex>(original.vertices.Select(cloner.Clone));
       arcs = new HashSet<IArc>(original.arcs.Select(cloner.Clone));
+
+      // add the arcs to the newly cloned vertices
+      foreach (var arc in arcs) {
+        arc.Source.AddArc(arc);
+        arc.Target.AddArc(arc);
+      }
     }
 
@@ -89 +95 @@
 
     public virtual void AddVertex(IVertex vertex) {
-      vertices.Add(vertex);
-      // register event handlers
-      vertex.ArcAdded += Vertex_ArcAdded;
-      vertex.ArcRemoved += Vertex_ArcRemoved;
-      OnVertedAdded(this, new EventArgs<IVertex>(vertex));
+      if (!vertices.Contains(vertex) && vertex.Degree > 0)
+        throw new ArgumentException("New vertices cannot have any arcs.");
+
+      if (vertices.Add(vertex)) {
+        // register event handlers
+        vertex.ArcAdded += Vertex_ArcAdded;
+        vertex.ArcRemoved += Vertex_ArcRemoved;
+        OnVertedAdded(this, new EventArgs<IVertex>(vertex));
+      }
     }