1

我有一个功能

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 触发”，但它不能编译：

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

2 回答 2

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、仿函数等作为回调。

这是更改其余代码以使其工作的一种方法（注意：这不是唯一的方法；但它是easiestIMO 具有通用性和灵活设计的方法。完整的源代码在这里。）

将第一行更改为newEvent：

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

并这样称呼它：

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

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

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

于 2013-10-12T15:13:39.910 回答
0

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

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

你称之为：

newEvent(&EventListener::Event1, "Event1");
于 2013-10-12T14:51:09.533 回答