Arduino.ru

#include <CustomStepper.h>
CustomStepper stepper(8, 9, 10, 11);
boolean x = false;
boolean y = false;
const int pinPhoto = A0;
const int led = 13;
int raw = 0;

void setup() {
  stepper.setRPM(12);                 // Устанавливаем кол-во оборотов в минуту
  stepper.setSPR(4075.7728395);
  pinMode( pinPhoto, INPUT );
  pinMode( led, OUTPUT );
}

void loop() {
  raw = analogRead( pinPhoto );                             // raw > 400, следовательно света нет
// Если света нет и штора не опущена, то штора опускается
  if (stepper.isDone() && x == false && raw>400){
    stepper.setDirection(CW);                                //Направление
    stepper.rotate(1);                                       //Кол-во оборотов
    x=true;
    y=false;
  }
//Если выполнилось Х оборотов, то стоп
  if (stepper.isDone() && x==true){
    stepper.setDirection(STOP);
  }
// Если есть свет и штора не поднята, то штора поднимается
  if (stepper.isDone() && y==false && raw<400){
    stepper.setDirection(CCW);
    stepper.rotate(1);
    y=true;
    x=false;
  }
// Если двигатель прокрутился на Х оборотов, то стоп
  if (stepper.isDone() && y==true){
    stepper.setDirection(STOP);
  }
stepper.run();
}

Здравствуй жопа новый год http://arduino.ru/forum/programmirovanie/vypolnenie-tsikla-1-raz-do-izmeneniya-parametrov. Я же сказал, пока вы не освоите программирование через цифровой автомат, у вас будут косяки лететь стаями.

Вы бы вставили перед каждым if комментарий, где словами было бы написано что именно делается. Только в терминах задачи (типа "если попал свет, то включаем на подъём") а не пересказом того же if.

И сам может поймёте проблему, и нам легче будет.

Хорошо, попробую освоить.

Тем более CustomStepper Это тоже цифровой автомат

/**/
const int pinPhoto = A0;
/*состояние светло/темно*/
enum state_t {LIGTH, DARK} state;

void setup() {
  Serial.begin(9600);
  pinMode( pinPhoto, INPUT );
  state = DARK;
  Serial.println("DARK");
}
void loop() {
  int raw = analogRead( pinPhoto );
  // raw > 400, следовательно света нет
  if (raw > 400 && state == LIGTH) {
    state = DARK;
    Serial.println("DARK");
  }
  if (raw <= 400 && state == DARK) {
    state = LIGTH;
    Serial.println("LIGTH");
  }
}

При использовании этого кода, выдаёт последовательно LIGHT DARK LIGHT DARK.

Это нормально?

Так происходит даже если отключить датчик.

/**/
const int pinPhoto = A0;
/*состояние светло/темно*/
enum state_t {LIGTH, DARK} state;

