übergeben Sie Zeichenfolgen, die durch Verweis in C

Eins, was jeder Blick ist, dass Sie den Aufruf strtok auf ein array gespeichert const Speicher. strtok schreibt in das array, das Sie es weiterleiten, so stellen Sie sicher, dass Sie Kopie, um ein temporäres array vor dem Aufruf von strtok oder nur reservieren, das original mag:

char mybaz[] = "hello:world";

Ooh ja, kleines problem gibt es.

Regel, wenn Sie gehen, um die Manipulation von strings in einer Funktion, die Speicher für strings besser werden außerhalb der Funktion. Der einfache Weg um dies zu erreichen, ist zu deklarieren arrays außerhalb der Funktion (z.B. in main()) und das übergeben des arrays (was automatisch zu Zeiger auf Ihre Anfänge), um die Funktion. Dies funktioniert gut, so lange, wie Sie Ihre Ergebnis-strings nicht überlauf des zugewiesenen Platzes in die Felder.

Ihnen gegangen sind, die vielseitiger, aber etwas schwieriger route: verwenden Sie malloc() um Platz zu schaffen für Ihre Ergebnisse (gut so weit!) und dann versuchen, weisen die malloc hatte Platz, um die Hinweise, die Sie übergeben. Dass, ach, wird nicht funktionieren.

Den Zeiger kommen, ist ein Wert; Sie kann es nicht ändern. Die Lösung ist, übergeben Sie einen Zeiger an einen Zeiger und verwenden Sie es innerhalb der Funktion zu ändern, welche der Zeiger verweist.

Wenn du den hast, ist das Super. Wenn nicht, Fragen Sie bitte für weitere Klärung.

den code wahrscheinlich segfaults, weil Sie zuweisen von Speicherplatz für den string aber zu vergessen, dass ein string hat eine extra-byte auf das Ende der null-terminator.

Auch Sie sind nur die übergabe eines Zeigers in. Da ein Zeiger ist eine 32-bit-Wert (auf einem 32-bit-Maschine) Sie sind einfach übergeben Sie den Wert der unitialised Zeiger in den "fn". In der gleichen Weise würden Sie nicht expact eine Ganzzahl, die einer Funktion übergeben werden zurückgegeben an die aufrufende Funktion (ohne Sie explizit Rückgabe) Sie können nicht erwarten, dass der Mauszeiger, das gleiche zu tun. So ist die neue pointer-Werte sind nie wieder zurück in die main-Funktion. Gewöhnlich tun Sie dies, indem ein Zeiger auf einen Zeiger in C.

Vergessen Sie auch nicht frei von dynamisch zugewiesenen Speicher!!

void
fn(char *baz, char **foo, char **bar)
{
     char *pch;

     /* this is the part I'm having trouble with */

     pch = strtok (baz, ":");
     *foo = malloc(strlen(pch) + 1);
     strcpy(*foo, pch);

     pch = strtok (NULL, ":");
     *bar = malloc(strlen(pch) + 1);
     strcpy(*bar, pch);

     return;
}

int
main(void)
{
     char *mybaz, *myfoo, *mybar;

     mybaz = "hello:world";

     fn(mybaz, &myfoo, &mybar);

     fprintf(stderr, "%s %s", myfoo, mybar);

     free( myFoo );
     free( myBar );
}

Das wesentliche problem ist, dass, obwohl Speicher ist immer reserviert (mit malloc()) für die Ergebnisse, die Sie versuchen, um return myfoo und mybar die Zeiger auf diese Zuweisungen nicht tatsächlich zurückgegeben main(). Als ein Ergebnis, das später nennen zu printf() ist durchaus wahrscheinlich, um einen Speicherauszug.

Die Lösung ist zu erklären, dass die Argumente als ponter, um Zeiger auf char ab und übergeben die Adressen von myfoo und mybar zu fn. So etwas (ungetestet) sollte den trick tun:

void
fn(char *baz, char **foo, char **bar)
{
     char *pch;

     /* this is the part I'm having trouble with */

     pch = strtok (baz, ":");
     *foo = malloc(strlen(pch)+1);  /* include space for NUL termination */
     strcpy(*foo, pch);

     pch = strtok (NULL, ":");
     *bar = malloc(strlen(pch)+1);  /* include space for NUL termination */
     strcpy(*bar, pch);

     return;
}

int
main(void)
{
     char mybaz[] = "hello:world";
     char *myfoo, *mybar;

     fn(mybaz, &myfoo, &mybar);
     fprintf(stderr, "%s %s", myfoo, mybar);
     free(myfoo);
     free(mybar);
}

Vergessen Sie nicht, die frei jeden zugewiesenen string zu einem späteren Zeitpunkt, oder erstellen Sie Speicher-Lecks.

Sowohl die malloc() und strcpy() in einen Aufruf, es wäre besser, verwenden strdup(), wie es sich auch an, um zu reservieren Zimmer für das abschließende NUL, die Sie Links aus deinem code. *foo = strdup(pch) ist viel klarer und leichter zu pflegen, dass die alternative. Da strdup() ist POSIX und nicht ANSI-C, möglicherweise müssen Sie es selbst implementieren, aber der Aufwand ist auch getilgt durch die daraus resultierende Klarheit für diese Art der Nutzung.

Andere traditionelle Möglichkeit zum zurückgeben einer Zeichenfolge aus einer C-Funktion ist für den Anrufer zuweisen, die Lagerung und geben Sie die Adresse der Funktion. Dies ist die Technik, die von sprintf() zum Beispiel. Es leidet unter dem problem, dass es keine Möglichkeit, einen solchen Anruf Website völlig sicher vor buffer-overrun-Fehlern verursacht, die durch die aufgerufene Funktion die übernahme von mehr Raum zugewiesen wurde, als tatsächlich verfügbar ist. Die traditionellen Reparatur für dieses problem ist zu verlangen, dass ein buffer-length-argument übergeben werden, und prüfen Sie sorgfältig, ob sowohl die ist-Allokation und die Länge behauptete auf der call-Seite in einem code-review.

Edit:

Den tatsächlichen segfault, die Sie bekommen ist wahrscheinlich im inneren strtok(), nicht printf() weil dein Beispiel, wie geschrieben versucht, übergeben Sie eine Zeichenfolge Konstante strtok() muss in der Lage sein, die Zeichenfolge zu ändern. Das ist offiziell nicht definiertes Verhalten.

Den fix für dieses Problem ist, um sicherzustellen, dass bybaz ist deklariert als ein array initialisiert, und nicht als Zeiger auf char. Das array initialisiert wird sich im beschreibbaren Speicher, während die string-Konstante dürfte sich im nur-lese-Speicher. In vielen Fällen, string-Konstanten gespeichert sind, in dem gleichen Teil des Speichers verwendet, um den ausführbaren code selbst, und moderne Systeme, die alle versuchen, zu machen es schwierig für ein Programm zu ändern, seinen eigenen code.

In der embedded-Systeme, ich arbeite für ein Leben, der code ist wahrscheinlich zu sein, gespeichert in einem ROM einer Art, und nicht physisch geändert.