Почему не работает с char[]?

Question

[Sergei Abramov]

struct MqttConfig {
    char host[50] = "192.168.1.94";
    unsigned short port = 1883;
    char user[10] = "mqtt";
    char password[20] = "password";
    char client[20] = "LightController";
};

MqttConfig mqttConfig;
WiFiClient espClient;
PubSubClient mqttClient(espClient);
mqttClient.setServer(mqttConfig.host, configmqttConfig.port);

Но mqtt библиотека не подключается (да, там дальше вызов mqttClient.connect, это для компактности). Методом тыка, понял, что дело в mqttConfig.host, если его заменить (и остальные user/password) на явные: mqttClient.setServer("192.168.1.94", 1883); то все работает.

Почему так происходит и как быть? Если выводить Serial.println(mqttConfig.host); то там тот же хост, что и указан. В общем, я в тупике. Структура мне нужна чтобы хранить настройки в EEPROM.

Comments

[mayton2019]

В этом вопросе нет ни кода ошибки ни стектрейса.

[Sergei Abramov]

mayton2019, при вызове mqttClient.state() получаю ошибку MQTT_CONNECT_FAILED (#define MQTT_CONNECT_FAILED -2) без каких либо подробностей. В логах MQTT сервера вижу, что железка к нему даже не пытается стучаться. Как дебажить esp8266 и получать с нее трейс еще не разобрался, не дошел до этого.

[Евгений Шатунов]

Sergei Abramov, какая сигнатура у метода setServer класса PubSubClient?

[Sergei Abramov]

Евгений Шатунов,

PubSubClient& PubSubClient::setServer(const char * domain, uint16_t port)

Так, я сдампил "192.168.1.94" и mqttConfig.host в сериал порт в HEX. Получилось
31 : 39 : 32 : 2E : 31 : 36 : 38 : 2E : 31 : 2E : 39 : 34 : 0
31 : 39 : 32 : 2E : 31 : 36 : 38 : 2E : 31 : 2E : 39 : 34 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0 : 0

В общем-то понятно почему. Размер массива чаров 50, и он добивает их 0. Но как тогда правильно это реализовать? Хост может быть любой длины, 50 - брал с запасом.
Причем библиотека для подключения к WiFi с такими трудностями не сталкивается, для нее так же из структуры передаются настройки, и она видать обрезает эти строки.

[Евгений Шатунов]

Sergei Iamskoi, аккаунт Sergei Abramov является твоим вторым? Это запрещено в П6.5 регламента работы сервиса.
Ну вот теперь, когда комментарий того аккаунта удален, а этого аккаунта точно с тем же текстом - добавлен, сомневаться в нарушении уже не приходится.

[mayton2019]

Sergei Abramov, ты зачем в вопрос втащил аж 2 проблемы. Если хочешь разбираться с сетевым соединением - делай так

mqttClient.setServer("192.168.1.1", 1883);

и погнал разбираться. Кстати что за библиотеки ты используешь для протокола mqtt? Потому что для меня Arduino мало что значит.

Если ты хочешь узнать как компиллятор C/C++ (кстати тот еще вопрос с чем ты работаешь) аллоцирует строки ASCIIZ - тогда давай другой вопрос.

Короче мухи отдельно и котлеты отдельно. Утонем ведь в догадках ну емаё.

[Sergei Abramov]

Евгений Шатунов, да, с аккаунтами чуть позже разберусь, чтобы объединили. Так исторически сложилось, когда были разные сервисы. Согласно 6.5 аккаунт не использую для обхода каких-либо ограничений. Сорри :(

mayton2019, мне кажется, проблема связанна с хранением строк. По этому стараюсь дать как можно больше информации, тем более сам очень плаваю знаниях в C++.
mqttClient.setServer("192.168.1.1", 1883); - в таком виде все прекрасно работает и подключается, если мы явно строкой указываем адрес сервера. Но если туда подставить переменную типа char[50] с хостом, то все перестает работать.
Библиотека используется эта https://github.com/knolleary/pubsubclient

Какой именно компилятор это всё собирает не подскажу, так как не владею информацией как это посмотреть. IDE Clion с интеграцией Platformio. Используется CMake в проекте.

[mayton2019]

Sergei Abramov,

Какой именно компилятор это всё собирает не подскажу, так как не владею информацией как это посмотреть. IDE Clion с интеграцией Platformio. Используется CMake в проекте.

Дружище. Мастер должен знать свой инструмент. Название компиллятора (gcc/clang...). Версия. Разрядность и платформа. Что у тебя там на Ардуине? Есть выход на монитор как в Raspberry? Как вообще ты дебажишся?

Тут - непонятно.

struct MqttConfig {
    char host[50] = "192.168.1.94";

Что это за синтаксис? В каком языке такое разрешено?

[Евгений Шатунов]

Sergei Abramov, по разным аккаунтам ясно. Ладно.

PubSubClient& PubSubClient::setServer(const char * domain, uint16_t port)

Код оформляется как код. Не курсивом, не жирным, только как код. Этого в правилах не написано. Но если код написан не как код, это будет нервировать людей. Ты ведь пишешь не для того чтобы выделиться и заявить о своей уникальности, а для того чтобы тебя прочитали, поняли и помогли. Верно же? Значит тебе абсолютно незачем нервировать людей на ровном месте.
Любой код всегда пишется для читателя. И при написании кода нужно максимум усилий направить на то, чтобы именно читателю было удобнее.
Тебе стоит принять предложенную мной правку вопроса.

Теперь по сути. Сигнатура у метода принимает указатель на память с типом константного символа.
char password[20] = "password";.
Сырой массив неконстантных символов подразумевает что ты планируешь менять его содержимое. Но ты ведь не планируешь менять? Я подозреваю что не планируешь.
И при этом, массив неконстантных символов инициализируется строковым литералом.
Я думаю, тебе легче будет использовать тип const char* для своих полей. Пусть в них хранятся адреса размещений литералов, которыми ты инициализируешь поля сейчас. Т.е. в плане инициализации ничего не поменяется, поменять нужно только тип полей.

[Sergei Abramov]

mayton2019, так, впроде нашел в настройках: Clangd 11.0.0
Насчет синтаксиса структур: в уроках по C++ видел, значения по умолчанию.
Насчет дебага всё сложно. Это микроконтроллер, и для дебага у меня только вывод в Serial порт данных. Я уже искал информацию, как можно микроконтроллер по человечески дебажить, но находил только вопросы без ответов:)

Евгений Шатунов, я только отдельные команды не оформлял в виде кода. Но ок, поправил.
К сожалению, сменить тип на const char* я не могу, т.к. структура записывается в EEPROM, а при подключении питания и старта контроллера побайтово считывается из памяти и восстанавливается структура, по этому важно знать размер структуры, а с const char* я так понимаю, это невозможно, т.к. размер в памяти выдается при инициализации, и может плавать в зависимости от данных.

Насчет литерала - я так понимаю, речь о '\0' ? Я пробовал обрезать массив в размер, что-то вроде того:

int sizeS = 0;
    for (int i = 0; i < 50; i++) {
        if (configMqtt.host[i] == '\0') {
            sizeS = i;
            break;
        }
    }

    char host[sizeS];
    const char* constHost = host;

    mqttClient.setServer(constHost, 1883);

Оно тоже не работает :(

[Евгений Шатунов]

Sergei Abramov,

а с const char* я так понимаю, это невозможно

Давай руководствоваться не условным пониманием, а точным знанием.
Как минимум, почему для случая с полями const char* узнать точный размер структуры MqttConfig невозможно? Размер указателя фиксирован, структура из указателей будет правильно выравнена и будет иметь правильный фиксированный размер. Скорее всего в твоем случае твой конфиг будет занимать 20Б.

Возможно тебе стоит более детально изучить процесс запуска твоего мк и формат зашиваемого в него бинарного образа.

Насчет литерала - я так понимаю, речь о '\0' ?

Нет, речь о строковом литерале. Ноль - это терминальный символ, которым завершаются все строки в c++. В этом плане, после первого нуля в буфере строки дальнейший контент не важен, т.к. строка завершена. Я не совсем понимаю причин твоего беспокойства тем, что остаток твоих полей в конфиге забит нулями. Это - правильное поведение твоего транслятора.
Я так понимаю, ты не очень уверенно владеешь c++. Хорошо, это не проблема.

Строковой литерал - это строка справа от символа присвоения, используемая тобой для инициализации полей по месту их определения. Литералы, вообще все, изначально не имеют идентичности и размещения. У них невозможно взять адрес, получить неконстантную ссылку или определить их принадлежность к иерархии классов.
Более подробно об этом ты можешь узнать из статьи.
Строковой литерал немного отличается от прочих литералов тем, что в момент запроса размещения строкового литерала транслятор это размещение для литерала выделяет ровно в том объеме, в котором литерал требует.
Строковой литерал размещается в глобальной неизменяемой секции памяти и загружается до начала исполнения твоего кода, собственно, вместе с твоим кодом.
Ты ведь везде в своем коде, в любом его месте, можешь начать использовать строковой литерал даже не думая о механике его поведения, так почему же в конкретном месте инициализации полей ты внезапно чего-то забоялся?
Своими сырыми массивами ты сейчас только отнимаешь свободную память у себя же в будущем, когда тебе этой памяти станет не хватать.

Answer 1

[Wataru]

В коде какой-то ошибки не вижу. Скорее всего вы уже пытаетесь эту структуру засунуть в EEPROM и из-за этого что-то не работает.

Это же c++, судя по тегам? Сделайте тогда все поля структуры static const и не заводите ее экземпляр. Обращайтесь к константам так: MqttConfig::host. Константы обычно задают так, а не значениями по умолчанию полей структуры.

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