FlProg

ecoins писал(а): 10 мар 2023, 12:31 И не стоит избегать фигурных скобок (строки 3,4,8).

Dryundel писал(а): 10 мар 2023, 12:54 А в чем проблема? Если не секрет.

ecoins писал(а): 10 мар 2023, 12:31 Из видимого в первой строке расставил скобки - в разных контроллерах физически используются разные компиляторы C++, которые могут "подбрасывать"...
И не стоит избегать фигурных скобок (строки 3,4,8).

      curState = (digitalRead(_pin_A)) | (digitalRead(_pin_B)) << 1; 
      if (resetFlag && curState == 0b11) 
	  {
        if (prevState == 0b10) 
		{
			_ou++;
		}
        if (prevState == 0b01) 
		{
			_ou--;
		}
    _ou_= _ou;
    resetFlag = 0;
         }
      if (curState == 0b00) 
	  {
		  resetFlag = 1;
	  }
      prevState = curState;

Dryundel писал(а): 10 мар 2023, 13:51 Второе. digitalRead для AVR достаточно длительная операция с проверками. А их целых две в функции. (не смотрел что там в ESP)
Я же говорил, попробуй впихнуть в проект (для AVR) вместе с энкодером мой блок оптимизации. Может быть поможет.

ecoins писал(а): 10 мар 2023, 12:31 Думаю, что номера пинов _pin_A и _pin_B в разных контроллерах соответствует пинам для конкретного контроллера.

Ingwar писал(а): 10 мар 2023, 15:19 Сделал, "не взлетает"...

Ingwar писал(а): 10 мар 2023, 15:30 Можно эти операции с битами преобразовать в другой способ без потери условий?

Sancho писал(а): 10 мар 2023, 15:45 _ou_ - в обработчике прерывания и в коде - должна быть volatile !!!

ecoins писал(а): 10 мар 2023, 15:32 В цикле прерывания очень длинная функция:

Sancho писал(а): 10 мар 2023, 15:45 _ou_ - в обработчике прерывания и в коде - должна быть volatile !!!

Dryundel писал(а): 10 мар 2023, 16:22 Битовые операции самые быстрые.
Алгоритм построен именно на битовых данных.

Ingwar писал(а): 10 мар 2023, 12:02 Подскажите, почему этот код из функции работает на stm и rp2040, но не работает (хотя компилится без проблем) на ардуино и esp?

Dryundel писал(а): 10 мар 2023, 16:22 Возможно. Но не факт. Она же объявлена не в обработчике, а глобальная.
А вот если бы это был обработчик по прерыванию таймера, то точно надо было бы volatile в любом случае.

Ingwar писал(а): 10 мар 2023, 16:31 Также возможно, что используемая версия Arduino IDE или библиотеки не поддерживает использование << для битового сдвига, что может вызвать ошибки компиляции.

Sancho писал(а): 10 мар 2023, 16:39 Для правильности нужен весь код

Sancho писал(а): 10 мар 2023, 16:47 Игорь, ты хочешь получить максимальный энкодер, 4 числа на щелчок?

Ingwar писал(а): 10 мар 2023, 16:31 Также возможно, что используемая версия Arduino IDE или библиотеки не поддерживает использование << для битового сдвига, что может вызвать ошибки компиляции.

Ingwar писал(а): 10 мар 2023, 16:31 Если бы код не работал совсем, это одна история.

Sancho писал(а): 10 мар 2023, 16:42 Потому-что я не знаю, а Андрей догадывается, что

Ingwar писал(а): 10 мар 2023, 16:53 Я делал на одном, примитивный вариант, так там если медленно крутить, мог давать ложные повороты.

		//	new	new	old	old
		//	pin2	pin1	pin2	pin1	Result
		//	----	----	----	----	------
		//	0	0	0	0	no movement
		//	0	0	0	1	+1
		//	0	0	1	0	-1
		//	0	0	1	1	+2  (assume pin1 edges only)
		//	0	1	0	0	-1
		//	0	1	0	1	no movement
		//	0	1	1	0	-2  (assume pin1 edges only)
		//	0	1	1	1	+1
		//	1	0	0	0	+1
		//	1	0	0	1	-2  (assume pin1 edges only)
		//	1	0	1	0	no movement
		//	1	0	1	1	-1
		//	1	1	0	0	+2  (assume pin1 edges only)
		//	1	1	0	1	-1
		//	1	1	1	0	+1
		//	1	1	1	1	no movement

Dryundel писал(а): 10 мар 2023, 17:13 Для того, что бы хорошо работало, необходима серьезная отработка всех возможный инстансов действий энкодера и еще предыдущее состояние, а не только - "есть контакт, нет контакта". Утрирую конечно.

Dryundel писал(а): 10 мар 2023, 17:13 А на AVR хоть что то шевелится?

Ingwar писал(а): 10 мар 2023, 17:22 Ардуино и esp как в танке.

Dryundel писал(а): 10 мар 2023, 17:29 Вот что я подумал. А может ли вообще AVR читать пин находящийся под прерывание во время прерывания?
Я этого не знаю.

attachInterrupt(digitalPinToInterrupt(pin), ISR, mode)(рекомендуемые)
attachInterrupt(interrupt, ISR, mode)(не рекомендуется)
attachInterrupt(pin, ISR, mode)(Не рекомендуется. Кроме того, этот синтаксис работает только на платах Arduino SAMD, Uno WiFi Rev2, Due и 101.)

attachInterrupt(digitalPinToInterrupt(_pin_A), encIsr, CHANGE);
attachInterrupt(digitalPinToInterrupt(_pin_В), encIsr, CHANGE);