Skip to main content
se añadieron 726 caracteres en el cuerpo
Origen Enlace

Bienvenido a StackOVerflow, y bienvenido a la programacion :)

El problem en tu codigo es que personas.Count se va agrandando adentro del loop, porque vuelves a agregar la persona una y otra vez. o sea, tu codigo hace eso:

 for (int i = 0; i <= personas.Count; i++)
 {            
     personas.Add(per);
 }

Aqui un codigo de ejemplo de como podrias hacerlo. Hay mejores maneras de hacer esto, pero intente hacer el codigo de la manera mas sencilla posible :)

public void LimpiarListar(List<Persona> personas) 
{

    // Primero tienes que limpiar
    tablePersona.Controls.Clear();

    // Luego agregar todas las personas
    tablePersona.ColumnCount = 6;

    tablePersona.Columns[0].Name = "Nombre";
    tablePersona.Columns[1].Name = "Apellidos";
    tablePersona.Columns[2].Name = "Sexo";
    tablePersona.Columns[3].Name = "Fecha";
    tablePersona.Columns[4].Name = "Cuidad";
    tablePersona.Columns[5].Name = "Estado";

    foreach(Persona pers in personas)
    {    
        tablePersona.Rows.Add(per.Nombre,per.Apellidos,per.Sexo,per.Fecha,per.Cuidad,per.Estado);
    }
}

private void btnActualizar_Click(object sender, EventArgs e)
{            
    foreach(var persona in personas)
    {
        if (txtNombre.Text == persona.Nombre)
        {
            persona.Apellidos = txtApellidos.Text;
            persona.Sexo = txtSexo.Text;
            persona.Fecha = txtFecha.Text;
            persona.Cuidad = txtCiudad.Text;
            persona.Estado = txtEstado.Text;

            // persona ya esta en la lista, asi que no hay que actualizar la lista
        }

    }
    listar(per);
    limpiarlimpiarYlistar(personas);
}

Bienvenido a StackOVerflow, y bienvenido a la programacion :)

El problem en tu codigo es que personas.Count se va agrandando adentro del loop, porque vuelves a agregar la persona una y otra vez. o sea, tu codigo hace eso:

 for (int i = 0; i <= personas.Count; i++)
 {            
     personas.Add(per);
 }

Aqui un codigo de ejemplo de como podrias hacerlo. Hay mejores maneras de hacer esto, pero intente hacer el codigo de la manera mas sencilla posible :)

private void btnActualizar_Click(object sender, EventArgs e)
{            
    foreach(var persona in personas)
    {
        if (txtNombre.Text == persona.Nombre)
        {
            persona.Apellidos = txtApellidos.Text;
            persona.Sexo = txtSexo.Text;
            persona.Fecha = txtFecha.Text;
            persona.Cuidad = txtCiudad.Text;
            persona.Estado = txtEstado.Text;

            // persona ya esta en la lista, asi que no hay que actualizar la lista
        }

    }
    listar(per);
    limpiar();
}

Bienvenido a StackOVerflow, y bienvenido a la programacion :)

El problem en tu codigo es que personas.Count se va agrandando adentro del loop, porque vuelves a agregar la persona una y otra vez. o sea, tu codigo hace eso:

 for (int i = 0; i <= personas.Count; i++)
 {            
     personas.Add(per);
 }

Aqui un codigo de ejemplo de como podrias hacerlo. Hay mejores maneras de hacer esto, pero intente hacer el codigo de la manera mas sencilla posible :)

public void LimpiarListar(List<Persona> personas) 
{

    // Primero tienes que limpiar
    tablePersona.Controls.Clear();

    // Luego agregar todas las personas
    tablePersona.ColumnCount = 6;

    tablePersona.Columns[0].Name = "Nombre";
    tablePersona.Columns[1].Name = "Apellidos";
    tablePersona.Columns[2].Name = "Sexo";
    tablePersona.Columns[3].Name = "Fecha";
    tablePersona.Columns[4].Name = "Cuidad";
    tablePersona.Columns[5].Name = "Estado";

    foreach(Persona pers in personas)
    {    
        tablePersona.Rows.Add(per.Nombre,per.Apellidos,per.Sexo,per.Fecha,per.Cuidad,per.Estado);
    }
}

private void btnActualizar_Click(object sender, EventArgs e)
{            
    foreach(var persona in personas)
    {
        if (txtNombre.Text == persona.Nombre)
        {
            persona.Apellidos = txtApellidos.Text;
            persona.Sexo = txtSexo.Text;
            persona.Fecha = txtFecha.Text;
            persona.Cuidad = txtCiudad.Text;
            persona.Estado = txtEstado.Text;

            // persona ya esta en la lista, asi que no hay que actualizar la lista
        }

    }
    limpiarYlistar(personas);
}
se eliminaron 2 caracteres en el cuerpo
Origen Enlace

Bienvenido a StackOVerflow, y bienvenido a la programacion :)

El problem en tu codigo es que personas.Count se va agrandando adentro del loop, porque vuelves a agregar la persona una y otra vez. o sea, tu codigo hace eso:

 for (int i = 0; i <= personas.Count; i++)
 {            
     personas.Add(per);
 }

Aqui un codigo de ejemplo de como podrias hacerlo. Hay mejores maneras de hacer esto, pero intente hacer el codigo de la manera mas sencilla posible :)

