Question

I understand what the using clause does, but should it be used with unnamed parameters?

For example; should:

var navigator = new XPathDocument(new StringReader(request)).CreateNavigator();

be:

using (StringReader sr = new StringReader(request))
{
    var navigator = new XPathDocument(sr).CreateNavigator();
    ...
}

Answer 1

Best way to know how it works : test!

class Program
{
    public static void Main()
    {
        using (TestDisposable d = new TestDisposable())
        {
        } // Will trace "disposed"

        UseDisposable use = UseDisposable.Create(new TestDisposable());
        // Will trace "disposed"
    }
}

public class UseDisposable
{
    public TestDisposable Disposable;

    public static UseDisposable Create(TestDisposable disposable)
    {
        return new UseDisposable()
            {
                Disposable = disposable
            };
    }
}

public class TestDisposable : IDisposable
{
    private bool _disposed = false;

    #region IDisposable Membres

    public void Dispose()
    {
        Dispose(true);
        GC.SuppressFinalize(this);
    }

    public void Dispose(bool disposing)
    {
        if(!_disposed && disposing)
        {
            Trace.WriteLine("Disposed");
            _disposed = true;
        }
    }

    #endregion
}

In my opinion, you should always use the second method.