strtok Problem

Du musst den saveptr noch mitgeben:

man strtok

1

           for (j = 1, str1 = argv[1]; ; j++, str1 = NULL) {

2

               token = strtok_r(str1, argv[2], &saveptr1);

3

               if (token == N[LL)

4

                   break;

5

               printf("%d: %s\n", j, token);

6

7

               for (str2 = token; ; str2 = NULL) {

8

                   subtoken = strtok_r(str2, argv[3], &saveptr2);

9

                   if (subtoken == NULL)

10

                       break;

11

                   printf(" --> %s\n", subtoken);

12

               }

13

           }

Grüsse,
René

strtok speichert darin seine letzte Position. Sonst weiss er ja nicht, 
ab welcher Stelle er weitersuchen soll.

Also:

1

   char* saveptr;

Grüsse,
René

In Deinem Fall müsste das demnach so aussehen (einfach so getippt uns 
ungetestet):

1

typedef struct MD{

2

         char all[40];

3

         char *Name1;

4

         char *Name2;

5

         char Number;

6

    }DESCRIPTION;

7

8

struct Tree {

9

    DESCRIPTION MD;              ///< Module Description

10

} Tree;

11

12

13

char *buffer;

14

buffer=0;

15

16

char* saveptr;

17

18

Tree.MD.Name1 = strtok(Tree.MD.all, ",");

19

Tree.MD.Name2 = strtok(NULL, ",", &saveptr);

20

buffer = strtok(NULL, ",", &saveptr);

21

Tree.MD.Channel=*buffer;

Grüsse,
René

Martin Rox schrieb:
> und wieso strtok_r statt strtok? ich meine strtok funktioniert ja, ich
> glaube nur dass irgendein pointer anstatt dem gewünschten inhalt
> ausgegeben wird.

Das war nur das Beispiel aus der man page. Du kannst natürlich strtok 
nehmen und brauchst nicht strtok_r.

Grüsse,
René

Aaargh

Da gehört natürlich der saveptr auch hin:

1

Tree.MD.Name1 = strtok(Tree.MD.all, ",", &saveptr);

Grüsse,
René

hmm...

da gehts:

1

#include <stdio.h>

2

#include <stdlib.h>

3

#include <stdint.h>

4

#include <string.h>

5

#include <math.h>

6

7

8

9

int main (int argc, char *argv[])

10

{

11

  char* saveptr;

12

  char* tok;

13

  char val[127];

14

15

  strcpy (val,  "12,34,56,78");

16

17

18

  tok = strtok_r(val, ",", &saveptr);

19

  fprintf (stdout, "Tok: %s\n", tok);

20

21

  tok = strtok_r(NULL, ",", &saveptr);

22

  fprintf (stdout, "Tok: %s\n", tok);

23

24

  return (0);

25

}

:~/gcctest> gcc -o tmp2 tmp2.c
:~/gcctest> ./tmp2
Tok: 12
Tok: 34

Grüsse,
René

Um es klarzustellen. Den saveptr gibt es nur bei strtok_r. strtok 
benötigt ihn nicht.

Martin Rox schrieb:
> Ich möchte dass in Tree.MD.Channel

Es gibt in Tree.MD kein Member mit Namen "Channel". Es wäre sinnvoll, 
den Code zu posten, den du tatsächlich probiert hast.


Martin Rox schrieb:
> token = strtok_r(Tree.MD.all, ",",&saveptr1);
>
>  error: assignment makes pointer from integer without a cast

Entweder hast du den Header vergessen, oder der Compiler ist in einen 
Modus geschaltet, in dem er strtok_r nicht kennt (z.B. C89 oder C99).

Dann gibt's da wohl kein strtok_r. Dann eben doch strtok.

Poste mal den ganzen Code. Du hast da bei der Zuweisung einen Fehler 
(struct definition).

Rolf Magnus schrieb:
> Um es klarzustellen. Den saveptr gibt es nur bei strtok_r. strtok
> benötigt ihn nicht.

Das ist korrekt, dann muss man das Handling selber machen und jeweils 
den letzten Pointer als erster Parameter mitgeben.

Grüsse,
René

Dann eben mit strtok (nicht schön programmiert, nur um aufzuzeigen wie):

1

#include <stdio.h>

2

#include <stdlib.h>

3

#include <stdint.h>

4

#include <string.h>

5

#include <math.h>

6

7

8

9

int main (int argc, char *argv[])

10

{

11

  char* saveptr;

12

  char* tok;

13

  char val[127];

14

15

  strcpy (val,  "12,34,56,78");

16

17

18

  tok = strtok(val, ",");

19

  fprintf (stdout, "Tok: %s\n", tok);

20

21

  tok = strtok(tok+strlen(tok)+1, ",");

22

  fprintf (stdout, "Tok: %s\n", tok);

23

24

  // etc.....

25

26

  return (0);

27

}

Grüsse,
René

Definiere: "das klappt aber auch nicht".

Grüsse,
René

Rene H. schrieb:
> Dann eben mit strtok (nicht schön programmiert, nur um aufzuzeigen wie):

warum nicht das richtige beispiel aus der doku verwenden, deine Lösung 
ist unsinn.

http://www.cplusplus.com/reference/cstring/strtok/

Peter II schrieb:
> warum nicht das richtige beispiel aus der doku verwenden, deine Lösung
> ist unsinn.

Stimmt, hast völlig recht :-)

Grtüsse,
René

Das geht auch so:

1

  tok = strtok(val, ",");

2

  fprintf (stdout, "Tok: %s\n", tok);

3

4

  tok = strtok(NULL, ",");

5

  fprintf (stdout, "Tok: %s\n", tok);

Grüsse,
René

PS: getestet