private void btnActualizar_Click(object sender, EventArgs e)
{            
    foreach(var persona in personas)
    {
        if (txtNombre.Text == persona.Nombre)
        {
            persona.Apellidos = txtApellidos.Text;
            persona.Sexo = txtSexo.Text;
            persona.Fecha;Fecha = txtFecha.Text;
            persona.Cuidad = txtCiudad.Text ;Text;
            persona.Estado = txtEstado.Text;

            // persona ya esta en la lista, asi que no hay que actualizar la lista
        }

    }
    listar(per);
    limpiar();
}

Bienvenido a StackOVerflow, y bienvenido a la programacion :)

El problem en tu codigo es que personas.Count se va agrandando adentro del loop, porque vuelves a agregar la persona una y otra vez. o sea, tu codigo hace eso:

 for (int i = 0; i <= personas.Count; i++)
 {            
     personas.Add(per);
 }

Aqui un codigo de ejemplo de como podrias hacerlo. Hay mejores maneras de hacer esto, pero intente hacer el codigo de la manera mas sencilla posible :)

private void btnActualizar_Click(object sender, EventArgs e)
{            
    foreach(var persona in personas)
    {
        if (txtNombre.Text == persona.Nombre)
        {
            persona.Apellidos = txtApellidos.Text;
            persona.Sexo = txtSexo.Text;
            persona.Fecha; = txtFecha.Text;
            persona.Cuidad = txtCiudad.Text ;
            persona.Estado = txtEstado.Text;

            // persona ya esta en la lista, asi que no hay que actualizar la lista
        }

    }
    listar(per);
    limpiar();
}

Bienvenido a StackOVerflow, y bienvenido a la programacion :)

El problem en tu codigo es que personas.Count se va agrandando adentro del loop, porque vuelves a agregar la persona una y otra vez. o sea, tu codigo hace eso:

 for (int i = 0; i <= personas.Count; i++)
 {            
     personas.Add(per);
 }

Aqui un codigo de ejemplo de como podrias hacerlo. Hay mejores maneras de hacer esto, pero intente hacer el codigo de la manera mas sencilla posible :)

private void btnActualizar_Click(object sender, EventArgs e)
{            
    foreach(var persona in personas)
    {
        if (txtNombre.Text == persona.Nombre)
        {
            persona.Apellidos = txtApellidos.Text;
            persona.Sexo = txtSexo.Text;
            persona.Fecha = txtFecha.Text;
            persona.Cuidad = txtCiudad.Text;
            persona.Estado = txtEstado.Text;

            // persona ya esta en la lista, asi que no hay que actualizar la lista
        }

    }
    listar(per);
    limpiar();
}
se añadieron 90 caracteres en el cuerpo
Origen Enlace

Bienvenido a StackOVerflow, y bienvenido a la programacion :)

El problem en tu codigo es que personas.Count se va agrandando adentro del loop, porque vuelves a agregar la persona una y otra vez. o sea, tu codigo hace eso:

 for (int i = 0; i <= personas.Count; i++)
 {            
     personas.Add(per);
 }

Aqui un codigo de ejemplo de como podrias hacerlo. Hay mejores maneras de hacer esto, pero intente hacer el codigo de la manera mas sencilla posible :)

private void btnActualizar_Click(object sender, EventArgs e)
{            
    foreach(var persona in personas)
    {
        if (txtNombre.Text == persona.Nombre)
        {
            persona.Apellidos = txtApellidos.Text;
            persona.Sexo = txtSexo.Text;
            persona.Fecha; = txtFecha.Text;
            persona.Cuidad = txtCiudad.Text ;
            persona.Estado = txtEstado.Text; 

            // persona ya esta en la lista, asi que no hay que actualizar la lista
        }

    }
    listar(per);
    limpiar();
}

Bienvenido a StackOVerflow, y bienvenido a la programacion :)

El problem en tu codigo es que personas.Count se va agrandando adentro del loop, porque vuelves a agregar la persona una y otra vez. o sea, tu codigo hace eso:

 for (int i = 0; i <= personas.Count; i++)
 {            
     personas.Add(per);
 }

Aqui un codigo de ejemplo de como podrias hacerlo. Hay mejores maneras de hacer esto, pero intente hacer el codigo de la manera mas sencilla posible :)

private void btnActualizar_Click(object sender, EventArgs e)
{            
    foreach(var persona in personas)
    {
        if (txtNombre.Text == persona.Nombre)
        {
            persona.Apellidos = txtApellidos.Text;
            persona.Sexo = txtSexo.Text;
            persona.Fecha; = txtFecha.Text;
            persona.Cuidad = txtCiudad.Text ;
            persona.Estado = txtEstado.Text;
        }

    }
    listar(per);
    limpiar();
}

Bienvenido a StackOVerflow, y bienvenido a la programacion :)

El problem en tu codigo es que personas.Count se va agrandando adentro del loop, porque vuelves a agregar la persona una y otra vez. o sea, tu codigo hace eso:

 for (int i = 0; i <= personas.Count; i++)
 {            
     personas.Add(per);
 }

Aqui un codigo de ejemplo de como podrias hacerlo. Hay mejores maneras de hacer esto, pero intente hacer el codigo de la manera mas sencilla posible :)

private void btnActualizar_Click(object sender, EventArgs e)
{            
    foreach(var persona in personas)
    {
        if (txtNombre.Text == persona.Nombre)
        {
            persona.Apellidos = txtApellidos.Text;
            persona.Sexo = txtSexo.Text;
            persona.Fecha; = txtFecha.Text;
            persona.Cuidad = txtCiudad.Text ;
            persona.Estado = txtEstado.Text; 

            // persona ya esta en la lista, asi que no hay que actualizar la lista
        }

    }
    listar(per);
    limpiar();
}
Origen Enlace
Loading