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

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 语句所花费的时间）。

但是，就"good coding practices"而言，这是设置可变函数调用的好方法吗？使用 "good" 我的意思是：（1）易于扩展，将来很容易有 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);

Answer 2

除了使用 map，我建议使用 std::function 和 lambda，这会给你更多的灵活性，语法也更友好（至少对我而言）：

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;
}

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

Am I using function pointers correctly?

c++

function-pointers