Changeset 7340 in orxonox.OLD

Timestamp:

Apr 19, 2006, 2:39:05 AM (18 years ago)

Author:

bensch

Message:

orxonox/trunk: fixed a ShellCommand-bug

Location:

trunk/src/lib

Files:

• shell/shell.cc (modified) (3 diffs)
• shell/shell_command.cc (modified) (13 diffs)
• util/substring.h (modified) (1 diff)

trunk/src/lib/shell/shell.cc

@@ -148 +148 @@
 
 /**
- * deactiveates the Shell.
+ * @brief deactiveates the Shell.
  */
 void Shell::deactivate()
@@ -175 +175 @@
 
 /**
- * sets the File to load the fonts from
+ * @brief sets the File to load the fonts from
  * @param fontFile the file to load the font from
  *
@@ -192 +192 @@
 
 /**
- * sets the size of the text and spacing
+ * @brief sets the size of the text and spacing
  * @param textSize the size of the Text in Pixels
  * @param lineSpacing the size of the Spacing between two lines in pixels

trunk/src/lib/shell/shell_command.cc

@@ -44 +44 @@
   this->executor->setName(commandName);
 
-//  this->classID = classID;
+  //  this->classID = classID;
   this->shellClass = ShellCommandClass::getCommandClass(className); //ClassList::IDToString(classID);
   if (this->shellClass != NULL)
@@ -83 +83 @@
 {
   /// FIXME
-/*  if (ShellCommandClass::commandClassList == NULL)
-    ShellCommandClass::initCommandClassList();
-
- const ShellCommandClass* checkClass = ShellCommandClass::isRegistered(className);
-
- if (checkClass != NULL)
-  {
-    std::list<ShellCommand*>::iterator elem;
-    for (elem = checkClass->commandList.begin(); elem != checkClass->commandList.end(); elem++)
-    {
-      if (!strcmp(commandName, (*elem)->getName()))
-      {
-        delete (*elem);
-        checkClass->commandList.remove(*elem);
-        break;
-      }
-    }
-
-    if (checkClass->commandList->size() == 0)
-    {
-      ShellCommandClass::commandClassList->remove(checkClass);
-      delete checkClass;
-    }
-  }*/
+  /*  if (ShellCommandClass::commandClassList == NULL)
+      ShellCommandClass::initCommandClassList();
+
+   const ShellCommandClass* checkClass = ShellCommandClass::isRegistered(className);
+
+   if (checkClass != NULL)
+    {
+      std::list<ShellCommand*>::iterator elem;
+      for (elem = checkClass->commandList.begin(); elem != checkClass->commandList.end(); elem++)
+      {
+        if (!strcmp(commandName, (*elem)->getName()))
+        {
+          delete (*elem);
+          checkClass->commandList.remove(*elem);
+          break;
+        }
+      }
+
+      if (checkClass->commandList->size() == 0)
+      {
+        ShellCommandClass::commandClassList->remove(checkClass);
+        delete checkClass;
+      }
+    }*/
 }
 
@@ -138 +138 @@
       }
     }
-   return false;
+    return false;
   }
   else
@@ -161 +161 @@
   BaseObject* objectPointer = NULL;            //< a pointer to th Object to Execute the command on
   bool emptyComplete = false;                  //< if the completion input is empty string. e.g ""
-  unsigned int fktPos = 1;                     //< the position of the function (needed for finding it)
-//  long completeType = SHELLC_NONE;           //< the Type we'd like to complete.
-  SubString inputSplits(executionString, " \t\n,");
-
-  if (inputSplits.size() == 0)
-    return false;
+  //  long completeType = SHELLC_NONE;           //< the Type we'd like to complete.
+  SubString inputSplits(executionString, SubString::WhiteSpacesWithComma);
+
+
+  // if we do not have any input return
+  if (inputSplits.empty())
+    return false;
+
+  // if we only have one input (!MUST BE AN ALIAS)
   if (inputSplits.size() >= 1)
   {
@@ -181 +184 @@
           if (objectList != NULL)
           {
-            if (inputSplits.size() > 1)
-            {
-
-              (*(*alias)->getCommand()->executor)(objectList->front(), inputSplits.getSubSet(1).join()); /// TODO CHECK IF OK
-            }
-            else
-              (*(*alias)->getCommand()->executor)(objectList->front(), "");
-           return true;
+            (*(*alias)->getCommand()->executor)(objectList->front(), inputSplits.getSubSet(1).join()); /// TODO CHECK IF OK
+            return true;
           }
-        }
-      }
-    }
+          /// TODO CHECK FOR STATIC functions.
+        }
+      }
+    }
+
     // looking for a Matching Class
     if (likely(ShellCommandClass::commandClassList != NULL))
