Как исправить код утечки памяти в ESP8266/NodeMCU, вызванный концентрацией строк?

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

serial_data_read.concat(character); 

Когда вы создаете экземпляр строкового объекта с помощью String serial_data_read = "", класс Arduino String создает выделение памяти, используя динамическое выделение памяти в куче, и, поскольку вы инициализировали его с помощью"", он, следовательно, создает минимальный размер выделения. Все это хорошо, если вы знаете, как создается динамическая память и как она будет использоваться.

Проблема появилась, когда вы начинаете делать конкатенацию строк с линией serial_data_read.concat(character), оригинал распределения для глобальных переменных serial_data_read уже не хватает места для составной переменной, так это создать временную переменную в куче, а не конкатенации, а затем в конечном итоге поместить результат обратно к недавно выделенной памяти для serial_data_read в куче, оставляя отверстие в кучи для оригинального serial_data_read переменной (и переменную Temp).

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

Краткосрочное исправление - не выполняйте связывание строк в цикле

Класс String не всегда является злым, если вы знаете, что происходит и как его использовать. Одно из быстрых решений вашей проблемы заключается в том, чтобы просто не выполнять связывание строк в цикле. И если вы используете его в функции, сохраняйте его локальным. Это сведет к минимуму фрагментацию кучи.

if ((last_weight_time < millis() - 1000)) {
   last_weight_time = millis();
   if (Serial.available() > 0) {
      String serial_data_read = Serial.readStringUntil('\n');
   }
}

Кстати, вы можете добавить ESP.getFreeHeap() в свой цикл (), чтобы проверить, есть ли еще утечка памяти.

Долгосрочное исправление - Не используйте класс String

Для лучшего исправления, и особенно для кого-то, кто не знаком с программированием или C/C++, не используйте класс String и узнайте, как использовать строку c и массив. Вот версия без использования класса String.

length = 80;
char buffer[length] = {0};
if ((last_weight_time < millis() - 1000)) {
   last_weight_time = millis();
   while(Serial.available()) {
      Serial.readBytesUntil('\n', buffer, length);
   }
}

Другие ресурсы, о которых можно узнать

https://www.cplusplus.com/reference/cstring/ все функции для управления строками c и массивами для замены класса строк Arduino.

https://hackingmajenkoblog.wordpress.com/2016/02/04/the-evils-of-arduino-strings/