Arduino.ru

Что должно происходить, если новая команда m поступит во время этих 5 секунд?

Ничего, игнорировать эту команду. Я пытаюсь имитировать зажигание, крутить стартер 5 секунд при поступлении команды "м". 

Читайте внимательно.

После того, как команда m пришла в первый раз, и всё отработало, что будет происходить при следующем проходе loop?

Поскольку нового ничего не пришло, условие в строке №22 будет ложным и строка №24 не отработает.

А что будет с условием в строке №27? Чему сейчас равна choice? Оппа ... условие-то будет истинным и строки №№29-31 сработают! И будут срабатывать ВСЕГДА.

Вот оно у Вас и срабатывает.

Переделал код, но теперь код работает только один раз. При поступлении новой команды "м" второй раз реле молчит.

SoftwareSerial BTserial(12,13);

char choice;

const int loopDelay = 50;

int IgnitionPin = 10;
unsigned long startTime;
unsigned long ignitionInterval = 30000;
unsigned long ignitionCountTime = 0;

void setup() 
{
  BTserial.begin(115200);  

  digitalWrite(IgnitionPin, HIGH);
  pinMode(IgnitionPin, OUTPUT);
}

void loop()
{
  if (BTserial.available())
  {
    choice = BTserial.read();
  }

  if( choice == 'm' )
  {
    digitalWrite(IgnitionPin, HIGH);
    ignitionCountTime = millis();
  }

  if (ignitionCountTime - startTime >= ignitionInterval)
  {
    digitalWrite(IgnitionPin, LOW);
  }

 delay(loopDelay);
}

33 строка - трэшь какой-то

SoftwareSerial BTserial(12,13);

char choice;

const int loopDelay = 50;

int IgnitionPin = 10;
unsigned long startTime;
unsigned long ignitionInterval = 30000;
unsigned long ignitionCountTime = 0;

void setup() 
{
  BTserial.begin(115200);  

  digitalWrite(IgnitionPin, HIGH);
  pinMode(IgnitionPin, OUTPUT);
}

void loop()
{
  if (BTserial.available())
  {
    choice = BTserial.read();
  }

  if( choice == 'm' )
  {
    digitalWrite(IgnitionPin, HIGH);
    ignitionCountTime = millis();
  }

  if (ignitionCountTime - startTime >= ignitionInterval)
  {
    digitalWrite(IgnitionPin, LOW);
  }

 delay(loopDelay);
}

Мля, давайте с начала. Уже по новому коду

Т.е. ignitionCountTime всегда будет обновляться свежим millis

Вы не с того конца взялись. Вы пытаетесь выразить мысль, которую Вы не до конца сформулировали на языке, который Вы не до конца понимаете.

Начните с того, что сядьте и по-русски напишите, что должно происходить в loop. детально, "если ... иначе .. то" но по-русски, внятно. Убедитесь, что это правильно, и только потом пытайтесь писать код.

До 33 мы с ним ещё не добрались :-)

Правильно ли я понимаю, что в конце цикла мне нужно "обнулить" choice? В моем цикле после поступления команды "м" он постоянно "м", пока не прийдет новое значение?

Тогда вопрос, как обнулить?

У меня только один вопрос: А почему эта тема не в песочнице? ))

Бинго. Так и есть. Обнулить - значит присвоить значение '\0'. Или вообще любое значение, кроме 'm'.

Например, присвоить 'Y' ))

Не знаю насколько это правильно, но получилось как-то так.

SoftwareSerial BTserial(12,13);

char choice;

int IgnitionPin = 10;
uint32_t timeout = 5 * 60;
uint8_t flag = 0;

void setup() 
{
  BTserial.begin(115200);  

  pinMode(IgnitionPin, OUTPUT);
  digitalWrite(IgnitionPin, LOW);
}

void loop()
{
  if (BTserial.available())
  {
    choice = BTserial.read();
  }

  if (choice == "m") 
   { 
    timeout = 5 * 60;
    flag = 1;
    digitalWrite(IgnitionPin, HIGH); 
   } 
   else if ((timeout > 0) && (flag == 1))
    {
            timeout--;
    }
   else 
    { 
       digitalWrite(IgnitionPin, LOW);
       flag = 0;
    }

choice = '\0';
}

