¿Es una mala práctica escribir manipuladores de eventos en línea

Question

¿Es una mala práctica escribir manejadores de eventos en línea?

Para mí, prefiero usarlo cuando quiera utilizar una variable local en el controlador de eventos como el siguiente:

Yo prefiero esta:

// This is just a sample 
private void Foo() 
{ 
    Timer timer = new Timer() { Interval = 1000 }; 
    int counter = 0; // counter has just this mission 
    timer.Tick += (s, e) => myTextBox.Text = (counter++).ToString(); 
    timer.Start(); 
} 

lugar de esto:

int counter = 0; // No need for this out of Boo & the event handler 

private void Boo() 
{ 
    Timer timer = new Timer() { Interval = 1000 }; 

    timer.Tick += timer_Tick; 
    timer.Start(); 
} 

void timer_Tick(object sender, EventArgs e) 
{ 
    myTextBox.Text = (counter++).ToString(); 
} 

Answer 1

Está absolutamente bien, aunque hay dos advertencias:

• Si está modificando una variable local desde un cierre, debe asegurarse de comprender lo que está haciendo.
• Usted no será capaz de dejar de recibir el evento

Normalmente yo sólo en línea realmente controladores de eventos simples - para nada más involucrados, utilizo expresiones lambda (o métodos anónimos) a suscribir con una llamada a un método con un método más apropiado:

// We don't care about the arguments here; SaveDocument shouldn't need parameters 
saveButton.Click += delegate { SaveDocument(); }; 

Answer 2

Junte las dos muestras. Está claro que la segunda opción (que no prefieres) es la más legible.

La legibilidad del código y el mantenimiento son muy importantes. Mantenga las cosas simples, lo más fácil posible de entender. Las expresiones de Lambda generalmente son consideradas más difíciles de entender por la mayoría de las personas. Incluso si son una segunda naturaleza para ti, para otros podría no serlo.

Answer 3

En la mayoría de los casos yo preferiría tener los métodos separadas como “timer_Tick()”, sin embargo, debería más bien ser llamado OnTimerTick() como:

• Cuando leí la clase, que es el trigo más clara es pasando. El "Encendido" me dice que puede manejar el evento.
• Es más fácil establecer un punto de inflexión en el método en el caso "en línea".
• El evento se inició mucho después de que el contratista "Foo" haya regresado, y no creo que esté funcionando dentro del alcance del contratista.

Sin embargo, si el evento sólo será despedido antes de que el método se declara retornos en línea y el objeto del evento se establece en tiene un alcance lo que se limita al método de declarar, a continuación, creo que el “en línea "versión es mejor. Por lo tanto, me gusta usar "en línea" para que el delegado de comparación pase a un método de "clasificación".