0

Hago una calculadora y quiero simplificar el codigo con el tema de los eventos al hacer click en un button Para no tener un evento Click por cada boton realizar un solo evento y aignarlo a todos los buttony asi que me detecte que button fue presionado y coloque su texto correspondiente en el label

LblPantallaPrincipal.Text = this.Text.ToString();

Dentro de el evento Click de un button pero me asigna el texto del formulario

La idea es no tener el

LblPantallaPrincipal.Text = btn1.Text.ToString();

En cada button cambiando el btn2 - btn3 - btn4 y asi

¿Hay alguna forma de hacerlo? ya que tengo 10 botones y un evento por cada uno, quisiera todos funcionen con el mismo evento y en dicho evento colocar un switch que me indique que button fue

2
  • No creo que lo que pretendes hacer sea correcto(una cosa es que funcione y otra cosa es que tenga sentido). Cada botón tiene su propio código diferente al resto, como pretendes que un mismo evento o método o lo que sea tenga el código propio de cada botón, ese hipotético método sería enorme y rompería con el principio de alta cohesión
    – Japv
    Commented el 16 feb. 2021 a las 1:35
  • Seria un código sencillo por lo que quise ver si había un modo de hacerlo... Ya que el boton solo colocaria el valor de su texto a un label, en vez de tener 10 botones con el codigo .Text.ToString(); Tener un solo evento para dichos botones Commented el 16 feb. 2021 a las 2:14

1 respuesta 1

2

Podrías obtenerlo de los parámetros del mismo evento:

private void onClick(object sender, System.EventArgs e)
{
   var btn = (Button)sender;
   LblPantallaPrincipal.Text = btn.Text.ToString();
}

Si tienes que realizar lógica más complicada:

private void onClick(object sender, System.EventArgs e)
{
   var btn = (Button)sender;
   switch (btn.Name) 
   {
        case "button1":
            ...
            break;
        case "button2":
            ...
            break;
        case "button3":
            ...
            break;
        default:
            break;
    }
}

Ten en cuenta que en el caso de que sean muchísimos más botones, el switch se vuelve inmantenible. En ese caso, quizá herencia y polimorfismo serian la opción a elegir pero depende totalmente de tu problema a resolver.

7
  • No puedo colocar el "btn" ya que no es un objeto tengo que colocar el objeto en ese caso el button (btn1 al 9) y si coloco e button como tal me va a colocar el texto del mismo... Commented el 16 feb. 2021 a las 0:57
  • 1
    @Javier Gonzalez ¿al menos probaste la respuesta de @Carlos antes de decir que no se puede?. Te recomiendo investigar qué es el sender en un evento y el significado de la instrucción var btn = (Button)sender; Commented el 16 feb. 2021 a las 1:11
  • Esta solución no es óptima y si fueran 100 botones, harias un Switch con 100 cases y si fueran 1000?
    – Japv
    Commented el 16 feb. 2021 a las 1:30
  • En caso de ser tantos botones si seria un problema, pero siendo 10 como en mi caso serviría, la cosa es que en C# debo declarar la variable en un tipo especifico, ejemplo un entero o flotante o string y me lo indicaron en Java asi que ahi no lo supe aplicar... Puede que no entienda alguna cosas soy bastante nuevo Commented el 16 feb. 2021 a las 1:35
  • @JavierGonzalez, a la larga eso va a ser un dolor de cabeza
    – Japv
    Commented el 16 feb. 2021 a las 1:38

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.