有关： N T ALMOND 写的CBuffer 类存在的问题

有关：  N T ALMOND  写的CBuffer 类应用的很广泛
对于这个类的实现，本人有很大的异议


如果不清楚，可以google 搜索 N T ALMOND 获得约 303,000 条结果
搜索 N T ALMOND Buffer 获得约 23,600 条结果
可见应用多少广泛。

 

或者上codeproject 找完成端口的例子。

 

我觉得他这个类在内存COPY时根本没有考虑到性能问题。明明只需要COPY  1  字节，可能COPY了1024字节。

而这样的类应用在多线程的完成端口中。能不有影响吗？

UINT CBuffer::ReAllocateBuffer(UINT nRequestedSize)
{
 if (nRequestedSize < GetMemSize())
  return 0;

 // Allocate new size
 UINT nNewSize = (UINT) ceil(nRequestedSize / 1024.0) * 1024;

 // New Copy Data Over
 PBYTE pNewBuffer = (PBYTE) VirtualAlloc(NULL,nNewSize,MEM_COMMIT,PAGE_READWRITE);

 UINT nBufferLen = GetBufferLen();
 CopyMemory(pNewBuffer,m_pBase,nBufferLen);

 if (m_pBase)
  VirtualFree(m_pBase,0,MEM_RELEASE);

 // Hand over the pointer
 m_pBase = pNewBuffer;

 // Realign position pointer
 m_pPtr = m_pBase + nBufferLen;

 m_nSize = nNewSize;

 return m_nSize;
}

这断代码很好，我没有异议。

但对于他的Read 的实现，有异议。
UINT CBuffer::Read(PBYTE pData, UINT nSize)
{
 // Trying to byte off more than ya can chew - eh?
 if (nSize > GetMemSize())
  return 0;

 // all that we have
 if (nSize > GetBufferLen())
  nSize = GetBufferLen();

  
 if (nSize)
 {
  // Copy over required amount and its not up to us
  // to terminate the buffer - got that!!!
  CopyMemory(pData,m_pBase,nSize);
  
  // Slide the buffer back - like sinking the data
  MoveMemory(m_pBase,m_pBase+nSize,GetMemSize() - nSize);

  m_pPtr -= nSize;
 }

 DeAllocateBuffer(GetBufferLen());

 return nSize;
}

1。 
 // all that we have
 if (nSize > GetBufferLen())
  nSize = GetBufferLen();

如果想要读取比当前已有数据更长的数据，这样的行为是没有意思，而且肯定是错误的。

而此实现，只是为了安全。却没有检查错误操作的作用。

2。 // Slide the buffer back - like sinking the data
  MoveMemory(m_pBase,m_pBase+nSize,GetMemSize() - nSize);
这是完全没有必要。移动与COPY内存在实现上相差无几，但为什么要移动哪些根本没有用过的内存。

GetMemSize() - nSize  必须改为：GetBufferLen() - nSize 可以改进性能。

 

另外可以取消GetBufferLen()这个API。