C 高质量编程点滴

　一、您需要一个函数将一个数组赋值为等差数列，并将会在函数的外部使用他。　　

　　不合理：　

int *GetArray( int n ) 　　{ 　　 int *p = new int[n]; 　　 for ( int i = 0; i < n; i ) 　　 { 　 　　p[i] = i; 　　 } 　　 return p; 　　}

　　合理：　

void GetArray( int *p, int n ) 　　{ 　　 for ( int i = 0; i < n; i ) 　　 { 　　 p[i] = 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[i] = 3 / a[i]; 　　 } 　　 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[i] ) 　　 { 　　 return false; 　　 } 　　 PtrGuard .get()[i] = 3 / a[i]; 　　 } 　　 m_pPtr = PtrGuard.release(); 　　 return true; 　　}

　　解析：　　

　　在循环中，当参数数组a中的某一个值为0时，将会产生除0异常，那么，这将会导致您在上面为m_pPtr申请的内存不能合理的释放。为了解决这个问题，我们写了一个auto_array作为卫兵来看守企图逃逸的指针。在auto_array对象PtrGuard析构的时候他会同时删除附加在他身上的内存指针。我们首先用PtrGuard来进行任何的指针操作，在确定操作完全结束的最后，把指针再赋给真正的变量，并使PtrGuard放弃对该指针的附加，这样我们就得到了一个最安全的结果。另外需要注意的是，C 的STL库里本来有一个和auto_array功能很相似的模版类auto_ptr，但是他只支持单个对象的内存，不支持数组，写这样一个auto_array也是不得已而为之。

　　四、您需要开辟一段内存来存放和管理一个4 x 4的矩阵，并单位化之。

　　不合理：

int aMatrix[4][4]; 　　for ( int i = 0; i < 4; i ) 　　{ 　　 for ( int j = 0; j < 4; j ) 　　 { 　　 if ( i == j ) 　　 { 　　 aMatrix[i][j] = 1; 　　 } 　　 else 　　 { 　　 aMatrix [i][j] = 0; 　　 } 　　 } 　　}

　　合理：

标签：

版权申明：本站文章部分自网络，如有侵权，请联系：west999com@outlook.com
特别注意：本站所有转载文章言论不代表本站观点，本站所提供的摄影照片，插画，设计作品，如需使用，请与原作者联系，版权归原作者所有

int aMatrix[4 * 4]; 　　for ( int i = 0; i < 4; i ) 　　{ 　　 for ( int j = 0; j < 4; j ) 　　 { 　　 if ( i == j )　 　　 { 　　 aMatrix[ i * 4 j ] = 1; 　　 } 　　 else 　　 { 　　 aMatrix [ i * 4 j ] = 0; 　　 } 　　 } 　　}