@@ -199 +198 @@
       for (commandClassIT = ShellCommandClass::commandClassList->begin(); commandClassIT != ShellCommandClass::commandClassList->end(); commandClassIT++)
       {
-        if ((*commandClassIT)->getName() && inputSplits.getString(0) == (*commandClassIT)->getName())
+        if ((*commandClassIT)->getName() && inputSplits[0] == (*commandClassIT)->getName())
         {
           //elemCL->getName();
@@ -210 +209 @@
     }
 
+    // Second Agument. (either Object, or Function)
     if (commandClass != NULL && inputSplits.size() >= 2)
     {
+      int fktPos = 1; // The position of the Function (either at pos 1, or 2)
+      // If we have an ObjectList.
       if (objectList != NULL)
       {
@@ -218 +220 @@
         for (object = objectList->begin(); object != objectList->end(); object++)
         {
-          if ((*object)->getName() != NULL && inputSplits.getString(1) == (*object)->getName() )
+          if ((*object)->getName() != NULL && inputSplits[1] == (*object)->getName())
           {
             objectPointer = (*object);
@@ -224 +226 @@
             break;
           }
-         }
-
-      //
+        }
+
+        // if we did not find an Object with matching name, take the first.
         if (objectPointer == NULL)
           objectPointer = objectList->front();
       }
+
       // match a function.
       if (commandClass != NULL && (fktPos == 1 || (fktPos == 2 && inputSplits.size() >= 3)))
@@ -236 +239 @@
         for (cmdIT = commandClass->commandList.begin(); cmdIT != commandClass->commandList.end(); cmdIT++)
         {
-          if (inputSplits.getString(fktPos) == (*cmdIT)->getName())
+          if (inputSplits[fktPos] == (*cmdIT)->getName())
           {
             if (objectPointer == NULL && (*cmdIT)->executor->getType() & Executor_Objective)
               return false;
-            if (inputSplits.size() > fktPos+1)
-              (*(*cmdIT)->executor)(objectPointer, inputSplits.getSubSet(2).join()); /// TODO CHECK IF OK
             else
-              (*(*cmdIT)->executor)(objectPointer, "");
-            return true;
+            {
+              (*(*cmdIT)->executor)(objectPointer, inputSplits.getSubSet(fktPos+1).join()); /// TODO CHECK IF OK
+              return true;
+            }
           }
         }
@@ -250 +253 @@
     }
   }
+  return false;
 }
 
@@ -260 +264 @@
   if (this == NULL)
     return NULL;
- else
- {
-   this->description = description;
-   return this;
- }
+  else
+  {
+    this->description = description;
+    return this;
+  }
 }
 
@@ -302 +306 @@
  */
 ShellCommand* ShellCommand::defaultValues(const MultiType& value0, const MultiType& value1,
-                                          const MultiType& value2, const MultiType& value3,
-                                          const MultiType& value4)
+    const MultiType& value2, const MultiType& value3,
+    const MultiType& value4)
 {
   if (this == NULL || this->executor == NULL)

trunk/src/lib/util/substring.h

@@ -55 +55 @@
 
   // retrieve Information from within
+  /** @returns true if the SubString is empty */
+  inline bool empty() const { return this->strings.empty(); };
+  /** @returns the count of Strings stored in this substring */
   inline unsigned int size() const { return this->strings.size(); };
-const std::string& getString(unsigned int i) const { return (i < this->strings.size()) ? this->strings[i] : emptyString; };
-  const std::string& operator[](unsigned int i) const { return this->getString(i); };
+  /** @param i the i'th String @returns the i'th string from the subset of Strings */
+  const std::string& operator[](unsigned int i) const { return (i < this->strings.size()) ? this->strings[i] : emptyString;return this->getString(i); };
+  /** @param i the i'th String @returns the i'th string from the subset of Strings */
+  const std::string& getString(unsigned int i) const { return (*this)[i]; };
 
   // the almighty algorithm.