Arduino.ru

функцию в студию

Может банально места в памяти не хватает ?

Так яж код весь выложил.

void publish()
{
  if (!client.connected())
    reconnect();
  else
  {
    String bufer;
    bufer = "{\n\t\"Air temperature\" : " +
            (String)display[TEMPERATURE_AIR].data +
            ",\n\t\"Air pressure\" : " +
            (String)display[PRESSURE].data +
            ",\n\t\"Air humidity\" : " +
            (String)display[HUMIDITY].data +
            ",\n\t\"Water temperature\" :" +
            (String)display[WATER_TEMPERATURE].data +
            ",\n\t\"Water flow\" : " +
            (String)display[WATER_FLOW].data +
            ",\n}";
    uint8_t msgLen = bufer.length();
    Serial.print(bufer);
    client.beginPublish(MQTT_TOPIC, msgLen, false);
    client.print(bufer);
    client.endPublish();
  }
}

То что памяти не хватает уже задумывался. 

и чо Serial.print(buffer)  пишет?

При компиляции bufer это указатель на строку. А в момент работы - под строку выделяется реальная память и судя по всему не малого размера.

Как вариант вынести объявление bufer из функции и разу задать максимальный размер.

В функции ни чего, в лупе выводит.

Скорее всего, он несколько раз пытается перехапать память, ибо строка растёт, ты ж ей максимальное значение не задал, так она и перевыделяется кусками по X байт (не помню скока).  А потом её не хватает....

Хреналысого.  Это сразу локальный (стековый) обьект. 

DetSimen с размером указателя же. 

Я про то, что в "DATA:    [=======   ]  69.7% (used 1427 bytes from 2048 bytes)" не учтен размер строки от слова СОВСЕМ.

anarch - вариант решения - переписать функцию без использования String - памяти нужно будет раз в 10 меньше

Урезал строку и все заработало 

void publish()
{
  if (!client.connected())
    reconnect();
  else
  {
    String bufer;
    bufer = "{\n\t\"Air temperature\" : " +
            (String)display[TEMPERATURE_AIR].data +
            ",\n\t\"Air pressure\" : " +
            (String)display[PRESSURE].data +
            ",\n\t\"Air humidity\" : " +
            (String)display[HUMIDITY].data +
            ",\n\t\"Water temperature\" :" +
            /* (String)display[WATER_TEMPERATURE].data +
            ",\n\t\"Water flow\" : " +
            (String)display[WATER_FLOW].data + */
            ",\n}"; 
    uint8_t msgLen = bufer.length();
    Serial.print(bufer);
    client.beginPublish(MQTT_TOPIC, msgLen, false);
    client.print(bufer);
    client.endPublish();
  }
}

Не удивлен.

Если решения со String не найдется буду переписывать.

А проверить вариант с объявлением вне функции и сразу достаточного размера ?

Нет. Размером класса сразу  но на стеке. Он обьявлен не как 

String *buffer;

Вкурил - String, это не string.

любое решение со String - временное, лучше сразу писать правильно

Вот так вот переписал 

    char *temp = new char();
    char *buf = new char();
    strcat(buf, "{\n\t\"Air temperature\" : ");
    sprintf(temp, "%f", display[TEMPERATURE_AIR].data);
    strcat(buf, temp);
    strcat(buf, ",\n\t\"Air pressure\" : ");
    sprintf(temp, "%f", display[PRESSURE].data);
    strcat(buf, temp);
    strcat(buf, ",\n\t\"Air humidity\" : ");
    sprintf(temp, "%f", display[HUMIDITY].data);
    strcat(buf, temp);
    strcat(buf, ",\n\t\"Water flow\" : ");
    sprintf(temp, "%f", display[WATER_FLOW].data);
    strcat(buf, temp);
    strcat(buf, ",\n\t\"Water temperature\" :");
    sprintf(temp, "%f", display[WATER_TEMPERATURE].data);
    strcat(buf, temp);
    strcat(buf, ",\n}");
    uint8_t msgLen = strlen(buf);
    Serial.println(buf);
    client.beginPublish(MQTT_TOPIC, msgLen, false);
    client.print(buf);
    client.endPublish();
    delete[] temp;
    delete[] buf;

Выводит полностью. Но на выходе не получаю значений.

{
        "Air temperature" : ?,
        "Air pressure" : ?,
        "Air humidity" : ?,
        "Water flow" : ?,
        "Water temperature" :?,
}

Как преобразовать поле структуры float  в строку.

И о том что меньше занимает места я бы не сказал.

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

значения теряются. потому что вы используете тип float. В ардуино sprintf с float не работает

dtostrf() наше все. 

