Нужна помощь по массиву данных

sanik, замечания по коду нужны или "работает и ладно"?

 

В работе не проверял замечания лишними не будут

 

вот этот кусок интересный

Код (C++):

 if (t.sec  == 00) {
    if (t.min == 10) { // если в текущем времени  10 минут
      if (t.hour == 00) { //  если в текущем времени  0 часов
        t = rtc.getTime(); // Get data from the DS3231
        w = t.dow - 1; // получаем день недели и отнимаем еденицу так как массив начинается с нуля

- во первых. что это за "00" в первой строке? t.sec - это целое число. если вы хотите, чтобы там был ноль -так и пишите, что такое 00 никто не знает. То же самое относится к третьей строчке
Ну и дальше - смотрите в первой. второй, третьей и пятой строке вы используете разные части структуры времени - t. Но читаете эту структуру почему-то между третьей и пятой строкой. Если у вас уже есть текущее время - почему вы день недели не берете из него? А если времени еще нет - то откуда берутся секунды. минуты и часы выше?

 

Спасибо забыл из своего кода про строку чтения времени Сейчас отредактирую

 

С комментариями согласен, но так ли это принципиально в плане корректности работы кода?

 

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

 

А что значит неинициализированной структуры касательно этого примера?

 

сначала удивился вопросу, а потом посмотрел - а ТС-то код уже переписал
Поэтому Вы и не можете понять, что я имею в виду..

Посмотрите кусок кода в моем сообщении #23 - первое чтение времени с часов было уже ПОСЛЕ проверки часов. минут и секунд

 

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

 

Код (C++):

uint8_t dow_flags[week] |= _BV(dow);