Cleaning up the get_command_type() function based on merge review comments
from Monty. This cleanup changed the command_vector to be a container of
std::string instead of const char *.

This allowed me to remove the need for a function object since I can now use
std::find instead of std::find_if