第一次使用内存池,请大家指正

时间:2022-04-23 23:20:15
本人很菜,只是喜欢写代码,在实际工作中第一次用到内存池,紧张高兴之余发出来请大家指正
CPtrList m_listBuf; // 定义指针链表

struct BMPDATA // 数据
{
BYTE *pBuf;
SYSTEMTIME systime;//取当前时间
};

// 建立内存池
//m_BufSize是很大的1个未知数,我遇到的是5M,
int m_BufSize = 3 * nBmpWidth * nBmpHeight;
BYTE *pArrayBuf = new BYTE[10 * m_BufSize];//内存池有10个buf反复使用,10个就是50M内存
for(int i = 0; i < 10; i++)
{
BMPDATA *pDATA = new BMPDATA;
pDATA->pBuf = pArrayBuf + i * m_BufSize;
m_listBuf.AddTail(pDATA);
}

//使用内存池
//实际情况:此函数为回调函数,要求每秒执行无限接近15次,为了满足效率,当然此时CPU未100%,现CPU太厉害了
void OnCaptureBackDay(BYTE * pBuffer)
{
//............很多处理

if(/*条件满足*/)//此条件满足出现不规律
{
SaveToThread(BYTE * pBuffer);//这是个存盘操作,比较耗时
}

//............很多处理
}

void SaveToThread(BYTE * pBuffer)//存盘操作
{
static int idx = 0;
if(idx == 10)
{
idx = 0;
}
//接顺序取1块内存来用,假设10块内存足够使用
BMPDATA *pDATA = (BMPDATA *)(m_listBuf.GetAt(m_listBuf.FindIndex(idx++)));
memcpy(pDATA->pBuf, pBuffer, m_BufSize);//copy内存
GetLocalTime(&(pDATA->systime));//取时间
//扔到线程中不管了,存盘在其它线程中完成
CloseHandle(CreateThread(NULL, 0, ThreadFunc, pDATA, 0, NULL));
}

26 个解决方案

#1


你太谦虚了
呵呵
如果就一个线程的话应该没有什么需要注意的

#2


呵呵,有代码就先标记

#3


up up up 

#4


内存本来就是在池子里存取东西。
其他的仅仅是手段而已。
路过路过,传说而已!

#5


Mark.........

#6


建立16个MemPool,<=8字节的内存申请由0号MemPool分配,<=16字节的内存申请由1号MemPool分配,<=24字节的内存有2号MemPool分配,以此类推。最后,>128字节的内存申请由普通的malloc分配。

#7


if (m_pFreeNodeHeader == NULL)
    {
       const int nCount = m_nMemBlockSize/m_nItemSize;
        _MemBlock* pNewBlock = new _MemBlock;
        pNewBlock->data[0].pPrev = NULL;
        for (int i = 1; i < nCount; ++i)
            pNewBlock->data[i].pPrev = &pNewBlock->data[i-1];
        m_pFreeNodeHeader = &pNewBlock->data[nCount-1];
        pNewBlock->pPrev = m_pMemBlock;
        m_pMemBlock = pNewBlock;
    }
    void* pFreeNode = m_pFreeNodeHeader;
    m_pFreeNodeHeader = m_pFreeNodeHeader->pPrev;

#8


建立16个MemPool,<=8字节的内存申请由0号MemPool分配,<=16字节的内存申请由1号MemPool分配,<=24字节的内存有2号MemPool分配,以此类推。最后,>128字节的内存申请由普通的malloc分配。

if (m_pFreeNodeHeader == NULL)
  {
  const int nCount = m_nMemBlockSize/m_nItemSize;
  _MemBlock* pNewBlock = new _MemBlock;
  pNewBlock->data[0].pPrev = NULL;
  for (int i = 1; i < nCount; ++i)
  pNewBlock->data[i].pPrev = &pNewBlock->data[i-1];
  m_pFreeNodeHeader = &pNewBlock->data[nCount-1];
  pNewBlock->pPrev = m_pMemBlock;
  m_pMemBlock = pNewBlock;
  }
  void* pFreeNode = m_pFreeNodeHeader;
  m_pFreeNodeHeader = m_pFreeNodeHeader->pPrev;

#9


codeproject上有不少内存池的类,写的不错

#10


你太谦虚了

#11


up..

#12


标记!!!!
楼主水平比我高多了.

#13


学习来了··

#14


是来学习和接分的!!

#15


up ,标记!

#16


谦虚使人进步 
看不懂

#17


要是某种特殊情况下,一个5M的内存不够用会怎样,没看到健壮性代码,这时能将两块内存合一起使用么?

#18


我是来打酱油的

#19


不错。学习了

#20


引用 17 楼 xwsn007 的回复:
要是某种特殊情况下,一个5M的内存不够用会怎样,没看到健壮性代码,这时能将两块内存合一起使用么?

确实没有做,,,,,,
每次用5M,我一共建用了50M
用10个buf在实际测试中感觉已经够(基本是4块就足够用了),
不过话说回来,如果真不够用,就是说这10个Buf都已在使用,我再调用其中任何一块一定会出现问题的
如果要保证不出错的话现在想想应该很麻烦的,因为不知道哪块buf会第一个空闲出来,为了保证其他代码的健壮,目前先做这里,偷懒~

#21


    static int idx = 0;
    if(idx == 10)    
    {
        idx = 0;
    }
    idx++;
