Изменение значения в цикле loop

Зачем его комментировать? В конце идет обработка else! Если метка не приложена флаг обнуляется. Понимаете? Мы приложили метку и флаг = 1. Если сделать, как вы говорите и убрать метку с датчике флаг останется = 1...

 

Нафиг ненужная переменная.
Реле щелкает здесь

Код (C++):

void indication(int led)
{
  digitalWrite(led, HIGH);
  delay(1000);
  digitalWrite(led, LOW);
}

 

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

 

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

 

Всем большое спасибо за попытки помочь, но Вы явно меня не поняли. У меня переменная не сохраняет новое значение! Такое иногда бывает, по запарке, при использовании ссылок на переменную.
Вопрос: почему? Тут, судя по-всему, с этой проблемой никто не сталкивался. Я уже и сам склоняюсь к тому, что проблема где-то в железе. Может во время работы микросхема не может переписывать данные в памяти или еще что-то... В любом случае, ардуинка осталась на работе и доступ к ней будет только в понедельник. Проверю код еще раз и обязательно скину видеоролик. До понедельника.Всех благ.

 

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

 

Расскажите подробнее.

 

Вам три человека уже написали подробно и как надо. Это уже не смешно.

 

Добавьте ёще отладочной информации типа:

Спойлер: пример

Код (C++):

/* MOSI: Pin 11 / ICSP-4
* MISO: Pin 12 / ICSP-1
* SCK: Pin 13 / ISCP-3
* SS: Pin 10
* RST: Pin 9
*/

#include <SPI.h>
#include <RFID.h>

#define SS_PIN 10
#define RST_PIN 9
#define BLUE_LED 6
#define YELLOW_LED 7


RFID rfid(SS_PIN, RST_PIN);

unsigned char reading_card[5]; //for reading card
unsigned char master[5] = {160,112,85,128,5}; // allowed card
unsigned char i;
int isAllow;

void indication(int led);
void allow();
void denied();

void setup()
{
  Serial.begin(9600);
  SPI.begin();
  rfid.init();
  pinMode(BLUE_LED, OUTPUT);
  pinMode(YELLOW_LED, OUTPUT);
  digitalWrite(BLUE_LED, HIGH);
  digitalWrite(YELLOW_LED, HIGH);
  isAllow = 0;

  //DEBUG
  Serial.print("setup(): isAllow = ");
  Serial.println(isAllow);
  //DEBUG END
}

void loop()
{
   if (rfid.isCard())
   {
     Serial.println(analogRead(YELLOW_LED));
     if (rfid.readCardSerial())
     {
        /* Reading card */
        Serial.println(" ");
        Serial.println("Card found");
        Serial.println("Cardnumber:");
        for (i = 0; i < 5; i++)
        {
           Serial.print(rfid.serNum[i]);
           Serial.print(" ");
           reading_card[i] = rfid.serNum[i];
        }
        Serial.println();
        //verification
        for (i = 0; i < 5; i++)
        {
           if (reading_card[i]!=master[i])
           {
              break;
           }
        }
        if (i == 5)
        {
           if(isAllow == 0)
           {
              allow();
           }
        }
        else
        {
           denied();
        }
     }
  }
  else
  {
    digitalWrite(YELLOW_LED, HIGH);
    isAllow = 0;
    // rfid.halt();

   //DEBUG
   Serial.print("loop(): isAllow = ");
   Serial.println(isAllow);
   //DEBUG END
  }
}

void allow()
{
  Serial.println("Access accept!"); //доступ получен
  digitalWrite(YELLOW_LED, LOW);
  //delay(1000);
  isAllow = 1;
 
  //DEBUG
  Serial.print("allow(): isAllow = ");
  Serial.println(isAllow);
  //DEBUG END
}
void denied()
{
  Serial.println("Access denied!"); //доступ закрыт
  digitalWrite(YELLOW_LED, HIGH);
  delay(1000);
  isAllow = 0;

  //DEBUG
  Serial.print("debied(): isAllow = ");
  Serial.println(isAllow);
  //DEBUG END
}

Когда разберётесь в чём дело, отладку удалите.

 

Спасибо за дельный совет, но, к сожалению, логирование уже делал. Не так досконально, как вы, но вывод в терминал показал, что переменная isAllow НЕ изменяется. Более того, сначала вместо флага делал вариант с чтением пина digitalRead, который должен вернуть LOW или HIGH.

Код (C++):

if(digitalRead(LED) == HIGH)
{
  digitalWrite(LED, LOW)
}
 

