[分享]：清除xdata 防止内存泄漏

fylinwater

以前论坛上摘录了一个清除xdata的函数，今天用用，发现有个隐蔽的内存泄漏

1. 
   
2. void removeXdata()
   
3. {
   
4.     ads_name eNam;
   
5.         ads_point pt;
   
6.         int i = acedEntSel ("\nPick an object:", eNam, pt);
   
7.         if (RTNORM != i)       
   
8.         {       
   
9.                 return;       
   
10.         }
    
11.         AcDbObjectId ObjId;
    
12.         acdbGetObjectId(ObjId, eNam);
    
13.         AcDbEntity *pEnt;
    
14.         acdbOpenAcDbEntity(pEnt, ObjId, AcDb::kForWrite);
    
15.         resbuf *xdata = pEnt->xData(NULL);
    
16.         if (xdata)       
    
17.         {
    
18.                 resbuf *nextbuf=xdata->rbnext;
    
19.                 xdata->rbnext = NULL;
    
20.                 pEnt->setXData(xdata);
    
21.                 acutRelRb(xdata);
    
22.                 acutRelRb(nextbuf);   [COLOR=red]//注意这里释放， 否则导致内存泄漏         [/COLOR]        
    
23.         }
    
24.        
    
25.         pEnt->close();
    
26. }
    

复制代码

lzxlzx

ObjectArx Samples中好象没有使
用acutRelRb(nextbuf);

ishou

楼主给的程序仍然有问题，该程序仅仅清除xdata中的第一个AppName对应的扩展数据。
实际上，实体(Entity)中可能不止包含一个AppName的扩展数据, 必须使用下面程序才能
彻底清除实体上的所有扩展数据：

1. 
   
2. AcDbObjectId ObjId;
   
3. acdbGetObjectId(ObjId, eNam);
   
4. AcDbEntity *pEnt;
   
5. resbuf  Tmp, *xp;
   
6. 
   
7. acdbOpenAcDbEntity(pEnt, ObjId, AcDb::kForWrite);
   
8. resbuf *xdata = pEnt->xData(NULL);
   
9. if (xdata)
   
10. {
    
11.       Tmp.rbnext = NULL;
    
12.       Tmp.restype = 1001;
    
13. 
    
14.        for (xp = xdata; xp; xp=xp->rbnext)
    
15.       {
    
16.              if (xp->restype  ==  1001)
    
17.              {
    
18.                      Tmp.resval.rstring = xp->resval.rstring;
    
19.                       pEnt->setXData(&Tmp);
    
20.              }
    
21.        }
    
22.       acutRelRb(xdata);
    
23. }
    
24.        
    
25. pEnt->close();
    
26. 
    

复制代码