5

我有以下代码：

#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

为什么我不能保留旧数据也不在stroroldstr中，我做错了什么，我怎么能不更改或保留数据？

4

4 回答 4

29

当您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'和0diffident（在字符串 6270 中，图中的 char 用括号括起来，'其中\00 是数字）

当您使用它打印 str 时，首先%s打印字符\0John

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

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

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

于 2013-06-14T09:11:26.683 回答
3

因为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. 而不是事先固定fnameetc 的大小，您可以在它们的位置放置指针并根据需要分配内存，不多也不少。这样你就可以避免越界写入缓冲区......

另一个想法：将结果分配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;
}
于 2013-06-14T09:12:41.593 回答
1

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
于 2013-06-14T09:10:26.010 回答
0

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

char *oldstr = str; // just copy of the address not the string itself!
于 2013-06-14T09:09:48.187 回答