Changeset 4508 in orxonox.OLD for orxonox/trunk/src/util

Timestamp:

Jun 4, 2005, 2:29:11 PM (20 years ago)

Author:

bensch

Message:

orxonox/trunk: implemented a better backloop-check in the track-system
also implemented a new function in tList: inList() that returns tru, if a certain element is already in the List and false otherwise

Location:

orxonox/trunk/src/util/track

Files:

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

orxonox/trunk/src/util/track/track_manager.cc

@@ -149 +149 @@
 
 /**
-   \brief checks if there are any BackLoops in the Track (Backloops only 
-   \param trackElem the trackElement to check about
-   \param depth the depth to search in
+   \brief checks if there are any BackLoops in the Track
    \returns true if NO loop was found, false Otherwise
    You actually have to act on false!!
-   it simply does this by looking if the current trackElem is found again somewhere else in the Track
-
-   \todo this has to be reimplemented
-*/
-bool TrackElement::backLoopCheck(const TrackElement* trackElem, unsigned int depth) const
-{
-  if(depth == 0 || this != trackElem)
-    {
-      if (this->children)
-        {
-          tIterator<TrackElement>* iterator = this->children->getIterator();
-          TrackElement* enumElem = iterator->nextElement();
-          while (enumElem)
-            {
-              if(!enumElem->backLoopCheck(trackElem, depth + 1))
-                return false;
-              enumElem = iterator->nextElement();
-            }
-          delete iterator;
-        }
-    }
-  else
-    return false;
-
+*/
+bool TrackElement::backLoopCheck(void) const
+{
+  tList<const TrackElement>* trackList = new tList<const TrackElement>;
+  
+  this->backLoopCheckAtomic(trackList);
+  
+  delete trackList;
   // only returns if everything worked out
   return true;
 }
+
+/**
+   \brief checks if there are any BackLoops in the Track.
+   \param trackList A list of stored tracks, to search in.
+   \returns true if NO loop was found, false Otherwise
+   You actually have to act on false!!
+*/
+bool TrackElement::backLoopCheckAtomic(tList<const TrackElement>* trackList) const
+{
+  if (trackList->inList(this))
+    return false;
+
+  trackList->add(this);
+
+  if (this->children)
+    {
+      tIterator<TrackElement>* iterator = this->children->getIterator();
+      TrackElement* enumElem = iterator->nextElement();
+      while (enumElem)
+        {
+          if (!enumElem->backLoopCheckAtomic(trackList))
+            return false;
+        }
+      delete iterator;
+    }
+  return true;
+}
+
 
 /**
@@ -264 +274 @@
     PRINT(0)("   is Joined at the End\n");
   
-  if(!this->backLoopCheck(this)) /* this should not happen */
+  if(!this->backLoopCheck()) /* this should not happen */
     PRINT(2)(" THERE IS A BACKLOOP TO THIS ELEMENT\n");
 }
@@ -916 +926 @@
   // checking if there is a back-loop-connection and ERROR if it is.
   tmpTrackElem = this->firstTrackElem->findByID(trackIDs[0]);
-  if (!tmpTrackElem->backLoopCheck(tmpTrackElem))
+  if (!tmpTrackElem->backLoopCheck())
     {
       PRINTF(2)("Backloop connection detected at joining trackElements\n -> TRACK WILL NOT BE JOINED\n");

orxonox/trunk/src/util/track/track_manager.h

@@ -48 +48 @@
   TrackElement* findByID(unsigned int trackID);
   TrackElement* findByName(const char* trackName);
-  bool backLoopCheck(const TrackElement* trackElem, unsigned int depth = 0) const;
+  bool backLoopCheck(void) const;
 
   TrackElement* getChild(int childNumber) const;
@@ -55 +55 @@
   inline const char* getName(void) const { return this->name; };
 
-
+ private:
+  bool backLoopCheckAtomic(tList<const TrackElement>* trackList) const;
+
+ public:
   // atributes
   bool                  isFresh;              //!< If no Points where added until now