Почему не получается остановить анимацию?

Question

[Михаил]

Всем привет. Есть класс с двумя методами:

async public void DisplayAnimation(Canvas canvas)
{
     this.IsShowAnimation = true;
     while (this.IsShowAnimation)
     {
         this.Draw(this.Angle, StartPoint, canvas);
         await Task.Delay(20);
         canvas.Children.Clear();
         this.Angle += 3;
     }
}

public void StopAnimation(Canvas canvas)
{
     this.IsShowAnimation = false;
}

Если я где-то в коде вызываю DisplayAnimation(), то анимация начинает правильно работать. Но если я через какое-то время вызываю StopAnimation(), то почему-то она продолжает работать, хотя по моей логике она должна прекратиться. В чём может быть проблема?

Если что, то вот ссылка на исходный код

P. S. Все модификаторы доступа настроены правильно. Все обработчики правильно вызываются в коде

Answer 1

[eRKa]

Дайте ссылку в исходном коде где именно код в том варианте, как привели. То, что я увидел в коде и в Вашем примере, это не одно и тоже. В исходнике запускается и останавливается таймер. Ваш пример вырван из контекста, поэтому по нему трудно что-то понять.

С гитхаба Ваш код себе не клонировал, но заметил, что ChebyshevMachine это структура, а не класс. Не знаю причины такого выбора, но могу предположить, что именно в этом может быть проблема. Поставьте брейкпоинт на методе StopAnimation и проверьте, что это тот же объект на котором Вы запустили таймер.

Comments

[Михаил]

Спасибо. Я уже решил проблему, переделав анимацию на DispatcherTimer. И вы посмотрели уже на исправленный и работающий код.

А что плохого в том, что это структура? Я думал, что лучше использовать структуры, чем классы, если есть такая возможность

[eRKa]

Михаил, структуры можно использовать, если объект не меняет состояние. Например как DateTime. Используя структуры, нужно помнить, что передавая их как параметр, он будут создавать копии, а не ссылки на объект, соответственно, если Вы передадите машину как параметр или просто присвоите другой переменной, то это будет новый объект, т.е. если будет что-то подобное

var mach = new ChebyshevMachine();
mach.StartAnimation();
var mach2 = mach;
mach2.StopAnimation();

то StopAnimation Вы будете вызывать уже на другом объекте, а не на том где стартовали.
Если же это был бы экземпляр класса, то при таком коде и старт, и остановка будет вызываться на одном и том же объекте.
А в предыдущем варианте он как раз хранил состояние векторов, булевское и т.д., и поэтому остановку Вы могли вызывать не на том же объекте, а на копии, но объект то другой, поэтому останавливали Вы что-то другое.