Всем спасибо! 

    char temp[7];
    char *buf = new char;
    strcat(buf, "{\n\t\"Air temperature\" : ");
    dtostrf(display[TEMPERATURE_AIR].data,6,1,temp);
    strcat(buf, temp);
    strcat(buf, ",\n\t\"Air pressure\" : ");
    dtostrf(display[PRESSURE].data,6,1,temp);
    strcat(buf, temp);
    strcat(buf, ",\n\t\"Air humidity\" : ");
    dtostrf(display[HUMIDITY].data,6,1,temp);
    strcat(buf, temp);
    strcat(buf, ",\n\t\"Water flow\" : ");
    dtostrf(display[WATER_FLOW].data,6,1,temp);
    strcat(buf, temp);
    strcat(buf, ",\n\t\"Water temperature\" :");
    dtostrf(display[WATER_TEMPERATURE].data,6,1,temp);
    strcat(buf, temp);
    strcat(buf, ",\n}");
    uint8_t msgLen = strlen(buf);
    client.beginPublish(MQTT_TOPIC, msgLen, false);
    client.print(buf);
    client.endPublish();
    delete[] buf;

    char temp[7];
    char *buf = new char;
    

Я ах..еваю, дорогая редакция! ;))) Я вааще фигею от здорового аптимизьмуса!

Что бы куча не скучала ...

wdrakula,нормально.

Чапаев нарисовал фальшивые 15 рублей, а сразу не сообразил, что такой купюры нет. Что делать? Послал Петьку разменять.
- Ну что, разменял?
- Ага.
- И что дали?
- 7 и 8

wdrakula, готов выслушать ваши предложения по решению данного момента мне то же не нравиться.

Родное сердце! Если ты поймешь, почему я глумился, то это уже "путь к исцелению".

1. что, по твоему, делает строка:

char *buf = new char;

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

2. Вообще  кто, где и когда выделяет память под твои строки и где указывает потребный размер памяти?

Выделяет 1 байт памяти. 

Тоды задам по другому вопрос. Как выделять память не зная размер строки?

Вы сами-то себя услышали? "Как выделить кусок памяти, не зная какого размера кусок нужен?"

Похоже, существуют люди, уверенные, что компьютер умнее, чем они.

И возможно даже, они правы. (((((((((((

Т.е. если я не знаю какой длины у меня строка, я должен выделить кусок памяти с запасом, поработать с ним и очистить, а если не хватило попросить еще. На верном пути?

Вы выделили память под один байт, а пихаете туда ... Хвост строки живет своей жизнью. Вам просто повезло что под этим хвостом не оказались важные данные. В Вашем случае очень легко вычисляется необходимый размер буфера под всю строку плюс один байт под 0. И ещё первой командой должна быть не strcat, а strcpy.

весело тут у вас)) жду свою ардуинку и тоже буду вопросы глупые задавать)

baby_in_Arduin для старта плата даже не нужна - достаточно Proteus !

а для глупых вопросов не нужно ни того, ни другова. 

Умные вопросы то же не стоит задавать :x

И так если я знаю размер буфера, что мне мешает сделать так:

char buf[200];

после того как блок отработает переменная удалиться (или просто исчезнет из области видимости?).

Т.е. смысл или когда нужно выделять динамически память?

Если это локальная переменная функции - то удалица, если глобальная то нет.

имеет смысл, если у вас потребности в памяти постоянно меняются. Например, в большинстве случаев вам хватает строчки в 30 символов, а раз в месяц проскакивает запрос в 400. В этом случае нет смысла всегда держать буфер в 400 символов.

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

Динамическая память для тех случаев, когда неизвестно количество даже близко (от 0 до ... размеров доступной памяти) ! У Вас же явно можно посчитать сколько символов в тексте и сколько позиций надо под цифры.

char buf[200];

так и надо делать если объем данных невелик это намного быстрее работы с динамической кучей

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

вопрос опытным ардуинщикам какой максимальный размер буфера на стеке? вообще каков размер стека? какая область памяти "за стеком"? что мы потрем в случае если превысим максимальный размер буфера?

Стек растет вниз и может переехать даже через порты IO. Если же локальная переменная в стеке "перерастет" свои границы, то первым делом наедет на адрес возврата (иногда это делают осознанно) и возврат будет ХЗ куда.

так этож тогда ассемблер получается )))

У нас статистики такой нету. Те, у каво стек переполнился, получают психологическую травму, несовместимую с нормальным функционирование арганизма, и на форумах больше не пишуть.

DetSimen надо взять на заметку как способ отписаться от форумов !!!

вот про ардуино уно пишут ОЗУ 2КБ в этих 2КБ и стек и куча? 

baby_in_Arduino плюс глобальные переменные :-)

вот почему я за тебя искать должен?  Ты ленив, или просто сын депутата? 

http://robocraft.ru/blog/arduino/531.html

а нет какого метода прикрутить внешнюю память?

научиться писать софт при ограниченных ресурсах железа

Есть и даже на этом форуме описано. Великий как-то прикручивал. Доприкрутил ли - не помню, но постов было много.