Stcmp abfrage geht nicht

International Deutsch
1

Hallo wieder mal,

zuerst schönes Wochenende für alle die frei haben!

und wie immer eine dumme Anfänger Frage.
Hab versucht das zu googeln aber anscheinend versteh ich das nicht obwohl ich sogar c Bücher runter geladen habe.

Warum funktioniert die if nicht?

#include "BluetoothSerial.h"

BluetoothSerial SerialBT;

String abf = "";
char* abf_c;
String send_data = "";
char*  vergl = "send";


void setup() {
  Serial.begin(115200);
  SerialBT.begin("ESP32test"); //Name des ESP32
  Serial.println("Der ESP32 ist bereit. Verbinde dich nun über Bluetooth.");
}

void loop() {
  //String a = "vergleich ok";
  //uint8_t buf[a.length()];
  //memcpy(buf, a.c_str(), a.length());

  if (Serial.available()) {
    SerialBT.write(Serial.read());
  }
  if (SerialBT.available()) {
    abf = SerialBT.readString();
    //Serial.println(abf);
  }

  char abf_c [abf.length()];
  Serial.print("Länge abf: ");
  Serial.println(abf.length());
  abf.toCharArray(abf_c, abf.length());
  Serial.print("abf: ");
  Serial.println(abf);
  Serial.print("abf_c: ");
  Serial.println(abf_c);
  Serial.print("Vergleich: ");
  Serial.println(strcmp(abf_c, vergl));

  if (strcmp(abf_c, vergl) == 0) {
    Serial.println("Vergleich ok");
    //SerialBT.write(buf, a.length());
  }
  delay(25);
}

Danke

Lg

2

Was wird das wenn es fertig ist?
Und was möchtest du genau machen?
Beschreibe exakt welche Funktionalität erwünscht ist.
Beschreibe exakt was aktuell nicht funktioniert.

Schau dir doch mal die Serial Input Basics an wie man Sachen von einer Seriellen Schnittstelle einliest.
Dauert ne Stunde - dann klappt es.

3

Die funktioniert.
Der Vergleich in der Bedingung gibt Dir einen Wert != 0 zurück.
Hast Du die Warnungen eingeschaltet?
Dann schau nach dem compilieren mal unten in die Ausgabe.

Und wenn Du genau wissen willst, was da beim Vergleich drin steht, mach Dir einen else auf:

  if (strcmp(abf_c, vergl) == 0)
  {
    Serial.println("Vergleich ok");
    //SerialBT.write(buf, a.length());
  }
  else
  {
    for (byte b = 0; b < sizeof(abf_c); b++)
    {
      Serial.print(abf_c[b], HEX);
      Serial.print(' ');
    }
    Serial.println();
    for (byte b = 0; b < sizeof(vergl); b++)
    {
      Serial.print(vergl[b], HEX);
      Serial.print(' ');
    }
    Serial.println();
  }
4

ne seids mir nicht bös, aber das bringt so nichts.

Eingabe von

send

13:10:36.537 -> Länge abf: 0
13:10:36.537 -> abf: 
13:10:36.537 -> abf_c: 
13:10:36.537 -> Vergleich: -115
13:10:37.522 -> sLänge abf: 3
13:10:38.559 -> abf: end
13:10:38.559 -> abf_c: en
13:10:38.559 -> Vergleich: -14
13:10:39.548 -> Länge abf: 3
13:10:39.548 -> abf: end
13:10:39.548 -> abf_c: en
13:10:39.548 -> Vergleich: -14
13:10:40.586 -> Länge abf: 3
13:10:40.586 -> abf: end
13:10:40.586 -> abf_c: en
13:10:40.586 -> Vergleich: -14
13:10:41.624 -> Länge abf: 3

erst mal die Grundlagen fürs Seriell einlesen erarbeiten,
dann beschreiben was man will,
dann einen Sketch schreiben.

5

Hast Du die Warnungen abgeschalten?

6

nein.

7

Siehst Du keine Warnung?

8

doch.
Worauf möchtest du raus?

9

Nimm meinen Elsezweig.
Einmal mit Warnung und einmal berichtigen und ohne Warnung.

10

ich sehe nicht was der else Zweig am grundsätzlichen Ergebnis verändern soll.
Warum soll ich das tun?

11

Um zu sehen, was Du vergleichst?
Oder anders: zu sehen, das Du im Vergleichstring was anderes drin hast, als Du erwartest?

12

das seh ich im code auch.
Ändert nichts daran, dass der TO das einlesen anders lösen muss.

13

