我是否正确使用了函数指针

Am I using function pointers correctly?

本文关键字：函数 指针 是否      更新时间：2023-10-16

我有一个函数，在伪代码中看起来像这样：

std::string option = "option1" // one of n options, user supplied
for (int i = 0; i < 100000; i++) {
if (option == "option1") {
doFunction1a();
} else if (option == "option2") {
doFunction2a();
} else if (option == "option3") {
doFunction3a();
}
//  more code...
if (option == "option1") {
doFunction1b();
} else if (option == "option2") {
doFunction2b();
} else if (option == "option3") {
doFunction3b();
}
}

然而，我可以通过这样做来避免循环中重复的if语句：

std::string option = "option1" // one of n options, user supplied
int (*doFunctiona)(int, int);
int (*doFunctionb)(int, int);
if (option == "option1") {
doFunctiona = doFunction1a;
doFunctionb = doFunction1b;
} else if (option == "option2") {
doFunctiona = doFunction2a;
doFunctionb = doFunction2b;
} else if (option == "option3") {
doFunctiona = doFunction3a;
doFunctionb = doFunction3b;
}
for (int i = 0; i < 100000; i++) {
doFunctiona();
//  more code...
doFunctionb();
}

我意识到这对性能几乎没有影响(函数花费的时间占执行if语句所需时间的主导地位)。

然而，就"良好的编码实践"而言，这是设置变量函数调用的好方法吗？"好"的意思是：(1)易于扩展，未来可能有20种选择；2) 产生可读代码。我希望有某种标准的方法来实现这一点。如果没有，请随意以意见为基础结束。

只需使用unordered_map，并省去if-else-if-orgy:

std::unordered_map<std::string, std::vector<int (*)(int, int)>> functions;
functions.insert({ "option1", { doFunction1a, doFunction1b } });
...
const auto& vec = functions["option1"];
for(auto& f : vec) f(1, 2);

除了使用map之外，我建议使用std::function和lambdas，这将给您带来更大的灵活性，语法也更友好(至少对我来说)：

std::unordered_map<std::string, std::function<void()>> functions {
{
"option1", 
[] { 
functionA();
functionB();
}
},
{
"option2", 
[] { 
functionC();
functionD();
}
}
};
auto optionFuncIt = functions.find("option1");
if (optionFuncIt != functions.end()) {
optionFuncIt->second();
} else {
std::cerr << "Invalid option name" << std::endl;
}
相关文章：