Как исправить ошибку в коде?

Question

[boOogdan]

Дан код. Светлодиод включает, но не выключает.

x = GetX();
if(x>-5000 && x<5000 && !isAction)
{
   isAction = true;
   digitalWrite(13, LOW);
}
if(x>-17000 && x<-13000 && isAction)
{
   isAction = false;
   digitalWrite(13, HIGH);
   delay(100);
   digitalWrite(13, LOW); //не выключает
   Serial.println("+");
}
delay(100);

Comments

[Армянское Радио]

Покажите всю прошивку. Пока вижу, что все включается и выключается, только очень быстро.

[DimiDr0lik]

if(x>=17000 && x<=13000 && isAction) знак = а не "-"

Answer 1

[Армянское Радио]

При каких условиях зажигать, при каких гасить? Телепаты все на конгрессе по гаданию на женских ножках, помочь не могут.

Comments

[boOogdan]

если x>-17000 && x<-13000 && isAction, то светодиод включается на 100 миллисекунд

[pi314]

Если этот код будет выполняться в цикле, то будет эффект PWM. Для проверки гипотезы достаточно вместо 100мс вставить большее значение - будет гореть "ярче".

А вообще, за такой код нужно гнать с Тостера поганой метлой! Уважайте, если уж не себя, то, по крайней мере тех, кто будет читать вопрос и ломать глаза об это непотребство... при вставке кода пользуйтесь кнопочкой с тремя точками, и код будет выглядеть вот так

x = GetX();
if(x>-5000 && x<5000 && !isAction){
  isAction = true;
  digitalWrite(13, LOW);
}
if(x>-17000 && x<-13000 && isAction){
  isAction = false;
  digitalWrite(13, HIGH);
  delay(100);
  digitalWrite(13, LOW); //не выключает
  Serial.println("+");
}
delay(100);

[boOogdan]

Серожа: после включения/выключения светодиода есть Serial/println("+"); , и он работает

[boOogdan]

pi314: сори за код. попробовал увеличить задержку - ничего не изменилось

[pi314]

boOogdan: Ну, тут - одно из двух. Либо поставить брекпоинт и самому проверить в пошаговом режиме, либо привести больше кода. Иначе мы можем только продолжать гадать на кофейной гуще :)

[boOogdan]

pi314: просто вместо GetX() написан код получения координаты X с акселерометра :D

[Армянское Радио]

boOogdan: выкладывайте весь код, чтобы нам не гадать.