[讨论]：朋友们是如何处理 Acad::ErrorStatus 的?

李四

写健壮的程序是程序员追求的目标，然而我发现在arx开发中对每一个Acad::ErrorStatus进行处理似乎变得不可能。
请看下面的代码：

2. 
   
3. // Function name    : addEntityToModelSpace
   
4. // Description      : 将 AcDbEntity 添加到 ModelSpace
   
5. // Return type      : Acad::ErrorStatus
   
6. // Argument         : AcDbEntity* pEntity
   
7. // Argument         : BOOL bClose // AcDbEntity 加入ModelSpace后是否立即关闭
   
8. Acad::ErrorStatus addEntityToModelSpace(AcDbEntity* pEntity,BOOL bClose)
   
9. {
   
10.     Acad::ErrorStatus     es = Acad::eOk;
    
11.     AcDbBlockTable        *pBlockTable;
    
12.     AcDbBlockTableRecord  *pBlockTableRecord;
    
13.    
    
14.     /* es = */acdbHostApplicationServices()->workingDatabase()->getBlockTable(pBlockTable,AcDb::kForRead);
    
15.    
    
16.     /* es = */pBlockTable->close();
    
17.    
    
18.     /* es = */pBlockTable->getAt(ACDB_MODEL_SPACE, pBlockTableRecord,AcDb::kForWrite);
    
19.    
    
20.     es = pBlockTableRecord->appendAcDbEntity(pEntity);
    
21.    
    
22.     /* es = */pBlockTableRecord->close();
    
23.     if (bClose)
    
24.     {
    
25.         /* es = */pEntity->close();
    
26.     }
    
27.         
    
28.     return es;
    
29. }
    

<br /> // Function name    : addEntityToModelSpace<br /> // Description      : 将 AcDbEntity 添加到 ModelSpace<br /> // Return type      : Acad::ErrorStatus <br /> // Argument         : AcDbEntity* pEntity<br /> // Argument         : BOOL bClose // AcDbEntity 加入ModelSpace后是否立即关闭<br /> Acad::ErrorStatus addEntityToModelSpace(AcDbEntity* pEntity,BOOL bClose)<br /> {<br />     Acad::ErrorStatus     es = Acad::eOk;<br />     AcDbBlockTable        *pBlockTable;<br />     AcDbBlockTableRecord  *pBlockTableRecord;<br />     <br />     /* es = */acdbHostApplicationServices()->workingDatabase()->getBlockTable(pBlockTable,AcDb::kForRead);<br />     <br />     /* es = */pBlockTable->close();<br />     <br />     /* es = */pBlockTable->getAt(ACDB_MODEL_SPACE, pBlockTableRecord,AcDb::kForWrite);<br />     <br />     es = pBlockTableRecord->appendAcDbEntity(pEntity);<br />     <br />     /* es = */pBlockTableRecord->close();<br />     if (bClose)<br />     {<br />         /* es = */pEntity->close();<br />     }<br />         <br />     return es;<br /> }<br />

[/COLOR]
如果对每一个 es 进行处理，程序会变成怎样一个模样？！
代码的可读性会变得极差。
我只处理了一个。

朋友们你们通常是如何处理的？

:)
看了fyje 的C++ 异常有感，其实是困惑我多时的疑问.

XDSoft

最初由 李四 发布
[B]写健壮的程序是程序员追求的目标，然而我发现在arx开发中对每一个Acad::ErrorStatus进行处理似乎变得不可能。
请看下面的代码：
[code]
// Function name    : addEntityToModelSpace
// Descriptio... [/B]

我一般也是仅仅处理了 eOk ，进行两个分支，其他的都没有考虑。

比如对于打开对象操作，如果不返回eOk, 那么ACAD也没有运行下去的必要的：）

不过都是在程序完全调试好后的。

在调试的过程中，如果有问题，还是要看看es返回是什么的，比如wasOpenForWrite 等等

ymd

有时可能需要多看帮助或多测试
我在R14下遇到下列问题，
ads_handent(hand,en);
er=acdbGetObjectId(id, en);
if(er!=Acad::eOk) {
  return;//注1：此处必须！
  }
er=acdbOpenObject(pE, id, AcDb::kForRead);
//acdbOpenObject同样返回Acad::ErrorStatus
//若没有注1，只要hand合法，不管实体存在与否
//er均为Acad::eOk;

rave

用智能指针 smart pointer， 可以有效处理

i_want_2

rave 斑竹能否给段演示代码

rave

最初由 i_want_2 发布
[B]rave 斑竹能否给段演示代码 [/B]

1. 
   
2. [color=blue]#include "dbobjptr.h"[/color]  
   
3. //...
   
4. Acad::ErrorStatus addToModelSpace(AcDbEntity* pEntity,BOOL bClose)
   
5. {
   
6.         Acad::ErrorStatus es;
   
7. 
   
8.         [color=blue]AcDbBlockTableRecordPointer spBlkRcd(ACDB_MODEL_SPACE, acdbHostApplicationServices()->workingDatabase(), AcDb::kForWrite);[/color]
   
9.         es = spBlkRcd.openStatus();
   
10.         if (es == Acad::eOk){
    
11.                 es = spBlkRcd->appendAcDbEntity(pEntity);
    
12.                 if ((es == Acad::eOk) && (bClose))
    
13.                         es = pEntity->close();
    
14.         }
    
15. 
    
16.         return es;
    
17. }
    

复制代码

含错误处理，代码也比较简洁

照猫画虎

AcDbBlockTableRecordPointer 在那定义的，怎么没有啊