Arduino.ru

А сами-то как думаете?

Есть конечно предположение, что будет, но... я только учусь....

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

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

Как я понимаю, многопоточности в Ардуино нет...

Тот код что Вы написали равносилен ДЕЛАЙ, ибо пока не выполнится фор 800000 раз программа дальше не пойдет

Так смотрите и учитесь, чего нас-то спрашиваете? Вы на своих ошибках учиться собрались, или на наших?

А где она есть? Если вычислительно ядро одно, то её, как таковой, нигде нет. Есть она только в мультиядерных системах. Но моделировать её можно везде.

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

#define delay for (int a =0; a<100;a++){};

=) шутка юмора =)

И тем не менее даже в предложенном синтаксисе можно поместить нечто внутрь фигурных скобок.

А причем тут ваши ошибки? Использовать for попробывал я и ваши ошибки я не рассматривал.

Так я об этом же =)) я плохо знаком с плюсами не знаю как использовать переменные в макросах) но суть была что в предложеном синтаксисе если что то находится меж скобок то можно заставить delay выполнять это что то =)

В Ваших словах есть резон но на мой взгляд это извращение.

Пишите номально код с миллис и будет все хорошо!

Так а к чему тут был вопрос тогда? вы пробовали? разницу не заметили - зачем задали вопрос?

А вообще учитесь писать код без delay и пустых циклов =)

Да с millis то понятно, у меня и построено смена экранов на нем, примерно так:

if ((millis() / 4000) % 2 == 0) // интервал 
{
led1; // Первый экран
}
else
{
led2; // Второй экарн
}

...

Уже лучше =) а теперь еще лучше сделайте =) 

(millis() && 0x100 == 0)

Присоединяюсь. Но это маразм из-за глубого не знания всего. Пишите нормально код и все. Да и millis() не всегда в выводе на экран нужен.

 ПС Все что в теме похоже на это.

А кто Вам сказал, что я это использую? Я только написал в чем разница.

если места мало и все просто, то миллис.

если для меги и много всего , то SimpleTimer и типамногозадачность, а по сути завернутый миллис.

например на сиплетаймере удобно опрашивать далласы. В сетапе запустил, потом первый раз через секунду вызвал , считал  и снова запустил. Потом через секунду пришел, или чере скольконадо. Я на этой библтотеке реализовывал каскадное регулирование четырех чиллеров ( блэкджеком и дамами с вэбом и аяксом) , работает однако уже года три.

Да, ладно! Не только можно, а там уже всё, что надо помещено. Вот он Ваш делэй

void delay(unsigned long ms)
{
	uint32_t start = micros();

	while (ms > 0) {
		yield();
		while ( ms > 0 && (micros() - start) >= 1000) {
			ms--;
			start += 1000;
		}
	}
}

Определите у себя функцию void yield(void)  и она будет вызываться из делэй за милую душу. А в ней уж помещайте чего хотите.

Так при том. Сам написал - сам и анализируй что получилось. Именно этот процесс анализа и есть учёба на ошибках. А Вы анализ на нас перекладываете.

Давайте рассуждать логически.

Какой код выполняет Ардуина во время цикла?

Да понятно, что процесс, будет ждать, пока цикл не прокрутится.

Анализ, анализом, я вопрос тут задал, для подтверждения, своего вывода. А не для того чтоб, за меня кто то что то решил.

Я не зря в самом первом посте написал: "Сейчас читаю форум "по порядку"" и насмотрелся на реакцию, некоторых индивидумов, которые себя позиционируют "всезнайками" и которым лень сказать, что - то по существу, когдау них просят совет, но зато как они любят разглагольствовать, мол ничего нет, а ты хочешь от нас, чтоб мы тебе все сделали.

Я просто хотл услышать подтверждение своих выводов, да, нет.

И еще, я обращался, лично вам ЕвгенийП ?

Нет.

Нет желания, помочь. Идем мимо. Зачем устраивать балаган кочив из себя "всезнайку".

Есть те , кто адекватно реагирует.

Интересно бы посмотреть на вас в реальной жизни, как вы общаетесь с окружающими вас людьми?

