排查小公司(对于国内的公司来说,很多都是如此)的代码,总有各种奇怪的惊喜,刚接手时还在感慨这个架构好, 我以前没学懂,这次要好好看懂。可是真运行起来发现错误的地方都是初学者常犯的错误,对这种架构高手的膜拜也就到此为止了。简化一下,放在这里,提醒一下。
代码不全,仅为说明
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是需要的,一般来说,我总是看所有人的头文件,如果头文件里有指针,一定要仔细看实现及使用类的过程;如果没有指针,那么出现内存泄漏,只能说这样的开发没有好的习惯。
无论架构是否优秀,还需要做好扎实的基础开发,否则,耽误的是大家宝贵的时间。