Форум arduino.ua

Ви користуєтесь Arduino IDE? Зайдіть у налаштування та увімкніть "Show verbose output during: compile". Я х.з. як воно в локалізованому варіанті, щось типу "Показывать подробный вывод при: компиляции".

Тег

[code]
[==C++==]

тут ваш код

[/code]

https://forum.arduino.ua/help.php#bbcode

Для початку скопіюйте текст помилки і вставте у повідомлення текстом. Також бажано код вставити текстом, щоб було, зрозуміло де в коді який номер рядка.
Ще би бажано описати, яке призначення вашого коду, що ви намагаєтесь реалізувати.
Так більше шансів отримати допомогу.

Може бути, що перед тим як зникнути сигнал тіліпається туди-сюди (той самий брязкіт), і відбувається перехід із високого рівня в низький, потім назад у високий. Я ж не знаю, що там за датчики і який характер сигналу на вході. Код розрахований на чіткі фронти.
Але додавати туди ще й фільтрацію по таймінгам - дійсно, простіше все з нуля переписати. У вигляді нормального конечного автомата.

Може бути і якась інша причина. Повставляйте Serial.print() у ключові точки та подивіться, що там відбувається.

PS: Опитування кнопок у вас чомусь під "if (input != input1_prev)" попало. Воно має бути ззовні блока if.

Яка саме помилка? Скопіюйте сюди її повний текст.
Куди і замість якої частини цю частину вставили? Який вигляд має loop() після зміни?

У вас є вхід з двома станами: низький / високий рівень (можливий брязкіт контактів поки до уваги не берем).
Є вихід з двома станами: реле вимкнене / увімкнене.
Є внутрішній стан таймера: активний / неактивний.

Це вже дає 8 можливих комбінацій (частина з них потім редукується). Для початку намалюйте діаграму або таблицю переходів. Для кожного переходу визначте умову (стан входу, стан таймера), а також дії, які мають виконатись (увімкнути/вимкнути реле, запустити таймер і т.п.).

Поки реле увімкнене, повторний перехід з високого на низький рівень на вході має перезапускати таймер? Чи стан вхіду має значення тільки після вимкнення реле?

В тому скетчі одразу впадає в очі як мінімум один баг: TubeDisplay4Bit() виконується в контексі переривання і читає та пише int relay*_time_left. В loop() ці змінні також модифікуються без будь-якої синхронізації.

Також uchar OneSecondCnt як глобальна змінна ініціалізована нулем, потім

  OneSecondCnt--;
  if(OneSecondCnt==0) ...

Тобто переходить через переповнення - навряд чи так було спеціально задумано.

Ще купа інших недоліків у тому коді, але про них поки промовчу.

Так у вас умова вмикання реле - це низький рівень на вході:

if (digitalRead(INPUT1)==LOW) { ... digitalWrite(j1, HIGH);digitalWrite(led, HIGH); } ...

А умова вимикання - закінчення відліку лічильника

if (relay1_time_left>0) { ... } else { digitalWrite(j1, LOW); relay1_display = relay1_delay_time;}

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

Тобто змінна year має значення 165. Воно може бути отримане від bcdToDec() при аргументі 255: 15*10 + 15.
У вас або Wire.read() повертає -1 (немає даних в буфері), або відвалилась SDA і читаються всі одиниці.

Спершу додайте перевірку результату Wire.read() на -1. Якщо що, Wire.read() повертає int, а не byte.

Якщо там проблема зі зʼєднанням по I²C, то аналізувати подальший код немає сенсу, поки цю проблему не буде виправлено. Нажаль, Wire - жахливо написана бібліотека, і з нею діагностувати помилки комунікації не так то просто.

Доречі, назви функцій bcdToDec() та decToBcd() невдалі, BCD саме по собі Dec. Назви мали би бути щось типу bcdToInt() та intToBcd().

Не дуже розумію, навіщо читати значення в BCD, конвертувати його в число, а потім число знову в BCD, щоб відображати на дисплеї. Але то таке.

Так само як читаєте регістри годинника та календаря, тільки всі 19 регістрів:

  Wire.beginTransmission(DS3231_I2C_ADDRESS);
  Wire.write(0);
  Wire.endTransmission();

  size_t len = Wire.requestFrom(DS3231_I2C_ADDRESS, 19);
  Serial.print("requestFrom() = ");
  Serial.println(len);
  for (int i = 0; i<19; i++) {
     Serial.print(i, HEX);
     Serial.print("h: ");
     Serial.println(Wire.read());
  }

Значить якось записалось. Це найперша очевидна причина зупинки осцилятора при живленні від батарейки.

Відʼєднайте зовнішнє живлення та батарейку. На пару секунд закоротіть контакти гнізда куди вона підключається, щоб розрядити ємності. Вставте батарейку назад.

Цікаво, як цей код може видавати "2165 рік", коли

byte ... year;

...

Serial.print(year);

Або це не той код, або щось недоговорюєте.

Та мабуть же код для нано непортабельний, покладається на особливості 8-бітної AVR. А на 32-розрядній системі виліз страшний Undefined Behavior.

Не треба на BMS подавати 9В, якщо в BMS немає функції обмеження зарядного струму та зупинки заряду. Хіба ця BMS містить в собі зарядний контролер?

Звідки пристрій, до якого не підключено нічого, може давати 8 вольт? Мабуть на P+ P- все ж таки щось подаєте?

Те ж саме, звідки на P+ P- може бути напруга, якщо нічого не підключено?

8205A - то мосфети. Контролер BMS - це ота маленька шестинога U1.

Зміна стану пінів порта - це запис значень в регістр ввода-вивода, який знаходиться за певною адресою. Присвоєння значення звичайній змінній фундаментального типу (яким є unsigned char aka byte) - це запис значення в іншу адресу, де ця змінна розташована. Хоча фактичного запису в ОЗУ може і не бути, якщо компілятор так вирішить

Якщо ви хочете перехоплювати звернення до певних адрес ОЗУ, то на процесорі без MMU, яким є Atmega, це неможливо.

Якщо ви хочете, щоб при присвоєнні виконалась певна _дія_ (бітові операції над значенням та запис в регістр ввода-вивода), то створюйте клас і перевантажуйте оператор присвоєння, як у прикладі вище.

A = (D >> 3) & 1;

Також можна:

A = (D & 8) >> 3;

що можна записати як

A = (D & (1u << 3)) >> 3;

Загальний випадок для будь-якого біта n:

A = (D >> n) & 1;