Если сегодняшнюю или подобную ситуацию, в которых вы были неоднакратно "замечены", экстраполировать на "семью": папа, а что будет если я ...? Сынок, тебе надо...., а от меня отстань, я то уже это знаю, мне не до тебя, но я тебе сейчас по этому поводу буду выносить моз, пустой информацией, которая к твоему вопросц не имеет ни малейшего отношения.

Хотя делая анализ, вашего поведения и отношения к людям - счастливая семья, много друзей?

Нет это не ваше.

Это затаенная детская, мелочная обида, на весь мир... 

Вы пишите, что не всегда нужен millis при выводе на экран.

Вероятно вы имели ввиду есть еще timeout  или SimpleTimer?

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

Экран 2004, на него поочереди выводится: текущее время, параметр установленный, параметр текущий 1, параметр текущий 2. На втором экране, показания с 4 других датчиков. И "экраны" с заданным интервалом меняются. В случаии "аварии" экран меняется функцией "if" на экран на котором выводится информация о событии. Но это уже другая история, так как этот "экран"(с информациеей о событии), можно переключить(сбросить) только в ручном режиме, так как ситуация обязывает, в любом случае, присутствие человека.

Так есть другой способ?

Если есть, был бы рад, "пинку" в нужном направлении. :)

Например система состояний, или система событий.

Система состояний, это "boolean", я правильно понял?

А система событий это прерывание по событию.

Прерывание хорошо подойдет, когда в приведенном мной примере, при скажем так внештатной ситуации, экран переключается, для отображения, состояния. Оно быстор переключит экран, вне зависимости на каком этапе находится выполнение скетча.

Но как прерывание или состояние будет управлять переключением экрана в штатном режиме? Отображение каждого экрана 4 - 5 секунд.

Отслеживать продолжительность в любом случае нужно.

А предложенные вами методы, на сколько мне известны срабатывают по событию.

Опять же я учусь и многого не знаю.

Конечно можно переключать экраны с помощью "boolean" и if но как отсчитывать время?

Если не использовать millis.

Нет, ты понял неправильно. Почитай, что такое конечные автоматы. Если вкратце: есть  набор состояний автомата, система может одновременно находится в одном из них. Как пример, в псевдокоде:

uint32_t timer = 0;
typedef enum
{
	stage1,
	stage2,
	
} MachineState;

MachineState stages = stage1;

void state1()
{
	// обрабатываем первое состояние
}

void state2()
{
	// обрабатываем второе состояние
}

void setup()
{
	
}

void loop()
{
	switch(stages)
	{
		case stage1:
		{
			state1();
			if(millis() - timer > 2000)
			{
				timer = millis();
				stages = stage2;
			}
		}
		break;
		
		case stage2:
		{
			state2();
		}
		break;
	}
}

В указанном примере система 2 секунды будет вызывать функцию state1, потом, бесконечно - функцию state2. При этом по каким-то условиям внутри state2 можно всегда переключиться на другое состояние конечного автомата, попав в другую ветку кода. Состояния можно добавлять до посинения. Короче - почитай теорию, она интересная, правда ;)

https://youtu.be/tt_EUA7aQNk

Просто настраиваете периодичность событий и они происходят, правда во время одного не вызывается другое, потому интервалы не обязательно будут соблюдаться абсолютно точно, и применением делаев или циклов без выхода все равно можно все завесить (как всегда). Но например опрос кнопок можно сделать в основном цикле, а реагирование на их состояние (например вывод на экран) в "сопроцессе", который периодически всплывает "сам".

Уже ответили за меня что не так.

Прерывание по таймеру? не слышали?

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

Вам никто не мешает сделать свою систему событий.

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

Можно но не так явно, выше вам человек привел код как сделать систему "состояний". Это на много более "правильно" нежели прям в коде лепить if(millis())... 

вот смотри.  Есть метеостанция.  Она показывает в основном время, но через некоторые промежутки показываются так же Температура и Влажность + еще может Давление и прочая х-ня.  Ясно, что дисплей в каждый определенный промежуток времени показывает что то одно, то есть находится в одном из детерминированных состояний.  Просто опиши их 

