Оптимизация кода на С ++

Question

Я разрабатываю классы для своего приложения (сетевой инструмент). I этот базовый класс:

class Descriptor 
{ 
    // ... 

    public: 
     virtual void data_read (void); 
     virtual void data_write (void); 
     virtual void data_error (void); 

    protected: 
     int socket_descriptor; 
    // ... 
} 

class TcpClient : 
    public Descriptor 
{ 
    // ... 
} 

Многие классы основаны на дескрипторе. Я отслеживаю события сокетов с помощью epoll. Когда я хочу, чтобы посмотреть события на объекте TcpClient добавить сокет и указатель объекта для этого объекта Epoll, код:

epoll_event descriptor_events; 

descriptor_events.events = EPOLLIN; 
descriptor_events.data.fd = this->socket_descriptor; 
descriptor_events.data.ptr = this; 

epoll_ctl (epoll_descriptor, EPOLL_CTL_ADD, this->socket_descriptor, &descriptor_events); 

рассылает Epoll событие в отдельном потоке таким образом:

Descriptor *descriptor (NULL); 

// ... 

return_value = epoll_wait (epoll_descriptor, events, 64, -1); 

while (i < return_value) 
{ 
    descriptor = (Descriptor *) (events [i]).data.ptr; 

    if ((events [i]).events & EPOLLOUT) 
     descriptor->data_write(); 
    if ((events [i]).events & EPOLLIN) 
     descriptor->data_read(); 
    if ((events [i]).events & EPOLLERR) 
     descriptor->data_error(); 

    i++; 
} 

Программа будет обрабатывать множество данных в потоке epoll, так что это означает, что виртуальные функции будут называться много раз там. Мне интересно о стоимости выполнения этого решения.

Я также думаю о двух других реализаций (однако я не уверен, если the're гораздо быстрее):

typedef void (*function) (Descriptor *) EventFunction; 

class Descriptor 
{ 
    // ... 

    public: 
     EventFunction data_read; 
     EventFunction data_write; 
     EventFunction data_error; 

    protected: 
     Descriptor (EventFunction data_read, 
        EventFunction data_write, 
        EventFunction data_error); 

     int socket_descriptor; 
    // ... 
} 

или использовать CRTP.

Возможно, у вас есть другие идеи по реализации этого?

Answer 1

Если не доказано иное, ваш оригинальный дизайн выглядит хорошо для меня.

Первое правило оптимизации - сначала измерить, а затем исправить только горячие точки, которые действительно существуют. Вы будете удивлены, когда ваш код проводит свое время. Остановить различие между виртуальными функциями и указателями функций - это, безусловно, преждевременная оптимизация. В обоих случаях компилятор будет генерировать код для перехода к указателю на функцию, хотя с виртуальными функциями компилятору придется сначала искать vtable. Напишите идиоматический код на C++, чтобы делать то, что вы хотите, а затем профиль его, если у вас проблемы с производительностью.

(у меня есть один комментарий о классе Descriptor:. Если вы не планируете иметь общий data_read(), data_write(), и data_error() методы, которые я рекомендовал бы сделать их чисто виртуальные методы)

Answer 2

Под зонтиком don't reinvent the wheel я бы предложил посмотреть на Boost.Asio, так как он предлагает большую часть функциональности, описанную в вашем примере кода.

Answer 3

Честно говоря, ваш лучший выбор для оптимизации этого кода, вероятно, полностью заменить его Boost ASIO. Как описано, вы по существу перепрофилируете тщательно изученную и хорошо протестированную библиотеку ASIO. Если вы не уверены в том, что вам нужно перевернуть свою собственную библиотеку ввода-вывода, вы, вероятно, сэкономите себе огромное количество времени разработки &, просто используя существующее решение.