Das runterladen von Büchern, hilft nicht dem C Verständnis auf die Sprünge.
Dazu muss man diese Bücher auch lesen.

Zudem ist Arduino in der Hauptsache C++.
Es wäre also völlig ok, ein C++ Buch zu lesen, wenn man C++ lernen will.

Zu deinem eigentlichen Problem:
Leider habe ich nicht verstanden, was du "wirklich" erreichen möchtest.
Ich vermute, du möchtest einen Parser bauen!
Falls das der Fall sein sollte, auch dazu gibt es Literatur.

14

Sorry war essen.
Um das mal zu erklären, bei mir kommt im Seriel monitor das:

abf_c:: send
Vergleich: 13
Länge abf: 6
abf:: send

also sind ja meine variablen die selben und für mich sieht es so aus al wenn die if nicht funktioniert obwohl eben strcmp schon was falsches gibt, jedoch sagen mir die variablen das bei beiden send drin steht.

Und ja das ganze kommt erst nachdem ich von handy an den esp32 was gesendet habe.
War auch meine erste Idee und ist vlt nicht gut, aber deshalb bitte ich ja hier um hilfe ob jemand so nett wäre und zeit hätte mir tipps zu geben.
Hier sind halt die Profis und an die versuch ich mich zu wenden, da ihr es am besten von allen wisst

Und nein ich bekomme im comiler keine warunung

Danke

Lg

15

Und Du wirst nicht stutzig, wenn für "send" die Länge 6 steht?

Gruß Tommy

16

Ladest du Bücher runter und liest sie dann nicht?

Aber danke für den hilfreichen tipp!

Lg

17

Bin ich eh aber ich hab keine idee woher das kommen kann?
+1 ist ja verständlich wegen /0 am ende aber 6?

Lg

18

Deshalb ist der Vorschlag, Dir mal alles als Hex ausgeben zu lassen äußerst hilfreich, wenn man ihn nutzt.
Keine Sorge, if und strcmp funktionieren garantiert. Der Fehler sitzt vor dem Monitor.

Gruß Tommy

19

Ok dann mach ich das mal und schau ob ich schlauer werde.

Ich gebe dir da recht das sicher ich der fehler bin, damit hab ich kein problem.

Wie gesagt ich frage ja nach tipps von den profis um schlauer zu werden.
Und bin auch für alles was mir hilft dankbar, ist nicht selbstcverständlich immer jemanden anders zu helfen.

Lg

20

So bin jetzt schlauer das ich am ende /n bekomme bei serieler eingabe und das mittels trim() wegbekomme, dann hab ich aber das problem von post#4
Wenn ich die länge um 1 erweitere dann sind jetzt beide gleich lang aber trotzdem funktioniert strcmp noch immer nicht.

Hätte jemand wieder einen tipp der mich in die richtige richtung bringt oder woran das genau liegt?

#include "BluetoothSerial.h"

BluetoothSerial SerialBT;

String abf = "";
char* abf_c;
String send_data = "";
char*  vergl = "send\n";
boolean newData = false;
int length_trim;
String abf_c_str = "";


void setup() {
  Serial.begin(115200);
  SerialBT.begin("ESP32test"); //Name des ESP32
  Serial.println("Der ESP32 ist bereit. Verbinde dich nun über Bluetooth.");
}

void loop() {
  //String a = "vergleich ok";
  //uint8_t buf[a.length()];
  //memcpy(buf, a.c_str(), a.length());

  if (Serial.available()) {
    SerialBT.write(Serial.read());
  }
  if (SerialBT.available() > 0 && newData == false) {
    abf = SerialBT.readString();
    //Serial.println(abf);
  }

  abf.trim();
  char abf_c [abf.length() + 1];
  Serial.print("Länge abf: ");
  Serial.println(abf.length());
  abf.toCharArray(abf_c, abf.length() +1 );
  Serial.print("abf: ");
  Serial.println(abf);
  Serial.print("abf_c: ");
  Serial.println(abf_c);
  abf_c_str = String(abf_c);
  Serial.print("abf_c_str: ");
  Serial.println(abf_c_str);
  Serial.print("Länge abf_c_str: ");
  Serial.println(abf_c_str.length());
  Serial.print("Vergleich: ");
  Serial.println(strcmp(abf_c, vergl));

  if (strcmp(abf_c, vergl) == 0) {
    Serial.println("Vergleich ok");
    //SerialBT.write(buf, a.length());
  }
  delay(25);
}

Länge abf: 4
abf: send
abf_c: send
abf_c_str: send
Länge abf_c_str: 4
Vergleich: -10

Danke

lg