Простой скетч arduino не работает как надо?

Question

[Vas Terkin]

Здравствуйте, я новичок в сфере arduino, и купил книгу Белова А.В. "arduino от азов программировани до создания практических устройств". В данной книге есть пример "Бегущие огни с одним общим циклом".
В соответствии со скетчем светодиоды должны загораться и гаснуть друг за другом, но у меня горят светодиод подключенный ко 2-му выводу постоянно, подскажите, пожалуйста, что может быть не так, благодарю!

const int buttonPin = 13;
const int loopTime = 200;

int loopStart = 2;
int loopStop = 10;
int loopStep = 1;

int thisPin;
unsigned long lastTime;

void setup () {
  for (int thisPin = 2; thisPin < 10; thisPin++ ){
    pinMode(thisPin, OUTPUT);
    digitalWrite(thisPin, 1);
  }
  pinMode(buttonPin, INPUT);
  thisPin = loopStart;
  lastTime = 0;
}
void loop (){
  if(lastTime == 0){
    digitalWrite(thisPin, LOW);
    lastTime = millis();
  }
  if (millis() - lastTime > loopTime){
    digitalWrite (thisPin, 1);
    thisPin += loopStep;
    if(thisPin == loopStop){
      if (digitalRead(buttonPin) == 1){
        loopStart=2; loopStop = 10; loopStep= 1;}
        else{loopStart= 9; loopStop=1; loopStep= -1;}
}
    thisPin = loopStart;
  }
  lastTime = 0;
}

Comments

[kalapanga]

Код желательно отформатировать и обернуть в тег code. И схемку бы для наглядности привести, кто там куда подключен.
Кстати, удобный вариант - на WokWi проект с вашими схемой и скетчем сделать. И ссылку на него дать. А может пока делаете уже и все ошибки найдёте!

Answer 1

[Григорий Боев]

Судя по коду
lastTime = 0;
должно быть внутри

if (millis() - lastTime > loopTime){
// ...
};

Иначе оно раз в цикл принимает значение millis(), потом сразу проскакивает цикл и обнуляется.

Comments

[Vas Terkin]

const int buttonPin = 11;
const int loopTime = 200;

int loopStart = 2;
int loopStop = 7;
int loopStep = 1;

int thisPin;
unsigned long lastTime;

void setup () {
for (int thisPin = 2; thisPin < 8; thisPin++ ){
pinMode(thisPin, OUTPUT);
digitalWrite(thisPin, 1);
}
pinMode(buttonPin, INPUT);
thisPin = loopStart;
lastTime = 0;
}
void loop (){
if(lastTime == 0){
digitalWrite(thisPin, LOW);
lastTime = millis();
}
if (millis() - lastTime > loopTime){
digitalWrite (thisPin, 1);
thisPin += loopStep;
if(thisPin == loopStop){
if (digitalRead(buttonPin) == 1){
loopStart=2; loopStop = 7; loopStep= 1;}
else{loopStart= 9; loopStop=1; loopStep= -1;}
}
thisPin = loopStart;
lastTime = 0;
}
}

в данном примере происходит то же самое что и было:(
я решил изменить 13 пин кнопки на 11 , так как подумал,ч то может это связано с встроенном на ардуинке светодиодом, но нет.

[Vas Terkin]

const int buttonPin = 13;
const int loopTime = 200;

int loopStart = 2;
int loopStop = 9;
int loopStep = 1;

int thisPin;
unsigned long lastTime;

void setup () {
for (int thisPin = 2; thisPin < 10; thisPin++ ){
pinMode(thisPin, OUTPUT);
digitalWrite(thisPin, 1);
}
pinMode(buttonPin, INPUT);
thisPin = loopStart;
lastTime = 0;
}
void loop (){
if(lastTime == 0){
digitalWrite(thisPin, LOW);
lastTime = millis();
}
if (millis() - lastTime > loopTime){
digitalWrite (thisPin, 1);
thisPin += loopStep;
if(thisPin == loopStop){
if (digitalRead(buttonPin) == 1){
loopStart=2; loopStop = 9; loopStep= 1;}
else{loopStart= 9; loopStop=1; loopStep= -1;
}
thisPin = loopStart;

}
lastTime = 0;
}
}

Здравствуйте, нашел ошибку! тег закрытия } не там поставил, благодарю!
Но появилась проблема дальгше:( при нажатии кнопки , бег дорожки не бежит в обратном направлении

[Григорий Боев]

Судя по коду бежать в обратном будет только при достижении края. Т.е. кнопку надо держать, до тех пор пока не сменится направление

if(thisPin == loopStop){ // условие раз
if (digitalRead(buttonPin) == 1){ // условие два