c++ - 我是否正确使用函数指针？

Question

我有一个伪代码看起来像这样的函数:

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)易于扩展， future 很容易有 20 个选项； 2) 生成可读代码。我希望存在某种标准方法来实现这一点。如果没有，请随时根据意见关闭。

Answer 1

只需使用 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);