Конкатенации строк и символов

Новичок в Arduino, и я пытаюсь объединить строку и символы (ну, в python это совсем другое).

Я пытаюсь опубликовать сообщение MQTT, построенное в следующем формате, в client.publish(outTopic, "["+outTopic+"]"+ msg)

в то время как outTopic — это переменная char*, а msg — это char, но я получаю сообщение об ошибке для такой конкатенации.

пожалуйста, помогите

#include <ESP8266WiFi.h>
#include <PubSubClient.h>


// GPIO на плате SONOFF
int LED_SONOFF = 13;
int REL_SONOFF = 12;
int PIN_SW = 14;

const char* ssid = "HomeNetwork_2.4G";
const char* password = "pswd";
const char* mqtt_server = "192.168.2.113";

WiFiClient espClient;
PubSubClient client(espClient);


char msg[50];
const char* outTopic = "HomePi/Dvir/Messages";
const char* inTopic = "HomePi/Dvir/Windows/SONOFF";
const char* inTopic2 = "HomePi/Dvir/Windows/All";
const char* clientID = "Sonoff#1";
String m1, m2;

void start_wifi() {
  WiFi.mode(WIFI_STA);
  WiFi.begin(ssid, password);

  while (WiFi.status() != WL_CONNECTED) {
    delay(500);
    Serial.print(".");
  }
  Serial.println("");
  Serial.println("WiFi connected");  
  Serial.print("IP address: ");
  Serial.println(WiFi.localIP()); 
}

void setup() {
  start_wifi(); // запускаем службу Wi-Fi
  Serial.begin(115200); // запускаем последовательный сервис
  delay(10);

  // определяем sonoff gpios
  pinMode(LED_SONOFF, OUTPUT);
  pinMode(REL_SONOFF, OUTPUT);
  pinMode(PIN_SW, INPUT_PULLUP); // GPIO14 имеет подтягивающий резистор
  digitalWrite(LED_SONOFF,HIGH); // Высокий светодиод выключен
  digitalWrite(REL_SONOFF,LOW); 

  // запускаем mqtt
  client.setServer(mqtt_server, 1883);
  client.setCallback(callback);  
}

void switch_on() {
  digitalWrite(REL_SONOFF,HIGH);
  digitalWrite(LED_SONOFF, LOW); // светодиод горит
  pub_mqtt("Switch ON");
}

void switch_off() {
  digitalWrite(REL_SONOFF,LOW);
  digitalWrite(LED_SONOFF, HIGH); // светодиод выключен
  pub_mqtt("Switch OFF");
}

void callback(char* topic, byte* payload, unsigned int length) {
  Serial.print("Message arrived [");
  Serial.print(topic);
  Serial.print("] ");
  for (int i = 0; i < length; i++) {
    Serial.print((char)payload[i]);
  }
  Serial.println();

  if ((char)payload[0] == '1') {
    switch_on();   
  } 
  if ((char)payload[0] == '0') {
    switch_off();
    Serial.print("payload: ");
    Serial.println(payload[0]);
  }

}

void reconnect() {
  while (!client.connected()) {
    Serial.print("Attempting MQTT connection...");
    if (client.connect("ESP8266Client")) {
      Serial.println("connected");
      client.publish(outTopic, "Connected");

      client.subscribe(inTopic);
      m1="Subscribed to: "+inTopic;
      client.publish(outTopic, m1);
      if (inTopic2 !=""){
        client.subscribe(inTopic2);
        m2="Subscribed to: ";//+ String(inTopic2);
        client.publish(outTopic, m2);
      }

    } else {
      Serial.print("failed, rc=");
      Serial.print(client.state());
      Serial.println(" try again in 5 seconds");
      delay(5000);
    }
  }
}

void pub_mqtt(char* msg) {
  client.publish(outTopic, msg);
}

void loop() {
  if (!client.connected()) {
    reconnect();
  }
  client.loop();

  if (digitalRead(PIN_SW)==1) {
    if (digitalRead(REL_SONOFF)!=LOW) {
      switch_off();
    }
  }
  else if (digitalRead(PIN_SW)==0) {
    if (digitalRead(REL_SONOFF)!=HIGH){
      switch_on();
    }
  }
   delay(100);
}

, 👍0

Обсуждение

Вероятно, будет лучше, если вы создадите новую строку, в которую поместите то, что хотите передать вызываемому методу. Предположительно, вы спросили, может ли компилятор C/C++ сделать это неявно, попросив его организовать извлечение битов строки из флэш-памяти (const outTopic и символы «[», которые вы добавили в строку кода) с помощью битов. из оперативной памяти (строка msg)., @st2000


1 ответ


Лучший ответ:

2

Вы не можете просто конкатенировать строки C таким образом. Вместо этого вы должны встроить их в новую строку. Для такого форматирования проще всего использовать sprintf:

char outMessage[strlen(outTopic) + strlen(msg) + 3]; // Место для темы, сообщения, [ и ] и завершения \0
sprintf(outMessage, "[%s]%s", outTopic, msg);
client.publish(outTopic, outMessage);

Другой метод — с помощью strcpy и strcat:

char outMessage[strlen(outTopic) + strlen(msg) + 3]; // Место для темы, сообщения, [ и ] и завершения \0
strcpy(outMessage, "[");
strcat(outMessage, outTopic);
strcat(outMessage, "]");
strcat(outMessage, msg);
client.publish(outTopic, outMessage);

Вы должны убедиться, что ваша «удерживающая» строка outMessage достаточно велика, чтобы вместить все, что вы хотите отправить, плюс завершающий символ NULL, иначе ваша программа выйдет из строя. .

Еще один способ создать "удерживающую" переменную — использовать alloca(). Это выделяет байты в стеке для переменной — точно так же, как malloc() делает в куче, но не требует free()ing, потому что он находится в стеке и восстанавливается при выходе из функции:

char *outMessage = (char *)alloca(strlen(outTopic) + strlen(msg) + 3);
,

спасибо - я финансирую первый метод, более короткий и разумный. имеет ли значение, что outTopic это char*?, @Guy . D

%s подключается к типу данных char *., @Majenko

Я имел в виду char outMessage[strlen(outTopic) + strlen(msg) + 3];, @Guy . D

char *foo и char foo[x] по существу одинаковы. Второй — это char * с добавленным выделением памяти., @Majenko

@Mejnko - не могли бы вы объяснить, зачем нужен +3, поскольку для объединения строк он не нужен - подходит ли он для отправки?, @Guy . D

Один для [, один для ] и один для завершающего символа NULL. Всего три., @Majenko

Я отредактировал свой комментарий после вашего ответа :) - я хотел спросить, нужна ли обычная конкатенация (просто любые другие 2 строки) этот NULL char?, @Guy . D

Вам всегда нужен нулевой символ., @Majenko