StreamReader.cl ose and StreamWriter.cl ose

[Eran.Yasso]

Hi,

In the MSDN the sample doesn't use the close() method. But I know that
in most languages you do need to use the close() method after reading
and writing to a file.

from MSDN:

public static void Main()
{
string path = @"c:\temp\MyTes t.txt";

try
{
if (File.Exists(pa th))
{
File.Delete(pat h);
}

using (StreamWriter sw = new StreamWriter(pa th))
{
sw.WriteLine("T his");
sw.WriteLine("i s some text");
sw.WriteLine("t o test");
sw.WriteLine("R eading");
}

using (StreamReader sr = new StreamReader(pa th))
{
while (sr.Peek() >= 0)
{
Console.WriteLi ne(sr.ReadLine( ));
}
}
}
catch (Exception e)
{
Console.WriteLi ne("The process failed: {0}",
e.ToString());
}
}

does the "Using" replaces the needing for Close() method?

[Jon Skeet [C# MVP]]

<Er********@gma il.comwrote:

In the MSDN the sample doesn't use the close() method. But I know that
in most languages you do need to use the close() method after reading
and writing to a file.

<snip>

does the "Using" replaces the needing for Close() method?

Yes. "using" is equivalent to a try/finally block which calls Dispose()
in the finally block.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too

[Ollie Riches]

The simple answer to your question is yes the 'using' statement does replace
the need for a 'Close'...

check out the definitions of the using keyword in c#

http://www.c-sharpcorner.com/UploadF...Statement.aspx

The 2 classes you mention below - StreamReader & StreamWriter both implement
the IDisposable interface and you can use any class that implements this
interface in a 'using' statement.

These particular classes call 'Close' on the underlying stream object when
the Dispose method is called by the 'using' statement.

A 'using' statement is often called a try - catch - finally block, as in the
variable inside the 'using' statement is automatically wrapped into it's own
try - catch - finally block and the Dispose mehtod of the IDisposable
interface is always called in the finally statement.

HTH

Ollie Riches

<Er********@gma il.comwrote in message
news:11******** **************@ k78g2000cwa.goo glegroups.com.. .

Hi,

In the MSDN the sample doesn't use the close() method. But I know that
in most languages you do need to use the close() method after reading
and writing to a file.

from MSDN:

public static void Main()
{
string path = @"c:\temp\MyTes t.txt";

try
{
if (File.Exists(pa th))
{
File.Delete(pat h);
}

using (StreamWriter sw = new StreamWriter(pa th))
{
sw.WriteLine("T his");
sw.WriteLine("i s some text");
sw.WriteLine("t o test");
sw.WriteLine("R eading");
}

using (StreamReader sr = new StreamReader(pa th))
{
while (sr.Peek() >= 0)
{
Console.WriteLi ne(sr.ReadLine( ));
}
}
}
catch (Exception e)
{
Console.WriteLi ne("The process failed: {0}",
e.ToString());
}
}

does the "Using" replaces the needing for Close() method?

[Laurent Bugnion [MVP]]

Hi,

Jon Skeet [C# MVP] wrote:

<Er********@gma il.comwrote:

>In the MSDN the sample doesn't use the close() method. But I know that
in most languages you do need to use the close() method after reading
and writing to a file.

<snip>

>does the "Using" replaces the needing for Close() method?

Yes. "using" is equivalent to a try/finally block which calls Dispose()
in the finally block.

About that, a question: I have this (pseudo)code that I want to refactor
using the "using" clause:

StreamWriter writer = null;

try
{
// ...
}
catch ( Exception ex )
{
logger.Log( ex.Message );
}
finally
{
if ( writer != null )
{
writer.Close();
writer.Dispose( );
}
}

How to code that with a "using"? How can I do something special in case
an Exception occurs?

Greetings,
Laurent
--
Laurent Bugnion [MVP ASP.NET]
Software engineering, Blog: http://www.galasoft-LB.ch
PhotoAlbum: http://www.galasoft-LB.ch/pictures
Support children in Calcutta: http://www.calcutta-espoir.ch

[Jon Skeet [C# MVP]]

Laurent Bugnion [MVP] <ga*********@bl uewin.chwrote:

About that, a question: I have this (pseudo)code that I want to refactor
using the "using" clause:

<snip>

How to code that with a "using"? How can I do something special in case
an Exception occurs?

Either put a try/catch within the using statement, or outside it.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too

[Laurent Bugnion [MVP]]

Hi Jon,

Jon Skeet [C# MVP] wrote:

Laurent Bugnion [MVP] <ga*********@bl uewin.chwrote:

>About that, a question: I have this (pseudo)code that I want to refactor
using the "using" clause:

<snip>

>How to code that with a "using"? How can I do something special in case
an Exception occurs?

Either put a try/catch within the using statement, or outside it.

Thanks. I thought of that already, just wanted to confirm. Question
though: The code would become something like:

using ( StreamWriter writer = new StreamWriter( ... ) )
{
try
{
// ...
}
catch ( Exception ex )
{
logger.Log( ex.Message );

throw; // Is that correct??
}
}

I think that I shouldn't rethrow the Exception, is that correct? The
"using" clause is a try/finally, not try/catch/finally?

Thanks,
Laurent
--
Laurent Bugnion [MVP ASP.NET]
Software engineering, Blog: http://www.galasoft-LB.ch
PhotoAlbum: http://www.galasoft-LB.ch/pictures
Support children in Calcutta: http://www.calcutta-espoir.ch

[=?ISO-8859-1?Q?G=F6ran_Andersson?=]

Laurent Bugnion [MVP] wrote:

Hi Jon,

Jon Skeet [C# MVP] wrote:

>Laurent Bugnion [MVP] <ga*********@bl uewin.chwrote:

>>About that, a question: I have this (pseudo)code that I want to
refactor using the "using" clause:

<snip>

>>How to code that with a "using"? How can I do something special in
case an Exception occurs?

Either put a try/catch within the using statement, or outside it.

Thanks. I thought of that already, just wanted to confirm. Question
though: The code would become something like:

using ( StreamWriter writer = new StreamWriter( ... ) )
{
try
{
// ...
}
catch ( Exception ex )
{
logger.Log( ex.Message );

throw; // Is that correct??
}
}

I think that I shouldn't rethrow the Exception, is that correct? The
"using" clause is a try/finally, not try/catch/finally?

Thanks,
Laurent

If you rethrow the exception or not depends on if you completely handled
it or not. If the code that called this method needs to know about the
exception, you should rethow it.

I haven't investigated if a using clause has a catch or not, but if it
has, it rethrows the exception. The point is that the using clause does
not consume exceptions.

--
Göran Andersson
_____
http://www.guffa.com

[Laurent Bugnion [MVP]]

Hi,

Göran Andersson wrote:

If you rethrow the exception or not depends on if you completely handled
it or not. If the code that called this method needs to know about the
exception, you should rethow it.

I haven't investigated if a using clause has a catch or not, but if it
has, it rethrows the exception. The point is that the using clause does
not consume exceptions.

That confirms what I actually observed. Thanks.

Laurent
--
Laurent Bugnion [MVP ASP.NET]
Software engineering, Blog: http://www.galasoft-LB.ch
PhotoAlbum: http://www.galasoft-LB.ch/pictures
Support children in Calcutta: http://www.calcutta-espoir.ch