Changeset 6077 in orxonox.OLD for trunk/src/lib

Timestamp:

Dec 13, 2005, 2:53:22 AM (19 years ago)

Author:

bensch

Message:

orxonox/trunk: better and faster BaseObject/ClassList

Location:

trunk/src/lib/lang

Files:

• base_object.cc (modified) (2 diffs)
• base_object.h (modified) (1 diff)
• class_list.cc (modified) (6 diffs)
• class_list.h (modified) (3 diffs)

trunk/src/lib/lang/base_object.cc

@@ -101 +101 @@
  * @returns true if it is, false otherwise
 */
-bool BaseObject::isA (long classID) const
+bool BaseObject::isA (ClassID classID) const
 {
   // if classID is a derivable object from a SUPERCLASS
@@ -132 +132 @@
 bool BaseObject::isA (const char* className) const
 {
-  long classID = ClassList::StringToID(className);
+  ClassID classID = ClassList::StringToID(className);
   if (classID != CL_NULL)
     return this->isA(classID);

trunk/src/lib/lang/base_object.h

@@ -35 +35 @@
   inline int getClassID() const { return this->classID; };
 
-  bool isA (long classID) const;
+  bool isA (ClassID classID) const;
   bool isA (const char* className) const;
   void whatIs() const;
 
   bool operator==(const char* objectName);
+  /** @param classID comparer for a ClassID @returns true on match, false otherwise */
+  bool operator==(ClassID classID) { return this->isA(classID); };
 
  protected:

trunk/src/lib/lang/class_list.cc

@@ -37 +37 @@
  *  Creates a new ClassList
 */
-ClassList::ClassList(const long& classID, const char* className)
+ClassList::ClassList(ClassID classID, const char* className)
 {
   this->className = className;
@@ -94 +94 @@
     if (objectPointer->isA((*cl).classID))
     {
-      std::list<BaseObject*>::iterator bo;
-      for (bo = (*cl).objectList.begin(); bo != (*cl).objectList.end(); bo++)
-      {
-        if ((*bo) == objectPointer)
-        {
+      std::list<BaseObject*>::iterator bo = std::find ((*cl).objectList.begin(), (*cl).objectList.end(), objectPointer);
+      if (bo != (*cl).objectList.end())
           (*cl).objectList.erase(bo);
-          break;
-        }
-      }
     }
   }
@@ -175 +169 @@
 }
 
-
+/**
+ * !!PRIVATE!!
+ * @param classID the ClassID to search for
+ * @returns the ClassList with classID as specifyer, or NULL if not
+ */
 ClassList* ClassList::getClassList(ClassID classID)
 {
@@ -183 +181 @@
 
 
+/**
+ * !!PRIVATE!!
+ * @param className the ClassName to search for
+ * @returns the ClassList with className as specifyer, or NULL if not
+ */
 ClassList* ClassList::getClassList(const char* className)
 {
@@ -288 +291 @@
  * @return the ClassID. CL_NULL, if the class was not found.
  */
-long ClassList::StringToID(const char* className)
+ClassID ClassList::StringToID(const char* className)
 {
   ClassList* cl = ClassList::getClassList(className);
@@ -314 +317 @@
  * @param classID the class that should be displayed (if CL_NULL (default) all classes will be displayed)
  */
-void ClassList::debug(unsigned int debugLevel, long classID)
+void ClassList::debug(unsigned int debugLevel, ClassID classID)
 {
   if (debugLevel > 3)

trunk/src/lib/lang/class_list.h

@@ -33 +33 @@
 
   public:
-    ClassList(const long& classID, const char* className);
+    ClassList(ClassID classID, const char* className);
     virtual ~ClassList();
 
@@ -51 +51 @@
 
     static const char*                    IDToString(ClassID classID = CL_NULL);
-    static long                           StringToID(const char* className);
-    static void                           debug(unsigned int debugLevel = 0, long classID = CL_NULL);
+    static ClassID                        StringToID(const char* className);
+    static void                           debug(unsigned int debugLevel = 0, ClassID classID = CL_NULL);
     static void                           debugS(const char* className = NULL, unsigned int debugLevel = 0);
 
@@ -65 +65 @@
   private:
 
-    long                            classID;                //!< ClassID stored in this ClassList \see ClassID
+    ClassID                         classID;                //!< ClassID stored in this ClassList
     const char*                     className;              //!< Name of the Class Stored here