Thread: Parsing commands , istringstream

Hi,


I got this coding working but i want to improve it.


I want to make sure that the frist word i read is a string without numbers.


Then i want to make sure the second / third word are int / floats...

I was thinking in using isalpha but i don´t know if is the best way.

And maybe change the code to switchs so i don´t have 20 (If elses)...

Code:

int main() {


	//// The Map will be 60x60 (Example)
	//Mundo myMundo(60);
	////  going to focus the board on 15 / 15
	//myMundo.drawTerrain(60, 60);




	string command, parse;
	istringstream is; // Going to get the input from the user. (String)


	// While i got things to processsing i will do it one by one from the object not from the keyboard
	do {


		cout << "Command: ";
		getline(cin, command); //Get the full string


		// Need to convert the object before use it.
		is.str(command);


		// Going to transfer everything i receive in lowercase
		transform(command.begin(), command.end(), command.begin(), tolower);


		// If the string is diferent from "exit" i will go execute the commands of the strings.
		if (command != "sair") {
			while (is >> parse) {
				if (command == "defmundo") {
					cout << "defmundo" << endl;
					defmundo(3);
				}
				else if (command == "defen")
				{
					cout << "defen" << endl;
				}
				else if (command == "defpc")
				{
					cout << "defpc" << endl;
				}
				else {
					cout << "Command not valid" << endl;
				}
			}
			// I just finished to use my string i need to clean it up to i can receive other clean one.
			is.clear();
		}
		// ... Other 20 options....		
	} while (command != "sair");




	cout << "exit of the cycle" << endl;


	getchar();
	return 0;
}

I want to treat the sentence now, i need to extract the next values...

Example: defmundo 5 2 hard (int int string)
Example: defxpto hard 2 ( string int)
Example: defasta hada ( string)

Code:

// If the string is diferent from "exit" i will go execute the commands of the strings.
        if (command != "sair") {
            while (is >> parse) {                
                if (parse == "defmundo") {
                    cout << "defmundo is going to being call..." << endl;
                    // Now i need to extract the nexts values.
                    defmundo(value1,value2,string);
                }
                else if (parse == "defen")
                {
                    cout << "defen" << endl;
                }
                else if (parse == "defpc")
                {
                    cout << "defpc" << endl;
                }
                else {
                    cout << "Command not valid" << endl;
                }
            }
            // I just finished to use my string i need to clean it up to i can receive other clean one.
            is.clear();
        }
        // ... Other 20 options....        
    } while (command != "sair");

Start with a parse class to take a string and produce a list of tokens.
With member functions like
- getNumberOfTokens
- getToken

If you want to parse parameters passed with program execution, I'd suggest taking argc and argv into account using a for-loop:

Code:

int value = 0;

if (argc > 1)
{
  for (int i = 1; i < argc; i++)
  {
    if (strcmp("-foo", argv[i]) == 0 && i < (argc - 1))
    {
      value = atoi(argv[i + 1]);
      i++;
    }
    else if // ... the rest
    else
      printf("Unknown parameter: %s\n", argv[i]);
    }
  }
}