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

本人很菜，只是喜欢写代码，在实际工作中第一次用到内存池，紧张高兴之余发出来请大家指正

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也会有问题