Большие массивы приводят к сбою Arduino

В отредактированной версии вашего вопроса вы написали:

Я использую memcpy_P(), чтобы скопировать массивы из PROGMEM в ОЗУ.

Вся цель PROGMEM — избежать необходимости хранить данные в ОЗУ. Если вы скопируете массивы в ОЗУ, вы победите PROGMEM. Твой У Arduino гораздо больше флэш-памяти, чем оперативной памяти, и она выйдет из строя, если вы ее съедите. вся оперативная память. Вот почему оптимизация PROGMEM полезна всякий раз, когда у вас большие массивы постоянных данных.

Правильный способ использования этих массивов — копировать в ОЗУ по одному элементу. время: получите данные для следующей ноты, которую вам предстоит сыграть, затем сыграйте ее, затем переходите к следующему. Например:

for (int i = 0; i < melody_length; i++) {
    int note = pgm_read_word(&Crash_Melody[i]);
    // и т. д...
}

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

---

Изменить: во втором редактировании вашего вопроса есть следующее:

float duration = pgm_read_word(&My_noteDurations[i]);

Что это делает:

• прочитайте первые два байта My_noteDurations[i] (который является четырехбайтовое количество, поэтому вы его усекаете)
• интерпретировать это как целое число (а это не так)
• преобразуйте это в float

Результат будет иметь мало общего с исходным числом. Верное способ чтения числа с плавающей запятой из PROGMEM — использовать pgm_read_float().

Теперь я вижу, что вы используете это число в качестве параметра для задержка(). Учитывая, что эта функция принимает целое число, нет смысла при сохранении задержки в виде числа с плавающей запятой. Это только добавит (несколько дорогостоящих) преобразование чисел с плавающей точкой в целое число в вашу программу без каких-либо дополнительных преимуществ. Кроме того, поскольку задержки, скорее всего, составят менее 65 секунд, 16-битного целого числа без знака должно быть достаточно.

Далее у вас есть это:

for (int thisNote = 0; thisNote < number_of_notes; thisNote++)
{
  tone(music_pin, melody, noteDuration);
  delay(pauseDuration);
  noTone(music_pin);
}

Я не понимаю цели этого цикла. Вы повторяете каждую ноту количество раз, равное общему количеству нот в песня. Почему?

---

Ответ на редактирование 3: в этой версии я вижу несколько проблем. кода.

1. Согласованность типов данных: в предыдущей версии это выглядело как будто длительность паузы была плавающей. Теперь они целые числа. У вас есть чтобы убедиться, что вы используете функцию pgm_read_*(), подходящую для фактический тип данных массива.

2. Чтение побайтово: нет смысла читать 16-битное целое число побайтно: используйте pgm_read_word(), чтобы прочитать все сразу. Чтение байт за байтом только усложняет код и увеличивает вероятность ошибок. Фактически, ваш способ восстановления чисел таков: глючит.

3. Цикл в playMelodyExtended2() для меня не имеет смысла. Пожалуйста попробуйте удаляя его.

Вот что я предлагаю на данный момент:

for (int i = 0; i < number_of_stored_values_in_MelodyArray; i++)
{
    int note = pgm_read_word(&My_Melody[i]);
    float duration = pgm_read_float(&My_noteDurations[i]);
    uint16_t pause_duration = pgm_read_word(&My_Delay[i]);
    tone(10, note, duration);
    delay(pause_duration);
}

В этом фрагменте сделаны некоторые предположения относительно остальной части вашего кода, и в частности, о том, как определяются ваши массивы. Если эти предположения неверны, это не сработает. Таким образом, я поддерживаю комментарий @orthena. и призываем вас опубликовать полную, тестируемую версию вашего эскиза (три примечания было бы достаточно).