BREW优化代码以避免堆存储碎片

优化代码以避免堆存储碎片优化代码以避免堆存储碎片 define MSG SIZE 100 sizeof AECHAR static void CMyApp FragmentTheHeap CMyApp pme AECHAR p1 AECHAR p2 byte pDest pTemp AECHAR szFormat 10 STR TO WSTR s s szFormat sizeof szFormat pTemp AECHAR MALLOC MSG SIZE if pTemp return WSPRINTF pTemp MSG SIZE szFormat p1 p2 i WSTRLEN pTemp 1 sizeof AECHAR pDest WSTRDUP pTemp FREE pTemp 此代码有什么问题 尽管此代码实际上在初次分配时检查 NULL 但它会导致堆中出现存储碎片 即 孔 这可能使以后的分配无故失败 pTemp 的内存仅在分配新值 pDest 后释放 更佳的解 决方案如下所示 define MSG SIZE 100 sizeof AECHAR static void CMyApp FragmentTheHeap CMyApp pme AECHAR p1 AECHAR p2 byte pDest int nLen if p1 p2 return pDest AECHAR MALLOC WSTRLEN p1 WSTRLEN p2 1 sizeof AECHAR if pDest return WSTRCPY pDest p1 WSTRCAT pDest p2 所有这些错误都非常严重 简言之 它们将一个可用于显示消息框的简单函数转换成一个 引起异常的函数 检查检查 NULL 和不必要的分配和不必要的分配 Static void CMyApp HowManyMistakesCanIMakeInOneFunction CmyApp pme const AECHAR pszError AECHAR pMsg int i i WSTRLEN pszError 1 sizeof AECHAR pMsg AECHAR MALLOC i WSTRCPY pMsg pszError ISHELL MessageBox pme a m pIShell pMsg FREE pMsg 此代码有什么问题 此函数中有几个非常严重的错误 它包括 无法检查确保 pszError 在使用前为 NULL 无法检查 MALLOC 的 NULL 返回值 不需要时分配 pMsg pszError 值可以直接传递给 ISHELL MessageBox 所有这些错误都非常严重 简言之 它们将一个可用于显示消息框的简单函数转换成一个 失效的函数 BREW 应用程序必须检查内存分配例程的返回值 NULL 返回类实例的值等 不合理地分配大块不合理地分配大块 static void CMyApp AllocateTooMuch CMyApp pme const byte pIn int nSize int nNewSize nNewSize pme m nBuffFilled nSize if pme m nBuffSize nNewSize if nNewSize m nBuffSize 0 x4000 else pme m nBuffSize nNewSize 0X4000 while pme m pBuff REALLOC pme m pBuff pme m nBuffSize NULL pme m nBuffSize 0 x100 if pme m nBuffSize m nBuffSize nSize pme m pBuf