enum class enumDisplayState : byte  {Normal, Temperature, Humidity, Pressure, Error};

в состоянии Normal мы показываем часы, в состоянии Error - ошибки чтения всяких даччиков, остальное ясно из названия.   

и создай переменную этого типа:  enumDisplayState DisplayState;

Теперь, если у нас есть функция вывода на экран, ставим переключашку, чтоб выводить нужную информацию, в зав-ти от состояния

void Display(void) {

  switch (DisplayState) {

    case enumDisplayState::Normal: /* выводим часы */; break;

    case enumDisplayState::Temperature: /* выводим температуру */; break;

    case enumDisplayState::Humidity: /* выводим влажность */; break;

    case enumDisplayState::Pressure: /* выводим давление */; break;

    case enumDisplayState::Error: /* выводим сапщение об ашыпке */; break;

    default: /* выводим неведомую х-ню */;

}

}

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

void setDisplayState(enumDisplayState newState) {

 if (DisplayState == newState) return;

 DisplayState = newState;

 Display();

}

и сё.

Вот ответы, так ответы.

Благодарю.

А за видео с канала Инженерка "#070 Ардуино Библиотека SimpleTimer Libraries Arduino" отдельная благодарность.

Канал на Ютубе "Инженерка" Подписался.

Там много интересного. 

Всем начинающим советую.

Так ладно, пошел смотреть, читать.................

Katapuf, господа выше все правильно описали. Просто добавлю. Я давно забил на правку скетчей от новичков. Как только я начинаю править их по мелочам, то все равно потом их приходится переписывать а попутно и перепроектировать. И все потому что там море скрытых багов.

 1- новички пытаются весть код разместить в loop(). И вся простыня скетча это длиный loop(), в котором очень сложно ловить баги. А все потому что забывают о таком инструменте как функция. Достаточно куски кода выделять в функции,а лучше в методы класса.

2- вывод на экран лучше организовать в отдельной функии, а не разбросаными по скетчу print . тогда элементы на экране не будут запаздывать при показе. Итак. Надо вывести 1 экран запустили функцию vievScreen1(), а если 2 экран то vievScreen2(). Ну и так далее.

3- иногда надо менять экран не по времени чередуя 1 экран со вторым, а при нажатии кнопки в меню. Ведь человек нажимает кнопки не по не по функции delay, millis или всяким for , а по своим непонятным МК законам. Так что в обработчиках кнопки желательно будет организовать команду . Display.refrech()  /*обновить текущий экран*/  или  Display.refrech(/*номер экрана*/ 2)  /*обновить экран 2*/ . Осталось просто организовать правильно работу объекта Display.

Да наверное, в текущем "проекте", у меня получился давольно таки длинный  loop().

void loop()
{
  ModuleKey();       // Модуль Key RFID
  ModuleCall();      // Модуль Call
  ModuleSMS();       // Модуль SMS
  ModuleMotion();    // Модуль датчика движения
  ModuleKalitka();   // Модуль геркон 1
  ModuleVorota();    // Модуль геркон 2
  ModulePower();     // Модуль наличия сетевого напряженния
  ModuleVoltmeter(); // Модуль Вольтметр
  ModuleMQ2();       // Модуль датчика MQ-2
  ModuleMQ6();       // Модуль датчика MQ-6
  ModuleMQ7();       // Модуль датчика MQ-7
  ModulePlamya();    // Модуль Пламя
  ModuleDHT22_1();   //  Модуль датчика DHT22_1
  ModuleDHT22_2();   //  Модуль датчика DHT22_2
  ModuleNoTone();    //  Модуль выключения бузера
}

Это там еще нет "модулей" LCD, часов реального времени и еще пару модулей.

И еше несколько "модулей" которые стоят в самом низу скетча и "объявляются" в нужном месте в теле скетча, 

типа: 

"void BUZ_KLYUCH() ",

"void POLOVINA_KLYUCHA()"

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

По ходу дела надо то это и вот это прилепть. А вот без этого вообще не обойтись.

И с NANO пришлось переходить на MEGA. 

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