void setup() {
  Serial.begin(9600);
  pinMode( pinPhoto, INPUT );
  state = DARK;
  Serial.println("DARK");
}
void loop() {
  int raw = 200;//< а вот так отключать не пробовал?
  // raw > 400, следовательно света нет
  if (raw > 400 && state == LIGTH) {
    state = DARK;
    Serial.println("DARK");
  }
  if (raw <= 400 && state == DARK) {
    state = LIGTH;
    Serial.println("LIGTH");
  }

Выведите значения raw в Serial, проверьте, что датчик подключен именно к A0.

Квон еще до гистеререзиса не доучил, хотя попытку сделал. Так что если raw болтается около 400ед, то так и будет флапать.

Да, подключен к A0. Если вывести в serial с этим скетчем, то просто пишет 14 14 14 и т.д, а с другими всё ок.

IVoxelI, ну, вставьте же Вы наконец комментарии и выложите код с ними! Вас что, умолять надо? Ну, вот непонятно что Вы там имели в виду, а объяснять Вы отказываетесь!

Так лучше?

Честно говоря не совсем, но хоть догадываться можно.

Я пока не могу понять причину такого поведения и потому предлагаю Вам оттрассировать программу. И кстати, всегда так поступайте, когда поведение программы непонятно. Это можно сделать примерно вот так.

#include <CustomStepper.h>
CustomStepper stepper(8, 9, 10, 11);
boolean x = false;
boolean y = false;
const int pinPhoto = A0;
const int led = 13;
int raw = 0;

void setup() {
  Serial.begin(115200);
  stepper.setRPM(12);                 // Устанавливаем кол-во оборотов в минуту
  stepper.setSPR(4075.7728395);
  pinMode( pinPhoto, INPUT );
  pinMode( led, OUTPUT );
}

void loop() {
  raw = analogRead( pinPhoto );  // raw > 400, следовательно света нет
  
  Serial.print("millis()="); Serial.print(millis());
  Serial.print("; raw="); Serial.print(raw);
  Serial.print("; x="); Serial.print(x);
  Serial.print("; y="); Serial.print(y);
  Serial.print("; stepper.isDone()="); Serial.println(stepper.isDone());

  // Если света нет и штора не опущена, то штора опускается
  if (stepper.isDone() && x == false && raw > 400) {
    Serial.println("Point #1");
    stepper.setDirection(CW);   //Направление
    stepper.rotate(1);          //Кол-во оборотов
    x = true;
    y = false;
  }
  //Если выполнилось Х оборотов, то стоп
  if (stepper.isDone() && x == true) {
    Serial.println("Point #2");
    stepper.setDirection(STOP);
  }
  // Если есть свет и штора не поднята, то штора поднимается
  if (stepper.isDone() && y == false && raw < 400) {
    Serial.println("Point #3");
    stepper.setDirection(CCW);
    stepper.rotate(1);
    y = true;
    x = false;
  }
  // Если двигатель прокрутился на Х оборотов, то стоп
  if (stepper.isDone() && y == true) {
    Serial.println("Point #4");
    stepper.setDirection(STOP);
  }
  stepper.run();
  Serial.println("--------------------");
}

Только делайте всё быстро, поскольку трасса будет печататься с огромной скоростью.

Если после изучения трассы Вам не станет легче, то выложите трассу сюда.

И да, кстати, к Вашей текущей проблеме это не относится, но имейте в виду, что эту библиотеку лучше не изсользовать. Она написана достаточно безграмотно.

Вы так пошутили? Надеюсь, что нет, иначе ... мне не нравится, когда надо мною так шутят.

Скорость порта в строке 10 и в нижнем правом углу монитора порта одинаковую выставьте. Жолжен выводиться нормальный текст.

Нет, не шутил. Теперь выводится так :

millis()=216946; raw=207; x=1; y=0; stepper.isDone()=0

а по-моему все-таки шутите. Причем злобно.

Зачем вы выкладываете трассу спустя почти 4 минуты после рестарта системы? (после 216 сек) . По коду и так очевидно, что если система попала в состояние raw=200; x=1; y=0; stepper.isDone()=0 - то система в нем застрянет навечно. Вам-то важно знать, как она в это состояние попала! Интересными являются разве что первый десяток проходов loop() , в вы выложили примерно 40тысячный оборот... смеетесь7

Трасса нужна сначала.

Ну, Вы место на сервере экономите? Побольше нельзя?

Ладно, скажите хоть stepper.isDone()=0 - всегда 0? Она должна иногда становиться 1 и я должен видеть это. Если уж сильно много, ну зипните и положите на файлообменник

Просто нет смысла, т.к. дальше строчки одинаковые

Это ответ на мой вопрос? stepper.isDone()=0 больше никогда не становится 1 - проверьте внимательно! Это очень странно

Ага, понятно. Вы библиотеку отсюда скачивали? https://playground.arduino.cc/Main/CustomStepper ?

Если отсюда, isDone у Вас никогда не станет истинной. Он там становится истинной только после создания объекта и в методе disable.

Потому у Вас ничего и не делается. что все условия в if'ах всегда ложны.

Если не оттуда скачивали, что скажите откуда.

Да, отсюда. Мне поменять библиотеку?

Не знаю, это Ваше дело.

Вы проверяете isDone(), я посмотрел в текст (можете и Вы посмотреть) и увидел, что истиной она бывает только в двух случаях: сразу в самом начале и после исполнения метода disable(). Если Вы выполнили хоть одну операцию rotate или rotateDegrees, она будет ложью пока не исполните disable.

Сами думайте, менять Вам библиотеку или disable вызывать. Это Ваше решение.