Foro de elhacker.net

Hola, tengo un listbox en el que se almacenarán nombres. Entonces para agregar los nombres he creado un botón "añadir" que al presionarlo carga un formulario simple con un textBox en el que se introduce el nombre y al darle a aceptar quiero que se añada al listBox.

Como puedo hacerlo??? porque no consigo ver desde el método del botón aceptar del nuevo formuladio el listBox....



gracias de antemano.

dejame ver si te entendi

capturas el nombre en un textbox y cuando le des clic en aceptar se añada al listbox
es muy facil
aki te dejo el codigo

 private void button1_Click(object sender, EventArgs e)
        {
            listBox1.Items.Add(textBox1.Text);
 
        }
private void button2_Click(object sender, EventArgs e)
        {
            textBox1.Text = "";
        }
 

espero te sirve
saludos

En la forma que lo haces tú (DaNuK), sería la forma correcta si estuviese dentro del mismo formulario. Pero como bien lo dijo Hartigan, es un formulario aparte. Intentad buscar 'Instanciar formularios en c-sharp'. Será algo como estó:

 
   frmFormulario1 frmForm1 = new frmFormulario1();
   frmForm1.Show();
 
 

Recomendación: No usen "" para indicar un String vacío. Usen String.Empty. Ya que es estático y es una sola instancia de "".

Saludos!

como no te entiendo D4n3R sobre string.empty podrias poner un ejemplo porfas

Supongamos que quieres hacer algo como estó:

if(strVacio != ""){
  //Acciones
}

La mejor forma de hacerlo con String.Empty es así:

if(strVacio != String.Empty){
  //Acciones
}

Aunque creo que da igual la forma en que lo uses, no le veo la diferencia entre usar una u otra. Tal vez sea estilo de programación.

Gracias amigo

Hola,

Debes usar "" cuando lo comparas con un valor en loop, ya que el runtime conoce el valor de "" justo antes del JIT.

Acuerdense que en .Net todo es un objeto por lo que cada vez que usas "" creas uno. Por lo cual es un poco más eficiente usar String.Empty que "".

Caemos en un dilema, un loop de 10.000, si usamos "" creamos 10.000 objetos más; si usamos String.Empty no los creamos, pero cada vez que se acceda a la propiedad se pierde un poco de tiempo.

Es cuestión de evaluar.

Saludos!

Por cierto la opción más rápida es .Length == 0

Bueno pues volviendo un poco al tema, con respecto a el problema de Hartigan parece que lo que se necesita es una variable publica en el formulario 2 para que haga referencia al listbox del formulario 1

Bueno yo lo haria asi:

Dentro de la clase de mi formulario 2 se declara una variable publica del tipo ListBox:

public ListBox MyList;
 

Despues al instanciar el formulario 2 desde el formulario 1 hacemos esto para asignarla:

Form2 frmNombres = new Form2();
frmNombres.MyList = listBox1;
frmNombres.Show();
 

Donde el ListBox1 es donde se quieren agregar los nombres.

Y ya en el codigo del boton que agrega el nombre en el formulario 2 hacemos esto:

        private void button1_Click(object sender, EventArgs e)
        {
            MyList.Items.Add(textBox1.Text);
        }
 

En teoria debe de funcionar, lo que se hace es pasar el objeto listbox del form 1 como referencia al  form 2 mediante una variable publica, al momento de asignarlo son 2 variables de distintas clases que hacen referencia al mismo objeto, ya que jamas creas la instancia nueva con "new", al agregar los nombres desde el form2 debera aparecer en el listbox del form 1.

Nota: Si nunca haces la asignacion frmNombres.MyList = listBox1 tendras un error de referencia nula.

Saludos.

Si, pero igual vuelvo yo jeje..

No uses miembros públicos, utiliza propiedades para acceder a éstos.

public ListBox MyListBox
{
     set { m_myListBox = value; }
     get { return m_myListBox; }
}

Gacias a todos por las respuestas pero me da el siguiente error haciendo lo que dice Xtriker:


Como sería usando la propiedad D4N93R????  esque aun soy bastante novato en esto y no las controlo muy bien. xDD

Salu2.

Asi tal y como lo puso D4N93R, declaras tu miembro privado y accedes a el desde la propiedad:

private Listbox m_myListBox;
 
public ListBox MyListBox
{
     set { m_myListBox = value; }
     get { return m_myListBox; }
 
}
 

lo de la referencia nula te lo marca por que MyListBox tiene el valor Null y no esta asociada con ninguna instancia, pero no se exactamente donde este el problema recien lo probe y me funciono bien, tal ves si pones la parte de tu codigo para checarlo podemos verificar que esta mal.

Por que no colocas un punto de interrupcion en donde asignas el Listbox al miembro publico o a la propiedad, y checas los valores de las variables hasta ese momento y vas ejecutando paso por paso para ver por que se queda tu objeto como null.

Este es el código:

En el formulario 1, tengo esto:

namespace Agenda_1._0.IGU
{    
    public partial class Interfaz_NuevoContacto : Form
    {
 
        public Interfaz_NuevoContacto()
        {            
            InitializeComponent();
 
            Interfaz_NuevoEmail frmNuevoEmail = new Interfaz_NuevoEmail();
            frmNuevoEmail.myListBox = listBox_Email;
            frmNuevoEmail.Show();
 
        }
 
