Define tu variable timer a nivel de clase para que no estes creando otra instancia al iniciar/detener el timer. De esta forma al cambiar el valor de flag al llamar el método nuevamente, podras detener/iniciar el timer sin problema.
//Define variable timer.
private System.Timers.Timer timer = new System.Timers.Timer(TimeSpan.FromSeconds(5).TotalMilliseconds);
//Metodo para iniciar/detener el timer
private void btnControlaTimer()
{
if (flag == true) {
Trace.WriteLine("entra a flag == true");
timer.Elapsed += async (sender, ea) =>
{
prueba();
};
timer.Start();
flag = false;
}
else
{
timer.Stop();
flag = true;
}
}
Incluso para que no se repita el llamado a "prueba()" , debes inicializar en otro lugar timer.Elapsed, ejemplo:
private bool flag = true;
private System.Timers.Timer timer = new System.Timers.Timer(TimeSpan.FromSeconds(5).TotalMilliseconds);
public MainWindow()
{
InitializeComponent();
timer.Elapsed += async (sender, ea) =>
{
prueba();
};
}
private void btnStart_Click(object sender, RoutedEventArgs e)
{
if (flag == true) {
/*timer.Elapsed += async (sender, ea) =>
{
prueba();
};*/
timer.Start();
flag = false;
}
else
{
timer.Stop();
flag = true;
}
}