using оператор FileStream и / или StreamReader - предупреждения Visual Studio 2012

Ниже показано, как Microsoft рекомендует это делать. Он длинный и громоздкий, но безопасный:

FileStream fs = null;
try
{
    fs = new FileStream(filePath, FileMode.Open, FileAccess.Read, FileShare.ReadWrite);
    using (TextReader tr= new StreamReader(fs))
    {
        fs = null;
        // Code here
    }
}
finally
{
    if (fs != null)
        fs.Dispose();
}

Этот метод всегда гарантирует, что все будет удалено, несмотря на то, какие исключения могут быть выброшены. Например, если конструктор StreamReader выдает исключение, FileStream все равно будет правильно удален.

Visual studio «предупреждает» меня, что я избавляюсь от файловой системы более одного раза.

Да, но это нормально. В документации для IDisposable.Dispose говорится:

Если метод Dispose объекта вызывается более одного раза, объект должен игнорировать все вызовы после первого. Объект не должен вызывать исключение, если его метод Dispose вызывается несколько раз.

Исходя из этого, предупреждение является фиктивным, и я бы предпочел оставить код как есть и подавить предупреждение.

Поскольку ответ Дэна работает только со StreamWriter, я считаю, что это может быть наиболее приемлемым ответом. (Ответ Дэна по-прежнему будет содержать дважды удаленное предупреждение с StreamReader - как упоминает Дэниел Хилгарт и exacerbatedexpert, StreamReader удаляет файловый поток)

using (TextReader tr = new StreamReader(new FileStream(filePath, FileMode.Open, FileAccess.Read, FileShare.ReadWrite)))
{
    string line;
    while ((line = tr.ReadLine()) != null)
    {
        // Do work here
    }
}

Это очень похоже на ответ Дэниела Хилгарта, измененный для вызова dispose с помощью оператора Using в StreamReader, поскольку теперь ясно, что StreamReader вызовет dispose в FileStream (согласно всем другим сообщениям, документация упоминается)

Обновлять:

Я нашел этот пост. За что стоит. Закрывает ли поток при удалении streamreader?

Да, правильным способом было бы использовать вашу первую альтернативу:

using (FileStream fs = new FileStream(filePath, FileMode.Open,
                                      FileAccess.Read, FileShare.ReadWrite)) 
{ 
    TextReader tr = new StreamReader(fs); 
    // do stuff here 
} 

Причина в следующем:
Удаление StreamReader удаляет только FileStream, так что на самом деле это единственное, что вам нужно удалить.

Ваш второй вариант (просто внутреннее «использование») не является решением, так как он оставит FileStream нерасположенным, если внутри конструктора StreamReader возникнет исключение.

Это потому, что способ, которым вы использовали StreamReader, удаляет поток при его удалении. Итак, если вы также удаляете поток, он удаляется дважды. Некоторые считают это недостатком _2 _ - но, тем не менее, он там есть. В VS 2012 (.NET 4.5) есть опция в StreamReader, чтобы не удалять поток с новым конструктором: http://msdn.microsoft.com/en-us/library/gg712952

Два решения:

A) Вы доверяете Reflector или Documentation и знаете, что *Reader и *Writer закроют лежащий в основе *Stream. Но предупреждение: это не сработает в случае сгенерированного исключения. Так что это не рекомендуемый способ:

using (TextReader tr = new StreamReader(new FileStream(filePath, FileMode.Open, FileAccess.Read, FileShare.ReadWrite)))
{
    // Code here
}

B) Вы игнорируете предупреждение, поскольку в документации указано The object must not throw an exception if its Dispose method is called multiple times. Это рекомендуемый способ, так как это хорошая практика - всегда использовать using, и безопасно в случае сгенерированного исключения:

[SuppressMessage("Microsoft.Usage", "CA2202:Do not dispose objects multiple times")]
internal void myMethod()
{
    [...]
    using (FileStream fs = new FileStream(filePath, FileMode.Open, FileAccess.Read, FileShare.ReadWrite))
    using (TextReader tr = new StreamReader(fs))
    {
        // Code here
    }
}

Учитывая всю чушь, которую породил этот (совершенно законный!) Вопрос, я бы предпочел следующее:

FileStream fs = null;
TextReader tr= null;
try
{
    fs = new FileStream(filePath, FileMode.Open, FileAccess.Read, FileShare.ReadWrite);
    tr= new StreamReader(fs);
    // Code here
}
finally
{
    if (tr != null)
        tr.Dispose();
    if (fs != null)
        fs.Dispose();
}

Ссылки ниже иллюстрируют совершенно допустимый синтаксис. ИМО, этот синтаксис «использования» намного предпочтительнее вложенного «использования». Но я признаю - это не решает исходный вопрос:

• http://blogs.msdn.com/b/ericgu/archive/2004/08/05/209267.aspx

• .NET - замена вложенных операторов using на один оператор using

ПО МОЕМУ МНЕНИЮ...