Золотое правило Адуинской программистской механики:  текст loop() должен полностью влазить в 1 экран.

паматри, кстати, мои таймеры на GitHub-е.  Вдруг пригадЯца.  

Пока влазит :)

Надеюсь на этом и останавлюсь(с этим скетчем).

На GitHub-е? По запросу у меня импортный дядька, а я его не понимаю.

Вы обращались ко всем, в том числе и ко мне, так что не надо.

А вот я обращался лично к Вам с вопросом "А сами-то как думаете?". Ответа не получил. Откуда сделал вывод, что никак.

Впрочем, мне пофиг.

2 Qwone, немного критики...

1) если нет необходимости в классе(хоть даже эстетической) - не нужно использовать его, используйте функции

2) англицкий, тяжко читать такие опечатки... и смысл понятен, и написано что глаз дерет....(View, refresh)

3) Логика работы рефреш в корне не верна =) 

  3а) Не нужно передавать в рефреш состояние, состояние должно менятся отдельно от рисования кадра для этого состояния, смена состояния вызвана ведь событием, и смена картинки является следствием смены состояния, а не причиной по которой нужно изменить состояние.

  3б) Не нужно(вообще!!!) делать функцию рефреш, нужно делать функцию инвалидации, которая скажет классу дисплея примерно следующее: "у тебя неверная картинка, следующий кадр строй с нуля". разница в том что при рефреше(как оно должно пониматся) - идет непосредственная перерисовка кадра, а при инвалидации - перерисовка кадра пойдет штатно, в положеное время.

Теперь по 3б подробней:

1) У вас идет отображение информации на двух экранах(Sc1 и Sc2) и так же в случае ошибок отображается экран ошибки(Er1).. Считаем что вывести ошибку - самый приоритетный для нас экран.

под экранами я понимаю "картинку"... 

2) Смена экранов Sc1 и Sc2 происходит по нажатию на кнопку(по таймеру, или еще как угодно).

3) Во время смены экрана Sc1->Sc2 происходит ошибка(аппаратная, прерывание например) и пользователю нужно показывать уже Er1.

4) Что должно произойти в случае Refresh? правильно экран моргнет отрисовав Sc2 и сразу следующим кадром Er1, т.к. оба раза были рефреши - вы использовали 2 раза отрисовку разных кадров.

5) Что произойдет в случае с инвалидацией? Вы 2 раза скажете дисплею "следующий раз рисуй с нуля", но дисплей нарисуется только единожды, с нужным экраном Er1. 

п.ы.с.ы. если я неверно понял ваш Refresh, и он означал ту же инвалидацию, то простите, в больших настольных системах это разные вещи которые реализуются именно как я описа - рефреш : немедленное перестроение картинки, инвалидация : штатное построение картинки в штатном режиме.

https://github.com/DetSimen/Arduino-

А этот ответ:

Не считается?

И еще.

Посмотрите, сколько людей откликнулось на мой вопрос и не просто так, а с советами и объяснениями, с сылками на обучающее виде.

Они даже спорят меж собой как лучше сделать.

...

Благодарю, скачал, буду посмотреть.

Здравия.

void loop()
{
  ModuleKey();       // Модуль Key RFID
  ModuleCall();      // Модуль Call
  ModuleSMS();       // Модуль SMS
  ModuleMotion();    // Модуль датчика движения
  ModuleKalitka();   // Модуль геркон 1
  ModuleVorota();    // Модуль геркон 2
  ModulePower();     // Модуль наличия сетевого напряженния
  ModuleVoltmeter(); // Модуль Вольтметр
  ModuleMQ2();       // Модуль датчика MQ-2
  ModuleMQ6();       // Модуль датчика MQ-6
  ModuleMQ7();       // Модуль датчика MQ-7
  ModulePlamya();    // Модуль Пламя
  ModuleDHT22_1();   //  Модуль датчика DHT22_1
  ModuleDHT22_2();   //  Модуль датчика DHT22_2
  ModuleNoTone();    //  Модуль выключения бузера
}

Длинный, короткий.... главное неверный =) 

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

