Changeset 10309

Timestamp:

04/07/11 15:46:15 (14 years ago)

Author:

tbretz

Message:

Added typedef for StateNames; changed the getter such that they can now be declared const.

Location:

trunk/FACT++/src

Files:

• StateMachineImp.cc (modified) (3 diffs)
• StateMachineImp.h (modified) (2 diffs)

trunk/FACT++/src/StateMachineImp.cc

@@ -650 +650 @@
 //!    as string.
 //!
-const string StateMachineImp::GetStateName(int state) /*const*/
-{
-    const string &str = fStateNames[state].first;
-    return str.empty() ? Form("%d", state) : str;
+const string StateMachineImp::GetStateName(int state) const
+{
+    const StateNames::const_iterator i = fStateNames.find(state);
+    return i==fStateNames.end() || i->second.first.empty() ? Form("%d", state) : i->second.first;
 }
 
@@ -666 +666 @@
 //!    returned.
 //!
-const string StateMachineImp::GetStateDesc(int state) /*const*/
-{
-    return fStateNames[state].second;
+const string StateMachineImp::GetStateDesc(int state) const
+{
+    const StateNames::const_iterator i = fStateNames.find(state);
+    return i==fStateNames.end() ? "" : i->second.second;
 }
 
@@ -685 +686 @@
 //!
 //
-const string StateMachineImp::GetStateDescription(int state) /*const*/
+const string StateMachineImp::GetStateDescription(int state) const
 {
     const string &str = GetStateName(state);

trunk/FACT++/src/StateMachineImp.h

@@ -31 +31 @@
     int fCurrentState;   /// Current state of the state machine
 
+    typedef std::map<const int, std::pair<std::string, std::string>> StateNames;
+
     /// Human readable names associated with the states
-    std::map<const int, std::pair<std::string, std::string>> fStateNames;
+    StateNames fStateNames;
 
     std::vector<EventImp*> fListOfEvents; /// List of available commands as setup by user
@@ -121 +123 @@
 
 
-    const std::string GetStateName(int state) /*const*/;
-    const std::string GetStateName() { return GetStateName(fCurrentState); }
-
-    const std::string GetStateDesc(int state) /*const*/;
-    const std::string GetStateDesc() { return GetStateDesc(fCurrentState); }
-
-    const std::string GetStateDescription(int state) /*const*/;
-    const std::string GetStateDescription() { return GetStateDescription(fCurrentState); }
+    const std::string GetStateName(int state) const;
+    const std::string GetStateName() const { return GetStateName(fCurrentState); }
+
+    const std::string GetStateDesc(int state) const;
+    const std::string GetStateDesc() const { return GetStateDesc(fCurrentState); }
+
+    const std::string GetStateDescription(int state) const;
+    const std::string GetStateDescription() const { return GetStateDescription(fCurrentState); }
 };