又一个内存泄漏

        排查小公司（对于国内的公司来说，很多都是如此）的代码，总有各种奇怪的惊喜，刚接手时还在感慨这个架构好， 我以前没学懂，这次要好好看懂。可是真运行起来发现错误的地方都是初学者常犯的错误，对这种架构高手的膜拜也就到此为止了。简化一下，放在这里，提醒一下。

代码不全，仅为说明
LoadFile.h

#pragma once
......   //#include 都省略

class LoadFile:
{
public:
	LoadFile();
	virtual ~LoadFile();
	size_t Length();
	const char* const Data();
	bool Load(const QString& strFilename);
	void Release();
protected:
	size_t m_nLength;
	char* m_pData;
};


LoadFile.cpp

LoadFile::LoadFile()
{
	m_pData = nullptr;
	m_nLength = 0;
}

LoadFile::~LoadFile()
{
}
size_t LoadFile::Length()
{
	return m_nLength;
}

const char * const LoadFile::Data()
{
	return m_pData;
}

bool LoadFile::Load(const QString& strFilename)
{
	bool bRet = false;
	
	ifstream is;
	is.open(strFilename.toLocal8Bit().constData(), ios_base::binary, ios_base::_Openprot);

	if (!is.fail())
	{

	    is.seekg(0,ios::end);
	    m_nLength = is.tellg();
	    is.seekg(LEN_FILEHEADER, ios::beg);
	    m_pData = new char[m_nLength];
	    is.read(m_pData, m_nLength);
	    is.close();
	}
	return bRet;
}
void LoadFile::Release()
{
	delete this;
}

内存泄漏就这样被高手给制造出来了，真心不想说什么

1 类里面有定义私有的指针，但是析构函数呢？？简直就是忽略最基本的概念。

2 Load函数看起来没问题，但是注意里面的new，如果该类实例化之后，只调用一次这个Load函数是没有问题的，但是如果你多次调用这个函数，就出问题了。

怎么修改就不讲了

 

代码review是需要的，一般来说，我总是看所有人的头文件，如果头文件里有指针，一定要仔细看实现及使用类的过程；如果没有指针，那么出现内存泄漏，只能说这样的开发没有好的习惯。

 

无论架构是否优秀，还需要做好扎实的基础开发，否则，耽误的是大家宝贵的时间。