strtok Problem

[Martin R. (m-joy)]

Hallo,

ich möchte eine zeichenkette getrennt von "," aufteilen,
dafür mache ich folgendes:

typedef struct MD{
         char all[40];
         char *Name1;
         char *Name2;
         char Number;
    }DESCRIPTION;

struct Tree {
    DESCRIPTION MD;              ///< Module Description
} Tree;


char *buffer;
buffer=0;


Tree.MD.Name1 = strtok(Tree.MD.all, ",");
Tree.MD.Name2 = strtok(NULL, ",");
buffer = strtok(NULL, ",");
Tree.MD.Channel=*buffer;

Tree.MD.all wird eingelesen. Kann zb "BLA,BLA123,3" sein....
Ich möchte dass in Tree.MD.Channel jetzt der dritte abschnitt nach den 
komma steht. da steht aber irgendwie immer was anderes drin. Wenn ich 
Name1 und Name2 printe steht da aber das richtige drin.... Auch wenn ich 
Channel ohne buffer auslese und ausgebe steht da das richtige drin..
Ich möchte aber den Wert von Channel haben.

Grüße

[Rene H. (Gast)]

Du musst den saveptr noch mitgeben:

man strtok

           for (j = 1, str1 = argv[1]; ; j++, str1 = NULL) {
               token = strtok_r(str1, argv[2], &saveptr1);
               if (token == N[LL)
                   break;
               printf("%d: %s\n", j, token);

               for (str2 = token; ; str2 = NULL) {
                   subtoken = strtok_r(str2, argv[3], &saveptr2);
                   if (subtoken == NULL)
                       break;
                   printf(" --> %s\n", subtoken);
               }
           }

Grüsse,
René

[Martin R. (m-joy)]

Huhu, was ist denn ein &saveptr1 ?

[Rene H. (Gast)]

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

Also:

   char* saveptr;

Grüsse,
René

[Martin R. (m-joy)]

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

[Rene H. (Gast)]

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

typedef struct MD{
         char all[40];
         char *Name1;
         char *Name2;
         char Number;
    }DESCRIPTION;

struct Tree {
    DESCRIPTION MD;              ///< Module Description
} Tree;


char *buffer;
buffer=0;

char* saveptr;

Tree.MD.Name1 = strtok(Tree.MD.all, ",");
Tree.MD.Name2 = strtok(NULL, ",", &saveptr);
buffer = strtok(NULL, ",", &saveptr);
Tree.MD.Channel=*buffer;

Grüsse,
René

[Rene H. (Gast)]

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é

[Rene H. (Gast)]

Aaargh

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

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

Grüsse,
René

[Martin R. (m-joy)]

char *token;
char *saveptr1;


token = strtok_r(Tree.MD.all, ",",&saveptr1);


 error: assignment makes pointer from integer without a cast

[Rene H. (Gast)]

hmm...

da gehts:

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



int main (int argc, char *argv[])
{
  char* saveptr;
  char* tok;
  char val[127];

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


  tok = strtok_r(val, ",", &saveptr);
  fprintf (stdout, "Tok: %s\n", tok);

  tok = strtok_r(NULL, ",", &saveptr);
  fprintf (stdout, "Tok: %s\n", tok);

  return (0);
}

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

Grüsse,
René

[Rolf Magnus (Gast)]

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).

[Frank M. (ukw) (Moderator)]

Martin Rox schrieb:
>  error: assignment makes pointer from integer without a cast

Kein Wunder, denn Du hast offenbar ein

#include <string.h>

vergessen.

[Martin R. (m-joy)]

komisch. bei mir nicht... benutze MPLAB 8 von microchip mit C32 
compiler...

[Martin R. (m-joy)]

string.h ist included.

[Rolf Magnus (Gast)]

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

[Rene H. (Gast)]

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é

[Rene H. (Gast)]

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

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



int main (int argc, char *argv[])
{
  char* saveptr;
  char* tok;
  char val[127];

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


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

  tok = strtok(tok+strlen(tok)+1, ",");
  fprintf (stdout, "Tok: %s\n", tok);

  // etc.....

  return (0);
}

Grüsse,
René

[Martin R. (m-joy)]

Huhu, also in meiner string.h gibt es:
char  *strtok_r (char *, const char *, char **);

ich habe zum testen deinen code kopiert, der klappt aber auch 
nicht...ich teste nochmal...

[Rene H. (Gast)]

Definiere: "das klappt aber auch nicht".

Grüsse,
René

[Peter II (Gast)]

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/

