Делаю беззвучную стиралку со звуком

Всем привет! Купил стиральную машину БЕКО, а там нет звука. Очень непривычно, решил сделать простенькую пищалку. Идея подключится питанием к экрану стиралки, а тригеры стартап-мелодии и финишной получать с соответсвующих светодиодов (лок дверцы - загорается после выбора программы стирки и ее запуска, завершение стирки/отмена) с помощью оптронов PC817. Глянье код пожалуйста, может что упустил?

Спойлер

#include <Arduino.h>

// Тут записаны ноты, которые мы будем использовать
#define NOTE_D5  587
#define NOTE_A5  880
#define NOTE_FS5 740
#define NOTE_E5  659
#define NOTE_G5  784
#define NOTE_C5  523
#define NOTE_G5  784

#define REST      0

// Мелодия окончания
int tempo = 85;
int finishMelody[] = {
  NOTE_D5, -4, NOTE_A5, 8,  NOTE_FS5, 8, NOTE_D5, 8,
  NOTE_E5, -4, NOTE_FS5, 8,  NOTE_G5, 4,
  NOTE_FS5, -4, NOTE_E5, 8,  NOTE_FS5, 4,
  NOTE_D5, -2,
  NOTE_D5, -4, NOTE_A5, 8,  NOTE_FS5, 8, NOTE_D5, 8,
  NOTE_E5, -4, NOTE_FS5, 8,  NOTE_G5, 4,
  NOTE_FS5, -1,
};

// Мелодия включения
int startupTempo = 85;
int startupMelody[] = {
  NOTE_C5, 6, NOTE_C5, 6, NOTE_G5, 4
};

// Настраиваем пины для устройств
int buzzer = 4;  // Бузер подключен сюда
const int finishTriggerPin = 12;  // Финиш-сигнал
const int startupTriggerPin = 10; // Стартап-сигнал

// Флаги и таймеры для окончания стирки
bool finishTriggered = false;
bool playedFirst = false;
bool playedSecond = false;
unsigned long firstPlayTime = 0;
unsigned long finishOffTime = 0;

// Флаги и таймеры для начала стирки
bool startupTriggered = false;
unsigned long startupOffTime = 0;

// Функция для воспроизведения финиш-мелодии
void playFinishMelody() {
  int wholenote = (60000 * 4) / tempo;
  int divider = 0, noteDuration = 0;
  int totalNotes = sizeof(finishMelody) / sizeof(finishMelody[0]) / 2;
  for (int i = 0; i < totalNotes * 2; i += 2) {
    divider = finishMelody[i + 1];
    if (divider > 0) {
      noteDuration = wholenote / divider;
    } else if (divider < 0) {
      noteDuration = wholenote / abs(divider) * 1.5;
    }
    tone(buzzer, finishMelody[i], noteDuration * 0.9);
    delay(noteDuration);
    noTone(buzzer);
  }
}

// Функция для воспроизведения стартап-мелодии
void playStartupMelody() {
  int wholenote = (60000 * 4) / startupTempo;
  int divider = 0, noteDuration = 0;
  int totalNotes = sizeof(startupMelody) / sizeof(startupMelody[0]) / 2;
  for (int i = 0; i < totalNotes * 2; i += 2) {
    divider = startupMelody[i + 1];
    if (divider > 0) {
      noteDuration = wholenote / divider;
    } else if (divider < 0) {
      noteDuration = wholenote / abs(divider) * 1.5;
    }
    tone(buzzer, startupMelody[i], noteDuration * 0.9);
    delay(noteDuration);
    noTone(buzzer);
  }
}

void setup() {
  pinMode(buzzer, OUTPUT);
  pinMode(finishTriggerPin, INPUT_PULLUP);
  pinMode(startupTriggerPin, INPUT_PULLUP);
}

