本周学习心得

1,《高质量C++/C 编程指南》第5章第4节讲到类中的常量时，作者是推荐用enum来实现

class  MyClass
{
public :
    MyClass()
    {
         for  ( int  i = 0 ; i < SIZE;  ++ i)
        {
            array[i]  =  i + 1 ;
        }
    }
     void  PrintArray()
    {
         for  ( int  i = 0 ; i < SIZE;  ++ i)
        {
            cout  <<  array[i]  << endl;
        }
    }
private :
     enum  { SIZE  =   100  };
     int  array[SIZE];
};

但我个人更喜欢用static const成员来实现：

class  MyClass
{
public :
    MyClass()
    {
         for  ( int  i = 0 ; i < SIZE;  ++ i)
        {
            array[i]  =  i + 1 ;
        }
    }

     void  PrintArray()
    {
         for  ( int  i = 0 ; i < SIZE;  ++ i)
        {
            cout  <<  array[i]  << endl;
        }
    }
private :
     static   const   int  SIZE  =   100 ;
     int  array[SIZE];
};

2,不要将正常值和错误标志混在一起返回，正常值用输出参数获得，错误标志用return语句返回。

3,在使用内存前检查指针是否为NULL

BYTE  * pBuffer  =   new  BYTE[len];
if  (pBuffer  ==  NULL)
{ // 分配内存失败
    delete [] pBuffer;
    pBuffer  =  NULL;  // 防止成为“野指针”
     return ;
} // end of if

4,如果函数的参数是一个指针，则不要指望用该指针去申请动态内存，下面示例中str依旧是NULL.原因在于调用函数时总是会为每个参数制作临时副本，指针也不例外，因此GetMemory里申请了内存，只是把str指针的副本所指的内存地址改变了，但str本身没变。而每执行一次GetMemory就会泄露一块内存，因为那个副本指针指向的内存没有free。

void  GetMemory( char   * p,  int  num)
{
    p  =  ( char * ) malloc( sizeof ( char )  *  num);
}

int  main()
{
     char   * str  =  NULL;
    GetMemory(str,  100 );
    strcpy(str,  " hello world " );
     return   0 ;
}

可以改成如下：

void  GetMemory( char   ** p,  int  num)
{
     * p  =  ( char * ) malloc( sizeof ( char )  *  num);
}

int  main()
{
     char   * str  =  NULL;
    GetMemory( & str,  100 );
    strcpy(str,  " hello world " );
    cout  <<  str  << endl;
    free(str);
    str  =  NULL;
     return   0 ;
}

5，在客户现场测试时发现ActiveX控件加载运行后，出现内存泄露的问题，几番测试后发现是是调用 CxImage库的Encode方法时犯了个错误，没有释放图像数据缓冲区，修改后代码如下：

CString Scan::EncodeImage()
{ // 对生成的图片进行Base64编码
    ZBase64 zBase;
     // 图片编码
    CxImage  image;    //  定义一个CxImage对象    
    image.Load( this -> m_strImgPath, CXIMAGE_FORMAT_JPG);    // 先装载jpg文件,需要指定文件类型
     long  size = 0 ; // 得到图像大小
    BYTE *  buffer = 0 ; // 存储图像数据的缓冲
    image.Encode(buffer,size,CXIMAGE_FORMAT_JPG); // 把image对象中的图像以type类型数据copy到buffer
     string  strTmpResult = zBase.Encode(buffer,size);  // 对生成的图片进行Base64编码
     if  (buffer  !=  NULL)
    { // 释放缓冲区
        delete [] buffer;
        buffer  =  NULL;
    }
// 返回编码后的图片数据
    CString result;
    result  =  strTmpResult.c_str();
     return  result;
}

6，《C++标准程序库》第6章第5节，运行时指定排序准则。

#include  < iostream >
#include  < set >
using   namespace  std;

template  < class  T >
class  RuntimeCmp
{
public :
     enum  cmp_mode {NORMAL,REVERSE};
private :
    cmp_mode mode;
public :
    RuntimeCmp(cmp_mode m  =  NORMAL) : mode(m)
    {

    }

     bool   operator  () ( const  T &  t1,  const  T &  t2)
    {
         return  mode  ==  NORMAL  ?  t1  <  t2 : t2  <  t1;
    }

     bool   operator   ==  ( const  RuntimeCmp &  rhs)
    {
         return  mode  ==  rhs.mode;
    }
};

typedef  set < int , RuntimeCmp < int >   >  IntSet;

void  fill(IntSet &   set )
{
     set .insert( 4 );
     set .insert( 7 );
     set .insert( 5 );
     set .insert( 1 );
     set .insert( 6 );
     set .insert( 2 );
     set .insert( 5 );
}
void  PrintElements(IntSet &   set )
{
    IntSet::iterator iter;
     for  (iter  =   set .begin(); iter  !=   set .end();  ++ iter)
    {
        cout  <<   * iter  <<  endl;
    }
}
int  main()
{
    IntSet coll1;
    fill(coll1);
    PrintElements(coll1);

    RuntimeCmp < int >  reverse_order(RuntimeCmp < int > ::REVERSE);

    IntSet coll2(reverse_order);
    fill(coll2);
    PrintElements(coll2);

    coll1  =  coll2;
    coll1.insert( 3 );
    PrintElements(coll1);

     if  (coll1.value_comp()  ==  coll2.value_comp())
    {
        cout  <<   " same sorting criterion "   <<  endl;
    }
     return   0 ;
}

 

7,我讨厌C++有两个原因，一个是字符串，一个是内存泄漏。这不，今天又犯了一个错误，复审代码时，读到下面的AllocSysString()时，需要调用方用SysFreeString来释放，而ActiveX控件嵌入IE后，在JavaScript代码中又该如何释放呢？

BSTR CCardScanCtrl::GetName( void )
{
    AFX_MANAGE_STATE(AfxGetStaticModuleState());
    CString strResult;
    strResult  =  Scan::getInstance().m_strName;
     return  strResult.AllocSysString();
}

  于是，改为如下代码：

BSTR CCardScanCtrl::GetName( void )
{
    AFX_MANAGE_STATE(AfxGetStaticModuleState());
    CString strResult;
    strResult  =  Scan::getInstance().m_strName;
    CComBSTR bstrResult(strResult);
     return  bstrResult;
}

可是，这样的后果是在JavaScript中访问Name属性时，出现内存访问错误的bug,原因就是CComBSTR的析构函数：

~ CComBSTR() 
{ 
::SysFreeString(m_str); 
}