[Martin R. (m-joy)]

ich glaube strtok_r ist verbuggt in meinem compiler. habe im microchip 
forum gelesen dass da paar string funktionen wohl buggy sind. ich widme 
mich also dem normalen strtok ^^ sooo mal gucken ob ich das 
hinbekomme....

[Rene H. (Gast)]

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é

[Rene H. (Gast)]

Das geht auch so:

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

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

Grüsse,
René

PS: getestet

[Rolf Magnus (Gast)]

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 R. (m-joy)]

huhu, jaaa das mit dem strtok geht bei mir auch. aber der trick ist ja, 
dass ich aus dem string die einzelnen wörter in variablen speicher will. 
und da speichert der irgendwie nichts gutes rein.
Beispiel:

String -> "Bla1,Bla2,123"

jetzt will ich dass 123 in meiner variablen "Nummer" steht...
wenn ich aber

 tok = strtok(val, ",");
Nummer=*tok;

mache, steht in nummer etwas anderes.... Also genauer ist der code am 
anfang den ich gepostet habe... Der fehler scheint bei der zuweisung zu 
entstehen, kann es sein dass hier:
Nummer=*tok;

nicht der inhalt ab dem komma, sondern nur der zeiger übergeben wird?

[Rene H. (Gast)]

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

Äaaah? Was ist Nummer? Ein int?

  uint8_t nummer = atoi(tok);

Grüsse,
René

[Martin R. (m-joy)]

Renee !!! ES KLAPPT das war der trick ich raste aus....

[Rene H. (Gast)]

lol

Glückwunsch :-).

Grüsse,
René

[Martin R. (m-joy)]

und wie ist der trick wenn ein Text im tok steht?

[Martin R. (m-joy)]

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;

[Rolf Magnus (Gast)]

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.

[Rene H. (Gast)]

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

    char a;

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

   char string[128];

   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:

    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:

    char    zahlenWert;
    uint8_t einByte;

    char tokenA[16];
    char tokenB[16];

    char* strZahl;

    strcpy (tokenA, "abc");
    strcpy (tokenB, "123");

    zahlenWert = *tokenA; // ergibt ASCII 0x61 oder 'a' als Zeichen
    zahlenWert = atoi(tokenA); // falsch, weil in Token A steht abc

    einByte = atoi(tokenB); // ergibt einByte = 123 als Zahl
    zahlenWert = *tokenB; // ergibt ASCII 0x31 oder '1' als Zeichen

Vielleicht hilft das etwas.

Grüsse,
René

[DirkB (Gast)]

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.

[Martin R. (m-joy)]

Hallo Rene, ich glaube auch dass ich da was durcheinander wurschtel... 
danke für den beitrag, werde ich mir heute anschauen =)

grüße

[Martin R. (m-joy)]

oah jetzt hab ichs hinbekommen.

Also erstmal ist es so, dass dieser strtok befehl irgendwie meinen 
strink zerhackt. Also habe ich den string vorher in eine neue variable 
kopiert, und die größen den chars definiert:

typedef struct MD{
         char all[40];
         char Name1[10];
         char Name2[20];
    }MD;

char all[40],*buffer;
strcpy(all,Tree.MD.all); //hier mache ich eine kopie um den string zu schützen

dann fange ich an:

buffer = strtok((char*)all, ",");
strcpy(Tree.MD.Name1,buffer);    //in meine variable speichern

und dann gehts weiter mit den nächsten teilen:

buffer = strtok(NULL, ",");
strcpy(Tree.MD.Name2,buffer);

und so weiter. dann habe ich den text in meinen variablen.
Spitze, danke für die hilfe.

[Rolf Magnus (Gast)]

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 R. (m-joy)]

hallo rolf, meinstu, dass ich meine zeichenketten lieber als u8 als char 
definieren sollte?

Grüße

[Rolf Magnus (Gast)]

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.

[Rolf Magnus (Gast)]

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.

[Klaus W. (mfgkw)]

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

Das ist das erwartete Verhalten von strtok(), steht auch in jeder Doku 
dazu drin.

Ansonsten würden die gelieferten Zeiger auf die einzelnen Token ja nicht 
auf jeweils nullterminierte Strings zeigen - die dafür notwendige 
Terminierung der einzelnen Token muß von strtok() in den String 
geschrieben werden anstelle des gefundenen Trennzeichens.

[Klaus W. (mfgkw)]

Fast wörtliche Übereinstimmung, gut gemacht! :-)

[Rene H. (Gast)]

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é