Нет, Вы неправильно понимаете.

Что нужно делать правильно, я Вам написал

Вы решили проигнорировать? Ваше право, пожалуйста. Только по-другому это не работает. Либо кто-то просто напишет код за Вас из жалости, либо Вы начнёте делать это правильно, а именно - сначала добьётесь (от себя) внятной и строгой формулировки алгоритма.

В #12 ты такты считаешь, а не время. Воспользуйся подсказкой Евгения Петровича, иначе тут без вариантов.

В чем проблема Вашей логики - Евгений Петрович пояснит. 
По поводу миллис - да, начинается отсчёт от включения, но не прекращается и даже переполняется (можно грубо сказать обнуляется) и все это пока мк работает. Но эти значения можно получить, можно, присвоить и даже сверить.) И не нужно писать новый код пока Евгений Петрович не скажет что делать дальше.

С millis() время не нужно обнулять, нужно запоминать текущее значение в момент включения реле, и от этого момента отсчитывать нужные вам 5 секунд

SoftwareSerial BTserial(12,13);

char choice;


int IgnitionPin = 10;

void setup() 
{
  BTserial.begin(115200);  

  pinMode(IgnitionPin, OUTPUT);
  digitalWrite(IgnitionPin, LOW);
}

void loop()
{
  if (BTserial.available())
  {
    choice = BTserial.read();
    if(choice == 'm')
  {
   digitalWrite(IgnitionPin, HIGH);
   delay(5000);
   digitalWrite(IgnitionPin, LOW);
  }
  }

  

  
}

ТС ваши скетчи не компилируются, вот вам с делеем, без миллис и не мучайтесь логикой - она проста "если пришёл символ и этот символ м, то включаем реле, ждём 5 секунд, выключаем реле" :-)

lilik, ничего личного - но "прерывать обучение" как минимум не этично. Пусть учится, если хочет. Вроде желание есть.

Дилей будет весь процесс задерживать, мне надо чтобы не влияло на исполнение кода, т.к. это только маленькая его часть. 

Это не обучение, а бесцельное времяпрепровождение :-)

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

Это суждение преподавателя (в Вашем лице)?

SoftwareSerial BTserial(12,13);

char choice;

int ignitionState;
unsigned long ignitionInterval = 50;
unsigned long ignitionCountTime = 0;

void setup() 
{
  BTserial.begin(115200);  

  pinMode(IgnitionPin, OUTPUT);
  digitalWrite(IgnitionPin, LOW);
}

void loop()
{
  if (BTserial.available())
  {
    choice = BTserial.read();
  }

  if(choice == 'm')
  {
    ignitionState = HIGH;
    digitalWrite(IgnitionPin, ignitionState);
    ignitionCountTime = 0;
  }

  if (ignitionState == HIGH) 
  {
    ignitionCountTime = ignitionCountTime + 1;

    if (ignitionCountTime >= ignitionInterval) 
    {
      ignitionState = LOW;
    }

    digitalWrite(IgnitionPin, ignitionState);
  }

choice = '\0';
}

Уважаемый, я Вам уже ДВАЖДЫ говорил

Говорю это в третий и в последний раз. Если Вы твёрдо решили игнорировать то, что я говорю, не вижу смысла бить пальцы об клавиши - они (пальцы) у меня не сменные.

В том, что Вы пытаетесь его писать не имея алгоритма, т.е. не понимая что именно Вы пишете.

Давайте посмотрим, что Вы называете "логикой"

Это не алгоритм, это хотелки или, если хотите, "чувственные идеи". Чтобы написать какой-то код по таким хотелкам, нужна очень высокая квалификация программиста.  Как говорил граф Калиостро: «Материализация чувственных идей - есть труднейшая задача научной магии»

Попробуйте таки написать так, как я Вам говорил

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

интересно, кто кроме меня из присутствующих понимает, что именно имел ввиду Калиостро )))

Боюсь, что никто. Тут один специалист по научной магии :-)

а если попытать с пристрастием, может есть скрытые? )))

Вот и не сомневайтесь. Он как никак преподаватель. И далеко не каждого он берется учить, как вас ;)