Buen día,
Utilizo el siguiente código para llenar un combobox, vengo tomando un proyecto de un antiguo programador que no realizaba buen uso de "las buenas practicas", en mi caso, me gusta apegarme a ellas, por lo cual, quisiera saber como envolver el siguiente código en una instrucción USING para eliminar aquel codigo innecesario y dejar que mi aplicación haga trabajo.
cls_general myCls_general = new cls_general();
myCls_general.conexion_mssql("1");
strSQL = "SELECT [IdContacto],[fld_Nomb_Apel_Razon_Social] FROM OTROS_CONFIG.dbo.vi_list_contactos WHERE [IdContacto] <> @contacto_id " +
"ORDER BY [fld_Nomb_Apel_Razon_Social] ASC";
myCboReferentesDataAdapter = new SqlDataAdapter(strSQL, myCls_general.myConnection);
myCboReferentesDataAdapter.SelectCommand.Parameters.AddWithValue("@contacto_id", txt_cod_contacto.Text);
myCboReferentesDataSet = new DataSet();
myCboReferentesDataAdapter.Fill(myCboReferentesDataSet, "vi_list_contactos");
cbo_contacto_referente.Properties.DataSource = myCboReferentesDataSet.Tables["vi_list_contactos"];
cbo_contacto_referente.Properties.DisplayMember = "fld_Nomb_Apel_Razon_Social";
cbo_contacto_referente.Properties.ValueMember = "IdContacto";
myCboReferentesDataAdapter.SelectCommand.Parameters.Clear();
myCboReferentesDataAdapter.Dispose();
myCboReferentesDataSet.Dispose();
myCls_general.cerrar_conexion();
Actualmente no utilizo mi clase "cls_general" para traer la cadena de conexión, ya que ahora almaceno dicha cadena en la configuración de mi aplicación, lo que me interesa aprender, ¿es cuando debo anidar las instrucciones using en un código?
Es esta la mejor forma de cargar un combobox? Que puedo mejorar? Está correcta la forma de utilizar USING para que limpie los objectos y cierre la conexión?
using (SqlConnection sqlconn = new SqlConnection(Settings.Default.OTROS))
{
sqlconn.Open();
string sqlquery = "SELECT [rubro_id],[rubro] FROM BienesDeUsoRubros ORDER BY[rubro] ASC";
using (SqlCommand sqlcomm = new SqlCommand(sqlquery, sqlconn))
{
using (SqlDataAdapter sda = new SqlDataAdapter(sqlcomm))
{
DataTable dt = new DataTable();
sda.Fill(dt);
lookUpEdit1.Properties.ValueMember = "rubro_id";
lookUpEdit1.Properties.DisplayMember = "rubro";
lookUpEdit1.Properties.DataSource = dt;
}
}
}
.Dispose()
ycerrar_conexion
Por otro lado, ¿qué código crees que es innecesario?