Question

When I use a using clause on an object should I dispose this object before exiting the using block?

            using (var transaction = TransactionUtils.CreateTransactionScope())
            {
                try
                {
                    _entity.Save(entity);
                    transaction.Complete();
                }
                catch // or better finally
                {
                    transaction.Dispose(); // Is this try-catch usefull?
                    throw;
                }
            }

Note : A similar question has already been asked but I find the example and the answers strange.

Answer 1

It is redundant to dispose the object.

using (ResourceType resource = CreateResource())
{
    DoStuffWith(resource);
}

is equivalent to

ResourceType resource = CreateResource();

try
{
    DoStuffWith(resource);
}
finally
{    
    if (resource != null)
    {
        ((IDisposable)resource).Dispose();
    }
}

For non-nullable value types the null-check is omitted and dynamic is handled slightly different, too. See 8.13 in the C# Language Specification for more details.