ах да и главное, код шикарно откоментирован!!!! (сарказм если что, загляните в литературу по программированию на любом ЯП и почитайте как должен выглядеть комментарий и сколько их надо)

ModuleNoTone - из названия данной функции(а что она вообще тут делает? ее не должно быть в этом месте кода, и я на полном серьезе, ее тут быть не должно!) Мне совершенно не понятно что это Модуль, и что он отключает генерацию Шим для пищалки... вы это для кого комментили в коде? 

Мне ваши комментарии напомнили как я курсовые делал для студентов, тот же уровень, и напрашивается вывод или вы писали код для идиота, или просто переборщили с комментариями....

Нальёшь, при случае. 

void loop()
{
  ModuleKey();       // Модуль Key RFID
  ModuleCall();      // Модуль Call
  ModuleSMS();       // Модуль SMS
  ModuleMotion();    // Модуль датчика движения
  ModuleKalitka();   // Модуль геркон 1
  ModuleVorota();    // Модуль геркон 2
  ModulePower();     // Модуль наличия сетевого напряженния
  ModuleVoltmeter(); // Модуль Вольтметр
  ModuleMQ2();       // Модуль датчика MQ-2
  ModuleMQ6();       // Модуль датчика MQ-6
  ModuleMQ7();       // Модуль датчика MQ-7
  ModulePlamya();    // Модуль Пламя
  ModuleDHT22_1();   //  Модуль датчика DHT22_1
  ModuleDHT22_2();   //  Модуль датчика DHT22_2
  ModuleNoTone();    //  Модуль выключения бузера
}

Вообще то это писал для себя и коментарии я писал для себя.

Еще раз повторю, я эти закорючки вижу второй календарный месяц.

Это не значит, что я этим занимаюсь, все два месяца без отрыва.

А еще из этого времени, недели две ждал пока придет заказ с комплектухой.

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

Поэтому, сколько мне потребуется коментариев, столько я и напишу. 

И по поводу: "любому кто будет с этим работать на уровне кода - будет не понятно...." еще раз я это делаю для себя, возможно выложу, кому будет не понятно, объясню. Хотя прочитав коментарии, только ... не поймет, что к чему.

Вот кусок кода, что тут не понятного?

