Selbstgemachter Split will nicht so ....

[A Former User]

ich möchte eine zeichenkette in ein array aus pointern umwandeln. dazu habe ich das hier programmiert.

vector<char*> split (char* in) 
{
    vector<char*> result;
    result.push_back((char*)"-dummy-");
    char* s = strtok(in, " ");
    while (s != NULL)
    {
        result.push_back(s);
        s = strtok(NULL, " ");
    }
    return result;
}

aufrufen tue ich es so:

int main()
{
    char arg[] = "move back motherfucker";
    vector<char*> vs = split(arg);
    for (auto i : vs) cout << i << endl;

nun bekomme ich

-dummy-
m
back
motherfucker

aber es sollte doch

-dummy-
move
back
motherfucker

sein???
woran kann das liegen?

[wob]

@Bushmaster sagte in Selbstgemachter Split will nicht so ....:

char arg[] = "move back motherfucker";
vector<char*> vs = split(arg);

Hast du kompiliert?

Ansonsten: brrr...

[A Former User]

@wob sagte in Selbstgemachter Split will nicht so ....:

@Bushmaster sagte in Selbstgemachter Split will nicht so ....:

char arg[] = "move back motherfucker";
vector<char*> vs = split(arg);

Hast du kompiliert?

habe ich, sonst hätte ich ja keine ausgaben gesehen. aber witzigerweise funktioniert es jetzt. lag wohl an der umgebung, dass das konsolenfenster oder der debugger die paar zeichen verschluckt hat. echt seltsam.

@wob sagte in Selbstgemachter Split will nicht so ....:

Ansonsten: brrr...

ich brauche letztlich ein array aus char*. wenn ich den vector als &vs[0] nehme, bekomme ich das gewünschte:

char** arr = &split(arg)[0];

deswegen ein vector aus char*. oder weißt du einen einfacheren weg?

[Schlangenmensch]

@Bushmaster ich würde das mit std::string, std::string::find(), und std::string::substr() lösen.

Wenn es mal funktioniert und mal nicht, klingt das nicht gut.

[A Former User]

@Schlangenmensch sagte in Selbstgemachter Split will nicht so ....:

std::string, std::string::find(), und std::string::substr() lösen.

das wäre sicher mehr c++like, aber wäre es auch kurz und effizient?
ich bin leider darauf angewiesen einen char** zu konstruieren, um damit die main-funktion eines anderen programms zu füttern.

@Schlangenmensch sagte in Selbstgemachter Split will nicht so ....:

Wenn es mal funktioniert und mal nicht, klingt das nicht gut.

vor allem weil computer sonst deterministisch arbeiten.
ich denke mal, das war ein temporäres problem. ich muss mal wieder einen virenscanner über meine kiste laufen lassen.

[Schlangenmensch]

@Bushmaster wie rufst du denn das andere Programm auf? Ich würde wahrscheinlich mit system("program.exe -params") arbeiten, also mit einem c-style string. Da muss dann nichts gesplittet werden.

[A Former User]

@Schlangenmensch sagte in Selbstgemachter Split will nicht so ....:

@Bushmaster wie rufst du denn das andere Programm auf? Ich würde wahrscheinlich mit system("program.exe -params") arbeiten, also mit einem c-style string. Da muss dann nichts gesplittet werden.

ich rufe direkt seine main-funktion auf.

extern "C" int samMain(int argc, char** argv);

https://github.com/s-macke/SAM

[A Former User]

@Bushmaster sagte in Selbstgemachter Split will nicht so ....:

ich muss mal wieder einen virenscanner über meine kiste laufen lassen.

Das ist so ziemlich der unwahrscheinlichste Grund für dein Problem.

[A Former User]

@manni66
aber was es war werde ich nie erfahren. der effekt trat nicht mehr auf.

[Jockelx]

@Bushmaster sagte in Selbstgemachter Split will nicht so ....:

aber was es war werde ich nie erfahren. der effekt trat nicht mehr auf.

Gerüchten zur Folge kursieren hier im Forum in letzter Zeit einige Leute mit Zeitmaschinen. Vielleicht wendest du dich an die, um den alten Fehler nochmals zu reproduzieren?

[A Former User]

@Jockelx sagte in Selbstgemachter Split will nicht so ....:

@Bushmaster sagte in Selbstgemachter Split will nicht so ....:

aber was es war werde ich nie erfahren. der effekt trat nicht mehr auf.

Gerüchten zur Folge kursieren hier im Forum in letzter Zeit einige Leute mit Zeitmaschinen. Vielleicht wendest du dich an die, um den alten Fehler nochmals zu reproduzieren?

zum beispiel mümmel der lümmel. er weiß jetzt schon, dass er in 2 monaten tot ist obwohl er sich ganz gesund fühlt. jedenfalls körperlich.

[Quiche Lorraine]

@Bushmaster
Ich glaube strtok() macht nicht genau das was du möchtest. Kann es sein das du die Variable arg in der Zwischenzeit verändert hast?

Schau dir mal das folgende Beispiel an. Verändert man arg, so verändert man auch dein Ergebnis vs.
Online Beispiel

Die Lösung hierfür ist recht einfach. Packe deine Strings in std::string.

[A Former User]

@Quiche-Lorraine sagte in Selbstgemachter Split will nicht so ....:

Ich glaube strtok() macht nicht genau das was du möchtest. Kann es sein das du die Variable arg in der Zwischenzeit verändert hast?

ja, strtok arbeitet so. ich kann ihm kein const char geben, weil es den string selber als 'workspace' nutzt.

@Quiche-Lorraine sagte in Selbstgemachter Split will nicht so ....:

Schau dir mal das folgende Beispiel an. Verändert man arg, so verändert man auch dein Ergebnis vs.
Online Beispiel

danke. das schaue ich mir geich mal an.

[A Former User]

@Quiche-Lorraine sagte in Selbstgemachter Split will nicht so ....:

@Bushmaster
Ich glaube strtok() macht nicht genau das was du möchtest. Kann es sein das du die Variable arg in der Zwischenzeit verändert hast?

Schau dir mal das folgende Beispiel an. Verändert man arg, so verändert man auch dein Ergebnis vs.
Online Beispiel

Die Lösung hierfür ist recht einfach. Packe deine Strings in std::string.

du simulierst den fehler durch

   arg[1] = ' ';
    arg[2] = ' ';
    arg[3] = ' ';

gleiches würdest du erreichen mit:

   arg[1] = NULL;

strtok setzt nullen als trenner in das array.
kann irgendwas strtok dazu bewegt haben, eine null an die zweite position des input-arrays zu setzen?

[DirkB]

@Bushmaster sagte in Selbstgemachter Split will nicht so ....:

kann irgendwas strtok dazu bewegt haben, eine null an die zweite position des input-arrays zu setzen?

Ein strtok(arg, "o"); in main.

Das strtok den String verändert ist blöd.
Noch blöder ist aber, dass es seinen Zustand (den Zeiger auf das Zeichen hinter dem gesetzten '\0') intern merkt.
So kann man nicht gleichzeitig mehrere Strings parsen.

[A Former User]

@DirkB sagte in Selbstgemachter Split will nicht so ....:

Noch blöder ist aber, dass es seinen Zustand (den Zeiger auf das Zeichen hinter dem gesetzten '\0') intern merkt.
So kann man nicht gleichzeitig mehrere Strings parsen.

ja, aber das ist eine bekannte einschränkung. also ich kann mir gut vorstellen, dass eine multithreading-fähige libc den strtok-state im 'thread local storage' speichert.