我正在尝试扫描C
:中的以下格式的文本文件
key01 value01
key02 value02
key03 value03
我想把键和值字符串分配给我创建的结构:
struct Mapping
{
char key[sizeof(char) * 5];
char value[sizeof(char) * 7];
} MapElement;
我的代码如下:
void initialize_server_map(struct Mapping serverMap[])
{
char* fileName = "server1.txt";
FILE *fp = fopen(fileName, "r");
char *lineKey = (char*)malloc(sizeof(char) * 5);
char *lineValue = (char*)malloc(sizeof(char) * 7);
int i = 0;
while (fscanf(fp, "%s %s", lineKey, lineValue) != EOF) {
struct Mapping mapping;
memcpy(mapping.key, lineKey, sizeof(char) * 5);
memcpy(mapping.value, lineValue, sizeof(char) * 7);
printf("key: %s value: %sn", mapping.key, mapping.value);
serverMap[i] = mapping;
i++;
}
fclose(fp);
}
int main(void)
{
int size = 4;
struct Mapping serverMap[size];
initialize_server_map(serverMap);
}
我的输出如下:
key: key01value01 value: value01
key: key02value02 value: value02
key: key03value03 value: value03
key: key04value04 value: value04
当然,我想要这个输出:
key: key01 value: value01
key: key02 value: value02
key: key03 value: value03
key: key04 value: value04
我确信这与我分配内存的方式有关。有人能帮我解决这个问题吗?此外,有没有更有效的方法来做到这一点?
由于数组的越界访问,您遇到了未定义的行为。
为了存储"key01"
,您需要一个大小至少为6
-5
的字符数组和一个用于终止null字符的数组。
类似地,您需要一个大小至少为8
的数组来存储"value01"
。
您声明的数组大小分别为5
和7
。因此,当在中使用时,您正在修改超出范围的数组
while (fscanf(fp, "%s %s", lineKey, lineValue) != EOF) {
好吧,至少您已经忘记了字符串中尾部"\0"的空格。因此,如果您有5个字符的字符串,您应该分配6个字符。
但总体而言,整个方法相当脆弱。如果输入数据中的键长度超过5个字符,该怎么办?传统的方法是为scanf
使用"足够大"的缓冲区(比如几百字节),使用格式中的长度限制器(例如"%20s"而不是"%s")限制scanf
中可能的字符串长度,然后通过strdup
分配所需的空间。
所以整个区块看起来像:
struct Mapping {
char* key;
char* value;
};
...
char lineKey[51];
char lineValue[51];
while (fscanf(fp, "%50s %50s", lineKey, lineValue) != EOF) {
struct Mapping* mp = (struct Mapping*)malloc(sizeof(struct Mapping));
assert(mp);
mp->key = strdup(lineKey);
mp->value = strdup(lineValue);
...
}