Arduino.ru

Потому что не надо брать непонятно какие скетчи, надо самому писать.
Этот код стабильно работать не может по определению. Мало того что дребезг, еще и пин кнопки наверняка в воздухе болтается.
Как улучшить? Во первых подтянуть кнопку - программно или аппаратно. И во-вторых убрать изменение вращения из прерывания. В прерывании оставить флаг, а сам реверс включать в лупе, причем не чаще раз в полсекунды, для примера

Уважаемые! Кнопка подтянута аппаратно, а вот как подтянуть программно? Всё остальное меня устраивает. 

 

 

Берёшь подтяжку от штанов и между 34 и 36 строками её наматываешь.

Честно говоря, не совсем понятно, зачем здесь кнопка подключена через прерывания, но то дело хозяйское. А дребезг подавляется путем задержки реакции на изменение состояния кнопки. Т. е. состояние изменилось, запомнили его и через 50-80 милисекунд снова проверили - если состояние кнопки совпадает с изменившимся, значит имеем нажатие/отпускание кнопки, нет - игнорируем.
Про подтяжку вопрос был не про кнопку, а про подтяжку пина МК, к которому кнопка подключена. На нем при разомкнутой кнопке должен быть стабильный уровень, в вашем случае должен быть высокий уровень. Для этого вы должны либо соединить пин с VCC через резистор, либо установить пину соответствующий режим.

Кнопка подтягивается к плюсу или аппаратно, или програмно. Если кнопка подтянута аппаратно, то програмно ее подтягивать уже не нужно. Только вы точно понимаете, что это значит?
А если остальное вас устраивает - не понятно, зачем вы писали на форум

Да же и не представлял что есть такая функция:
digitalPinToInterrupt().
Ручками как то удобнее и нагляднее

/*
  Stepper Motor Test
  stepper-test01.ino
  Uses DM860H or similar Stepper Driver Unit
  Has speed control & reverse switch
  
  DroneBot Workshop 2019
  <a href="https://dronebotworkshop.com" title="https://dronebotworkshop.com" rel="nofollow">https://dronebotworkshop.com</a>
*/

// Defin pins

int reverseSwitch = 2;  // Push button for reverse
int driverPUL = 7;    // PUL- pin
int driverDIR = 6;    // DIR- pin
int spd = A0;     // Potentiometer

// Variables

int pd = 500;       // Pulse Delay period
boolean setdir = LOW; // Set Direction
unsigned long previousMillis = 0;
uint16_t t_drebezg = 80;

// Interrupt Handler

void revmotor (){
if(millis()-previousMillis > t_drebezg){
   setdir = !setdir;
   previousMillis = millis();
 }
}


void setup() {

  pinMode (driverPUL, OUTPUT);
  pinMode (driverDIR, OUTPUT);
  attachInterrupt(digitalPinToInterrupt(reverseSwitch), revmotor, FALLING);
  
}

void loop() {
  
    pd = map((analogRead(spd)),0,1023,2000,50);
    digitalWrite(driverDIR,setdir);
    digitalWrite(driverPUL,HIGH);
    delayMicroseconds(pd);
    digitalWrite(driverPUL,LOW);
    delayMicroseconds(pd);
 
}

 

есть мнение, что переменные setdir и previousMillis нужно объявлять как volatile

можно и объявить, но previousMillis зачем?

Затем, что обрабатывается в прерывании

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

Какая целевая функция делать её волатильной?

del
Похоже, был не прав ))