几个内存泄漏的例子

几个内存泄漏的例子

✧new和delete要成对使用

✧new和delete要匹配

经常看到一些C++方面的书籍中这样提及到内存泄漏问题，这样的说法的意思是比较明白，但对于初学C++程序员还是很难掌握，所以下面举几个反面的例子，希望对大家有帮助。

例一：错误处理流程中的return导致的内存泄漏

bool MyFun()

{

CMyObject* pObj = NULL;

pObj = new CMyObject();

…

if (…)

return false;

…

if(…)

return false;

…

if (pObj != NULL)

delete pObj;

return true;

}

注意：红色字体部分的return之前没有释放pObj，导致内存泄漏。

例二：exception改变了程序的正常流程，导致内存泄漏

情况1：

HRESULT MyFun()

{

HRESULT hr = S_OK;

try

{

CMyObject* pObj = NULL;

pObj = new CMyObject();

…

if (…)

{

hr = E_FAIL;

throw hr;

}

…

if(…)

{

hr = E_FAIL;

throw hr;

}

…

if (pObj != NULL)

delete pObj;

}

catch (HRESULT& eHr)

{

}

return hr;

}

情况2：

void OtherFun() // 可能是自己写的其他函数；

// 也可能是其他人写的函数；

// 也可能是系统的API；

{

…

if(…)

throw exception;

…

}

bool MyFun()

{

CMyObject* pObj = NULL;

pObj = new CMyObject();

…

OtherFun();

…

if (pObj != NULL)

delete pObj;

return true;

}

注意：上面的两种情况中的throw行为将导致程序的正常流程，一旦有throw的动作发生，pObj对象将不会被正确释放（delete）。

例三：忘记释放系统API创建的资源，导致内存泄露

bool CMyClass::MyFun()

{

HANDLE hHandle = CreateEvent(NULL,FALSE,TRUE,NULL);

…

if (…)

return false;

…

return true;

}

注意：系统API CreateEvent创建的HANDLE对象，需要自己释放，否则会导致内存泄漏。还有其他一些系统的API也是如此，如：CreateFile、CreateFileMapping 等等，所以我们在使用不熟悉的系统API时，一定要仔细阅读MSDN。

例四：PostMessage可能导致的内存泄漏

// 自定义的消息结构体

typedef struct tagMSG

{

int i;

float f;

}MSG;

// 发送消息的函数

void MyFun()

{

MSG* pMsg = new MSG;

…

PostMessage(m_hWnd, WM_MYEVENT, (WPARAM)pMsg, 0);

}

// 接收消息的函数

afx_msg void OnMessage(WPARAM wParam, LPARAM lParam)

{

MSG* pMsg = (MSG*)wParam;

m_i = pMsg->i;

m_f = pMsg->f;

}

注意：OnMessage函数中忘记释放pMsg，导致内存泄漏。

例五：函数返回new的对象而导致的内存泄漏

char* MyFun()

{

char* p = new char[10];

…

return p;

}

注意：调用MyFun程序的人可能不会注意到MyFun函数内部new出的对象，往往会忽略对象p的释放。

例六：不好的类结构也会导致内存泄漏

// MyClass.h文件

class MyClass

{

public:

MyClass();

virtual ~MyClass();

BOOL Init();

BOOL UnInit();

private:

char* m_pStr;

}

// MyClass.cpp文件MyClass::MyClass()

: m_pStr(NULL)

{

}

MyClass::~MyClass()

{

}

BOOL MyClass::Init()

{

m_pStr = new char[100];

…

if (…)

{

delete m_pStr;

m_pStr = NULL;

return FALSE;

}

return TRUE;

}

BOOL MyClass::UnInit()

{

if (m_pStr != NULL)

{

delete m_pStr;

m_pStr = NULL;

}

return TRUE;

}