通过带有成员函数指针的 QHash 调用成员函数的正确方法

Question

我有一个服务器类，它处理QJsonObject 数据并根据数据中设置的键对其进行处理。

目前，我使用一个大的 if-then-else 语句来决定像这样做什么：

const QString action = jsonObject.value(KEY_ACTION).toString();
if (action == SOME_ACTION) {
    // do something
} else if (action == SOME_OTHER_ACTION) {
    // do something else
}

等等。现在，与此同时，我有很多操作，对于每一个操作，我的服务器都必须检查所有案例，直到找到正确的案例。因此，我想知道是否有更好的方法来做到这一点。

我考虑过在不同的函数中处理数据，并有一个 QHash 与相应的函数指针指向每个操作的相应函数，如下所示：

在构造函数中：

const QHash<QString, void(Server::*)(const QJsonObject &)> processFunctionsMap {
    { SOME_ACTION, &Server::processSomeAction },
    { SOME_OTHER_ACTION, &Server::processSomeOtherAction }
}

以及各自的功能：

void Server::processSomeAction(const QJsonObject &data)
{
    ...
}

然后调用匹配函数：

if (! processFunctionsMap.contains(action)) {
    // Catch this case
}
(this->*processFunctionsMap.value(action))(jsonObject);

这似乎可行，但我不是 C++ 专业人士，所以我的问题是这是否是正确的方法。

Answer 1

您的方法是合理的，但您已将不匹配的情况从执行 else 块（可能什么都不做）更改为即时未定义的行为。

您需要将哈希查找与调用分开，以便您可以在两者之间插入成功查找的检查。使用 C++ 标准集合（std::map 是红黑树，std::unordered_map 是哈希表），这将是对返回迭代器的 find(key) 的调用......您将其与 map.end() 和如果它们相等，请确保不要取消引用。 QHash 或任何其他非标准哈希表肯定会提供类似的东西。

• Understanding what QHash does when key not found