C strtok() 将字符串拆分为标记，但保持旧数据不变

Question

我有以下代码：

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

int main (void) {
    char str[] = "John|Doe|Melbourne|6270|AU";

    char fname[32], lname[32], city[32], zip[32], country[32];
    char *oldstr = str;

    strcpy(fname, strtok(str, "|"));
    strcpy(lname, strtok(NULL, "|"));
    strcpy(city, strtok(NULL, "|"));
    strcpy(zip, strtok(NULL, "|"));
    strcpy(country, strtok(NULL, "|"));

    printf("Firstname: %s\n", fname);
    printf("Lastname: %s\n", lname);
    printf("City: %s\n", city);
    printf("Zip: %s\n", zip);
    printf("Country: %s\n", country);
    printf("STR: %s\n", str);
    printf("OLDSTR: %s\n", oldstr);

    return 0;
}

执行输出：

$ ./str
Firstname: John
Lastname: Doe
City: Melbourne
Zip: 6270
Country: AU
STR: John
OLDSTR: John

为什么我不能保留旧数据，也不能保留在str 或oldstr 中，我做错了什么，如何不更改或保留数据？

Answer 1

当您执行strtok(NULL, "|") strtok() 时，找到令牌并将null 放在适当的位置（将令牌替换为\0）并修改字符串。

你str，变成：

char str[] = John0Doe0Melbourne062700AU;
                 
  Str array in memory 
+------------------------------------------------------------------------------------------------+
|'J'|'o'|'h'|'n'|0|'D'|'o'|'e'|0|'M'|'e'|'l'|'b'|'o'|'u'|'r'|'n'|'e'|0|'6'|'2'|'7'|'0'|0|'A'|'U'|0|
+------------------------------------------------------------------------------------------------+
                 ^  replace | with \0  (ASCII value is 0)

考虑到图表很重要，因为 char '0' 和 0 不同（在字符串 6270 中，图中的 char 由 ' 括起来，其中 \0 0 是数字）

当您使用 %s 打印 str 时，它会打印字符到第一个 \0 即 John

要保持原始 str 不变，您应该先将 str 复制到某个 tempstr 变量中，然后在 strtok() 中使用该 tempstr 字符串：

char str[] = "John|Doe|Melbourne|6270|AU";
char* tempstr = calloc(strlen(str)+1, sizeof(char));
strcpy(tempstr, str);

现在在您的代码中使用这个tempstr 字符串代替str。

Answer 2

因为oldstr 只是一个指针，所以赋值不会复制你的字符串。

在将 str 传递给 strtok 之前复制它：

          char *oldstr=malloc(sizeof(str));
          strcpy(oldstr,str);

您的更正版本：

#include <stdio.h>
#include <string.h>
#include<malloc.h>
int main (void) {

   char str[] = "John|Doe|Melbourne|6270|AU";
   char fname[32], lname[32], city[32], zip[32], country[32];
   char *oldstr = malloc(sizeof(str));
   strcpy(oldstr,str);

    ...................
    free(oldstr);
return 0;
}

编辑：

正如@CodeClown 提到的，在您的情况下，最好使用strncpy。而不是事先固定fname 等的大小，您可以在它们的位置放置指针并根据需要分配内存，不多也不少。这样就可以避免越界写入缓冲区......

另一个想法： 将strtok 的结果分配给指针*fname、*lname 等，而不是数组。似乎strtok 被设计为在看到接受的答案后以这种方式使用。

注意：通过这种方式，如果您进一步更改str，这也会反映在fname,lname 中。因为，它们只是指向str 数据而不是新的内存块。因此，请使用oldstr 进行其他操作。

#include <stdio.h>
#include <string.h>
#include<malloc.h>
int main (void) {

    char str[] = "John|Doe|Melbourne|6270|AU";
    char *fname, *lname, *city, *zip, *country;
    char *oldstr = malloc(sizeof(str));
    strcpy(oldstr,str);
    fname=strtok(str,"|");
    lname=strtok(NULL,"|");
    city=strtok(NULL, "|");
    zip=strtok(NULL, "|");
    country=strtok(NULL, "|");

    printf("Firstname: %s\n", fname);
    printf("Lastname: %s\n", lname);
    printf("City: %s\n", city);
    printf("Zip: %s\n", zip);
    printf("Country: %s\n", country);
    printf("STR: %s\n", str);
    printf("OLDSTR: %s\n", oldstr);
    free(oldstr);
return 0;
}

Answer 3

strtok 需要一个可写的输入字符串，它会修改输入字符串。如果你想保留输入字符串，你必须先复制它。

例如：

char str[] = "John|Doe|Melbourne|6270|AU";
char oldstr[32];

strcpy(oldstr, str);  // Use strncpy if you don't know
                      // the size of str

Answer 4

您只需将指针复制到字符串，而不是字符串本身。使用strncpy() 创建副本。

char *oldstr = str; // just copy of the address not the string itself!