Changeset 945 for library/bdm/base/bdmbase.h

Timestamp:

05/17/10 17:54:44 (14 years ago)

Author:

mido

Message:

patch of UI and log_levels to obtain nicer and mainly compilable source code.)

Files:

• library/bdm/base/bdmbase.h (modified) (4 diffs)

library/bdm/base/bdmbase.h

@@ -406 +406 @@
 //! This class stores a details that will be logged to a logger
 template<class T> class log_level_template : public log_level_base<T> {
-public:
+protected:
+        //! boolean flags related indicating which details will be logged to a logger
+        bitset<32> values;
+
+public:
+
+        //! read only operator for testing individual fields of log_level
+        //!
+        //! it is necessary to acces it with a proper enumeration type, thus this approach is type-safe
+        bool operator [] (const enum T::log_level_enums &log_level_enum ) const
+        {
+                return values[log_level_enum];
+        }
+
+        //! operator for setting an individual field of log_level 
+        //!
+        //! it is necessary to acces it with a proper enumeration type, thus this approach is type-safe
+        bitset<32>::reference operator [] (const enum T::log_level_enums &log_level_enum )
+        {
+                return values[log_level_enum];
+        }
+
+        //! Set log_levels according to the Setting element
+        void from_setting ( const Setting &element )
+        {
+                string raw_log_level;
+                UI::get( raw_log_level, element );
+                Array<string> loaded_log_level = string2Array( raw_log_level );
+        
+                values.reset();
+
+                for( int i = 0; i < loaded_log_level.length(); i++ )
+                        for( int j = 0; j < names().length(); j++ ){
+                                if( loaded_log_level(i) == names()(j)  ) 
+                                {
+                                        values[j] = true;
+                                        break;
+                                } 
+                        }
+        }
+
+        //! Store log_levels into the Setting element
+        void to_setting ( Setting &element ) const 
+        {
+                // HERE WE WANT NOT TO DELETE PREVIOUS DATA STORED BY OTHER LOG_LEVELS, SEE SPECIAL IMPLEMENTATION OF UI::GET(...) FOR THIS CLASS
+                string string_to_write =  ( const char* ) element;
+
+                for( unsigned int i = 0; i < values.size(); i++ )
+                        if( values[i] ) 
+                        {
+                                if( string_to_write.length() > 0 )
+                                        string_to_write = string_to_write + ',';
+                                string_to_write = string_to_write + names()(i);
+                        }
+                        
+                element = string_to_write;
+        }
+
         //! This method stores a vector to the proper place in registered logger 
         //!
@@ -430 +487 @@
         //!
         //! parameter \c enum_subindex identifies the precise position of setting in the case there is more settings registered to with this enum
-        //!
-        //! If this method was not templated, we could store whole body of this class in cpp file without explicitly touching registered_logger->log_setting(...) here.
-        //! (it would not be straightforward, though, still there are some enums which had to be converted into integers but it could be done without loosing type control)
-        //! This way, it would not be necessary to declare log_level_base<T> class and we could declare log_level_template<T>
-        //! before the logger class itself with a mere foroward declaration of logger. In our case, however, touching of registered_logger->log_setting
-        //! implies that forward declaration is not enough and we are lost in a circle. And just by cutting this circle we obtains log_level_base<T> class. 
         template<class U> void store( const enum T::log_level_enums log_level_enum, const U data, int enum_subindex = 0 ) const
         {                       
@@ -443 +494 @@
         }
 };
+//UIREGISTER IS FORBIDDEN FOR THIS CLASS,  AS IT SHOULD BE LOADED ONLY THROUGH THE SPECIALIZED UI::GET(...) METHOD
+
 
 /*!
@@ -459 +512 @@
   \ref ui
 */
-#define LOG_LEVEL(classname,...) public: enum log_level_enums { __VA_ARGS__ }; log_level_template<classname> log_level; private: friend class log_level_template<classname>; friend class log_level_base<classname>; static const Array<string> &log_level_names() { static const Array<string> log_level_names = log_level_base<classname>::string2Array( #__VA_ARGS__ ); return log_level_names; }
+#define LOG_LEVEL(classname,...) public: enum log_level_enums { __VA_ARGS__ }; log_level_template<classname> log_level; private: friend class log_level_base<classname>; static const Array<string> &log_level_names() { static const Array<string> log_level_names = log_level_template<classname>::string2Array( #__VA_ARGS__ ); return log_level_names; }