Changeset 290:f6899946c1ac in lemon-main for lemon/lgf_writer.h

Timestamp:

09/30/08 20:53:18 (16 years ago)

Author:

Balazs Dezso <deba@…>

Branch:

default

Phase:

public

Message:

Simplifying exceptions

• Using asserts instead of exceptions for unitialized parameters
• Only the IO exceptions are used in the lemon
• DataFormatError? is renamed to FormatError?
• The IoError? is used for file access errors

File:

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

lemon/lgf_writer.h

@@ -56 +56 @@
     template <typename T>
     bool operator<(const T&, const T&) {
-      throw DataFormatError("Label map is not comparable");
+      throw FormatError("Label map is not comparable");
     }
 
@@ -204 +204 @@
           _map.find(str);
         if (it == _map.end()) {
-          throw DataFormatError("Item not found");
+          throw FormatError("Item not found");
         }
         return it->second;
@@ -224 +224 @@
           ::const_iterator it = _map.find(val);
         if (it == _map.end()) {
-          throw DataFormatError("Item not found");
+          throw FormatError("Item not found");
         }
         return (_graph.direction(val) ? '+' : '-') + it->second;
@@ -463 +463 @@
     DigraphWriter(const std::string& fn, const Digraph& digraph)
       : _os(new std::ofstream(fn.c_str())), local_os(true), _digraph(digraph),
-        _skip_nodes(false), _skip_arcs(false) {}
+        _skip_nodes(false), _skip_arcs(false) {
+      if (!(*_os)) throw IoError(fn, "Cannot write file");
+    }
 
     /// \brief Constructor
@@ -471 +473 @@
     DigraphWriter(const char* fn, const Digraph& digraph)
       : _os(new std::ofstream(fn)), local_os(true), _digraph(digraph),
-        _skip_nodes(false), _skip_arcs(false) {}
+        _skip_nodes(false), _skip_arcs(false) {
+      if (!(*_os)) throw IoError(fn, "Cannot write file");
+    }
 
     /// \brief Destructor
@@ -1019 +1023 @@
     GraphWriter(const std::string& fn, const Graph& graph)
       : _os(new std::ofstream(fn.c_str())), local_os(true), _graph(graph),
-        _skip_nodes(false), _skip_edges(false) {}
+        _skip_nodes(false), _skip_edges(false) {
+      if (!(*_os)) throw IoError(fn, "Cannot write file");
+    }
 
     /// \brief Constructor
@@ -1027 +1033 @@
     GraphWriter(const char* fn, const Graph& graph)
       : _os(new std::ofstream(fn)), local_os(true), _graph(graph),
-        _skip_nodes(false), _skip_edges(false) {}
+        _skip_nodes(false), _skip_edges(false) {
+      if (!(*_os)) throw IoError(fn, "Cannot write file");
+    }
 
     /// \brief Destructor
@@ -1577 +1585 @@
     /// Construct a section writer, which writes into the given file.
     SectionWriter(const std::string& fn)
-      : _os(new std::ofstream(fn.c_str())), local_os(true) {}
+      : _os(new std::ofstream(fn.c_str())), local_os(true) {
+      if (!(*_os)) throw IoError(fn, "Cannot write file");
+    }
 
     /// \brief Constructor
@@ -1583 +1593 @@
     /// Construct a section writer, which writes into the given file.
     SectionWriter(const char* fn)
-      : _os(new std::ofstream(fn)), local_os(true) {}
+      : _os(new std::ofstream(fn)), local_os(true) {
+      if (!(*_os)) throw IoError(fn, "Cannot write file");
+    }
 
     /// \brief Destructor