C++如何使用更少的条件语句？

Question

对于我的任务，我正在存储用户登录信息。我正在接受一个作为命令的字符串。该命令可以是创建、登录、删除等。总共有 10 个选项，即可能有 10 个不同的字符串。谁能解释一种更有效的方法来编写这个而不是 10 个 if 和 else if 语句？基本上除了使用一堆if (string == "one")，else if (string == "two")之外，我应该如何格式化/构造东西。谢谢

Answer 1

我希望您的讲师希望您将函数提取到另一个可重用函数：

string action;
command = CreateAction(action);
command.Do(...);

当然，在您的 CreateAction 类中，您仍然需要有条件来确定需要创建哪些命令。

AbstractCommand CreateAction(action)
{
    if (action == "login")
        return LoginCommand();
    else if (action == "remove")
        return RemoveCommand();
    ..... etc etc
}

如果你真的想摆脱所有条件，你可以创建一些自注册命令，但这涉及更多的代码和类......

您应该查找 Command Pattern 和 Factory Pattern 之类的内容

Answer 2

您可以使用函数指针和查找表。

typedef void (*Function_Pointer)(void);
void Create(void);
void Login(void);
void Remove(void);

struct Function_Option_Entry
{
  const char * option_text;
  Function_Pointer p_function;
};

Function_Option_Entry option_table[] =
{
  {"one", Create},
  {"two", Login},
  {"three", Remove},
};
const unsigned int option_table_size =
  sizeof(option_table) / sizeof(option_table[0]);

//...
std::string option_text;
//...
for (i = 0; i < option_table_size; ++i)
{
  if (option_text == option_table[i].option_text)
  {
    option_table[i].p_function();
    break;
  }
}

Answer 3

使用switch 和一个简单的哈希函数。
您需要使用哈希函数，因为 C 和 C++ 只允许切换整数值。

template<size_t N> constexpr char myhash(const char &x[N]) { return x[0] ^ (x[1]+63); }
char myhash(const string& x) { return x.size() ? x[0] ^ (x[1]+63) : 0; }

switch(myhash(s)) {
case myhash("one"):
    if(s != "one") goto nomatch;
    // do things
    break;
case myhash("two"):
    if(s != "two") goto nomatch;
    // do things
    break;
default:
nomatch:
    // No match
}

如果您不使用std::string，则需要稍作调整。

Answer 4

我建议您为每个特定字符串创建一个函数。例如，如果你收到一个字符串“create”，你会调用函数doCreate()，如果你收到一个字符串“login”，那么你会调用函数doLogin()

对这些函数的唯一限制是它们都必须具有相同的签名。在上面的示例中，它是这样的：

typedef void (*func_t) ();

这个想法是创建一个从字符串到这些函数的 std::map。因此，您不必写 10 个 if 左右，因为您将能够通过特定字符串名称的名称从映射中简单地选择正确的函数。让我通过一个小例子来解释一下：

typedef void (*func_t) ();
void doCreate()
{
     std::cout << "Create function called!\n";
}

void doLogin()
{
     std::cout << "Login function called!\n";
}

std::map<std::string, func_t> functionMap;

void initMap()
{
    functionMap["create"] = doCreate;
    functionMap["login"] = doLogin;
}

int main()
{
     initMap();
     std::string str = "login";
     functionMap[str](); // will call doLogin()

     str = "create";
     functionMap[str](); // will call doCreate()

     std::string userStr;
     // let's now assume that we also can receive a string not from our set of functions
     std::cin >> userStr;
     if (functionMap.count(userStr))
     {
          functionMap[str](); // now we call doCreate() or doLogin()
     }
     else
     {
          std::cout << "Unknown command\n";
     }

     return 0;
 }

希望对你有所帮助=)

Answer 5

您可以使用地图为您进行比较。

类似这样的：

初始化地图：

std::map<std::string, std::function<void(std::string&)>> map;
map["login"]  = std::bind(&Class::DoLogin,  this, std::placeholders::_1);
map["create"] = std::bind(&Class::DoCreate, this, std::placeholders::_1);

接收消息：

map.at(rx.msg_type)(rx.msg_data);

处理程序：

void Class::DoLogin(const std::string& data)
{
   // do login
}

Answer 6

也许您可以创建一个std::map<std::string, int> 并使用地图查找来获取传递的命令的代码 - 您可以稍后打开该号码。或者创建一个enum Command 并拥有一个std::map<std::string, Command> 并使用开关。

例子：

enum Command
{
     CREATE,
     LOGIN,
     ...
};

std::map<std::string, Command> commandNameToCode;
// fill the map with appropriate values
commandNameToCode["create"] = Command::CREATE;

// somehow get command name from user and store in the below variable (not shown)    
std::string input;

// check if the command is in the map and if so, act accordingly
if(commandNameToCode.find(input) != commandNameToCode.end())
{
    switch(commandNameToCode[input])
    {
    case CREATE:
         // handle create
         break;
    ...
    }
}