Topic: Lock and unlock a document

[latour_g]

Hi,

I would like to know if using

Code: [Select]

using (doc.LockDocument())
{
.....
}

is better than using :

Code: [Select]

Document doc = AcadApp.DocumentManager.MdiActiveDocument;
DocumentLock loc = doc.LockDocument();
....
loc.Dispose();

Right now, I'm using doc.LockDocument() at a lot of place in my code.  I'm wondering if I choose change them all for the other method because user sometime get an error message when trying to save (Command still active).  I have checked everywhere in my code, there is always a loc.Dispose() but I might have miss something.

Thank youbefore printing this email.

[Andrey Bushman]

If you use the using block, then document will be unlocked even if an unhandled exception happens. If you use

Code: [Select]

DocumentLock loc = doc.LockDocument();then you are to use 'try\catch\finally' block and unlock document in the 'finally' block.

[latour_g]

Thanks for your answer.

Suppose I have this :

Code: [Select]

try { db.LoadLineTypeFile(clType, cLinFile); }
catch { }

Can I unlock the document in the catch since there is nothing ?

And one last question if I may.  If I want to replace this :

Code: [Select]

DocumentLock loc = doc.LockDocument();
using (Transaction tr = db.TransactionManager.StartTransaction())
{
DBObjectCollection objs = new DBObjectCollection();
BlockReference br = tr.GetObject(blkId, OpenMode.ForWrite) as BlockReference;
.....

with "using document lock"

should I declare using (doc.LockDocument()) inside or outside using (Transaction tr = db.TransactionManager.StartTransaction()) ?

Code: [Select]

using (doc.LockDocument())
{
        using (Transaction tr = db.TransactionManager.StartTransaction())
        {
                DBObjectCollection objs = new DBObjectCollection();
                BlockReference br = tr.GetObject(blkId, OpenMode.ForWrite) as BlockReference;
                ....
        }
}

or

Code: [Select]

using (Transaction tr = db.TransactionManager.StartTransaction())
{
        using (doc.LockDocument())
        {
                DBObjectCollection objs = new DBObjectCollection();
                BlockReference br = tr.GetObject(blkId, OpenMode.ForWrite) as BlockReference;
                ....
        }
}


[Andrey Bushman]

This is bad way. Why don't you need to do at in the finally?

[latour_g]

I thought "finally" was useless if there was nothing to be done in the catch.

I have edited my previous post, can you take a look at it please ?

[Andrey Bushman]

should I declare using (doc.LockDocument()) inside or outside using (Transaction tr = db.TransactionManager.StartTransaction()) ?

I think both variants are correct.

I thought "finally" was useless if there was nothing to be done in the catch.

Content of the finally block will be called always. Content of the catch block will be called only if exception occurs. Read about the finally in the C# specification, please.

[latour_g]

Yes you are right.  My bad, I was confused when I talked about try/catch !

Thanks for your answer !

[Andrey Bushman]

No problem. Welcome.

[gile]

Hi,

By my side I use to write :

Code - C#: [Select]

1. using (doc.LockDocument())
2. using (var tr = db.TransactionManager.StartTransaction())
3. {
4.     // ...
5. } // <- the transaction is disposed, then the document is unlocked (disposed)

[Keith Brown]

Hi,

By my side I use to write :

Code - C#: [Select]

1. using (doc.LockDocument())
2. using (var tr = db.TransactionManager.StartTransaction())
3. {
4.     // ...
5. } // <- the transaction is disposed, then the document is unlocked (disposed)

+1