Бесконечный цикл c Timer start/stop

private readonly System.Timers.Timer timer1 = new System.Timers.Timer();

timer1.Interval = 250;
timer1.Elapsed += RunCoinValidator;

public void StartDevice()
{
    timer1.Stop();
    RunCoinValidator();
}

private void RunCoinValidator(object sender = null, System.Timers.ElapsedEventArgs e = null)
{
    timer1.Stop();
    try
    {
        while (Running)
        {
            // тут 1 триллиард строк
        }
    }
    catch
    {
        Running = false;
    }
}

timer1.Stop(); // здесь вы таймер остановили
    try
    {
        while (Running) // а тут пошел бесконечный цикл который и тормозит всю UI
        {
            // тут 1 триллиард строк
        }
    }

while (Running)
...

private readonly IValidator coinValidator;
 
//инициализация в конструкторе
public ...
{
    coinValidator = new EmptyCoinValidatorManager();
    if(/*тыры-пыры*/)
    {
        coinValidator = CoinValidatorManager(/* тыры-пыры */);
    }
 
 
    public IValidator GetCoinValidator()
    {
        return coinValidator;
    }
 
}

//ну и запуск
validatorService.GetCoinValidator().ActiveAccept(); //это запускает StartDevice();

using System;
 
namespace CoinAcceptor
{
    /// <summary>Аргумент события изменения IsRunning.</summary>
    public class RunningChangedEventArgs : EventArgs
    {
        public bool IsRunning { get; }
 
        public RunningChangedEventArgs(bool isRunning)
        {
            IsRunning = isRunning;
        }
    }
}

using System;
 
namespace CoinAcceptor
{
    /// <summary>Аргумент события изменения состояния монетоприёмника.</summary>
    public class DataStateChangedEventArgs : EventArgs
    {
        public object DataState { get; }
 
        public DataStateChangedEventArgs(object dataState)
        {
            DataState = dataState;
        }
    }
}

using System;
using System.Timers;
 
namespace CoinAcceptor
{
    public partial class CoinAcceptorModel
    {
        /// <summary>Событие извещающее ою изменении состояния Модели: работает/не работает.</summary>
        public EventHandler<RunningChangedEventArgs> RunningChanged;
 
        /// <summary>Флаг работает/не работает.</summary>
        private bool isRunning;
        private void SetRunning(bool isRunning)
        {
            if (this.isRunning == isRunning)
                return;
 
            this.isRunning = isRunning;
            if (isRunning)
                timer.Start();
            else
                timer.Stop();
 
            RunningChanged?.Invoke(this, new RunningChangedEventArgs(isRunning));
        }
 
        public bool GetRunning() => isRunning;
 
        private readonly Timer timer = new Timer()
        {
            Interval = 250,
        };
 
        /// <summary>Вот этот метод будет вызываться таймером на 
        /// произвольном потоке из пула потоков.
        /// Каждый вызов будет на разном потоке.</summary>
        private void OnElapsed(object sender, ElapsedEventArgs e)
        {
            // Если есть опасность недопустимого наложения 
            // вызовов, то лучше при входе останавливать таймер,
            // а при выходе запускать по новой.
            timer.Stop();
 
            // Вызов метода для работы с саммим физическим устройством.
            // Если есть возможность исключения в его работе, то
            // вызов должен быть обёрнут в try-catch и исключение
            // должно быть здесь обработано.
            // Если этого не сделать, то будут баги или падение
            // приложения.
            CoinAcceptorExecute();
 
 
            // Запуск если поднят флаг isRunning
            if (isRunning)
                timer.Start();
        }
 
        public void StartRunning() => SetRunning(true);
        public void StopRunning() => SetRunning(false);
    }
}

using System;
 
namespace CoinAcceptor
{
    public partial class CoinAcceptorModel
    {
        public EventHandler<DataStateChangedEventArgs> DataStateChanged;
 
        /// <summary>Собственно сам метод работающий с монетоприёмником.</summary>
        private void CoinAcceptorExecute()
        {
            // Получение состояния монетоприёмника.
            object dataState = new object();
 
            // Сравнение с текущим состоянием
            if (Equals(dataStateCoinAcceptor,dataState))
            {
                // Если состояние изменилось, то его обработка
                // и вызов создание события
                dataStateCoinAcceptor = dataState;
                DataStateChanged?.Invoke(this, new DataStateChangedEventArgs(dataState));
            }
        }
 
        /// <summary>Поле хранящее состояние монетоприёмника.
        /// Типизировал просто object, так как для примера это не важно.
        /// В реале должен быть некий конейнер с данными.</summary>
        private object dataStateCoinAcceptor;
    }
}

namespace CoinAcceptor
{
    public partial class CoinAcceptorModel
    {
        /// <summary>Закрытый конструктор.</summary>
        private CoinAcceptorModel() => timer.Elapsed += OnElapsed;
 
        /// <summary>Единственственный экземпляр.</summary>
        public static CoinAcceptorModel Instance { get; } = new CoinAcceptorModel();
 
    }
}

public static CoinAcceptor Instance { get; } = new CoinAcceptor ();

        private static CoinAcceptor _instance = null;
 
        private static Object _mutex = new Object();
 
        public static CoinAcceptorGetInstance(string port, string adress, int[] inhibitCoins, string currency)
        {
            if (_instance == null)
            {
                lock (_mutex) // now I can claim some form of thread safety...
                {
                    if (_instance == null)
                    {
                        _instance = new CoinAcceptor (port, adress, inhibitCoins, currency);
                    }
                }
            }
 
            return _instance;
        }

public async Task<bool> Calculate(int coins)
{
    var res = await Task<bool>.Run(() => 
    {
        return hopper.NameOfMethod(coins);
    }
    return res;
}