String не сравнивается

lanket
Offline
Зарегистрирован: 03.02.2017

В Ардуинке пока новичок, но кое что уже получается.

Разобрался и добился прохода команд MQTT до ардуинки. Чтобы серво привод реагировал надо при прихождении определенного топика значение его отдавалось сервоприводу. Вроде куда проще. Как я понял топик попадает в переменную типа String. Надо сделать простое сравнение на предмет сходствна с нужным и отреагировать. Код:

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

Servo myservo;

const char* ssid = "MyWiFiPoint";
const char* password = "NepodbiraenuyParol";

//const char* mqtt_server = "test.mosquitto.org";
//const char* mqtt_server = "iot.eclipse.org";
const char* mqtt_server = "MQTTserverVinete";
const int mqtt_port = 12345; // Порт для подключения к серверу MQTT
const char *mqtt_user = "SamuyGlavnuy"; // Логи от сервер
const char *mqtt_pass = "1234"; // Пароль от сервера

WiFiClient espClient;
PubSubClient client(espClient);

void setup() {

  myservo.attach(2);
  // pinMode(0,OUTPUT);
  Serial.begin(115200);
  setup_wifi();
  // client.setServer(mqtt_server, 1883);
  client.setServer(mqtt_server, mqtt_port);
  client.setCallback(callback);
  reconnect();
}

void setup_wifi(){

  delay(10);
  // We start by connecting to a WiFi network
  Serial.println();
  Serial.print("Connecting to ");
  Serial.println(ssid);

  WiFi.begin(ssid, password);

  while (WiFi.status() != WL_CONNECTED) {
    delay(500);
    Serial.print(".");
  }

  Serial.println("");
  Serial.println("WiFi connected");
  Serial.println("IP address: ");
  Serial.println(WiFi.localIP());
}

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.print(topic);
  if(topic == "Kran/Strela") {
    Serial.print(" Srabotalo!");
    myservo.write(payload[0]);
  }
  
  
  //if((char)payload[0] == 'o' && (char)payload[1] == 'n') //on
  //  digitalWrite(2,HIGH);
  //else if((char)payload[0] == 'o' && (char)payload[1] == 'f' && (char)payload[2] == 'f') //off
  //  digitalWrite(2,LOW);
    
  Serial.println();
}

void reconnect() {
  // Loop until we're reconnected
  while (!client.connected()) {
    Serial.print("Attempting MQTT connection...");
    // Attempt to connect
    //if (client.connect("ESP8266Client")) {
    //if (client.connect("ESP8266Client").set_auth(mqtt_user, mqtt_pass)) {
    if (client.connect("arduinoClient2", mqtt_user, mqtt_pass)) {
      Serial.println("connected");
      // Once connected, publish an announcement...
      client.publish("Kran/connection status", "Connected!");
      // ... and resubscribe
      client.subscribe("Kran/Strela");
    } else {
      Serial.print("failed, rc=");
      Serial.print(client.state());
      Serial.println(" try again in 5 seconds");
      // Wait 5 seconds before retrying
      delay(5000);
    }
  }
}

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

в строчке №60 Serial.print(topic);

В терминал попадает надпись такая какая и ожидается "Kran/Strela"

А в 61 строчке сразуже после идет сравнение if(topic == "Kran/Strela") {

и почемуто оно не работает, нифега не понимаю. вроде все просто, и не работает. Я имею ввиду что условие ложное. Хотя предыдущая строка пишет в терминал что как раз условие должно быть Истиной. Чего я не допираю?

И второе. Как я понял из примера благодаря строчкам с 57 по 59 значение топика payload это массив почемуто, из которого надо собрать готовое число. Как я понял что отправив в топик чило например 100. То получатель получит массив [1,0,0]. 

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

Спасибо.

DetSimen
DetSimen аватар
Offline
Зарегистрирован: 25.01.2017

if(topic == "Kran/Strela")

 
потому что здесь сравниваются указатели на строки, а оне разные.  Используй или memcmp или храни строки в объекте String, у него переопределен оператор сравнения и строки сравниваются правильно. 
ЕвгенийП
ЕвгенийП аватар
Offline
Зарегистрирован: 25.05.2015

Вместо 

if (topic == "Kran/Strela")

надо

if (! strcmp(topic, "Kran/Strela"))

и обязательно разбиритесь с работой со строками

lanket
Offline
Зарегистрирован: 03.02.2017

Спасибо за помощь.

Почитал тут в метро описание по String.

Мое мнение - тихий ужас

Если я правильно понял то строковая переменная это массив букв. Прям бред какойто.

И все равно не понимаю:

Строка 55 - Serial.print(topic); Выводит в сериал текст

Строка 58 - Serial.print((char)payload[i]); результат тот же, правда символ а не строку, но у меня в сознании разница не должна быть

Исходя логики, из увиденного разницы нет.

Исходя из Вашего утверждения что это "указатели", всмысле? Объекты это?

А в строке 67 - if((char)payload[0] == 'o' && (char)payload[1] == 'n')

Вполне логично мне показалось, что выводя символы/текст в поток сравнительно одинаковым способом, и видя работающию строчку 67 НЕПОНИМАЮ хоть запинайте почему не работает if(topic == "Kran/Strela") {

Перечитал 5 вариантов туториалов про переменные так не понял почему данный стринг не стринг.

Ткните носом пож-та где нормально данный случай "указатель" описывается. Хочется понять а не получить готовое решение.

Но по любому большое спасибо за решение.

ЕвгенийП
ЕвгенийП аватар
Offline
Зарегистрирован: 25.05.2015

lanket пишет:

Почитал тут в метро описание по String.

Не путайте мягкое с круглым. Массив символов, которым Вы пользовались и класс String - абсолютно разные вещи.

lanket пишет:

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

Т.е. в Вашем сознании нет разницы между словом/фразой и отдельной буквой? Ну, ... имеете право.

lanket пишет:

Ткните носом пож-та где нормально данный случай "указатель" описывается. 

Читайте Кернигана и Ритчи. ТОлько внимательно, разбирая примеры, всё прояснится.

 

lanket
Offline
Зарегистрирован: 03.02.2017

ЕвгенийП пишет:

Читайте Кернигана и Ритчи. ТОлько внимательно, разбирая примеры, всё прояснится.

Спасибо, буду разбираться.

ua6em
ua6em аватар
Онлайн
Зарегистрирован: 17.08.2016

char* topic - Это указатель

andriano
andriano аватар
Offline
Зарегистрирован: 20.06.2015

lanket пишет:

видя работающию строчку 67 НЕПОНИМАЮ хоть запинайте почему не работает if(topic == "Kran/Strela") {

Так оно работает. Притом, работает правильно. Вот только не совсем так, как бы Вам хотелось.