[讨论]trans的两种用法

ahopera

在Kean的blog里，他喜欢采用

2. using(trans)
3. {
4. ...........
5. }

using(trans) { ........... }

我在CAD的DTV里，发现负责讲解的工程师，多采用

1. dim trans as  ......=
2. try
4.     'if success then commit trans
5.     trans.commit()
6. catch
7.     'add some code
8. finally
9.     'if get error then dispose trans
10.     trans.dispose()
11. end try

复制代码

谈一下我对两者的看法，using的好处是可以省掉一个trans.dispose()，但用try.....end try 则可以对错误进行判断
在我对ARX函数的学习过程中，发现对图元操作的函数，往住是带一个状态返回值。在.Net Api写进行arx托管程序时，我们一般采用对objectid的操作，需要大量使用trans，一般创建图元需要返回图元对象或objectid，操作对象则不需要返回值。
例如对图元移动，则可以用
sub MoveEnt(EntID as Objectid,StartPnt as Point3d,EndPnt as Point3d)
但是，有的这个函数会出错，例如需要对图形中的一些定义过的图元进行操作。
比如：你创建了一些曲线，把曲线的handler存在字典里，在以后用的时候把handler转换成objectid进行使用。有时候用户在操作图形的时候把这些曲线的图层锁定了，当你操作这一系列图元时，会导致一个eonlockedlayer的错误，如果用using trans,会直接出错然后退出。但是，很可能这些曲线是你经过检查后认为不需要操作或不能操作才锁定的，同时即使出现错误也可以进行对其它图元操作，你的程序将无法达到你想要的目的。
而如果你用了try的话，完全可以将过程（C#里的void Function）写成一个函数:
Function MoveEnt(EntID as Objectid,StartPnt as Point3d,EndPnt as Point3d) As Boolean
在转换时可以在这样写

1. try
2.     ent=trans.GetObject
3.     ''Add MoveEntCode
4.      trans.Commit()
5.     Return True '成功变换返回真
6. Catch
7.     Return False '变换失败返回假
8. Finally
9.     trans.Dispose()
10. End Try

复制代码

我觉得在逻辑上，用try要严密些
同时在调用上，也可以采用无返回值的调用，也可以采用有返回值的调用。

雪山飞狐_lzh

using语句内部一样可以用try的

2.         [CommandMethod("mm")]
3.         static public void MMove()
4.         {
5.             Document doc = Application.DocumentManager.MdiActiveDocument;
6.             Database db = doc.Database;
7.             Editor ed = doc.Editor;
8.             PromptSelectionResult res = ed.GetSelection();
9.             if (res.Status == PromptStatus.OK)
10.             {
11.                 using (Transaction tr = db.TransactionManager.StartTransaction())
12.                 {
13.                     foreach (ObjectId id in res.Value.GetObjectIds())
14.                     {
15.                         Move(
16.                             (Entity)id.GetObject(OpenMode.ForRead),
17.                             Point3d.Origin,
18.                             new Point3d(10,10,0)
19.                             );
20.                     }
21.                     tr.Commit();
22.                 }
23.             }
24.         }
26.         static public bool Move(Entity ent, Point3d startPoint, Point3d endPoint)
27.         {
28.             try
29.             {
30.                 ent.UpgradeOpen();
31.                 ent.TransformBy(Matrix3d.Displacement(endPoint - startPoint));
32.             }
33.             catch
34.             {
35.                 return false;
36.             }
37.             return true;
38.         }

        [CommandMethod("mm")]         static public void MMove()         {             Document doc = Application.DocumentManager.MdiActiveDocument;             Database db = doc.Database;             Editor ed = doc.Editor;             PromptSelectionResult res = ed.GetSelection();             if (res.Status == PromptStatus.OK)             {                 using (Transaction tr = db.TransactionManager.StartTransaction())                 {                     foreach (ObjectId id in res.Value.GetObjectIds())                     {                         Move(                             (Entity)id.GetObject(OpenMode.ForRead),                             Point3d.Origin,                             new Point3d(10,10,0)                             );                     }                     tr.Commit();                 }             }         }         static public bool Move(Entity ent, Point3d startPoint, Point3d endPoint)         {             try             {                 ent.UpgradeOpen();                 ent.TransformBy(Matrix3d.Displacement(endPoint - startPoint));             }             catch             {                 return false;             }             return true;         }

ahopera

那是,我只是觉得,当封装在函数里的时候,用using就会重复了,用我的想法,是

2.         [CommandMethod("mm")]
3.         static public void MMove()
4.         {
5.             Document doc = Application.DocumentManager.MdiActiveDocument;
6.             Database db = doc.Database;
7.             Editor ed = doc.Editor;
8.             PromptSelectionResult res = ed.GetSelection();
9.             if (res.Status == PromptStatus.OK)
10.             {
11.                 foreach (ObjectId id in res.Value.GetObjectIds())
12.                 {
13.                     Move(id,
14.                         Point3d.Origin,
15.                         new Point3d(10, 10, 0)
16.                         );
17.                 }
18.              }
19.         }
20.         static public bool Move(ObjectId entid, Point3d startPoint, Point3d endPoint)
21.         {
22.             Transaction tr = entid.Database.TransactionManager.StartTransaction();
23.             try
24.             {
25.                 Entity ent = (Entity)tr.GetObject(entid, OpenMode.ForWrite);
26.                 ent.TransformBy(Matrix3d.Displacement(endPoint - startPoint));
27.                 tr.Commit();
28.                 return true;
29.             }
30.             catch
31.             {
32.                 return false;
33.             }
34.             finally
35.             {
36.                 tr.Dispose();
37.             }
38.         }

复制代码

上面的的缺点是trans重新用了n次,效率较差,但是封装性较高
学到了一点,我用Objectid.GetObject总是出错,原来是要放在trans里面