converting char arrays in int, float, string

[Max M. (Firma: Herr) (maxtox)]

Hi,

und wieder eine Herasuvorderung :-)

habe es geschafft, doch ich frage nach einem eleganteren Weg, da ich 
kein Programmierer bin...

Ich habe Payload welches als byte* kommt, to int/float so umgewandelt:

//**********
int i;
for (i = 0; i<length; i++) {
message_buff[i] = payload[i];
}
message_buff[i] = '\0';
const char *p_payload = message_buff;

//********** converting array to int and float
got_double = strtod(p_payload, NULL);
got_int = atoi(p_payload);
got_float = atof(p_payload);
if (got_int == 0 || got_int<0) {
got_bool = 0;
}
else { got_bool = 1; }
//**********

danach habe ich Payload was so kommt xxx/sdfdfdjh/lksdfkd/sdjsdhf...

 in ein const char * umgewandelt, weil p_p wird als const char* erwartet 
wird

PubSubClient::publish(const char* topic, const char* payload, boolean retained) {
    return publish(topic,(const uint8_t*)payload,strlen(payload),retained);

hier die umwandlung:

//*******convert float value to Char expected by function
char temp[5];
dtostrf(rrr,4,3,temp); //rrr ist a float value
//*******************

for (i = 0; i<length; i++) {
p_buff[i] = p[i];
}
p_buff[i] = '\0';
const char p_p = p_buff; //
MQTTclient.publish(p_p, CHAR);
//*********

Velleicht kennt Ihr einen einfacheren und ELLEGANTEREN Weg es zu 
konvertieren?

Danke!

Thank you in Advance!

[Klaus W. (mfgkw)]

Max M. schrieb:
> ELLEGANTEREN

Eleganter wäre es, erstmal lesbaren Quelltext zu schreiben.
Bei dem oben springen ja die Frösche aus dem Teich. Ich lese sowas gar 
nicht im Detail - kann ja nichts taugen, wenn sich jemand sowenig Mühe 
gibt.

Davon abgesehen:

- Wieso konvertierst du in alles mögliche? Zuviel Rechenzeit? Oder 
brauchst du das wirklich?

- Wenn ich von irgendwo eine Zeichenkette bekomme, würde ich mich als 
erstes dafür interessieren, ob überhaupt etwas sinnvolles in einem 
korrekten Format drinsteht, und nicht einfach unreflektiert nach 
irgendwas umwandeln.
Was soll ich mit einer double, wenn der Text "Hilfe" war?

[Max M. (Firma: Herr) (maxtox)]

Da muss ich dir recht geben... aber bis jetzt habe ich nur das:

void callback(char* topic, byte* payload, unsigned int length) {
  //START********************************Convert Calback to int float and bool*************************************** 
  Serial.print("Message arrived [");
  Serial.print(topic);
  Serial.print("] ");
  Serial.print("Length:");
  Serial.println(length);
  Serial.print("Payload:");
  Serial.write(payload, length);
  Serial.println();

 //********** Copy the payload to the new buffer 
  byte* p = (byte*)malloc(length);
  memcpy(p,payload,length);
 //********** needs because payload is not deleted 
  
  
  char message_buff[length];
  char p_buff[length];
  double got_double;
  int got_int;
  float got_float;
  bool got_bool;

  int i, len;

  for (i = 0; i<length; i++) 
  {
    message_buff[i] = payload[i];
  }
  message_buff[i] = '\0';
  const char *p_payload = message_buff;

  Serial.println("p_payload");
  Serial.println(p_payload);

  got_double = strtod(p_payload, NULL);
  got_int = atoi(p_payload);
  got_float = atof(p_payload);
  if (got_int == 0 || got_int<0) {
    got_bool = 0;
  }
  else { got_bool = 1; }  
  Serial.println("BUFF");Serial.println(p_payload);
  Serial.println("FLOAT");Serial.println(got_float);
  Serial.println("INT");Serial.println(got_int);
  Serial.println("BOOL");Serial.println(got_bool);
 //**********convert float value to Char expected by function
  char temp[5];
  dtostrf(rrr,4,3,temp); //rrr ist a float value

//**********************  
  for (i = 0; i<length; i++) {
    p_buff[i] = p[i];
  }
  p_buff[i] = '\0';
  const char *p_p = p_buff; //p_p ist topic made from last payload
  Serial.println("p_p");
  Serial.println(p_p);
//  Serial.write(p,length);
//  MQTTclient.publish(p_p, temp);
//**********************
free(p);

Weiss halt noch nicht wie ich am paylaod erkennen kann was das für ein 
tyüe ist... sind ja "nur" bytes (byte array...
Hast du da einen Tipp???


Danke!

[Rolf M. (rmagnus)]

Max M. schrieb:
> Weiss halt noch nicht wie ich am paylaod erkennen kann was das für ein
> tyüe ist... sind ja "nur" bytes (byte array...
> Hast du da einen Tipp???

Dafür definiert man sich normalerweise ein Protokoll (oder nutzt ein 
existierendes). Das legt genau fest, wann welche Daten übertragen werden 
bzw. schickt die relevanten Infos mit, so dass man nicht beim Empfang 
raten muss, was man da wohl gerade empfangen haben könnte.
Um was mit den Daten anfangen zu können, musst du doch eh die Bedeutung 
kennen, und dann ergibt sich der Typ doch schon meist daraus.
Also was ist es, was du da empfängst?

[Volle22 (Gast)]

Ein serielles Protokoll mit variablem Inhalt, liefert auch eine 
Information mit die es erlaubt den Inhalt zu identifizieren.

Das kann sich aus der Länge ergeben
oder  z.B durch ein Schlüsselzeichen

[Max M. (Firma: Herr) (maxtox)]

Hi,

also ich empfange so was:

DOino_01/eins/zwei/drei/vier/fuenf - als topic
100 - als payload


vom server wird folgendes geschickt:

mqtt.0 Send to client [DOino_01] 'DOino_01/eins/zwei/drei/vier/fuenf': 
10

[Max M. (Firma: Herr) (maxtox)]

es können also nur:

bool
int
float
string

kommen..

deswegen habe ich gedacht ich konvertiere reinkommende daten in alles 
was es sein könnte und dann im code (wenn ich weiß was ich will) gehe 
ich und sage getint oder getfloat...

[Volle22 (Gast)]

zuerst musst du prüfen ob generell etwas gekommen ist das dem Protokoll 
entspricht. oder ob der Empfang gestört war.

Dann überprüfst du ob du der richtige Empfänger bist
in dem Beispiel: DOino_01  am Anfang bis zum ersten /

Die Nutzdaten sind in dem Beispiel auch einfach zu finden
kommen direkt nach dem Doppelpunkt


Bool   enthalten nur 0 oder 1
Float  einen Punkt

Wie Int "1" sich vom Bool unterscheiden kann ich an dem Beispiel nicht 
sagen


Konvertierungen von Ascii nach Zahlen benötigen sehr viel Rechenzeit
und macht man nur wenn man es muss.

Zeichenvergleich und Suche geht schnell

[Max M. (Firma: Herr) (maxtox)]

das mit dem punkt (.)

also ich suche im string nach einem punkt? und wenn ja, dann ist es ein 
float?

so meinst du das?

[Volle22 (Gast)]

Ich kenne das Protokoll nicht und das eine Beispiel zeugt es nicht

aber das westliche Merkmal einer Floatzahl ist der Punkt oder auf 
deutsch die Kommastelle.
 Es gibt natürlich viele Möglichkeiten diese Information zu übertragen.

[Mikro 7. (mikro77)]

Hallo Max Maxov,

offensichtlich geht es dir um Deserialisierung: 
https://en.wikipedia.org/wiki/Serialization

Also, ein Sender erstellt bspw. aus C-Skalaren (int, float, etc.) ein 
Byte-Array, überträgt dieses Array an den Empfänger, der daraus wieder 
die C-Skalare zurückgewinnt.

Um das Umzusetzen, gibt es sehr viele Möglichkeiten.

Deinem Quelltext nach zu urteilen, serialisierst du deine Daten als 
ASCII String. Das ist absolut ok. Du mußt noch entscheiden wie genau 
dieser ASCII-String aufgebaut sein soll: Wie sieht die Repräsentation 
der einzelnen Skalare in ASCII aus, und, ggf., wie sieht eine Liste von 
Skalaren aus?

Nehmen wir einen ganz einfachen Fall: Es wird lediglich ein Boolean 
übertragen. True als '1' (ASCII 0x31) und False als '0' (0x30). Die 
Nachricht ist somit immer genau ein Byte lang.

Jetzt kommst du. Was möchtest du da übertragen? Welche Kombinationen 
gibt es? Deinen Quelltext nach zu urteilen möchtest du entweder ein 
float, double, int oder boolean übertragen. Häufig benutzt man in diesem 
Fall ein Wertepaar: ein Diskriminator, der anzeigt um welchen Datentyp 
es sich handelt, und dann den Wert selbst.

Die Kodierung des Boolean ist simpel (wie bspw. oben gezeigt). Bei int, 
float und double gibt es allerdings ein paar Fallstricke...

[Max M. (Firma: Herr) (maxtox)]

S. J. schrieb:
> Die Kodierung des Boolean ist simpel (wie bspw. oben gezeigt). Bei int,
> float und double gibt es allerdings ein paar Fallstricke...

Vielen Dank,

versuche mich da durchzukäpfen!!!!

Danke für den Tipp!