Как реализовать: пока нажата кнопка — выполнять команду?

Question

[1bd_1l_Bari]

Доброго времени суток.
Суть : шаговый двигатель подключен к микрухе atmega, которая подключена к PC через интерфейс USART.
На компьютере написано приложение на C# , где 4 кнопки (вверх, вниз, вправо, влево). Нажимая на которые, контроллеру подаются байты: 1, 2, 3, 4. Необходимо , чтобы пока нажата кнопка (например вверх), комп посылал байт 1 постоянно на контроллер, т.е пока нажата кнопка- шаговик крутится.

Проблема: в функции, где нажимается кнопка, написал вечный цикл, который действует пока флаг == тру. Как только, отпускаем кнопку, флаг становится фолс, и нужно выйти из цикла, однако цикл продолжается, программа виснет, а шаговик крутится.

Как сделать правильно, не подскажете?

Вот код в C# (для контроллера всё сделано правильно 100% , поэтому выкладывать нет смысла)

void Button1MouseDown(object sender, MouseEventArgs e) //кнопка нажата
		{
			flag = true;
			while(flag)
			{
				Port_Write(1); //посылаем 1 байт на контроллер
			}
		}
		
		void Button1MouseUp(object sender, MouseEventArgs e) // кнопка отпущена
		{
			flag = false;
		}

Answer 1

[Станислав Силин]

Вы входитет в цикл в UI потоке и намертво его утилизируете этим же циклом. Если вы присмотритесь повнимательней, то заметите что кнопка на UI вовсе не отжимается после того как вы ее отпустили. Нужно вынести цикл в отдельный поток, например так:

void Button1MouseDown(object sender, MouseEventArgs e) //кнопка нажата
{
 Task.Run(() => {
      flag = true;
      while(flag)
      {
        Port_Write(1); //посылаем 1 байт на контроллер
      }
});
}

P.S. Я не в курсе микроконтролерной кухни, но может не стоит слать команды с такой частотой?

Comments

[1bd_1l_Bari]

Task.Run у меня не нашел такой метод. В моем случае вместо него сработал Task.Factory.StartNew (что вроде тоже самое, я в c# пока не очень). Всё работает теперь.

Спасибо большое, лайк + отмечаю решением.

Answer 2

[NonameProgrammer]

А вы то забыли указать доп.условие - как выйти из цикла? Как приложение догадается о том , что кнопка уже не нажата, если вечный цикл?, вот и зависает.

Comments

[1bd_1l_Bari]

Не забыл.

void Button1MouseUp(object sender, MouseEventArgs e) // кнопка отпущена
    {
      flag = false;
    }

Выше правильный ответ.

[NonameProgrammer]

1bd_1l_Bari, Вы меня не так поняли.
Можно вроде бы сделать по другому.
Вот примерно схема вашего кода
1. Нажата кнопка - войти в цикл
2. Пока ЦИКЛ (управление передается циклу), то есть пока выполняется цикл управление в ц-кле, завершается цикл - ВОЗВРАЩАЕТСЯ управление к кнопке, чтобы завершить нажатие
Из этого следует, что пока мы в цикле приложение не будет реагировать, пока он работает. А вот еслиб была доп. Проверка, то тогда все получилось, опять же button1mouseup.Is Pressed etc

[1bd_1l_Bari]

NonameProgrammer, понял вас. Скорее всего вы правы ))

[Дмитрий Башинский]

Может пригодится, посмотри RepeatButton
Он когда зажат вызывает событие клика, тогда получится без флага и без цикла

[1bd_1l_Bari]

Дмитрий Башинский, спасибо, не знал про это!

[NonameProgrammer]

Дмитрий Башинский, Я в GUI особо не силён) Моя деятельность в основном с этим не связана, но если и будет (планирую на Xamarin), то обязательно учту, спасибо.