ComboBox y DataSet

"Oscar M" wrote in message
news:%

uno de mis formularios tiene un datagridview y 20 combobox para realizar
filtros a la selección.

Para rellenar la lista de los combobox (Cada uno de ellos con un campo de
la tabla) he creado un enlace a datos para cada uno de ellos,
modificandoles luego el SQL para hacer un Distinct de cada elemento.

Pero claro esto me ha creado en mi formulario 20 DataSet, 20
bindingsourcer y 20 Tableadapter.

¿Hay alguna solución mas eficiente?

Es más eficiente cargarlos con un datareader, en lugar de un dataadapter
más un dataset. Si el formulario es Web, puedes poner el DataReader como
DataSource del combo. Si es un Winform, no puedes poner el datareader como
datasource, pero sí puedes escribir una pequeña subrutina que reciba como
argumentos el combo y el datareader y que haga un pequeño bucle leyendo el
datareader y metiendo cada linea en el combo. De esta manera te saltas un
paso respecto al dataadapter, ya que éste utiliza internamente un bucle
leyendo de un datareader para traer los datos al dataset, y luego el
databinding usa otro bucle para copiarlos del dataset al combo.

Si quieres añadir una optimización más, mete los 20 selects en un
procedimiento almacenado, ejecuta la llamada al procedimiento con un
ExecuteReader de un SqlCommand, y usa el método NextResult() del DataReader
para ir saltando de un resultado a otro. De esta forma haces una sola
llamada al servidor de base de datos, en lugar de 20.

Ok, lo intentaré como dices...

La idea, si no lo he entendido mal, es crear una rutina que reciba el
Combobox (que no estará asociado a ninguna dataset, unbound) y rellene su
lista buscando en el dataset del DateGridView.

Creo que servirá para dotar a mi formulario de las funcionalidades que
deseo.

Muchas gracias.

Un saludo... Y a pasarlo bien que es viernes...

byebye


"A.Poblacion" escribió en
el mensaje news:

"Oscar M" wrote in message
news:%

uno de mis formularios tiene un datagridview y 20 combobox para realizar
filtros a la selección.

Para rellenar la lista de los combobox (Cada uno de ellos con un campo de
la tabla) he creado un enlace a datos para cada uno de ellos,
modificandoles luego el SQL para hacer un Distinct de cada elemento.

Pero claro esto me ha creado en mi formulario 20 DataSet, 20
bindingsourcer y 20 Tableadapter.

¿Hay alguna solución mas eficiente?

Es más eficiente cargarlos con un datareader, en lugar de un dataadapter
más un dataset. Si el formulario es Web, puedes poner el DataReader como
DataSource del combo. Si es un Winform, no puedes poner el datareader como
datasource, pero sí puedes escribir una pequeña subrutina que reciba como
argumentos el combo y el datareader y que haga un pequeño bucle leyendo el
datareader y metiendo cada linea en el combo. De esta manera te saltas un
paso respecto al dataadapter, ya que éste utiliza internamente un bucle
leyendo de un datareader para traer los datos al dataset, y luego el
databinding usa otro bucle para copiarlos del dataset al combo.

Si quieres añadir una optimización más, mete los 20 selects en un
procedimiento almacenado, ejecuta la llamada al procedimiento con un
ExecuteReader de un SqlCommand, y usa el método NextResult() del
DataReader para ir saltando de un resultado a otro. De esta forma haces
una sola llamada al servidor de base de datos, en lugar de 20.

"Oscar M" wrote in message
news:

La idea, si no lo he entendido mal, es crear una rutina que reciba el
Combobox (que no estará asociado a ninguna dataset, unbound) y rellene su
lista buscando en el dataset del DateGridView.

No, lo has entendido mal. La idea es llenar los combobox _prescindiendo_
por completo de usar DataSets, y usar en su lugar DataReaders. Algo asi:

SqlCommand cmd = new SqlCommand("Select Campo from Tabla1", conexion);
conexion.Open();
SqlDataReader rdr = cmd.ExecuteReader();
dropdownlist1.Items.Clear();
while (rdr.Read())
{
dropdownlist1.Items.Add(rdr[0]);
}
conexion.Close();

Este trozo de código habría que repetirlo 20 veces, una por cada combo.
Para no escribir tanto es por lo que decía yo lo de escribir una subrutina
con el bucle y llamarla 20 veces.
Obviamente el combo estará "unbound", y se cargará de datos metiendo este
código en el form_load.

Ahora si que lo he entendido, muchas gracias, tiene mucha mejor pinta que lo
que yo decía.
Lo voy a intentar..

Un saludo.

Oscar

"A.Poblacion" escribió en
el mensaje news:

"Oscar M" wrote in message
news:

La idea, si no lo he entendido mal, es crear una rutina que reciba el
Combobox (que no estará asociado a ninguna dataset, unbound) y rellene su
lista buscando en el dataset del DateGridView.

No, lo has entendido mal. La idea es llenar los combobox
_prescindiendo_ por completo de usar DataSets, y usar en su lugar
DataReaders. Algo asi:

SqlCommand cmd = new SqlCommand("Select Campo from Tabla1", conexion);
conexion.Open();
SqlDataReader rdr = cmd.ExecuteReader();
dropdownlist1.Items.Clear();
while (rdr.Read())
{
dropdownlist1.Items.Add(rdr[0]);
}
conexion.Close();

Este trozo de código habría que repetirlo 20 veces, una por cada combo.
Para no escribir tanto es por lo que decía yo lo de escribir una subrutina
con el bucle y llamarla 20 veces.
Obviamente el combo estará "unbound", y se cargará de datos metiendo
este código en el form_load.

Estupendo, lo he probado y funciona perfectamente, solo he tenido que
cambiar SQL por OLEDB, ya que yo utilizo bases de datos de Access...

Muchas Gracias de nuevo...

"A.Poblacion" escribió en
el mensaje news:

"Oscar M" wrote in message
news:

La idea, si no lo he entendido mal, es crear una rutina que reciba el
Combobox (que no estará asociado a ninguna dataset, unbound) y rellene su
lista buscando en el dataset del DateGridView.

No, lo has entendido mal. La idea es llenar los combobox
_prescindiendo_ por completo de usar DataSets, y usar en su lugar
DataReaders. Algo asi:

SqlCommand cmd = new SqlCommand("Select Campo from Tabla1", conexion);
conexion.Open();
SqlDataReader rdr = cmd.ExecuteReader();
dropdownlist1.Items.Clear();
while (rdr.Read())
{
dropdownlist1.Items.Add(rdr[0]);
}
conexion.Close();

Este trozo de código habría que repetirlo 20 veces, una por cada combo.
Para no escribir tanto es por lo que decía yo lo de escribir una subrutina
con el bucle y llamarla 20 veces.
Obviamente el combo estará "unbound", y se cargará de datos metiendo
este código en el form_load.