void loop() {
  if (!startupTriggered && digitalRead(startupTriggerPin) == LOW) {
    playStartupMelody();
    startupTriggered = true;
  }
  if (startupTriggered) {
    if (digitalRead(startupTriggerPin) == HIGH) {
      if (startupOffTime == 0) {
        startupOffTime = millis();
      } else if (millis() - startupOffTime >= 10000) {
        startupTriggered = false;
        startupOffTime = 0;
      }
    } else {
      startupOffTime = 0;
    }
  }
  if (!finishTriggered && digitalRead(finishTriggerPin) == LOW) {
    playFinishMelody();
    playedFirst = true;
    finishTriggered = true;
    firstPlayTime = millis();
  }
  if (finishTriggered && playedFirst && !playedSecond && (millis() - firstPlayTime >= 60000)) {
    playFinishMelody();
    playedSecond = true;
  }
  if (finishTriggered && playedFirst && playedSecond) {
    if (digitalRead(finishTriggerPin) == HIGH) {
      if (finishOffTime == 0) {
        finishOffTime = millis();
      } else if (millis() - finishOffTime >= 10000) {
        finishTriggered = false;
        playedFirst = false;
        playedSecond = false;
        finishOffTime = 0;
      }
    } else {
      finishOffTime = 0;
    }
  }
}

Комментарии, именно там где они нужны.

1 лайк

Раздел для новичков, вопрос адекватно задал. Ты почему сразу хамишь?

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

3 лайка

Я вот тоже тут безалкогольный коктейль с коньячком хряпнул – потеплело :slight_smile:

Ну, про комментарии Вам уже сказали. В нескольких местах (строки №№ 50,58, 68, 76) Вы прошли в миллиметре от граблей и лишь случайно их не задели (уверен, что Вы их просто не видели). Аккуратность кода оставляет желать. Например, Вы в строке №52 делите на два, а в следующей строке “умножаете взад”. Да не просто умножаете, а делаете это в цикле (если оптимизатор не поправит). Ну и про две практически одинаковые функции … а если бы у Вас было не две, а 102 мелодии? 102 функции бы писали?

А чё так резко? Человек адекватно высказал своё мнение о Вашем коде. В чём Ваша проблема?

Я пока учусь и делаю как умею, но за наводку спасибо!

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

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

1 лайк

Лично я написал Вам вполне конструктивно, указал номера строк, где грабли, написал про неэффективность, про дублирование кода и т.п.

Однако, у меня сложилось впечатление, что фразой

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

1 лайк

Возможно Вы, как новичок, не в курсе, что по сути предложили форумчанам выполнить более 90% Вашей работы, тогда как сами сделали не более 10%.
Притом, выполнить - совершенно идиотским способом - посмотреть глазами.
И то и другое на данном форуме рассматривается как бестактное поведение.

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

Вот, кстати:

Неужели до этого трудно было дойти самостоятельно?

Может че по проще https://youtu.be/LkMATMANtLw?si=L12QBbLyIJQrWwlo

Хочу что бы было так:
Выбирается режим на стиралке, нажимается кнопка запуск стирки. В этот момент играет мелодия. (буду брать сигнал со светодиода блокировки дверцы). Потом стирка заканчивается, загорается светодиод окончания, с него беру сигнал окончания - играет вторая мелодия. По прошествии 5 минут мелодия повторяется. Свой код я протестил, работает все отлично. Стоит ли заморачиватся с его правками если итак все пашет?

нет!!

1 лайк

С точки зрения устройства, которое один раз сделали и больше в него никогда не полезете, вполне можно не заморачиваться. “Согласен с предыдущим оратором!” :slight_smile:
Но в образовательных целях, если собираетесь продолжать это занятие, стоит как минимум обратить внимание на замечания @ЕвгенийП. Устройство Ваше работает отчасти случайно, могло и не работать.

1 лайк

Не, ну где логика?
Если “нажимается кнопка”, то и сигнал надо брать с контактов этой кнопки.
А если “буду брать сигнал со светодиода”, то и писать нужно “зажигается светодиод…”.

1 лайк

О и правда! Спасибо за замечание, чето я совсем заморочился с этими оптронами. Хотя с другой стороны хз как там мозги машинки будут реагировать, я ведь 5V беру с ее мозгов (они на atmega128A). Безопасно ли к кнопке напрямую подключаться?

А проверял - оптрон точно открывается от светодиода?

Ну я 1.2 вольта подал, открылся. PC817

Это разные вещи.

Проверил - подкинул к машинке, открывается