Changeset 767:11c946fa8d13 in lemon-1.2 for lemon/hartmann_orlin.h

Timestamp:

08/11/09 22:52:35 (15 years ago)

Author:

Peter Kovacs <kpeter@…>

Branch:

default

Phase:

public

Message:

Simplify comparisons in min mean cycle classes (#179)
using extreme INF values instead of bool flags.

File:

• lemon/hartmann_orlin.h (modified) (9 diffs)

lemon/hartmann_orlin.h

@@ -151 +151 @@
     struct PathData
     {
-      bool found;
       LargeValue dist;
       Arc pred;
-      PathData(bool f = false, LargeValue d = 0, Arc p = INVALID) :
-        found(f), dist(d), pred(p) {}
+      PathData(LargeValue d, Arc p = INVALID) :
+        dist(d), pred(p) {}
     };
 
@@ -191 +190 @@
 
     Tolerance _tolerance;
+
+    // Infinite constant
+    const LargeValue INF;
 
   public:
@@ -246 +248 @@
       _gr(digraph), _length(length), _comp(digraph), _out_arcs(digraph),
       _best_found(false), _best_length(0), _best_size(1),
-      _cycle_path(NULL), _local_path(false), _data(digraph)
+      _cycle_path(NULL), _local_path(false), _data(digraph),
+      INF(std::numeric_limits<LargeValue>::has_infinity ?
+          std::numeric_limits<LargeValue>::infinity() :
+          std::numeric_limits<LargeValue>::max())
     {}
 
@@ -473 +478 @@
       }      
       for (int i = 0; i < n; ++i) {
-        _data[(*_nodes)[i]].resize(n + 1);
+        _data[(*_nodes)[i]].resize(n + 1, PathData(INF));
       }
       return true;
@@ -483 +488 @@
     void processRounds() {
       Node start = (*_nodes)[0];
-      _data[start][0] = PathData(true, 0);
+      _data[start][0] = PathData(0);
       _process.clear();
       _process.push_back(start);
@@ -518 +523 @@
           v = _gr.target(e);
           d = _data[u][k-1].dist + _length[e];
-          if (!_data[v][k].found) {
-            next.push_back(v);
-            _data[v][k] = PathData(true, _data[u][k-1].dist + _length[e], e);
-          }
-          else if (_tolerance.less(d, _data[v][k].dist)) {
-            _data[v][k] = PathData(true, d, e);
+          if (_tolerance.less(d, _data[v][k].dist)) {
+            if (_data[v][k].dist == INF) next.push_back(v);
+            _data[v][k] = PathData(d, e);
           }
         }
@@ -541 +543 @@
           v = _gr.target(e);
           d = _data[u][k-1].dist + _length[e];
-          if (!_data[v][k].found || _tolerance.less(d, _data[v][k].dist)) {
-            _data[v][k] = PathData(true, d, e);
+          if (_tolerance.less(d, _data[v][k].dist)) {
+            _data[v][k] = PathData(d, e);
           }
         }
@@ -562 +564 @@
       for (int i = 0; i < n; ++i) {
         u = (*_nodes)[i];
-        if (!_data[u][k].found) continue;
+        if (_data[u][k].dist == INF) continue;
         for (int j = k; j >= 0; --j) {
           if (level[u].first == i && level[u].second > 0) {
@@ -587 +589 @@
         for (int i = 0; i < n; ++i) {
           u = (*_nodes)[i];
-          pi[u] = std::numeric_limits<LargeValue>::max();
+          pi[u] = INF;
           for (int j = 0; j <= k; ++j) {
-            d = _data[u][j].dist * _curr_size - j * _curr_length;
-            if (_data[u][j].found && _tolerance.less(d, pi[u])) {
-              pi[u] = d;
+            if (_data[u][j].dist < INF) {
+              d = _data[u][j].dist * _curr_size - j * _curr_length;
+              if (_tolerance.less(d, pi[u])) pi[u] = d;
             }
           }