Skip to main content
Code Review

Return to Answer

replaced http://stackoverflow.com/ with https://stackoverflow.com/
Source Link

It seems to me that if File.AppendAllText throws an exception dbConn won't be closed. The dbConn.Close() should be in a finally block. (Exception inside catch block Exception inside catch block.)

It seems to me that if File.AppendAllText throws an exception dbConn won't be closed. The dbConn.Close() should be in a finally block. (Exception inside catch block.)

It seems to me that if File.AppendAllText throws an exception dbConn won't be closed. The dbConn.Close() should be in a finally block. (Exception inside catch block.)

added 170 characters in body
Source Link
palacsint
  • 30.4k
  • 9
  • 82
  • 157

It seems to me that if File.AppendAllText throws an exception dbConn won't be closed. The dbConn.Close() should be in a finally block. (Exception inside catch block .)

It seems to me that if File.AppendAllText throws an exception dbConn won't be closed.

It seems to me that if File.AppendAllText throws an exception dbConn won't be closed. The dbConn.Close() should be in a finally block. (Exception inside catch block .)

Source Link
palacsint
  • 30.4k
  • 9
  • 82
  • 157

It seems to me that if File.AppendAllText throws an exception dbConn won't be closed.

default

AltStyle によって変換されたページ (->オリジナル) /