Массовое объявление переменных

А где и как у вас эти строки используются?

 

Вот как все работает. loop вызывает функцию light каждую секунду.
В light определяется текущее время, переводится в секунды, затем из переменный lightOn1 и lightOff1 берется время включения и время отключения, переводится в секунды, далее сравнивается текущее время в секундах с временем включения и выключения, если попадает в этот диапазон, то включает реле

Код (C++):

void loop(){
  light();
  delay(1000);
}

void light(){
  // Определение текущего времени
  char *timeLight = rtc.getTimeStr();
  sscanf (timeLight,"%d:%d:%d", &hourNow, &minuteNow, &secondNow);
  secondTime = hourNow * 60 * 60 + minuteNow * 60 + secondNow;
 
  // Определение времени включения, выключение. Включение реле и отправка отчета
  if (lightOn1.length() > 0 && lightOff1.length() > 0) {
    char tempOn1[lightOn1.length()+1];
    lightOn1.toCharArray(tempOn1, lightOn1.length()+1);
    char tempOff1[lightOff1.length()+1];
    lightOff1.toCharArray(tempOff1, lightOff1.length()+1);  
    sscanf (tempOn1, "%d:%d:%d", &hourStart1, &minuteStart1, &secondStart1);
    secondTimeStart1 = hourStart1 *60 * 60 + minuteStart1 * 60 + secondStart1;
    sscanf (tempOff1, "%d:%d:%d", &hourStop1, &minuteStop1, &secondStop1);
    secondTimeStop1 = hourStop1 * 60 * 60  + minuteStop1 * 60 + secondStop1;
    if(secondTime >= secondTimeStart1 && secondTime <= secondTimeStop1){
      if(digitalRead(SHELF_LIGHT_1)==OFF) {
        digitalWrite(SHELF_LIGHT_1, ON);
        sendStatus(ID_SENSOR_LIGHT_1,1,0); //Отправляем данные на сервер
      }
    } else {
      if(digitalRead(SHELF_LIGHT_1)==ON) {
        digitalWrite(SHELF_LIGHT_1, OFF);
        sendStatus(ID_SENSOR_LIGHT_1,0,0); //Отправляем данные на сервер
      }
    }
  }
}

 

Это прямо какой-то "код наоборот"

А что за библиотека для RTC у вас? Дайте ссылку.

 

ОК, допустим что вот эта: https://physics.csuchico.edu/~eayars/code/DS3231.zip

Зачем вы берете время в виде строки и мучаетесь с разбором, когда в классе DS3231 есть методы, чтобы время читать нормально?

 

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

А библиотека та, что в вашей ссылке

Ну вот сейчас еще раз посмотрел. Варианта не нашел.
Сервер у меня передает время включения и выключения в формате hh:mm:ss
от DS3231 я могу получить время либо в Unix формате rtc.getUnixTime(rtc.getTime()) либо в том же формате, что и с моего сервера (hh:mm:ss). Поэтому и решил раскладывать и переводить в секунды

 

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

То, что приходит снаружи - да, единожды обрабатываете - преобразуете в числовое представление и потом с ним уже работаете.

 

А какой командой, я в примерах такую не нашел

 

Зачем примеры? Смотрим заголовочный файл с объявлением класса:
Куча методов get... на все случаи жизни.

Код (C++):

/*
* DS3231.h
*
* Arduino Library for the DS3231 Real-Time Clock chip
*
* (c) Eric Ayars
* 4/1/11
* released into the public domain. If you use this, please let me know
* (just out of pure curiosity!) by sending me an email:
* eric@ayars.org
*
*/

#ifndef DS3231_h
#define DS3231_h

#include <WProgram.h>
#include <Wire.h>

class DS3231 {
    public:
         
        //Constructor
        DS3231();

        // Time-retrieval functions
 
        // the get*() functions retrieve current values of the registers.
        // If you only need one element, use that one for simplicity; but
        // if you need the whole passel then use getTime() to avoid
        // the chance of rollover between reads of the different components.
        void getTime(byte& year, byte& month, byte& date, byte& DoW, byte& hour, byte& minute, byte& second);
        byte getSecond();
        byte getMinute();
        byte getHour(bool& h12, bool& PM);
            // In addition to returning the hour register, this function
            // returns the values of the 12/24-hour flag and the AM/PM flag.
        byte getDoW();
        byte getDate();
        byte getMonth(bool& Century);
            // Also sets the flag indicating century roll-over.
        byte getYear();
            // Last 2 digits only

