Changeset 201 for trunk/src/statistics.cpp

Timestamp:

05/04/2009 09:21:44 AM (3 years ago)

Author:

guyru

Message:

Re-factor the Statistics constructor to use exceptions

File:

• trunk/src/statistics.cpp (modified) (3 diffs)

trunk/src/statistics.cpp

@@ -34 +34 @@
 Statistics::Statistics(configuration::Configuration *backend)
 {
+        try {
+                load_data();
+        } catch (exception &e) {
+                reset();
+        }
+}
+
+void Statistics::load_data() 
+{
         string tmp;
         vector<string> tmp_vec;
@@ -46 +55 @@
         tmp = backend->get("statistics_score_distribution");
         split(tmp_vec, tmp, is_any_of(","));
-        if (tmp_vec.size() != score_distributions_slots) {
-                reset();
-                return;
-        }
+        if (tmp_vec.size() != score_distributions_slots)
+                throw BadConfiguration();
 
         BOOST_FOREACH(string i, tmp_vec) {
@@ -57 +64 @@
         istringstream i(tmp);
         i >> _last_reset;
+        if (!_last_reset)
+                throw BadConfiguration();
+                
 }