1

Estoy intentando agregar datos en un datagrid usando DataTable mediante una condición. Si el ID que traigo de un dataRow es diferente al ID que tengo en la celda de mi datagrid, agregar el dato nuevo en el datagrid si no solo actualizar algunos datos del datagrid.

Sí me agrega los datos, el problema es que mis datos se duplican. No sé si tengo algún error de lógica o de código. Pensé en utilizar un break en mi condición y un row.delete pero no funcionó.

 DataTable dt = _datos.llenaDataDatos(cadena,cbxDatabase.Text);
                if (dataStatus.Rows.Count-1 > 0)//si es mayor a 0 agrego por primera vez las filas
                {

                    foreach (DataRow row in dt.Rows)//recorrido del datatable
                    {
                        for (int i = 0; i < dataStatus.Rows.Count; i++)//recorro las filas de datagrid
                        {
                            if (Convert.ToInt32(dataStatus.Rows[i].Cells[2].Value) != Convert.ToInt32(row["pid"]))
                            {
                                dataStatus.Rows.Add("", "", row["pid"], row["usename"], row["application_name"], row["datname"], row["client_addr"], row.IsNull("backend_start") ? row["backend_start"] : "", row["state"], row.IsNull("query_start") ? row["query_start"] : "", row.IsNull("xact_start") ? row["xact_start"] : "", row["wait_event"], row["query"]);

                            }                            
                            else
                            {
                                dataStatus.Rows[i].Cells[8].Value = row["state"];
                                dataStatus.Rows[i].Cells[9].Value = row["query_start"];
                                dataStatus.Rows[i].Cells[11].Value = row["wait_event"];

                            }
                        }
                    }
                }
                else
                {
                    foreach (DataRow row in dt.Rows)
                    {
                            dataStatus.Rows.Add("", "", row["pid"], row["usename"], row["application_name"], row["datname"], row["client_addr"], row.IsNull("backend_start") ? row["backend_start"] : "", row["state"], row.IsNull("query_start") ? row["query_start"] : "", row.IsNull("xact_start") ? row["xact_start"] : "", row["wait_event"], row["query"]);
                    }
                }
1
  • Yo intentaria obtener todos los datos que contiene el datagrid, guardarlo en un arreglo y despues comparar el dato que se va agregar con el arreglo y ver si coincide con los datos del arreglo, si el nuevo dato se encuentra en el arreglo, no se añade al datagrid. el 29 jun. 2019 a las 19:35

1 respuesta 1

1

Esto es solo un problema de lógica.

si vos haces esto:

foreach (DataRow row in dt.Rows)//recorrido del datatable
{
    for (int i = 0; i < dataStatus.Rows.Count; i++)//recorro las filas de datagrid
    {
        ...
    }
}

Estas comprobando cada fila de la grilla, con cada fila de tus datos nuevos... y que pasa?

que al hacer esa comparacion, por cada fila de tus datos nuevos, o agregas o editas una fila.. pero por cada fila! y entonces no solo duplicas, si no que metes muchisimos datos de mas...

Lo que hay que hacer es otra cosa.. tu algoritmo tiene que comparar todos contra todos, pero ante cada iteración completa de comparación, no guardar directamente, si no despues, cuando sabe que tiene que hacer.

Para que se entienda, deberia ser algo asi:

foreach (DataRow row in dt.Rows)//recorrido del datatable
{
    int pos = 0;
    boolean esigual = false;
    for (int i = 0; i < dataStatus.Rows.Count; i++)//recorro las filas de datagrid
    {
        // Comparo
        if (Convert.ToInt32(dataStatus.Rows[i].Cells[2].Value) == Convert.ToInt32(row["pid"]))
        {
            esigual = true; // solo ocurre si es un id repetido
            pos = i;
        }
    }
    //ahora ya compare todas las filas, si hago lo que hacias antes adentro
    if (!esigual)
    {
        dataStatus.Rows.Add("", "", row["pid"], row["usename"], row["application_name"], row["datname"], row["client_addr"], row.IsNull("backend_start") ? row["backend_start"] : "", row["state"], row.IsNull("query_start") ? row["query_start"] : "", row.IsNull("xact_start") ? row["xact_start"] : "", row["wait_event"], row["query"]);
    }                          
    else
    {
        dataStatus.Rows[pos].Cells[8].Value = row["state"];
        dataStatus.Rows[pos].Cells[9].Value = row["query_start"];
        dataStatus.Rows[pos].Cells[11].Value = row["wait_event"];
    }
}
7
  • fue muy útil tu respuesta..
    – mica
    el 1 jul. 2019 a las 23:15
  • 1
    @mica si te sirvio recuerda aceptarla. Si no sirvio, dime que falla asi la arreglamos, o escribe tu propia respuesta con lo que hiciste para que funcionara.
    – gbianchi
    el 1 jul. 2019 a las 23:41
  • Solo agrege otro for en el else donde actualizo las filas y listo.. else { for (int i = 0; i < dataStatus.Rows.Count; i++)// { dataStatus.Rows[i].Cells[8].Value = row["state"]; dataStatus.Rows[i].Cells[9].Value = row["query_start"]; dataStatus.Rows[i].Cells[11].Value = row["wait_event"]; } }
    – mica
    el 1 jul. 2019 a las 23:54
  • No entiendo porque agregaste ese for.. si ya el que estaba deberia haber solucionado todo. Queres explicar eso en una respuesta
    – gbianchi
    el 1 jul. 2019 a las 23:58
  • lo agrege porque el for del primer recorrido termina donde se hace la comparacion de que si son iguales, al momento de hacer la comparacion de que si es diferente entra en otra sentencia donde el contenido del for pierde accesibilidad, y al momento de actualizar mis datos la variable i marca como no error, es por ello que tuve que volver a gregar.
    – mica
    el 2 jul. 2019 a las 0:02

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.