高质量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; &nb 相关新闻>>最新推荐更多>>>
|