Rene H. schrieb:
> Rolf Magnus schrieb:
>> Um es klarzustellen. Den saveptr gibt es nur bei strtok_r. strtok
>> benötigt ihn nicht.
>
> Das ist korrekt, dann muss man das Handling selber machen und jeweils
> den letzten Pointer als erster Parameter mitgeben.

Nein. Das ist im Prinzip genau das, was man bei strtok_r macht. Bei 
strtok übergibt man einfach NULL. Den letzten Pointer merkt sich die 
Funktion intern, weshalb sie nicht reentrant ist. Dafür steht dann auch 
das _r bei strtok_r.

Martin Rox schrieb:
> tok = strtok(val, ",");
> Nummer=*tok;

Äaaah? Was ist Nummer? Ein int?

1

  uint8_t nummer = atoi(tok);

Grüsse,
René

lol

Glückwunsch :-).

Grüsse,
René

Martin Rox schrieb:
> seltsam, eigentlich ist nummer auch ein char. die sind eigentlich alle
> char :D aber mit atoi klappt es mit der nummer. wenn text drin steht
> dann klappt es nicht mit
>
> Nummer=*tok;

Was willst du denn nun drin speichern? Ein char kannst du verwenden, um 
genau ein Zeichen zu speichern, oder um eine Zahl im Bereich bis 127 (je 
nach CPU, aber 127 ist der sichere Wert) zu speichern.
Wenndu also Nunmmer=*tok schreibst, speicherst du das erste Zeichen von 
tok in Nummer. Wenn du atoi benutzt, wird das, was in tok steht, 
umgewandelt in eine Zahl, und die wird in Nummer gespeichert. Du kannst 
halt nicht beides haben.

Hi Martin,

ich habe den Eindruck, das nicht strtok Dein Problem war/ist. Kann es 
sein, dass Du ein Durcheinander machst mit Zeichenketten und Zahlen?

Falls ich Dich damit unterfordere, einfach ignrorieren.

ein

1

    char a;

stellt ein ASCII Zeichen dar und kann den numerischen Wert von -127 bis 
+127 annehmen. Im Speicher ist das ein Byte üblicherweise.

1

   char string[128];

2

3

   strcpy (string, "Meine Zeichenkette");

ist ein Zeiger auf eine Zeichenkette, also eine Anreihung von char's. Im 
Speicher braucht sie 128 Byte, benutzt davon werden aber lediglich  19 
Byte (inkl. der abschliessenden Null-Terminierung).

Wenn Du nun die Zuweisung machst:

1

    char aChar = *string;

steht in aChar ein M. Oder als Zahlenwert: 0x4d. Du dereferenzierst 
lediglich den char*, somit das erste Zeichen.


Wie willst Du Deine Tokens weiter verabeiten? Mit welchem Datentyp?
Du hast die Wahl Deine Tokens als String oder Wert zu verarbeiten:

1

    char    zahlenWert;

2

    uint8_t einByte;

3

4

    char tokenA[16];

5

    char tokenB[16];

6

7

    char* strZahl;

8

9

    strcpy (tokenA, "abc");

10

    strcpy (tokenB, "123");

11

12

    zahlenWert = *tokenA; // ergibt ASCII 0x61 oder 'a' als Zeichen

13

    zahlenWert = atoi(tokenA); // falsch, weil in Token A steht abc

14

15

    einByte = atoi(tokenB); // ergibt einByte = 123 als Zahl

16

    zahlenWert = *tokenB; // ergibt ASCII 0x31 oder '1' als Zeichen

Vielleicht hilft das etwas.

Grüsse,
René

Das Problem wird die Unwissenheit über den Unterschied zwischen Ziffer 
und Zahl oder konkret zwischen 3 und '3' und "3"

Bzw. das eine Zeichenkette nicht durch eine einfache Zuweisung zu einem 
Zahlenwert führt sondern erst mit Hilfe einer Funktion gewandelt werden 
muss.

Das ist halt C und keine Typenlose Scriptsprache.

Rene H. schrieb:
> ein
>     char a;
>
> stellt ein ASCII Zeichen dar und kann den numerischen Wert von -127 bis
> +127 annehmen.

Ich weiß nicht, wie es beim C32 ist, aber prinzipiell sollte man char 
höchstens für den Bereich 0 bis 127, oder besser noch, einfach gar nicht 
für das Speichern von Zahlen verwenden. Es ist dem Compiler überlassen, 
ob char mit oder ohne Vorzeichen ist. Übrigens: Wenn es ein Vorzeichen 
hat, ist der Bereich meistens eher -128 bis 127.

> Im Speicher ist das ein Byte üblicherweise.

Es ist immer ein Byte, denn genau so ist in C das Byte definiert - als 
die Speichermenge, die ein char belegt.

Martin Rox schrieb:
> hallo rolf, meinstu, dass ich meine zeichenketten lieber als u8 als char
> definieren sollte?

Du kannst auch einfach signed oder unsigned char nehmen, also explzit 
angeben, ob Vorzeichen oder nicht.

Martin Rox schrieb:
> Also erstmal ist es so, dass dieser strtok befehl irgendwie meinen
> strink zerhackt.

Das ist so bei strtrok. Steht normalerweise auch in der Doku drin.

Rolf Magnus schrieb:
>> Im Speicher ist das ein Byte üblicherweise.
>
> Es ist immer ein Byte, denn genau so ist in C das Byte definiert - als
> die Speichermenge, die ein char belegt.

Ok, ich kannte die Definition nicht, resp. war mir unsicher, deshalb 
meine vorsichtige Aussage. Vorstellen hätte ich mir können, dass es 
irgendeinen Compiler gibt der per default oder Option w_char macht o.ä.
Ich meine, dass das hier auch schon Thema war.

Grüsse,
René