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 trigger”但它不会编译：

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

现在，你可以传入普通的函数指针，指向成员的指针，lambdas，functor等作为你的回调。

这是更改代码的其余部分以使其工作的一种方法（注意：这不是唯一的方法;但是IMO具有通用性和灵活设计的easiest方式。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功能，我建议使用旧的虚拟基类接口方式，使用特定的纯虚方法作为鼠标甚至回调，并传递指向对象的指针而不是指向成员函数的指针。

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

Answer 2

Event1是一个成员函数，因此您需要将其限定为：

void newEvent(void (EventListener::*onRun)(), std::string eventName)
//                  ^^^^^^^^^^^^^^^

你称之为：

newEvent(&EventListener::Event1, "Event1");

使用函数指针时无效使用成员函数

2 个答案: