Welcome to the Lounge

OriginalGriff wrote:

.. fancy dress party – I may or may not dress as Schroedinger

using (var outerStream = new MemoryStream(someData))
{
 using (var innerStream = new TextReader(outerStream))
 {
  // do something
 }
}

var outerStream = new MemoryStream(someData);
using (var innerStream = new TextReader(outerStream)
{
 // do something
}

// Create the streams used for decryption.
using (MemoryStream msDecrypt = new MemoryStream(cipherText))
 {
  using (CryptoStream csDecrypt = new CryptoStream(msDecrypt, decryptor, CryptoStreamMode.Read))
  {
   using (StreamReader srDecrypt = new StreamReader(csDecrypt))
   {
    // Read the decrypted bytes from the decrypting stream
    // and place them in a string.
    plaintext = srDecrypt.ReadToEnd();
   }
  }
 }

MemoryStream memStream = new MemoryStream(data);
CryptoStream decStream = new CryptoStream(memStream, decryptor, CryptoStreamMode.Read);

using (StreamReader reader = new StreamReader(decStream))
{
 decryptedValue = Encoding.UTF8.GetBytes(reader.ReadToEnd());
}

Foothill wrote:

However, the proper way is this

C#

var outerStream = new MemoryStream(someData);
using (var innerStream = new TextReader(outerStream)
{
 // do something
}

This is because the a stream will dispose of the underlying streams when you call Stream.Dispose(). I had code analysis bark at me all the time until I figured this one out.

var outerStream = new AStream();
using (var innerStream = new AnotherStream(outerStream)) {...}

{
    var outerStream = new MyStream();  // line 59
    using (var innerStream = new MyStream(outerStream)) {
        bool i = innerStream.CanRead;
        Console.WriteLine("Read: " + i);
    }
}
{
    using (var outerStream = new MyStream())
    {
        using (var innerStream = new MyStream(outerStream))
        {
            bool i = innerStream.CanRead;
            Console.WriteLine("Read: " + i);
        }
    }   // Line 73
}