c++ - 使用函数指针时成员函数的使用无效

Question

我有一个函数

void newEvent(void (*onRun)(), std::string eventName)

我这样称呼

newEvent((*Event1)(), "Event1");

其中 Event1() 显然是一个 void 函数。这是我的构建输出:

/home/mrasicci/Programming/SSCE/main.cpp||In member function ‘void MenuState::enter()’:|
/home/mrasicci/Programming/SSCE/main.cpp|45|error: invalid use of member function (did you forget the ‘()’ ?)|
||=== Build finished: 1 errors, 0 warnings ===|

我已经尝试将 newEvent((*Event1)(), "Event1"); 更改为其他内容，例如 newEvent((*Event1), "Event1"); 或 newEvent(Event1(), "Event1"); 或 newEvent(Event1, "Event1"); 没有一个有效，所有这些都给出了不同的错误.这是给出错误的示例代码，预期的输出将是“Event1 triggered”但它没有编译:

#include <iostream>
#include <vector>

namespace TGE
{
    class EventListener
    {
        public:
            typedef struct
            {
                void (*run)();
                bool triggered;
                std::string name;
            } Event;

            void newEvent(void (*onRun)(), std::string eventName)
            {
                Event newEvent;
                newEvent.name = eventName;
                newEvent.triggered = false;
                newEvent.run = onRun;
                eventStack.push_back(newEvent);
            }

            void trigger(std::string eventName)
            {
                std::vector<Event>::iterator itr;

                for(itr = eventStack.begin(); itr != eventStack.end(); itr++)
                {
                    if(itr->name == eventName) itr->triggered = true;
                }
            }

        protected:
            std::vector<Event> eventStack;
    };
}

class MenuState : protected TGE::EventListener
{
    public:
        void enter()
        {
            newEvent((*Event1)(), "Event1");
            trigger("Event1");

            std::vector<Event>::iterator itr;
            for(itr = eventStack.begin(); itr != eventStack.end(); itr++)
            {
                if(itr->triggered)
                {
                    itr->run();
                    itr->triggered = false;
                }
            }
        }

        void Event1()
        {
            std::cout << "Event1";
        }
};

int main()
{
    MenuState* menuState = 0;
    menuState = new MenuState();
    menuState->enter();

    return 0;
}

Answer 1

像这样编写代码 newEvent((*Event1)(), "Event1"); 是错误的，因为您正在取消引用函数名称。

将其写为 newEvent(Event1(), "Event1"); 意味着您正在尝试将 void 函数的返回值用作函数指针。

说 newEvent(Event1, "Event1"); 只是稍微好一点，因为您将尝试将“指向成员函数的指针”作为“指向函数的指针”传递。由于成员函数需要实例才能运行，因此这也行不通。

我不知道你更大的架构是什么样的，但更通用的解决方案是使用 std::function 而不是简单的函数指针:

struct Event
{
    std::function<void()> run; // requires <functional>
    bool triggered;
    std::string name;
};

现在，您可以传入普通函数指针、指向成员的指针、lambda、仿函数等作为回调。

这是更改其余代码以使其工作的一种方法(注意:这不是唯一的方法；但它是 IMO 具有通用性和灵活设计的最简单方法。Full source code here. )

将 newEvent 的第一行更改为:

template <typename F>
void newEvent(F onRun, std::string eventName)
{...}

然后这样调用它:

newEvent([=](){Event1();}, "Event1");
//       ^^^^^^^^^^^^^^^^
//       This is a lambda

这些是 C++11 的特性。如果您不熟悉或不舒服或由于某种原因无法使用 C++11 功能，我建议使用旧的虚拟基类接口(interface)方式，使用特定的纯虚拟方法作为鼠标甚至回调，并将指针传递给对象而不是指向成员函数的指针。

您可以传递并稍后调用指向成员函数的指针，但您也需要一个指向该特定类的实例的指针才能调用它。我认为设计会很快变得复杂并且不值得，但如果您愿意，可以探索这条路线。