下面介绍C++高质量编程。
一、你需要一个函数将一个数组赋值为等差数列，并将会在函数的外部使用它。
不合理：
int *GetArray( int n ) - { int *p = new int[n];
- for ( int i = 0; i < n; i++ ) {
- p = i; }
- return p;
- }
合理：
void GetArray( int *p, int n ) - { for ( int i = 0; i < n; i++ )
- { p = i;
- }
- }
解析：
检查内存泄露的最好办法，就是检查完全配对的申请和释放，在函数中申请而在外部释放，将导致代码的一致性变差，难以维护。而且，你写的函数不一定是你自己使用的，这样的函数别人会不知道该怎么适当的使用，如果它是一个DLL的导出函数，并且你在不同的平台下使用了，便会导致系统崩溃。最好的解决办法就是在函数调用的外面将内存申请好，函数只对数据进行复制。
二、你需要写一个类来为你管理一个指针，这个类将封装对指针的申请内存、释放和其它一些基本操作。
不合理：
class A - { public:
- A( void ) {} ~A( void ) { delete []m_pPtr; }
- void Create( int n ){ m_pPtr = new int[n]; } private:
- int *m_pPtr;
- };
合理：
class A - { public:
- A( void ) : m_pPtr(0){} ~A( void ) { Clear(); }
- bool Create( int n ){ if ( m_pPtr ) return false; m_pPtr = new int[n]; return ture; } void Clear( void ) { delete []m_pPtr; m_pPtr = 0; }
- private: int *m_pPtr;
- };
解析：
不合理的代码就在于当你重复调用Create的时候就会造成内存泄露，解决的办法就是在new之前判断一下指针是否为0。要能够有效的执行这个判断，则必须在构造的时候对指针进行初始化，并为这个类添加一个Clear函数来释放内存。
三、接上题的Create函数，你现在需要根据传入的参数做一些比较复杂的算法操作，并对申请的数组赋值。
不合理：
bool Create(int *a, int n ) - { if ( m_pPtr )
- return false; m_pPtr = new int[n];
- for ( int i = 0; i < n; i++ ) {
- m_pPtr = 3 / a; }
- return true;
- }
合理：
template - class auto_array {
- public: explicit auto_array(_Ty *pPtr=0)throw():m_Ptr(pPtr){}
- ~auto_array(){delete[]m_Ptr;} void reset(_Ty *pPtr=0){if(pPtr!=m_Ptr){delete[]m_Ptr;m_Ptr=pPtr;}}
- _Ty* release(void){_Ty *pTemp=m_Ptr;m_Ptr=0;return pTemp;} private:
- auto_array(const auto_array&other){} auto_array& operator=(const auto_array& other){}
- _Ty *m_Ptr; };
- bool A::Create(int *a, int n ) {
- if ( m_pPtr ) return false;
- auto_array PtrGuard( new int[n] ); for ( int i = 0; i < n; i++ )
- { if ( 0 == a )
- { return false;
- } PtrGuard .get() = 3 / a;
- } m_pPtr = PtrGuard.release();
- return true;
- }
|