这样写应该是不安全的,好像应该是用一个就从链表头删除掉指针,用完了再add回链表尾,
每次使用链表的时候检查一下链表是否为空,如果不为空取链表头指针,这样呢??

#22


学习学习。。。

#23


引用 21 楼 delphigis 的回复:
static int idx = 0;
  if(idx == 10)  
  {
  idx = 0;
  }
  idx++;
这样写应该是不安全的,好像应该是用一个就从链表头删除掉指针,用完了再add回链表尾,
每次使用链表的时候检查一下链表是否为空,如果不为空取链表头指针,这样呢??


感觉用链表更好些,而且既然能判断一个内存块是否使用完毕,为何不加个使用标记呢?

#24


默认M块内存,如果这M块都在使用,再需要内存时,就new一个新的,加到链表里
最好M放在配置文件里,这样可灵活修改M值,而无需修改程序

#25


谢谢xwsn007,

就new一个新的

很受启发

#26


不过想想也不能无限new,如果出现别的问题,不停的new也会有问题

#1


你太谦虚了
呵呵
如果就一个线程的话应该没有什么需要注意的

#2


呵呵,有代码就先标记

#3


up up up 

#4


内存本来就是在池子里存取东西。
其他的仅仅是手段而已。
路过路过,传说而已!

#5


Mark.........

#6


建立16个MemPool,<=8字节的内存申请由0号MemPool分配,<=16字节的内存申请由1号MemPool分配,<=24字节的内存有2号MemPool分配,以此类推。最后,>128字节的内存申请由普通的malloc分配。

#7


if (m_pFreeNodeHeader == NULL)
    {
       const int nCount = m_nMemBlockSize/m_nItemSize;
        _MemBlock* pNewBlock = new _MemBlock;
        pNewBlock->data[0].pPrev = NULL;
        for (int i = 1; i < nCount; ++i)
            pNewBlock->data[i].pPrev = &pNewBlock->data[i-1];
        m_pFreeNodeHeader = &pNewBlock->data[nCount-1];
        pNewBlock->pPrev = m_pMemBlock;
        m_pMemBlock = pNewBlock;
    }
    void* pFreeNode = m_pFreeNodeHeader;
    m_pFreeNodeHeader = m_pFreeNodeHeader->pPrev;

#8


建立16个MemPool,<=8字节的内存申请由0号MemPool分配,<=16字节的内存申请由1号MemPool分配,<=24字节的内存有2号MemPool分配,以此类推。最后,>128字节的内存申请由普通的malloc分配。

if (m_pFreeNodeHeader == NULL)
  {
  const int nCount = m_nMemBlockSize/m_nItemSize;
  _MemBlock* pNewBlock = new _MemBlock;
  pNewBlock->data[0].pPrev = NULL;
  for (int i = 1; i < nCount; ++i)
  pNewBlock->data[i].pPrev = &pNewBlock->data[i-1];
  m_pFreeNodeHeader = &pNewBlock->data[nCount-1];
  pNewBlock->pPrev = m_pMemBlock;
  m_pMemBlock = pNewBlock;
  }
  void* pFreeNode = m_pFreeNodeHeader;
  m_pFreeNodeHeader = m_pFreeNodeHeader->pPrev;

#9


codeproject上有不少内存池的类,写的不错

#10


你太谦虚了

#11


up..

#12


标记!!!!
楼主水平比我高多了.

#13


学习来了··

#14


是来学习和接分的!!

#15


up ,标记!

#16


谦虚使人进步 
看不懂

#17


要是某种特殊情况下,一个5M的内存不够用会怎样,没看到健壮性代码,这时能将两块内存合一起使用么?

#18


我是来打酱油的

#19


不错。学习了

#20


引用 17 楼 xwsn007 的回复:
要是某种特殊情况下,一个5M的内存不够用会怎样,没看到健壮性代码,这时能将两块内存合一起使用么?

确实没有做,,,,,,
每次用5M,我一共建用了50M
用10个buf在实际测试中感觉已经够(基本是4块就足够用了),
不过话说回来,如果真不够用,就是说这10个Buf都已在使用,我再调用其中任何一块一定会出现问题的
如果要保证不出错的话现在想想应该很麻烦的,因为不知道哪块buf会第一个空闲出来,为了保证其他代码的健壮,目前先做这里,偷懒~

#21


    static int idx = 0;
    if(idx == 10)    
    {
        idx = 0;
    }
    idx++;
这样写应该是不安全的,好像应该是用一个就从链表头删除掉指针,用完了再add回链表尾,
每次使用链表的时候检查一下链表是否为空,如果不为空取链表头指针,这样呢??

#22


学习学习。。。

#23


引用 21 楼 delphigis 的回复:
static int idx = 0;
  if(idx == 10)  
  {
  idx = 0;
  }
  idx++;
这样写应该是不安全的,好像应该是用一个就从链表头删除掉指针,用完了再add回链表尾,
每次使用链表的时候检查一下链表是否为空,如果不为空取链表头指针,这样呢??


感觉用链表更好些,而且既然能判断一个内存块是否使用完毕,为何不加个使用标记呢?

#24


默认M块内存,如果这M块都在使用,再需要内存时,就new一个新的,加到链表里
最好M放在配置文件里,这样可灵活修改M值,而无需修改程序

#25


谢谢xwsn007,

就new一个新的

很受启发

#26


不过想想也不能无限new,如果出现别的问题,不停的new也会有问题