有效防止代码内存泄漏的方法

　　这样行了，解决上面的情况。让成员变量成为const指针，这样设计也合理，避免指针无意被改变。

            Image * const pImg;          //图像
            Voice   * const pVoi;        //声音

　　那么这样只能用成员初始化列表为指针初始化，没有其他地方可以给const指针赋值了。

People::People( const std::string& n，const int& a，const int& h，const std::stirng& imgFileName，const std::string& vFileName )
:name(n)，age(a)，height(h)，
pImg( imgFileName !="" ? new Image( imgFileName ) : 0 )，
pVoi( vFileName != "" ? new Voice( vFileName ) : 0)
{}

　　如果这样重新回到上面所遇到的问题，即构造过程中抛出异常，指针可能无法正确的释放所占内存。那么我们可以进一步对代码进行改进，如下：

People::People( const std::string& n，const int& a，const int& h，const std::stirng& imgFileName，const std::string& vFileName )
:name(n)，age(a)，height(h)，
pImg( initImage( imgFileName ) )，
pVoi(  initVoice( vFileName ) )
{}

Image* People::initImage(const string& imgFileName)
{
    if(imgFileName !="") return new Image(imgFileName);
    else return 0;
}

Voice* People::initVoice(const string& vFileName)
{
    try
    {
        if(vFileName !="")return new Voice(vFileName)
        esle return 0;
    }
    catch(... )
    {
            delete pImg;
            throw;
    }
}
 

　　这样在调用构建Voice对象中加入try...catch...用于释放pImg所占用的内存空间。其实有一个比其更简单的方法是使用智能指针。

const auto_ptr<Image> pImg;
const auto_ptr<Voice>  pVoi;

People::People( const std::string& n，const int& a，const int& h，const std::stirng& imgFileName，const std::string& vFileName )
:name(n)，age(a)，height(h)，
pImg( imgFileName !="" ? new Image( imgFileName ) : 0 )，
pVoi( vFileName != "" ? new Voice( vFileName ) : 0)
{}
 

　　那么问题算解决了，因为当其中有一个创建失败，离开函数的时候，智能指针会自动删除已经创建的空间，防止内存泄漏了。