Changeset 1053:90f8696360b2 in lemon-0.x

Timestamp:

01/05/05 15:34:00 (18 years ago)

Author:

Mihaly Barasz

Branch:

default

Phase:

public

Convert:

svn:c9d7d8f5-90d6-0310-b91f-818b3a526b0e/lemon/trunk@1447

Message:

UndirGraphs?: invalid edge bug

Location:

src

Files:

• lemon/graph_utils.h (modified) (1 diff)
• lemon/undir_graph_extender.h (modified) (1 diff)
• test/undir_graph_test.cc (modified) (3 diffs)

src/lemon/graph_utils.h

@@ -104 +104 @@
   }
 
-  /// \brief Function to count the symmetric edges in the graph.
-  ///
-  /// This function counts the symmetric edges in the graph.
+  // Undirected edge counting:
+
+  template <typename Graph>
+  inline
+  typename enable_if<typename Graph::EdgeNumTag, int>::type
+  _countUndirEdges(const Graph &g) {
+    return g.undirEdgeNum();
+  }
+
+  template <typename Graph>
+  inline int _countUndirEdges(Wrap<Graph> w) {
+    return countItems<Graph, typename Graph::UndirEdgeIt>(w.value);
+  }
+
+  /// \brief Function to count the edges in the graph.
+  ///
+  /// This function counts the edges in the graph.
   /// The complexity of the function is O(e) but for some
   /// graph structure it is specialized to run in O(1).
-  template <typename Graph>
-  inline int countSymEdges(const Graph& _g) {
-    return countItems<Graph, typename Graph::SymEdgeIt>(_g);
-  }
+
+  template <typename Graph>
+  inline int countUndirEdges(const Graph& g) {
+    return _countUndirEdges<Graph>(g);
+  }
+
 
 

src/lemon/undir_graph_extender.h

@@ -49 +49 @@
         UndirEdge(ue), forward(_forward) {}
       /// Invalid edge constructor
-      Edge(Invalid i) : UndirEdge(i), forward(false) {}
+      Edge(Invalid i) : UndirEdge(i), forward(true) {}
 
       bool operator==(const Edge &that) const {

src/test/undir_graph_test.cc

@@ -7 +7 @@
 #include <lemon/full_graph.h>
 
+#include <lemon/graph_utils.h>
+
 #include "test_tools.h"
 
@@ -13 +15 @@
 using namespace lemon::concept;
 
-
-int main() {
+void check_concepts() {
   typedef UndirGraphExtender<ListGraphBase> UndirListGraphBase;
 
@@ -41 +42 @@
 
   checkConcept<UndirGraph, UndirGraph>();
+}
+
+typedef UndirListGraph Graph;
+typedef Graph::Node Node;
+typedef Graph::UndirEdge UEdge;
+typedef Graph::Edge Edge;
+typedef Graph::NodeIt NodeIt;
+typedef Graph::UndirEdgeIt UEdgeIt;
+typedef Graph::EdgeIt EdgeIt;
+
+void check_item_counts(Graph &g, int n, int e) {
+  check(countNodes(g)==n, "Wrong node number.");
+  check(countEdges(g)==2*e, "Wrong edge number.");
+}
+
+void print_items(Graph &g) {
+  cout << "Nodes" << endl;
+  int i=0;
+  for(NodeIt it(g); it!=INVALID; ++it, ++i) {
+    cout << "  " << i << ": " << g.id(it) << endl;
+  }
+
+  cout << "UndirEdge" << endl;
+  i=0;
+  for(UEdgeIt it(g); it!=INVALID; ++it, ++i) {
+    cout << "  " << i << ": " << g.id(it) 
+         << " (" << g.id(g.source(it)) << ", " << g.id(g.target(it)) 
+         << ")" << endl;
+  }
+
+  cout << "Edge" << endl;
+  i=0;
+  for(EdgeIt it(g); it!=INVALID; ++it, ++i) {
+    cout << "  " << i << ": " << g.id(it)
+         << " (" << g.id(g.source(it)) << ", " << g.id(g.target(it)) 
+         << ")" << endl;
+  }
+
+}
+
+int main() {
+  check_concepts();
+
+
+  Graph g;
+
+  check_item_counts(g,0,0);
+
+  Node
+    n1 = g.addNode(),
+    n2 = g.addNode(),
+    n3 = g.addNode();
+
+  UEdge
+    e1 = g.addEdge(n1, n2),
+    e2 = g.addEdge(n2, n3);
+
+  // print_items(g);
+
+  check_item_counts(g,3,2);
 
   return 0;