管道发送数据不断申请内存，另一边不断删除内存，会不会很耗性能？

jd808

1. void TServer::ReadEvent(Conn *pcon)
   
2. {
   
3.         if(pcon->is_down==1||pcon->m_Engine==NULL)return;
   
4. 
   
5.         char *p_cmd;
   
6.         int res;
   
7. 
   
8. 
   
9.         while ((res = pcon->m_Engine->ReadPack(&p_cmd))>0)
   
10.         {
    
11.                 S_CON_INFO s_info;
    
12.                 s_info.pcon= pcon;
    
13.                 s_info.run_code = 1;
    
14.                 s_info.fd=pcon->GetFd();
    
15.                 //读取句柄消息
    
16.                 s_info.data_buf = new char[res+1];
    
17.                 memcpy(s_info.data_buf, p_cmd, res+1);
    
18.                 s_info.data_buf_len=res;
    
19.                 int tid = rand()%m_ThreadCount;
    
20.                 if(write(m_pThreadInfo[tid].m_notifySendFd, &s_info, sizeof(S_CON_INFO))==-1){
    
21.                         printf("管道写入错误====pid:%d\n",tid);
    
22.                 }
    
23.         }
    
24. }

复制代码

这里s_info.data_buf = new char[res+1]; 不断new内存，管道另一边就得不断delete内存，是不是很浪费？有没有更好的办法
ReadEvent函数是libevent句柄触发的，只要有句柄有消息，就会调用，while ((res = pcon->m_Engine->ReadPack(&p_cmd))>0)主要负责将这个句柄的消息按照分包原则每个包读取出来，然后通过管道，向线程池哪里发送数据，线程池收到管道数据后，先拷贝数据，然后删除发送管道这边的内存，在继续处理数据罗辑部分。

lxyscls

分配一个固定大小的，里面带实际大小

jd808

lxyscls 发表于 2016-01-07 14:06
分配一个固定大小的，里面带实际大小

试过，不行啊，因为这边循环太快，管道读取的时候处理第一条的时候，写管道已经把第二条覆盖了，导致读的时候数据发送错误。

hellioncu

自己加一个内存池，重复利用

hanzhenlll

回复 3# jd808


    快速循环的操作，优先考虑 共享内存。。

实在需要FIFO的话，    FIFO本身只传递地址，  内存提前分配固定大小，每次使用标识一下

jd808

hanzhenlll 发表于 2016-01-08 18:33
回复 3# jd808

能不能给个列子看看？