集合对象的相互转换

雪山飞狐_lzh

哎，你不能把一个实体两次重复的加入数据库
这里加入一次
                ObjectId c1 = AddCircle(new Point3d(0, 0, 0), 10);
                ObjectId c2 = AddCircle(new Point3d(0, 0, 0), 15);

接着又加了一次
                CreateBlockDef("Test", objs);

上一个帖子的代码我不是直接贴了吗，汗

chpmould

你可以帮我调通吗?绘制圆的方式还是按以下方式...
ObjectId c1 = AddCircle(new Point3d(0, 0, 0), 10);
ObjectId c2 = AddCircle(new Point3d(0, 0, 0), 15);

河伯

Linq是个好东西，还没来得及研究哪。不过，这个好像是Lambda表达式，和Linq没关系吧？
Ids.Cast(Of ObjectId).ToList().ForEach(Sub(id) objs.Add(id.GetObject(OpenMode.ForRead)))

雪山飞狐_lzh

Cast、ToList就是System.Linq的扩展了
Linq和Lambda结合的还是挺多的

不过我的想法是
            ids
            .Cast<ObjectId>()
            .Select(id => id.GetObject(OpenMode.ForRead));
只返回一个查询，这个用到的更多些

河伯

2.         Dim lstObj As New List(Of T)
   
3.         Dim objs As New DBObjectCollection
   
4.         Using Trans As Transaction = db.TransactionManager.StartTransaction
   
5.             Array.ForEach(Ids, Sub(Id) objs.Add(Id.GetObject(OpenMode.ForRead)))
   
6.             lstObj = objs.OfType(Of T).ToList
   
7.             Trans.Commit()
   
8.         End Using
   

        Dim lstObj As New List(Of T)<br />         Dim objs As New DBObjectCollection<br />         Using Trans As Transaction = db.TransactionManager.StartTransaction<br />             Array.ForEach(Ids, Sub(Id) objs.Add(Id.GetObject(OpenMode.ForRead)))<br />             lstObj = objs.OfType(Of T).ToList<br />             Trans.Commit()<br />         End Using<br />

很好！前面函数又改了一下，果然简洁不少，看来用.NET真要有点新思维。

雪山飞狐_lzh

你还是写复杂了，呵呵
建议的方式，在子函数里不要新建事务，如果处理的实体较多，每开一个事务就会占耗时
正确的做法是传入事务作为参数，

2.         [CommandMethod("tttt")]
   
3.         public void tttt()
   
4.         {
   
5.             var db = HostApplicationServices.WorkingDatabase;
   
6.             var doc = Application.DocumentManager.GetDocument(db);
   
7.             var ed = doc.Editor;
   
8. 
   
9.             using (Transaction tr = doc.TransactionManager.StartTransaction())
   
10.             {
    
11. 
    
12.                 ObjectIdCollection ids = new ObjectIdCollection();
    
13.                 var cirs = GetEntitys<Circle>(tr, ids.Cast<ObjectId>());
    
14. 
    
15.             }
    
16.             
    
17.         }
    
18. 
    
19.         public IEnumerable<T> GetEntitys<T>(Transaction tr, IEnumerable<ObjectId> ids) where T : DBObject
    
20.         {
    
21.             return
    
22.                 ids
    
23.                 .Select(id => tr.GetObject(id, OpenMode.ForRead))
    
24.                 .OfType<T>();
    
25.         }

        [CommandMethod("tttt")]<br />         public void tttt()<br />         {<br />             var db = HostApplicationServices.WorkingDatabase;<br />             var doc = Application.DocumentManager.GetDocument(db);<br />             var ed = doc.Editor;<br /> <br />             using (Transaction tr = doc.TransactionManager.StartTransaction())<br />             {<br /> <br />                 ObjectIdCollection ids = new ObjectIdCollection();<br />                 var cirs = GetEntitys<Circle>(tr, ids.Cast<ObjectId>());<br /> <br />             }<br />             <br />         }<br /> <br />         public IEnumerable<T> GetEntitys<T>(Transaction tr, IEnumerable<ObjectId> ids) where T : DBObject<br />         {<br />             return<br />                 ids<br />                 .Select(id => tr.GetObject(id, OpenMode.ForRead))<br />                 .OfType<T>();<br />         }

河伯

方法不错，可借鉴。其实以我的经验，代码的可读性比性能更重要，呵呵。
参数多了，代码长了，经常是隔段时间自己都看不懂了，不容易维护。

雪山飞狐_lzh

你说的是一个方面
还有情况是不能把事务放到子函数的，
比如对内存数据库的操作
即Database = new Database()的情况，这时用你的函数会出错的
而且多事务操作不光耗时，有时会产生莫名其妙的错误
传入事务作为参数的方式在kean的代码里经常会看到的，
应该算是Autodesk推荐的方式吧

河伯

事物耗时的问题，我已注意到。具体的实现里面，并没有嵌套事务。当需要多重操作，通过重载函数来完成的。
前面的子函数，都是不同类里面的成员，并无主次之分，如果把事物作为参数，反而容易搞乱逻辑。
当然，这些还只用.NET的初步感觉，未来有N多代码要迁移，随着研习的深入，希望找到一个好的设计模式。

雪山飞狐_lzh

正确，这个只有做多了才会感觉到
我以前在论坛贴过的DBTransaction类算是这方面我自己的解决方案吧
http://bbs.mjtd.com/forum.php?mod=viewthread&tid=76123&extra=%26page%3D1&page=1
这个类的调用代码可以看看这里：
http://bbs.mjtd.com/forum.php?mod=viewthread&tid=75701
各人有个人的想法，不强求，你说的重载函数的方式我看就不错，呵呵