Это не сработало и поскольку это был мой первый опыт с ардуино, подумал, что сделал ошибку или неправильно использовал функцию - отказался от данного кода. Решил использовать флаг - старый проверенный метод. Опять не работает. Сделал логирование - переменная всегда выдает 0. Что за ерунду, подумал я?
Проверю еще раз в понедельник, может у меня крыша поехала или где-то ошибся. Конец недели, все-таки...

Нет, эта часть кода срабатывает, когда rc метка не приложена к датчику.

Код (C++):

if (rfid.isCard())
{
//Если приложенная карта определена
}
else
{
//а это место, про которое вы мне говорите - оно срабатывает НЕ ВСЕГДА, а когда НЕТ КАРТЫ!!!!
}
 

Спасибо за пример с флагами. Обратил внимание, что код у вас срабатывает ровно ОДИН раз. Например, у вас сработал контакт и начинается дозвон по заданному номеру, но если контакт сработает еще раз? Например, в следующем проходе цикла? Дозвона не будет, так как флаг не обнулен. Ваш флаг остается = 0. Получается одноразовая сигнализация. Владельцу придется приезжать и делать перезагрузку ардуино, после чего все переменные обнулятся и флаг будет = 1. Тоже самое с state. Остальной код будет срабатывать каждый раз, когда сработает контакт, но без телефона. К примеру, если поставить ваш код на лампочку и датчик освещения в подъезде, то свет загорится только один раз и больше загораться не будет и люди будут ходить в темноте...
В моем случае, если я подношу карту, то лампочка зажигается, если убираю карту, то лампочка гаснет, если опять подношу карту лампочка зажигается и так далее... Ниже, для пользователя Airbus, есть пример работы с обнулением флага.

Если вы запустите СВОЙ скетч, то увидите, что если приложить карточку и не убирать ее, то реле будет включаться каждую секунду. Если карточку НЕ убирать, то цикл loop будет включать реле каждую секунду.
Тут важно понять, что это НЕ разовое нажатие кнопки, а УДЕРЖИВАНИЕ. Пока держишь кнопку условие выполняется. Если отпускаешь - условие не выполняется.

Теперь к моим баранам. Я хочу, чтобы при прикладывании карты реле включилось только ОДИН раз, следовательно программа должна как-то узнать включено ли реле УЖЕ или не включено. Есть два способа: узнать состояние пина или ввести дополнительный флаг. Если мы используем флаг, то его надо СБРАСЫВАТЬ на исходное состояние, иначе он не сработает в дальнейшем.
Напишу для вас простой пример кода с флагом ниже:

Код (C++):

flag = 0;
void loop()
{
  if(Удерживаем кнопку)
  {
    //Если еще не включено, то флаг будет = 0
    if(flag == 0)
    {
      turnOn(); //включаем
      flag = 1; //делаем пометку, что включено - если будем держать кнопку и дальше этот код не выполнится
    }
  }
  else //НЕ удерживаем кнопку
  {
    //если до этого момента удерживали кнопку, то flag будет =1, но кнопку то уже отпустили!!!
    flag = 0; //делаем пометку, что выключили
    turnOff(); //выключаем - тут, по-хорошему, надо тоже сделать проверку, иначе код будет всегда выключать то, что уже выключено.
  }
}
 

 

Я и не смеюсь. В целом, вы уже дважды себя показали, что наводит на определённые размышление, однако, бог любит троицу: обоснуйте свою позицию

 

После второго ответа я потерял мысль. Но если отвечать на первый вопрос, то условие срабатывает 1 раз. С чего вы взяли что переменная не меняется? Вот это:

Код (C++):

int var = 0;

где обьявлено?

 

Это упрощенный вариант. Реальный код есть на первой странице, внизу.

 

Я ещё и сам не знаю, что хочу сделать. Сам факт того, что оно щелкает меня бесит.

А какой у вас скетч?

Намного лучше.

Ковыряйте сколько угодно. Этот скетч скачал откуда то. Посмотрел видео и загорелся попробовать. Я лишь добавил флаг. Спасибо за помощь.

 

Поясните Вашу высказывание, т. е. каждую секунду реле сначала выключается, а потом включается? Или Вы имеете ввиду, что команда на включение реле подается каждую секунду?

По поводу флага. Пробовали проверять значение i перед строкой:

Код (C++):

if (i == 5)

 

Да, скетч постоянно включает включенное.

i не проверял. Конструкция странная