        private void button_AañadirEmail_Click(object sender, EventArgs e)
        {
            Interfaz_NuevoEmail interfazNuevoEmail = new Interfaz_NuevoEmail();
            interfazNuevoEmail.Show();
        }
.....
 

en el formulario 2 tengo:

namespace Agenda_1._0.IGU
{
    public partial class Interfaz_NuevoEmail : Form
    {
        //Variable publica para copiar datos al Form Nuevo_contacto
        public ListBox myListBox;
 
 
        public Interfaz_NuevoEmail()
        {
            InitializeComponent();            
        }        
...
 
        private void button_AñadirCE_Click(object sender, EventArgs e)
        {
            myListBox.Items.Add(textBox_AñadirCE.Text);
        }
...
 
}
 
 

De momento lo tengo puesto con la variable pública, en cuanto me funcione lo cambio por la propieda.

Salu2.

Supongo que el error te lo da en myListBox.Items.Add(textBox_AñadirCE.Text);

Tienes que asegurarte de que en frmNuevoEmail.myListBox = listBox_Email;   listBox_Email no este nulo..

Efectivamente el error me lo da ahí. Pero como hago que no esté nulo si primeramente quiero que el listBox este vacio, es decir que no haya ningun email agregado...???

Bueno, al parecer el error esta aqui:

Al crear una nueva instancia de la clase Interfaz_NuevoEmail, myListBox esta con valor null, entonces en esta parte del codigo no le asignas nada a myListBox

 
        private void button_AañadirEmail_Click(object sender, EventArgs e)
        {
            Interfaz_NuevoEmail interfazNuevoEmail = new Interfaz_NuevoEmail();
            'Aqui faltaria interfazNuevoEmail.myListBox = listBox_Email 
            interfazNuevoEmail.Show();
        }
 
 

Por curiosidad por que creas una instancia del Interfaz_NuevoEmail en el constructor?

listBox_Email en un listbox en el formulario, o simplemente una variable tipo listbox?

Te toca mejorarlo pero puedes probar asi:

Form1:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
 
namespace Mail
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
            Form2 f2 = new Form2();
            DialogResult resul = new DialogResult();
            resul = f2.ShowDialog();
            if (resul == DialogResult.OK && f2.mail != string.Empty)
            {
                listBox1.Items.Add(f2.mail);
            }
        }
    }
}
 

Form2:

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
 
namespace Mail
{
    public partial class Form2 : Form
    {
        public string mail;
 
        public Form2()
        {
            InitializeComponent();
        }
 
        private void Form2_Load(object sender, EventArgs e)
        {
            button1.DialogResult = DialogResult.OK;
        }
 
        private void button1_Click(object sender, EventArgs e)
        {
            mail = textBox1.Text;
        }
    }
}
 

Lo hago para poder mostrar el segundo formulario. Es decir, el primer formulario lo llamo Interfaz_NuevoContacto y el segundo Interfaz_NuevoEmail. De esa manera cargo al pulsar el botón... xD


Es el nombre del listBox dentro del fomulario 1. osea del fomulario Interfaz_Nuevo contacto.



Gracias, lo pruebo y te comento.

Gracias a todos por la ayuda, probaré a ver lo que me decis a ver si funciona.

Bueno pues resulta que efectivamente había colocado mal las cosas y añadiendo lo que dice Xtriker me ha funcionado. xDD jajaja. asíque ya lo tengo.

43H4FH44H45H4CH49H56H45H lo de los Dialog no es para los cuadros de dialogo???. pregunto porque no lo se. xDD como ya he dicho soy aun novato en esto.

Otra pregunta, me han comentado que estas cosas se hacen mediante los delegados. Que son??? es esto que hemos hecho???. xDD


Muchas gracias a todos por la ayuda, sois unos cracks. xDD

Si utilizas DialogShow y dialogresult, te permite añadir items al listbox solo si presiona el boton con dialogresult del 2do form, asi puedes hacer todas las validaciones que necesites desde el primer form, la verdad tiene bastantes usos  ;D

Bueno chicos ya me funciona perfectamente usando una propiedad en lugar de la variable pública. xDD

Muchas gracias a todos.

La solución de como lo he hecho es poner lo mismo que me han dicho los compañeros.

salu2.

Otra vez yo acá jejeje

No coloques la palabra interfaz en los nombres de los formularios, utiliza otra palabra, como form, formulario, ventana, o lo que quieras, ya que una interfaz, interface en programación es otra cosa:

http://msdn.microsoft.com/en-us/library/ms173156.aspx (http://msdn.microsoft.com/en-us/library/ms173156.aspx)
http://msdn.microsoft.com/en-us/library/87d83y5b(VS.80).aspx (http://msdn.microsoft.com/en-us/library/87d83y5b(VS.80).aspx)

Un saludo!

mmm yo esque los llamo interfaz puesto que van a mostrar las interfaces de mi programa, es decir los menús y demás historias. pero quizás tengas razón y debería llamarlos de otra menera... xD

Entonces el nombre completo sería interfaz de usuario :)

Es solo una recomendación,

Un saludo!

Ok, gracias ;)

Tengo una nueva duda con un comboBox. La pregunto aquí para no andar creando otro hilo.

Como puedo hacer para que el comboBox muestre un elemento de la lista desde el principio????.

Salu2.

comboBox.SelectedIndex = al numero que quieras,  o comboBox.SelectedItem = al objeto, en caso de que metas en los Items puros objetos y entonces en la clase de esos objetos sobre escribes el método ToString y retornas un campo del objeto..

Oki, muchas gracias, me funcionó perfecto. xDD

Salu2.  ;-)

Que bueno!