MVVM бизнес логика и Model

        /// <summary>Метод произвольно вызвает методы модели</summary>
        private async Task RandomizeAsync(MonitorModel monitorModel)
        {
 
            while (isStarted)
            {
                // TODO: цикл прерывается, в логе: Вызвано исключение: "System.InvalidOperationException" в System.Private.CoreLib.dll
                // В Модели можно добавить internal метод возвращающий все Бизнес сущности
 
                // Коллекция по которой происходит перечисление, не должна изменяться на время работы перечисления.
                // Типовой способ развязки исходной коллекции и перечисления по ней - это создание её копии.
                foreach (var monitor in monitorModel.GetMonitors().ToArray())
                {
                    Debug.WriteLine("Старая конфигурация");
                    Debug.WriteLine("-------------------");
                    Debug.WriteLine(monitor.Name);
                    Debug.WriteLine(monitor.ColorConfiguration);
                    Debug.WriteLine(monitor.IsActive);
                    Debug.WriteLine(monitor.Height);
                    Debug.WriteLine(monitor.Width);
                    Debug.WriteLine("-------------------");
 
                    // Если монитор удалён, то к нему нельзя больше обращаться.
                    if (random.Next(0, 10) > 5) // Получется удаление с вероятностью 50% - это не слишком много? Может сравнивать с 3 или 2?
                    {
                        monitorModel.DeleteMonitor(monitor.Name);
                        // Переход к следйющему циклу
                        continue;
                    }
 
                    var boolrandom = random.Next(0, 10);
 
                    if (boolrandom >= 7)
                    {
                        monitor.IsActive = true;
                    }
                    else if (boolrandom <= 3)
                    {
                        monitor.IsActive = false;
                    }
 
                    if (random.Next(0, 10) > 5)
                    {
                        monitor.ColorConfiguration = random.NextDouble();
                    }
 
                }
 
                // Добавлениме Монитора никак не связано с циклом по СУЩЕСТВУЮЩЩИМ мониторам.
                // Поэтому его стоит делать отдельным блоком.
                if (random.Next(0, 10) > 5)
                {
                    monitorModel.AddMonitor($"ServiceAdded monitor {random.NextDouble()}");
                }
 
 
                foreach (var monitor in monitorModel.GetMonitors())
                {
                    Debug.WriteLine("Новая конфигурация");
                    Debug.WriteLine("-------------------");
                    Debug.WriteLine(monitor.Name);
                    Debug.WriteLine(monitor.ColorConfiguration);
                    Debug.WriteLine(monitor.IsActive);
                    Debug.WriteLine(monitor.Height);
                    Debug.WriteLine(monitor.Width);
                    Debug.WriteLine("-------------------");
                }
 
                await Task.Delay(1000);
            }
        }

                // Вывод это достаточно долгий метод.
                // Если в этот момент будет изменена коллекция мониторов, то цикл упадёт с исключением.
                // Поэтому лучше создавать её копию.
                // Возможно из-за это цикл иногда и падал во время добавления в момент вывода
                Debug.WriteLine("Новая конфигурация");
                foreach (var monitor in monitorModel.GetMonitors().ToArray())
                {
                    Debug.WriteLine(monitor);
                }
                Debug.WriteLine("-------------------");

        protected override void OnPropertyChanged(in string propertyName, in object oldValue, in object newValue)
        {
            base.OnPropertyChanged(propertyName, oldValue, newValue);
 
            if (propertyName == nameof(RandomizerState))
            {
                if ((bool)newValue)
                {
                    // асинхронное выполнение в UI потоке, чтобы не пропадало исключение
                    Dispatcher.InvokeAsync(async () => await randomizerModel.StartAsync(monitorModel));
                }

        // Класс для отмены задачи
        private class CanceledStart
        {
            public bool IsCanseled { get; private set; }
 
            public void Canseled()
            {
                IsCanseled = true;
            }
        }

        // Останавливать цикл в задаче надо неменого по иному.
        // Объект для отмены цикла запущенного в методе Start.
        private CanceledStart canceledStart;
 
        /// <summary>
        /// Запускает произвольные изменения классса настроек
        /// </summary>
        public async /* void */ Task StartAsync(IMonitorModel monitorModel)
        {
            // Предотваращение повторного старта
            if (this.canceledStart != null)
            {
                return;
            }
 
            CanceledStart canceledStart = new();
            this.canceledStart = canceledStart;
            // Определяем дружественна ли Модель
            if (monitorModel is MonitorModel model)
            {
                await Task.Run(() =>
                {
                    _ = RandomizeAsync(model, canceledStart).ConfigureAwait(false);
                });
            }

        /// <summary>
        /// Останавливает произвольные изменения классса настроек
        /// </summary>
        public void Stop()
        {
            canceledStart?.Canseled();
            canceledStart = null;
        }

        /// <summary>Метод произвольно вызвает методы модели</summary>
        private static async Task RandomizeAsync(MonitorModel monitorModel, CanceledStart canceledStart)
        {
 
            while (!canceledStart.IsCanseled)
            {

public class MonitorVM : BaseInpc
{
    public IMonitorConfigurationModel Monitor {get => ...; set => ....:}
    public bool IsSelected {get => ...; set => ....:}
    public bool IsExpanded {get => ...; set => ....:}
}