优化C++代码

Question

我正在为我的应用程序（网络工具）设计类。我这个基类：

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库。除非您完全确定您必须推出自己的I/O库，否则您只需使用现有的解决方案即可为您节省大量的开发优化时间。