如何减少事件总线实现中的耦合

Question

在我的应用程序中，我有几个模块不适合“is-a”或“has-a”关系，但仍然需要相互通信和传递数据。为了尝试松散地耦合这些模块，我实现了一个事件总线类，用于处理从“事件海报”到“事件侦听器”的消息传递。

如果类希望注册接收某些事件，它们可以实现< code>IEventListener。同样，如果需要将事件推到总线上，类可以调用< code > event bus::postEvent()。当调用< code>EventBus::update()时，EventBus处理预定消息的队列，并将它们路由到注册的侦听器。

EventBus.h

#pragma once

#include <queue>
#include <map>
#include <set>
#include <memory>


class IEvent
{
public:
    static enum EventType
    {
        EV_ENEMY_DIED,
        EV_ENEMY_SPAWNED,
        EV_GAME_OVER
    };

    virtual ~IEvent() {};
    virtual EventType getType() const = 0;
};


class IEventListener
{
public:
    virtual void handleEvent(IEvent * const e) = 0;
};


class EventBus
{
public:
    EventBus() {};
    ~EventBus() {};

    void update();
    void postEvent(std::unique_ptr<IEvent> &e);
    void registerListener(IEvent::EventType t, IEventListener *l);
    void removeListener(IEvent::EventType t, IEventListener *l);

private:
    std::queue<std::unique_ptr<IEvent>> m_eventBus;
    std::map<IEvent::EventType, std::set<IEventListener *>> m_routingTable;
};

事件总线.cpp

#include "EventBus.h"


using namespace std;


/**
 * Gives the EventBus a chance to dispatch and route events
 * Listener callbacks will be called from here
 */
void EventBus::update()
{
    while (!m_eventBus.empty())
    {
        // Get the next event (e_local now owns the on-heap event object)
        unique_ptr<IEvent> e_local(move(m_eventBus.front()));
        m_eventBus.pop();

        IEvent::EventType t = e_local->getType();
        auto it = m_routingTable.find(t);
        if (it != m_routingTable.end())
        {
            for (auto l : ((*it).second))
            {
                l->handleEvent(e_local.get());
            }
        }
    }
}

/**
 * Posts an event to the bus, for processing and dispatch later on
 * NB: The event bus will takes ownership of the on-heap event here
 */
void EventBus::postEvent(unique_ptr<IEvent> &e)
{
    // The EventBus now owns the object pointed to by e
    m_eventBus.push(unique_ptr<IEvent>(move(e)));
}

/**
 * Registers a listener against an event type
 */
void EventBus::registerListener(IEvent::EventType t, IEventListener *l)
{
    // Add this listener entry
    // If the routing table doesn't have an entry for t, std::map.operator[] will add one
    // If the listener is alredy registered std::set.insert() won't do anything
    m_routingTable[t].insert(l);
}

/**
 * Removes a listener from the event routing table
 */
void EventBus::removeListener(IEvent::EventType t, IEventListener *l)
{
    // Check if an entry for event t exists
    auto keyIterator = m_routingTable.find(t);
    if (keyIterator != m_routingTable.end())
    {
        // Remove the given listener if it exists in the set
        m_routingTable[t].erase(l);
    }
}

如您所见，在我当前的实现中，我为要传递的每种类型的事件创建具体的 IEvent 实现。我这样做是为了使每个事件都可以附加自定义数据（这是我的情况的要求）。不幸的是，这意味着我的 EventBus 系统必须了解系统的所有用户，从而增加了我的 EventBus 类与该类的用户之间的耦合。此外，IEvent 接口需要将所有事件类型的列表保存为枚举，这具有相同的问题（增加耦合）。

1. 有没有一种方法可以修改这个实现，使EventBus完全通用（不需要知道EventBus的用户），但仍然允许我在每个事件中传递自定义数据？我研究了C11可变模板函数，但不知道如何在这种情况下使用它们
2. 作为一个附带问题，我在这里是否正确使用了std:：unique_ptr

Answer 1

问题1“有没有办法修改这个实现，使EventBus完全通用”:

简短的回答，是的。

更长的回答:有很多html" target="_blank">方法可以实现这一点。这里描述一个:

事件的生产者和消费者都需要就类型/数据达成一致，但EventBus本身不需要知道。实现这一点的一种方法是使用＜code＞boost：：signals2:：signal

但是，这也是可以补救的。通过使事件类型＜code＞EventBus:：postEvent（）＜/code＞将＜code＞std：：function作为参数

boost::signals2::signal<int> signal;
...
eventbus.postEvent(boost::bind(signal, 42));
// note: we need to use boost::bind (not std::bind) for boost::signals to be happy

＜code＞EventBus＜/code＞将看到＜code＞std:：函数

问题2“我是否正确使用了< code>std::unique_ptr”:

几乎我将删除对＜code＞EventBus:：postEvent＜/code＞的引用，使其：

void EventBus::postEvent(std::unique_ptr<IEvent> e);

通过这样做，您可以强制调用者主动移动< code>std::unique_ptr

氯吡咯烷酮指南 R.32：

“使用unique_ptr参数表示函数拥有小部件的所有权”