Question about Using

[Jack Jackson]

I want to open a filestream and so some processing. If the open
throws an exception I want to throw a custom exception. If any other
exception occurs during the processing I want that exception to bubble
up.

The code without Using would be:

dim fs As IO.FileStream

try
fs = IO.File.Open(parmFileName, IO.FileMode.Create)
catch
<<Throw custom exception>>
end try

<< Code that writes using fs>>

fs.close()
fs.Dispose()


The only thing I can think of is this, but it seems kludgy to me:

Dim rethrow As Boolean = False
Try
Using fs = IO.File.Open(parmFileName, IO.FileMode.Create)
rethrow = True
<<Processing>>
fs.Close()
End Using
Catch
If rethrow
Throw
End If
<<Throw custom exception>>
End Try

 

[Tom Shelton]

I want to open a filestream and so some processing. If the open
throws an exception I want to throw a custom exception. If any other
exception occurs during the processing I want that exception to bubble
up.

The code without Using would be:

dim fs As IO.FileStream

try
fs = IO.File.Open(parmFileName, IO.FileMode.Create)
catch
<<Throw custom exception>>
end try

<< Code that writes using fs>>

fs.close()
fs.Dispose()

The only thing I can think of is this, but it seems kludgy to me:

Dim rethrow As Boolean = False
Try
Using fs = IO.File.Open(parmFileName, IO.FileMode.Create)
rethrow = True
<<Processing>>
fs.Close()
End Using
Catch
If rethrow
Throw
End If
<<Throw custom exception>>
End Try

It sounds as if you may not want to use a using block here. By the
way, if you do use using, calling fs.Close () inside is completly
redundant - since it will be called by dispose anyway.

You might want to do something like:

Dim fs As FileStream

Try
fs = File.Open (....)
Catch ex As Exception
Throw New CustomException (ex) ' you should set the inner exception
to the original
End Try

Try
' do cool stuff with fs
Finaly
fs.Close ()
End Try

 

[Jack Jackson]

It sounds as if you may not want to use a using block here. By the
way, if you do use using, calling fs.Close () inside is completly
redundant - since it will be called by dispose anyway.

You might want to do something like:

Dim fs As FileStream

Try
fs = File.Open (....)
Catch ex As Exception
Throw New CustomException (ex) ' you should set the inner exception
to the original
End Try

Try
' do cool stuff with fs
Finaly
fs.Close ()
End Try

That seems like the best solution. I was just thinking that I should
use the new whiz-bang Using.

 

[sloan]

Another variation:


using (someObject as IDisposable)

{

proxy1.SubmitPostSubmitProcessRequest(scoringSetID, dbInstanceName);

return;

}



I would also read and reread this:
http://blogs.msdn.com/kcwalina/archive/2005/03/16/396787.aspx