използване на израза 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 ме „предупреждава“, че изхвърлям fs повече от веднъж.

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

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

Въз основа на това предупреждението е фалшиво и моят избор би бил да оставя кода такъв, какъвто е, и да премахна предупреждението.

Тъй като отговорът на Дан изглежда работи само със StreamWriter, вярвам, че това може да е най-приемливият отговор. (Отговорът на Дан все пак ще даде предупреждение за изхвърляне два пъти със StreamReader - както Daniel Hilgarth и 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.

<?php
//Defining the variables
$a="[[*menutitle]]";
if ($a=="classpvp"){
echo "selected";
} else {
echo $a;
} ?>

<li class="navTab [[TabSelectClassPvP]]">
ClassPvP
</li>

<li class="navTab string(14) " classpvp"="">
ClassPvP
</li>

<?php
//Defining the variables
$a="[[*menutitle]]";
var_dump($a);
if ($a=="classpvp"){
echo "selected";
} else {
echo $a;
}

Това е така, защото начинът, по който сте използвали StreamReader, изхвърля потока, когато е изхвърлен. Така че, ако изхвърлите и потока, той ще бъде изхвърлен два пъти. Някои смятат това за недостатък в StreamReader--но все пак го има. Във VS 2012 (.NET 4.5) има опция в StreamReader да не се изхвърля потокът с нов конструктор: http://msdn.microsoft.com/en-us/library/gg712952

Две решения:

А) Доверявате се на 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();
}

Връзките по-долу илюстрират напълно легален синтаксис. IMO, този синтаксис "използване" е много за предпочитане пред вложеното "използване". Но признавам - това не решава първоначалния въпрос:

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

• .NET - Замяна на вложени изрази за използване с оператор за единично използване

ИМХО...