Skip to main content
se añadieron 10 caracteres en el cuerpo
Origen Enlace
Eduardo Reyes
  • 1.3k
  • 1
  • 5
  • 14

Otra cosa muy diferente seria agregar los nodos hijos, solo cuando el usuario expanda los nodosel nodo padre, para ello se puede usar el evento BeforeExpand del TreeView y agregar en ese evento solo sus nodos hijos, también es conveniente en este caso declarar las variables DataTable dataTableNodos, DataView dataViewNodosHijos, al nivel del formulario, cargándolas en el Load.

Otra cosa muy diferente seria agregar los nodos, solo cuando el usuario expanda los nodos, para ello se puede usar el evento BeforeExpand del TreeView y agregar en ese evento solo sus nodos hijos, también es conveniente en este caso declarar las variables DataTable dataTableNodos, DataView dataViewNodosHijos, al nivel del formulario, cargándolas en el Load.

Otra cosa muy diferente seria agregar los nodos hijos, solo cuando el usuario expanda el nodo padre, para ello se puede usar el evento BeforeExpand del TreeView y agregar en ese evento solo sus nodos hijos, también es conveniente en este caso declarar las variables DataTable dataTableNodos, DataView dataViewNodosHijos, al nivel del formulario, cargándolas en el Load.

Incluí la solución agregando los nodos, cuando el usuario, expende el nodo.
Origen Enlace
Eduardo Reyes
  • 1.3k
  • 1
  • 5
  • 14

Otra cosa muy diferente seria agregar los nodos, solo cuando el usuario expanda los nodos, para ello se puede usar el evento BeforeExpand del TreeView y agregar en ese evento solo sus nodos hijos, también es conveniente en este caso declarar las variables DataTable dataTableNodos, DataView dataViewNodosHijos, al nivel del formulario, cargándolas en el Load.

void treeView1_BeforeExpand(object sender, System.Windows.Forms.TreeViewCancelEventArgs e)
{
    //Solo se agregaran los nodos si no los tiene ya agregados
    if (e.Node.Nodes.Count == 0)
    {
        //Recuperamos la propiedad Tag del nodo donde hemos almacenamos su índice, como veras más abajo
        dataViewNodosHijos.RowFilter = dataTableNodos.Columns["IdPadre"].ColumnName + " = " + Int32.Parse(e.Node.Tag.ToString());

        foreach (DataRowView dataFilas in dataViewNodosHijos)
        {
            TreeNode nuevoNodo = new TreeNode();
            nuevoNodo.Text = dataFilas["Codigo"].ToString().Trim() + "  " + dataFilas["Nombre"].ToString().Trim();
            nuevoNodo.Name = dataFilas["Nombre"].ToString().Trim();
            //En la propiedad Tag almaceno el índice que lo asocia a la Fila correspondiente
            nuevoNodo.Tag = dataFilas["IdPCuentas"].ToString();
            e.Node.Nodes.Add(nuevoNodo);
        }
    }
}

No obstante te recomiendo probar lo primero y si no te satisface puedes aplicar la segunda solución.


Otra cosa muy diferente seria agregar los nodos, solo cuando el usuario expanda los nodos, para ello se puede usar el evento BeforeExpand del TreeView y agregar en ese evento solo sus nodos hijos, también es conveniente en este caso declarar las variables DataTable dataTableNodos, DataView dataViewNodosHijos, al nivel del formulario, cargándolas en el Load.

void treeView1_BeforeExpand(object sender, System.Windows.Forms.TreeViewCancelEventArgs e)
{
    //Solo se agregaran los nodos si no los tiene ya agregados
    if (e.Node.Nodes.Count == 0)
    {
        //Recuperamos la propiedad Tag del nodo donde hemos almacenamos su índice, como veras más abajo
        dataViewNodosHijos.RowFilter = dataTableNodos.Columns["IdPadre"].ColumnName + " = " + Int32.Parse(e.Node.Tag.ToString());

        foreach (DataRowView dataFilas in dataViewNodosHijos)
        {
            TreeNode nuevoNodo = new TreeNode();
            nuevoNodo.Text = dataFilas["Codigo"].ToString().Trim() + "  " + dataFilas["Nombre"].ToString().Trim();
            nuevoNodo.Name = dataFilas["Nombre"].ToString().Trim();
            //En la propiedad Tag almaceno el índice que lo asocia a la Fila correspondiente
            nuevoNodo.Tag = dataFilas["IdPCuentas"].ToString();
            e.Node.Nodes.Add(nuevoNodo);
        }
    }
}

No obstante te recomiendo probar lo primero y si no te satisface puedes aplicar la segunda solución.

Origen Enlace
Eduardo Reyes
  • 1.3k
  • 1
  • 5
  • 14

Pienso que puedes probar sacar las dos primeras lineas fuera de tu función CrearNodos esas dos lineas tal como están ejecutaran exactamente lo mismo en cada llamada que se haga a la función recursiva CrearNodos. Por tanto no tienen por que estar dentro función, deberías incluir esas lineas en el punto anterior en donde realizas la llamada a la función recursiva, es decir dentro del Load del formulario

 private void Form1_Load(object sender, EventArgs e)
 {
     DataTable dataTableNodos = mCuentas.RecogerDatos(nombreBDUsada);
     DataView dataViewNodosHijos = new DataView(dataTableNodos);
     //Aquí realizas la llamada a la función CrearNodos.
     //pasando como argumento los objetos dataTableNodos, dataViewNodosHijos
     CrearNodos(dataTableNodos, dataViewNodosHijos, 0, null); 
 }

private void CrearNodos(DataTable dataTableNodos, DataView dataViewNodosHijos, int indicePadre, TreeNode nodePadre)
{
        dataViewNodosHijos.RowFilter = dataTableNodos.Columns["IdPadre"].ColumnName + " = " + indicePadre;

        foreach (DataRowView dataFilas in dataViewNodosHijos)
        {
            TreeNode nuevoNodo = new TreeNode();
            nuevoNodo.Text = dataFilas["Codigo"].ToString().Trim() + "  " + dataFilas["Nombre"].ToString().Trim();
            nuevoNodo.Name = dataFilas["Nombre"].ToString().Trim();
            nuevoNodo.Tag = dataViewNodosHijos.Table.Rows.IndexOf(dataFilas.Row);

            if (nodePadre == null)
            {
                treeView1.Nodes.Add(nuevoNodo);
            }
            else
            {
                nodePadre.Nodes.Add(nuevoNodo);
            }
            CrearNodos(dataTableNodos, dataViewNodosHijos, Int32.Parse(dataFilas["IdPCuentas"].ToString()), nuevoNodo);
        }
}

En una función recursiva, es una regla elemental que no deben incluirse lineas de código, que ejecuten exactamente lo mismo en cada iteración, pues solo servirían para consumir tiempo de computo innecesario, ten en cuenta que si como dices tu Treeview tiene 352 nodos, esas dos lineas se estarían ejecutando 352 veces, tantas veces como se llame a la función recursiva.

Tal ves este arreglo no resuelva todos los problemas de optimización y la carga de los nodos no sea todo lo rápida que deseas, pero debe mejorar.