Python --> C/C++

[henkoegema]

Ik probeer dit stukje programma in (micro)Python om te zetten naar C/C++ voor een Arduino.

(micro)Python
Dit stukje programma leest de gegevens van een sensor.
De inhoud wordt gesplitst m.b.v. de  komma (,) in 4 variablelen. (q0, q1,q2 en q3)

Code: [Selecteer]

.............
.............
dataPacket=ad.readline()        # deze lijn leest de input van een sensor. b.v. "-0.01,0.55,-0.7,-1"
dataPacket=str(dataPacket,'utf-8')     
splitPacket=dataPacket.split(",")         
q0=float(splitPacket[0])
q1=float(splitPacket[1])
q2=float(splitPacket[2])
q3=float(splitPacket[3])
...........
...........

Tijdens het testen ben ik tot zover gekomen:

C/C++ (Arduino)

Code: [Selecteer]

//  https://www.codingame.com/playgrounds/14213/how-to-play-with-strings-in-c/string-split

void setup(){
  Serial.begin(115200);
}

void loop(){

  char dataPacket[] = "-0.01,0.55,-0.7,-1";
  char delim[] = ",";
  char *ptr = strtok(dataPacket, delim);

   while (ptr != NULL){
    Serial.println(ptr);
    ptr = strtok(NULL, delim);
   
   q0 = float .......???;   
   q1 = float .......???;   
   q2 = float .......???;   
   q3 = float .......???;   
  }
  // delay(1000);
}

Output van dit programma is:



Mijn probleem is hoe ik de waarden van de gesplitste string in C/C++ kan  toevoegen aan q0, q1,q2,q3. 

[erik1984]

Zoiets werkt als je zeker weet dat er in dataPacket[] altijd netjes 4 getallen zitten bij splitsen op komma. Ik ben zeker geen C-expert, dus voor eigen risico  C is een mijnenveld waarin je heel makkelijk een stapje verkeerd zet.

Code: [Selecteer]

#include <stdlib.h>
#include <string.h>
#include <stdio.h>

int main (int argc, char *argv[])
{
    char dataPacket[] = "-0.01,0.55,-0.7,-1";
    char delim[] = ",";   
       
    float q0 = atof(strtok(dataPacket, delim));
    float q1 = atof(strtok(NULL, delim));
    float q2 = atof(strtok(NULL, delim));
    float q3 = atof(strtok(NULL, delim));
   
    printf("q0 = %f\n", q0);
    printf("q1 = %f\n", q1);
    printf("q2 = %f\n", q2);
    printf("q3 = %f\n", q3);
   
    return 0;
}


[henkoegema]

Erik.

Het is bijna goed.   
De waarde van q0 is ok.
Echter de waarden van q1,q2,q3 zijn allemaal  0.00   

Code: [Selecteer]


void setup(){
  Serial.begin(115200);
}

void loop(){

  char dataPacket[] = "-0.01,0.55,-0.7,-1";
  char delim[] = ",";
  char *ptr = strtok(dataPacket, delim);

  float q0 = atof(strtok(dataPacket, delim));
  float q1 = atof(strtok(NULL, delim));
  float q2 = atof(strtok(NULL, delim));
  float q3 = atof(strtok(NULL, delim));
 
  Serial.println(q0);
  Serial.println(q1);
  Serial.println(q2);
  Serial.println(q3);
 
  delay(1000);
}


[henkoegema]

Zoiets werkt als je zeker weet dat er in dataPacket[] altijd netjes 4 getallen zitten bij splitsen op komma. Ik ben zeker geen C-expert, dus voor eigen risico  C is een mijnenveld waarin je heel makkelijk een stapje verkeerd zet.

Code: [Selecteer]

#include <stdlib.h>
#include <string.h>
#include <stdio.h>

int main (int argc, char *argv[])
{
    char dataPacket[] = "-0.01,0.55,-0.7,-1";
    char delim[] = ",";   
       
    float q0 = atof(strtok(dataPacket, delim));
    float q1 = atof(strtok(NULL, delim));
    float q2 = atof(strtok(NULL, delim));
    float q3 = atof(strtok(NULL, delim));
   
    printf("q0 = %f\n", q0);
    printf("q1 = %f\n", q1);
    printf("q2 = %f\n", q2);
    printf("q3 = %f\n", q3);
   
    return 0;
}


In jouw programma werkt het WEL. 
In mijn Arduino (nog niet)

[henkoegema]

Hoera, hoera. Het werkt. 
Bij deze mijn dank aan Erik voor de genomen moeite.   

[TopGear]

Wat was de uiteindelijke oplossing, Henk?

[henkoegema]

Wat was de uiteindelijke oplossing, Henk?

Code: [Selecteer]


void setup(){
  Serial.begin(115200);
}

void loop(){

  char dataPacket[] = "-0.01,0.55,-0.7,-1";
  char delim[] = ",";
  char *ptr = strtok(dataPacket, delim);   <====== moest deze lijn weglaten. (zoals erik1984 al aangaf)

  float q0 = atof(strtok(dataPacket, delim));
  float q1 = atof(strtok(NULL, delim));
  float q2 = atof(strtok(NULL, delim));
  float q3 = atof(strtok(NULL, delim));
 
  Serial.println(q0);
  Serial.println(q1);
  Serial.println(q2);
  Serial.println(q3);
 
  delay(1000);
}


[bart85]

waarom niet onderstaande?

Code: [Selecteer]

float q[4];
q[0] = atof(strtok(dataPacket, delim));

for(int i=1;i<4;i++{
    q[i]=atof(strtok(NULL, delim));
}

[henkoegema]

waarom niet onderstaande?

Code: [Selecteer]

float q[4];
q[0] = atof(strtok(dataPacket, delim));

for(int i=1;i<4;i++{
    q[i]=atof(strtok(NULL, delim));
}

Ja, had ook gekund.   

[bart85]

ik denk dat de python code wat mooier kan. Met onderstaande oneliner:

Code: [Selecteer]

q = [float(i) for i in str(ad.readline(), 'utf-8').split(",")]

Dit creeert een array met 4 elementen van het type float. Dat zijn 7 regels naar 1 regel gebracht.
Met een simpele for loop kun je q uitlezen.

[henkoegema]

ik denk dat de python code wat mooier kan. Met onderstaande oneliner:

Code: [Selecteer]

q = [float(i) for i in str(ad.readline(), 'utf-8').split(",")]

Dit creeert een array met 4 elementen van het type float. Dat zijn 7 regels naar 1 regel gebracht.
Met een simpele for loop kun je q uitlezen.

Er zijn altijd nog mensen die slimmer zijn dan ik.   

[bart85]

Mij lijkt in de python code een deel overbodig is. ad.readline() geeft waarschijnlijk al een string terug.

Code: [Selecteer]

q = [float(i) for i in ad.readline().split(",")]
Werkt deze regel ook zonder errors?

[henkoegema]

Mij lijkt in de python code een deel overbodig is. ad.readline() geeft waarschijnlijk al een string terug.

Code: [Selecteer]

q = [float(i) for i in ad.readline().split(",")]
Werkt deze regel ook zonder errors?

Ik heb het niet getest.
Had het project al afgesloten.