public static DataTable select(string Query)
        {
            DataTable dt = new DataTable();
            SqlConnection connection = new SqlConnection(Utility.Connection_String());

            connection.Open();

            SqlCommand sqlCmd = new SqlCommand(Query, connection);
            SqlDataAdapter sqlDa = new SqlDataAdapter(sqlCmd);

            sqlDa.Fill(dt);
            connection.Close();
            return dt;
            
        }

آیا در کلاس بالا  connection.Close(); هم لازم هست نوشته بشه؟

2-آیا جای connection.Close(); صحیح می باشد؟

یا باید جاش مثل کد زیر باشد

 public static DataTable select(string Query)
        {
            DataTable dt = new DataTable();
            SqlConnection connection = new SqlConnection(Utility.Connection_String());

            connection.Open();

            SqlCommand sqlCmd = new SqlCommand(Query, connection);
            SqlDataAdapter sqlDa = new SqlDataAdapter(sqlCmd);
            connection.Close();
            sqlDa.Fill(dt);
            
            return dt;
            
        }