cstring and switch problems

[bmxtrev]

#include <iostream>
#include <iomanip>
#include <cstring>
#include <cctype>    //used for toupper
using namespace std;


const int MAX_CITIES = 10;
const int NOT_FOUND = -1;
const int MAXLEN = 20;
const int MAXCOMMANDLEN = 9;

typedef char CityNameType[MAXLEN + 1];

enum CommandType { ADD, DELETE, UPDATE, OUTPUT, CLEAR, SUMMARIZE,
                   TOTAL, SORT, QUIT, UNKNOWN };
// This class handles the commands which find and store data for each
// city in the list.
class DestinationList 
{
   private:
      int num;                       // number of objects in the list
      Destination list[MAX_CITIES];  // array of destinations
   
   //This function searches the list for a city name, and 
   //determines if the name is already in the list or not.
   //params: in      
   int Search(CityNameType city)
   {
      for(int i = 0; i < num; i++)
      {
         if (list[i].DestinationHasName( city ))
            return i;
      }
      return NOT_FOUND;
   }
   
   void Swap( Destination & Dest1, Destination & Dest2)
   {
      Destination temp = Dest1;
      Dest1 = Dest2;
      Dest2 = temp;
   }
 
   public:
   
   //This constructor initializes num to 0.
   DestinationList()
   {
      num = 0;
   }
   
   // This function adds a city name to the list, only if the list
   // is not full, and the city does not already appear in the list.
   //params: in, out
   void Add (  )
   { 
      CityNameType city;
      cin >> city;
      ToUpper();
      Destination dest( city );
      int index = Search(city);
  
      if (num < (MAX_CITIES) && index == NOT_FOUND)
      {
         list[num] = dest;
         num++;
         cout << city << " added to the list.";
      }
      else if ( num >= (MAX_CITIES))
         cout << city << " not added. List is full.";
      else
         cout << city << " is already in the list.";
      cout << endl; 
   }
  
   // This function updates a city's number of packages and weight,
   // adding to the value totals.
   //params: in, out
   void Update ( )
   { 
      CityNameType city;
      cin >> city;
      int numpackages;
      float weight;
      cin >> numpackages >> weight;
      int index = Search(city);
      if (index == NOT_FOUND)
         cout << "Cannot update. " << city << " is not in the list." 
              << endl;
      else
      { 
         list[index].RecordShipment( numpackages, weight );
         cout << "Destination " << city << " updated with " 
              << numpackages << " packages weighing "
              << fixed << showpoint << setprecision(2) << weight 
              << " lbs." << endl;
      }
   }
  
   // This function displays the Average weight of the packages
   // in a certain city, or an error message.
   //params: none
   void Output (  )
   {
      CityNameType city;
      cin >> city;
      int index = Search(city);
      if (index == NOT_FOUND)
        cout << "Cannot output. " << city << " is not in the list." 
             << endl;
      else
         cout << "Average weight of all packages to " << city << ": " 
              << fixed << showpoint << setprecision(2) 
              << list[index].AverageWeight() << endl;

   }
  
   //This function ends the program.
   //params: none
   void Quit ( )
   {
      cout << "Normal termination.";
   }
  
   //This function clears the number of packages and weight for 
   //the specified city.
   //params: in, out
   void Clear ( )
   {
      CityNameType city;
      cin >> city;
      int index = Search(city);
      if (index == NOT_FOUND)
        cout << "Cannot clear. " << city << " is not in the list." 
             << endl;
      else
         list[index].ClearShipment();
   }
 
   //Get the command from the user input
   CommandType GetCommand()
   {
      char command [MAXCOMMANDLEN + 1];
      cin >> command;
      if (strcmp(command, "Add") == 0)
         return ADD;
      else if (strcmp(command, "Delete") == 0)
         return DELETE;
      else if (strcmp(command, "Update") == 0)
         return UPDATE;
      else if (strcmp(command, "Output") == 0)
         return OUTPUT;
      else if (strcmp(command, "Clear") == 0)
         return CLEAR;
      else if (strcmp(command, "Summarize") == 0)
         return SUMMARIZE;
      else if (strcmp(command, "Total") == 0)
         return TOTAL;
      else if (strcmp(command, "Sort") == 0)
         return SORT;
      else if (strcmp(command, "Quit") == 0)
         return QUIT;
      else 
         return UNKNOWN;
   }

   
   //Converts city to all uppercase
   void ToUpper()
   {
      CityNameType city;
      for ( int i = 0; i < strlen(city); i++ )
         city[i] = toupper(city[i]);
   }


int main ()
{
   DestinationList list;
   CommandType command;
    
   GetCommand();
   switch(command)
   {
      case ADD:
         Add().list;
         break;
      case 'DELETE':
         Delete();
         break;
      case 'UPDATE':
         Update();
         break;
      case 'OUTPUT':
      case 'CLEAR':
      case 'SUMMARIZE':
      case 'TOTAL':
      case 'SORT':
      case 'QUIT':
      case 'UNKNOWN':
   }
         
   list.Quit();       
}

I'm having trouble using the GetCommand function in main. leaving it as is, i get an error saying value returning functions can't be ingnored.

The main has to be able to take in things such as "Add New york" or "Update Chicago"

this is using cstring, so how can I get GetCommand to work?
*note a large portion of the program is missing from this post.

[S G H]

The issue is not cstring.

First issue, at the end:
This:
GetCommand()
Should be This:
GetCommand();

Function clear has no closing bracket.

Main should be int.

If issues persist, also copy us the lines of where this issue lies in.

[bmxtrev]

fixed all of those, problem is still there.

"The value from a value-returning function cannot be ignored" referring to GetCommand(); in the main

I do not know how to use GetCommand in the main.

[S G H]

Try:

CommandType command = GetCommand();

Also

Add().list; will fail to compile.

Use

Add();