void ModuleKey()
{
  if ( ! mfrc522.PICC_IsNewCardPresent()) // Если нет нового ключа, то...
  {
    return;                               // Возвращаемся в начало
  }
  if ( ! mfrc522.PICC_ReadCardSerial())   // Если нечего не считали, то...
  {
    return;                               // Возвращаемся в начало
  }
  for (byte i = 0; i < mfrc522.uid.size; i++) // Подсчет серийного номера ключа.
  {
    uidDecTemp = mfrc522.uid.uidByte[i]; // Получение номера ключа
    uidDec = uidDec * 256 + uidDecTemp;  // Переводим номер ключа в дисятичный вид
  }
  //// Первый ключ считывание /////////////////////////////////////////////////////////////////////////////////////////////////////
  if (uidDec == keyBoss1_1 && key1_1_State == false)     // Если считанный номер ключа равен номеру ключа 1_1 и статус ключа 1_1 выклчен, то...
  {
    Serial.println("Первая половина ключа № 1 принята"); // Выводим в COM порт сообщение "Первая половина ключа № 1 принята"
    POLOVINA_KLYUCHA(); key1_1_State = true;             // Проигрываем мелодию половинки ключа, переводим состояние ключа 1_1 в включено
    delay(200);
  }
  if (uidDec == keyBoss1_2 && key1_2_State == false)     // Если считанный номер ключа равен номеру ключа 1_2 и статус ключа 1_2 выклчен, то...
  {
    Serial.println("Вторая половина ключа № 1 принята"); // Выводим в COM порт сообщение "Вторая половина ключа № 1 принята"
    POLOVINA_KLYUCHA(); key1_2_State = true;             // Проигрываем мелодию половинки ключа, переводим состояние ключа 1_2 в включено
    delay(200);
  }
  //// Первый ключ включаем
  if (key1_1_State == true && key1_2_State == true && alarmState == false) // Если статус ключа 1_1 вклчен, статус ключа 1_2 вклчен и сигнализация отключена, то...
  {
    digitalWrite(AlarmLED, HIGH); // Включаем LED индикатор "Сигнализация влючена"
    alarmState = true; BUZ_KLYUCH(); uidDec = 0;             // Переводим состояние сигнализации в включено, проигрываем мелодию ключ подошол, обнуляем номер ключа
    key1_1_State = false; key1_2_State = false;              // Переводим состояние ключа 1_1 и ключа 1_2 в выключено
    Serial.println(AlarmOn_K1);      // Выводим в COM порт сообщение "Гараж поставлен на охрану ключом № 1!"
    sms.SendSMS(Nomer1, AlarmOn_K1); // Отправляем SMS-сообщение "Гараж поставлен на охрану ключом № 1!"
    //sms.SendSMS(Nomer2, AlarmOn_K1); // Отправляем SMS-сообщение "Гараж поставлен на охрану ключом № 1!"
    //sms.SendSMS(Nomer3, AlarmOn_K1); // Отправляем SMS-сообщение "Гараж поставлен на охрану ключом № 1!"
  }
  //// Первый ключ отключаем
  if (key1_1_State == true && key1_2_State == true && alarmState == true) // Если статус ключа 1_1 вклчен, статус ключа 1_2 вклчен и сигнализация включена, то...
  {
    digitalWrite(AlarmLED, LOW); // Выключаем LED индикатор "Сигнализация влючена"
    alarmState = false; BUZ_KLYUCH(); uidDec = 0;      // Переводим состояние сигнализации в включено, проигрываем мелодию ключ подошол, обнуляем номер ключа
    key1_1_State = false; key1_2_State = false;        // Переводим состояние ключа 1_1 и ключа 1_2 в выключено
    Serial.println(AlarmOff_K1);      // Выводим в COM порт сообщение "Гараж снят с охраны ключом № 1!"
    sms.SendS

А строки типа "Serial.println(AlarmOff_K1); // Выводим в COM порт сообщение "Гараж снят с охраны ключом № 1!"" для отладки. потом их закоментирую.

И да этот скетч, а вернее готовое изделие уже работает, просто, как и говорил, хочеся добавить это, это, а еще вот это. А еще, пришлось переделывать с ключей iButton на RFID.

Развелось много идиотов, с электошокерами. У всех у кого стояла сигнализация с ключами iButton, пожгли логику. 

Конечно, нет. Вопрос: "как думаете". Ответ: "есть предположение". Я же не спрашивал есть у Вас предположение или нет, я спросил "каково оно". А ведь я всего лишь хотел Вам что-то объяснить, но не про коня в вакууме, а отталкиваясь от Вашего понимания вопроса (т.е. объяснить именно Вам, а не спорить с другими). Но Вы предпочли не отвечать на мой вопрос, дело Ваше. Мне, повторяю, пофиг. 

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

Я вас не учу, а советую как лучше, и лучше учится сразу делать правильно.

Избыточне комментарии - захламление кода, к хорошему не приводит, проверено.

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

void ModuleKey()
{
  if ( ! mfrc522.PICC_IsNewCardPresent()) // Если нет нового ключа, то...
  {
    return;                               // Возвращаемся в начало
  }
  if ( ! mfrc522.PICC_ReadCardSerial())   // Если нечего не считали, то...
  {
    return;                               // Возвращаемся в начало
  }
  for (byte i = 0; i < mfrc522.uid.size; i++) // Подсчет серийного номера ключа.
  {
    uidDecTemp = mfrc522.uid.uidByte[i]; // Получение номера ключа
    uidDec = uidDec * 256 + uidDecTemp;  // Переводим номер ключа в дисятичный вид
  }
  //// Первый ключ считывание /////////////////////////////////////////////////////////////////////////////////////////////////////
  if (uidDec == keyBoss1_1 && key1_1_State == false)     // Если считанный номер ключа равен номеру ключа 1_1 и статус ключа 1_1 выклчен, то...
  {
    Serial.println("Первая половина ключа № 1 принята"); // Выводим в COM порт сообщение "Первая половина ключа № 1 принята"
    POLOVINA_KLYUCHA(); key1_1_State = true;             // Проигрываем мелодию половинки ключа, переводим состояние ключа 1_1 в включено
    delay(200);
  }
  if (uidDec == keyBoss1_2 && key1_2_State == false)     // Если считанный номер ключа равен номеру ключа 1_2 и статус ключа 1_2 выклчен, то...
  {
    Serial.println("Вторая половина ключа № 1 принята"); // Выводим в COM порт сообщение "Вторая половина ключа № 1 принята"
    POLOVINA_KLYUCHA(); key1_2_State = true;             // Проигрываем мелодию половинки ключа, переводим состояние ключа 1_2 в включено
    delay(200);
  }
  //// Первый ключ включаем
  if (key1_1_State == true && key1_2_State == true && alarmState == false) // Если статус ключа 1_1 вклчен, статус ключа 1_2 вклчен и сигнализация отключена, то...
  {
    digitalWrite(AlarmLED, HIGH); // Включаем LED индикатор "Сигнализация влючена"
    alarmState = true; BUZ_KLYUCH(); uidDec = 0;             // Переводим состояние сигнализации в включено, проигрываем мелодию ключ подошол, обнуляем номер ключа
    key1_1_State = false; key1_2_State = false;              // Переводим состояние ключа 1_1 и ключа 1_2 в выключено
    Serial.println(AlarmOn_K1);      // Выводим в COM порт сообщение "Гараж поставлен на охрану ключом № 1!"
    sms.SendSMS(Nomer1, AlarmOn_K1); // Отправляем SMS-сообщение "Гараж поставлен на охрану ключом № 1!"
    //sms.SendSMS(Nomer2, AlarmOn_K1); // Отправляем SMS-сообщение "Гараж поставлен на охрану ключом № 1!"
    //sms.SendSMS(Nomer3, AlarmOn_K1); // Отправляем SMS-сообщение "Гараж поставлен на охрану ключом № 1!"
  }
  //// Первый ключ отключаем
  if (key1_1_State == true && key1_2_State == true && alarmState == true) // Если статус ключа 1_1 вклчен, статус ключа 1_2 вклчен и сигнализация включена, то...
  {
    digitalWrite(AlarmLED, LOW); // Выключаем LED индикатор "Сигнализация влючена"
    alarmState = false; BUZ_KLYUCH(); uidDec = 0;      // Переводим состояние сигнализации в включено, проигрываем мелодию ключ подошол, обнуляем номер ключа
    key1_1_State = false; key1_2_State = false;        // Переводим состояние ключа 1_1 и ключа 1_2 в выключено
    Serial.println(AlarmOff_K1);      // Выводим в COM порт сообщение "Гараж снят с охраны ключом № 1!"
    sms.SendS

То что ваш код говорит одно, а комментарий обратное, это первое что непонятно в этом коде.

Например return никак не "Возвращаемся в начало", а диаметрально противоположное - завершаем выполнение функции и возвращаем значение, в вашем случае null.

Дальше непонятен не этот кусок кода в частности, а что именно происходит внутри программы, если появится баг - тому кто не знаком с кодом, придется перечитать весь код чтобы найти в чем проблемма, а не "оглавление" и сразу перейти к нужному участку(Привет ООП).

Дальше что мне не понятно, зачем вы расходуете ресурсы МК? если по незнанию то посмотрите описание оператора if, а точнее его полной версии if then else

Тоже неверный подход... оно конечно карашо просто закоментировать, но(!) мой вам совет, используйте условную компиляцию:

 В начале программы обьявляем #define DEBUG

Ваш код для отладки обрамляем(в синтаксисе могу немного ошибатся):

#if defined DEBUG

  Serial.println(AlarmOff_K1);

#endif

Тогда этот код будет компилироватся только пока есть директива #define DEBUG, если ее убрать - код с примера выше скомпилирован не будет, и в МК не попадет.

Еще замечание по коду - У вас в примере нечто похожее на систему состояний, используйте там Case.. читаемость кода будет выше.

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

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

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

Поэтму если нет причин использовать класс - лучше использовать функцию.

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

Ровно до момента "У меня нехватило бетона" и "Мне нужен МК с 200мгц и 30 мб памяти"

#if defined DEBUG

  Serial.println(AlarmOff_K1);

#endif

ИМХО, несколько утомительно вокруг каждого вывода в Сериал писать операторы условной компиляции. Вот такой вариант мне нравится больше - подсмотрел у кого-то из местных гуру:

// определяем отдельные операторы отладлчной печати
#ifdef DEBUG
 #define DEBUG_PRINT(x)     Serial.print(x)
 #define DEBUG_PRINTDEC(x)     Serial.print(x, DEC)
 #define DEBUG_PRINTLN(x)  Serial.println(x)
#else
 #define DEBUG_PRINT(x)
 #define DEBUG_PRINTDEC(x)
 #define DEBUG_PRINTLN(x)
#endif 
.....
// в коде программы отладочный вывод печатаем как

DEBUG_PRINTLN(AlarmOff_K1) ;

// определяем отдельные операторы отладлчной печати
#ifdef DEBUG
 #define DEBUG_PRINT(x)     Serial.print(x)
 #define DEBUG_PRINTDEC(x)     Serial.print(x, DEC)
 #define DEBUG_PRINTLN(x)  Serial.println(x)
#else
 #define DEBUG_PRINT(x)
 #define DEBUG_PRINTDEC(x)
 #define DEBUG_PRINTLN(x)
#endif 
.....
// в коде программы отладочный вывод печатаем как

DEBUG_PRINTLN(AlarmOff_K1) ;

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

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

А вот тут если можно то поподробней.

Это из-за того, что не использовал "else" или ....

if then else - да, из за того что вы его не использовали, хотя почему - не понятно, состояния у вас внутри конструкции не меняются настолько чтобы последовательно могли выполнится все if. 

В вашем случае у вас выполнятся все проверки условий(будет проверка всех условий, если гдето в условиях есть вызовы функций - они будут вызваны, и т.д.), если же поставить else - програма сразу же выскочит из функии выполнив первое подходящее условие.

Еще дальше было сказано что есть штука Switch case, почему не ее использовали? 

Давайте пройдемся по вашему коду..:

void ModuleKey()
{
  if(!(mfrc522.PICC_IsNewCardPresent() && mfrc522.PICC_ReadCardSerial())) return;

  for (byte i = 0; i < mfrc522.uid.size; i++) 
  {
    uidDecTemp = mfrc522.uid.uidByte[i]; 
    uidDec = uidDec * 256 + uidDecTemp;  
  }

  if(!key1_1_State || !key1_2_State)
  {
    if (uidDec == keyBoss1_1 && !key1_1_State)
    {
      Serial.println("Первая половина ключа № 1 принята");
      POLOVINA_KLYUCHA(); key1_1_State = true;
      delay(200);
    }else
    if (uidDec == keyBoss1_2 && !key1_2_State)
    {
      Serial.println("Вторая половина ключа № 1 принята");
      POLOVINA_KLYUCHA(); key1_2_State = true;
      delay(200);
    }
  } 
  if (key1_1_State && key1_2_State && !alarmState)

  {
    digitalWrite(AlarmLED, HIGH); 
    alarmState = true; BUZ_KLYUCH(); uidDec = 0;
    key1_1_State = false; key1_2_State = false;
    Serial.println(AlarmOn_K1);
    sms.SendSMS(Nomer1, AlarmOn_K1);
  }else
  {
    digitalWrite(AlarmLED, LOW);
    alarmState = false; BUZ_KLYUCH(); uidDec = 0;
    key1_1_State = false; key1_2_State = false;
    Serial.println(AlarmOff_K1);
    sms.SendS
  }
}

Сравнивайте, это "на коленке" и не вдаваясь в подробности, больше чем уверен код можно сделать еще более воспринимаемым если вчитыватся...

Опять мы друг друга не понимай =) Если причина использования класса - это организация верной структуры, то это уже очень веская причина..

Но делать класс ради использования один раз одного метода из него за весь код проэкта - не хорошая идея.