        // Time-setting functions
        // Note that none of these check for sensibility: You can set the
        // date to July 42nd and strange things will probably result.
     
        void setSecond(byte Second);
            // In addition to setting the seconds, this clears the
            // "Oscillator Stop Flag".
        void setMinute(byte Minute);
            // Sets the minute
        void setHour(byte Hour);
            // Sets the hour
        void setDoW(byte DoW);
            // Sets the Day of the Week (1-7);
        void setDate(byte Date);
            // Sets the Date of the Month
        void setMonth(byte Month);
            // Sets the Month of the year
        void setYear(byte Year);
            // Last two digits of the year
        void setClockMode(bool h12);
            // Set 12/24h mode. True is 12-h, false is 24-hour.

        // Temperature function

        float getTemperature();

        // Alarm functions
     
        void getA1Time(byte& A1Day, byte& A1Hour, byte& A1Minute, byte& A1Second, byte& AlarmBits, bool& A1Dy, bool& A1h12, bool& A1PM);
/* Retrieves everything you could want to know about alarm
* one.
* A1Dy true makes the alarm go on A1Day = Day of Week,
* A1Dy false makes the alarm go on A1Day = Date of month.
*
* byte AlarmBits sets the behavior of the alarms:
*    Dy    A1M4    A1M3    A1M2    A1M1    Rate
*    X    1        1        1        1        Once per second
*    X    1        1        1        0        Alarm when seconds match
*    X    1        1        0        0        Alarm when min, sec match
*    X    1        0        0        0        Alarm when hour, min, sec match
*    0    0        0        0        0        Alarm when date, h, m, s match
*    1    0        0        0        0        Alarm when DoW, h, m, s match
*
*    Dy    A2M4    A2M3    A2M2    Rate
*    X    1        1        1        Once per minute (at seconds = 00)
*    X    1        1        0        Alarm when minutes match
*    X    1        0        0        Alarm when hours and minutes match
*    0    0        0        0        Alarm when date, hour, min match
*    1    0        0        0        Alarm when DoW, hour, min match
*/
        void getA2Time(byte& A2Day, byte& A2Hour, byte& A2Minute, byte& AlarmBits, bool& A2Dy, bool& A2h12, bool& A2PM);
            // Same as getA1Time();, but A2 only goes on seconds == 00.
        void setA1Time(byte A1Day, byte A1Hour, byte A1Minute, byte A1Second, byte AlarmBits, bool A1Dy, bool A1h12, bool A1PM);
            // Set the details for Alarm 1
        void setA2Time(byte A2Day, byte A2Hour, byte A2Minute, byte AlarmBits, bool A2Dy, bool A2h12, bool A2PM);
            // Set the details for Alarm 2
        void turnOnAlarm(byte Alarm);
            // Enables alarm 1 or 2 and the external interrupt pin.
            // If Alarm != 1, it assumes Alarm == 2.
        void turnOffAlarm(byte Alarm);
            // Disables alarm 1 or 2 (default is 2 if Alarm != 1);
            // and leaves the interrupt pin alone.
        bool checkAlarmEnabled(byte Alarm);
            // Returns T/F to indicate whether the requested alarm is
            // enabled. Defaults to 2 if Alarm != 1.
        bool checkIfAlarm(byte Alarm);
            // Checks whether the indicated alarm (1 or 2, 2 default);
            // has been activated.

        // Oscillator functions

        void enableOscillator(bool TF, bool battery, byte frequency);
            // turns oscillator on or off. True is on, false is off.
            // if battery is true, turns on even for battery-only operation,
            // otherwise turns off if Vcc is off.
            // frequency must be 0, 1, 2, or 3.
            // 0 = 1 Hz
            // 1 = 1.024 kHz
            // 2 = 4.096 kHz
            // 3 = 8.192 kHz (Default if frequency byte is out of range);
        void enable32kHz(bool TF);
            // Turns the 32kHz output pin on (true); or off (false).
        bool oscillatorCheck();;
            // Checks the status of the OSF (Oscillator Stop Flag);.
            // If this returns false, then the clock is probably not
            // giving you the correct time.
            // The OSF is cleared by function setSecond();.

    private:

        byte decToBcd(byte val);
            // Convert normal decimal numbers to binary coded decimal
        byte bcdToDec(byte val);
            // Convert binary coded decimal to normal decimal numbers
        byte readControlByte(bool which);
            // Read selected control byte: (0); reads 0x0e, (1) reads 0x0f
        void writeControlByte(byte control, bool which);
            // Write the selected control byte.
            // which == false -> 0x0e, true->0x0f.

};

#endif