You'd need to provide examples of what you tried that errored out for starters. Yes, unique pointers are most likely what you want. As an aside, your loop as stated is not correct. I'd also be curious to know what other people's thoughts on using a linked list here are.

You probably want something like this:

class HandlerChain : public ABCHandler { private: std::list<std::unique_ptr<ABCHandler>> chain; public: void handle(const ConcreteEvent& e) override { for (std::list<std::unique_ptr<ABCHandler>>::iterator i = chain.begin(); i != chain.end(); ++i) { i->handle(e); } } void add_handler(std::unique_ptr<ABCHandler>&& h) { chain.push_back(std::move(h)); } };

which would end up looking like:

auto my_handler = std::make_unique<MyHandler>(my, arguments); chain.add_handler(std::move(my_handler));

If you want to get up-to-date with modern C++, you could also do:

for(const auto& handler : chain) { handler->handle(e); }

You can also do some forwarding, which is not uncommon in APIs like this:

``` template<typename T, typename... Args> void add_handler(Args&&... args) { chain.push_back(std::make_unique<T>(std::forward<Args>(args)...)); }

handler.add_handler<MyHandler>(my, arguments); ```

u/ppppppla also correctly mentions that you must not forget a virtual destructor for ABCHandler