C++ Указатели функций против подклассов

Question

Я могу выбирать между указателями функций и объектами подкласса. Чтобы было ясно, скажите, что я должен уведомить какой-то объект о каком-либо действии (например, таймер); обратитесь к следующим двум вариантам (очень базовый код для демонстрационных целей):

Version 1

typedef void TimerCallback(void *args); 
class Timer{ 
public: 
    Timer(); 
    ~Timer(); 
    void schedule(TimerCallback *callback, void *args, long timeout)=0; 
    void cancel(); 
}; 

Версию 2

class TimerTask{ 
    public: 
    TimerTask(); 
    virtual ~TimerTask(); 
    void timedout()=0; 
}; 
class Timer{ 
    public: 
    Timer(); 
    virtual ~Timer(); 
    void schedule(TimerTask *callback, long timeout)=0; 
    void cancel(); 
}; 

который один путь стандарта C++ и какой из них эффективен? Пожалуйста, дайте мне знать, если у вас есть другие предложения в этом отношении.

Пожалуйста, дайте мне знать, если я не понимаю в этом отношении.

Благодаря

Answer 1

Я бы сказал std::function и std::bind. Тогда не имеет значения, хотите ли вы использовать унаследованные классы, автономные функции, функции-члены или lambdas.

---

Кстати, если кому-то интересно, я сделал простое событие таймера обработки некоторое время назад, как ответ на другой вопрос. Это демонстрирует использование, например, std::function и std::bind: https://stackoverflow.com/a/11866539/440558.

Answer 2

Вы используете объектно-ориентированное программирование, и вы должны следовать объектно-ориентированной парадигме программирования.

На мой взгляд, использование объектов, а не указателей функций, является чище и вообще лучшим способом сделать.

Вы также можете попытаться использовать visitor pattern, чтобы сделать код еще лучше и более гибким.

Вы также можете рассмотреть publisher/subscriber pattern.

Answer 3

Я думаю, что лучше использовать boost(or std since C++11)::function для проведения обратного вызова и boost::bind, чтобы связать его аргументы или использовать boost::signal. Это было бы более общее и подробное решение по цене действительно небольшого штрафа.

http://www.boost.org/doc/libs/1_53_0/doc/html/signals2.html

Answer 4

Оба работают. Ваш первый - это «C-стиль» и где-то потребуется статическая функция. Вторая версия - «C++ style» и позволяет использовать экземпляр TimerTask.

Как правило, следует использовать версию 2, поскольку она устраняет необходимость в статической функции.

Answer 5

Функциональный указатель эффективно предотвращает использование замыканий - назначение методов обработчику событий (это не полностью true, но это будет ограничивать вас таким образом, что это решение не очень полезно).

Я бы проголосовал за объектно-ориентированный подход. Если вы используете C++ 11, вы можете значительно упростить код:

#include <cstdio> 
#include <functional> 

class Emitter 
{ 
private: 
    std::function<void(int)> eventHandler; 

public: 
    void SetEventHandler(std::function<void(int)> newEventHandler) 
    { 
     eventHandler = newEventHandler; 
    } 

    void EmitEvent() 
    { 
     eventHandler(42); // + error-checking 
    } 
}; 

class Handler 
{ 
private: 
    void HandleEvent(int i) 
    { 
     printf("Event handled with i == %d\n", i); 
    } 

public: 
    void AttachEmitter(Emitter & e) 
    { 
     e.SetEventHandler([this](int i) { HandleEvent(i); }); 
    } 
}; 

int main(int argc, char * argv[]) 
{ 
    Emitter e; 
    Handler h; 

    h.AttachEmitter(e); 
    e.EmitEvent(); 
}