C#

mark_migrated(Convert.ToInt32(MSSQLreader["file_no"]));

private void mark_migrated(int file_no)
{
    MSSQLconnection_Migrated = new SqlConnection(ConfigurationManager.ConnectionStrings["MSSQL"].ConnectionString);
    MSSQLconnection_Migrated.Open();

    // where is_migrated = false;
    MSSQLcommand_Migrated = new SqlCommand("update persons set is_migrated = 1 where file_no = " + file_no, MSSQLconnection_Migrated);
    int MSSQLreader_Migrated = MSSQLcommand_Migrated.ExecuteNonQuery();

    MSSQLcommand_Migrated.Dispose();
    MSSQLconnection_Migrated.Close();
}

jrahma wrote:

MSSQLconnection_Migrated = new SqlConnection(ConfigurationManager.ConnectionStrings["MSSQL"].ConnectionString);

try
{
  ...
  command.Dispose();
  connect.Close();
  // At this point ALL resources should be cleaned up for
  // a normal exit.
}
catch(Exception e)
{
   // Clean up resources for abnormal exit.

   // REPORT THE EXCEPTION!!!!!
   try
   {
     command.Dispose();
   }
   catch(Exception)
   {
     // Ignore it because we already have an exception
   }

   try
   {
     connection.Close();
   }
   catch(Exception)
   {
     // Ignore it because we already have an exception
   }

   // Repeat try/catch for all other resources.
 }  // End of first try

 // There is NO finally block.  All connection
 // resources were cleaned up by regular process flow or exception block.
 // finally could be used for resources that were NOT connection related.

Meysam Tolouee wrote:

It is absolutely fine, isn't it?

Meysam Tolouee wrote:

Now it really works fine.

Fine means it gives me true result.

ihoecken wrote:

By the way, I didn't downvote you and I wonder why someone did.