2

Necesito converitir un ComboBox al cual llame cmbmes. La realidad es que no me funciona con Convert.ToInt32() ni con Int.Parse(). El error que me da en especifico es System.FormatException.

Lo que quiero lograr es que al seleccionar un Item (Numero) del ComboBox me diga en un TextBox el Mes.

El codigo es este:

    private void Form1_Load(object sender, EventArgs e)
    {
        //Numero de Meses del Año
        for (int i = 1; i <= 12; i++)
        {
            cmbmes.Items.Add(i);
        }

        //Para seleccionar el Mes
        int numero;                       
        numero = int.Parse(cmbmes.Text);

        if (numero == 1)
        {
            txtmes.Text = "Enero";
        }
        else if (numero == 2)
        {
            txtmes.Text = "Febrero";
        }

    }

Si conocen otro metodo, se los agradeceria mucho.

1
  • 1
    Esto es winforms? wpf? mas alla de eso, porque queres convertir un numero de mes en el texto, cuando al usuario deberias mostrarle el texto y tener los value como numeros?
    – gbianchi
    Commented el 30 may. 2018 a las 15:45

3 respuestas 3

8

A pesar de que parece que lo has resuelto, voy a añadir una respuesta por si alguien se encuentra en la misma situación y no quiere reinventar la rueda. En .net existe el método GetMonthName(int) que te devuelve el nombre del mes pasándole un entero con el numero de mes. En tu caso el código se reduciría a:

//Hay que añadir este using
using System.Globalization;
...
private void cmbmes_SelectedIndexChanged(object sender, EventArgs e)
{
    txtmes.Text = CultureInfo.CurrentCulture.DateTimeFormat.GetMonthName(int.Parse(cmbmes.Text));
}

Si necesitas que la primera letra del mes esté en mayúsculas, simplemente añade debajo de txtmes.text = ... esta línea:

txtmes.Text = txtmes.Text.First().ToString().ToUpper() + txtmes.Text.Substring(1);
2
  • Solo como nota, vale muchisimo mas cargar el combo con los valores y los textos correspondientes y no hacer todos estos malabares despues.
    – gbianchi
    Commented el 30 may. 2018 a las 16:24
  • 3
    Joder. Si que eres bueno Commented el 30 may. 2018 a las 16:28
0
int numero;

private void Form1_Load(object sender, EventArgs e)
    {
        //Numero de Meses del Año
        for (int i = 1; i <= 12; i++)
        {
            cmbmes.Items.Add(i);
        }
    }

private void cmbmes_SelectionChangeCommitted(object sender, EventArgs e)
{
    //Para seleccionar el Mes                      
        numero = Convert.ToInt32(cmbmes.Text);

        if (numero == 1)
        {
            txtmes.Text = "Enero";
        }
        else if (numero == 2)
        {
            txtmes.Text = "Febrero";
        }
}

Prueba con esto... tengo rato que no toco c# pero es algo así.. te explico un poco el código

En el evento load del form añadimos los items a tu combobox

Y luego en el evento SelectionChangeCommitted de tu combobox recogemos el numero que hemos seleccionado de este, y ya solo comparamos y desplegamos en tu textbox el mes dependiendo del número...

-1

pero al fin logre encontrar la respuesta sin error.

    private void Form1_Load(object sender, EventArgs e)
    {
        //Numero de Meses del Año
        for (int i = 1; i <= 12; i++)
        {
            cmbmes.Items.Add(i);
        }
    }

    private void txtmes_TextChanged(object sender, EventArgs e)
    {           
    }

    private void cmbmes_SelectedIndexChanged(object sender, EventArgs e)
    {
        if (Convert.ToInt32(cmbmes.Text)==1)
        {
            txtmes.Text = "Enero";
        }
        else if (Convert.ToInt32(cmbmes.Text) == 2)
        {
            txtmes.Text = "Febrero";
        }
        else if (Convert.ToInt32(cmbmes.Text) == 3)
        {
            txtmes.Text = "Marzo";
        }
        else if (Convert.ToInt32(cmbmes.Text) == 4)
        {
            txtmes.Text = "Abril";
        }
        else if (Convert.ToInt32(cmbmes.Text) == 5)
        {
            txtmes.Text = "Mayo";
        }
        else if (Convert.ToInt32(cmbmes.Text) == 6)
        {
            txtmes.Text = "Junio";
        }
        else if (Convert.ToInt32(cmbmes.Text) == 7)
        {
            txtmes.Text = "Julio";
        }
        else if (Convert.ToInt32(cmbmes.Text) == 8)
        {
            txtmes.Text = "Agosto";
        }
        else if (Convert.ToInt32(cmbmes.Text) == 9)
        {
            txtmes.Text = "Septiembre";
        }
        else if (Convert.ToInt32(cmbmes.Text) == 10)
        {
            txtmes.Text = "Octubre";
        }
        else if (Convert.ToInt32(cmbmes.Text) == 11)
        {
            txtmes.Text = "Noviembre";
        }
        else if (Convert.ToInt32(cmbmes.Text) == 12)
        {
            txtmes.Text = "Diciembre";
        }
    }
}

}

2
  • 1
    Menuda cadena de if esa... Todo eso puede ser resumido a 3 lineas de código, como minimo...
    – NaCl
    Commented el 30 may. 2018 a las 16:22
  • 1
    Aunque todo esto es muy lindo, y la solucion de pikoh es muchisimo mejor que esta, un combo puede tener toda esta informacion de antemano sin necesidad de todo esto!
    – gbianchi
    Commented el 30 may. 2018 a las 16:23

Tu Respuesta

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

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