2

我有一个函数可以解析一些传入的数据。我的问题是，在使用 strncpy 之后，当我尝试打印它时会得到一些垃圾。我尝试使用 malloc 使 char 数组具有确切的大小。

代码：

void parse_data(char *unparsed_data)
{
char *temp_str;
char *pos;
char *pos2;
char *key;
char *data;
const char newline = '\n';

int timestamp = 0;

temp_str = (char*)malloc(strlen(unparsed_data));

g_print("\nThe original string is: \n%s\n",unparsed_data);


//Ignore the first two lines
pos = strchr(unparsed_data, newline);
strcpy(temp_str, pos+1);
pos = strchr(temp_str, newline);
strcpy(temp_str, pos+1);

//Split the line in two; The key name and the value
pos = strchr(temp_str, ':'); // ':' divides the name from the value
pos2 = strchr(temp_str, '\n'); //end of the line
key = (char*)malloc((size_t)(pos-temp_str)-1); //allocate enough memory
data = (char*)malloc((size_t)(pos2-pos)-1);

strncpy(key, temp_str, (size_t)(pos-temp_str));
strncpy(data, pos + 2, (size_t)(pos2-pos));

timestamp = atoi(data);

g_print("size of the variable \"key\" = %d or %d\n", (size_t)(pos-temp_str), strlen(key));
g_print("size of the variable \"data\" = %d or %d\n", (size_t)(pos2-pos), strlen(data));

g_print("The key name is %s\n",key);
g_print("The value is %s\n",data);
g_print("End of Parser\n");
  }

输出：

The original string is: 
NEW_DATAa_PACKET
Local Data Set 16-byte Universal Key
Time Stamp (microsec): 1319639501097446
Frame Number: 0
Version: 3
Angle (deg): 10.228428

size of the variable "key" = 21 or 22
size of the variable "data" = 18 or 21
The key name is Time Stamp (microsec)
The value is 1319639501097446
F32
End of Parser

再次运行它：

  The original string is: 
  NEW_DATAa_PACKET
  Local Data Set 16-byte Universal Key
  Time Stamp (microsec): 1319639501097446
  Frame Number: 0
  Version: 3
  Angle (deg): 10.228428

  size of the variable "key" = 21 or 25
  size of the variable "data" = 18 or 18
  The key name is Time Stamp (microsec)ipe 
  The value is 1319639501097446
  F
  End of Parser
4

5 回答 5

5

您的结果是因为strncpy没有在字符串末尾放置空字符。

于 2012-06-22T17:57:52.337 回答
2

您需要 malloc() +1 字节的字符串，因此当您执行 strcpy() 时它可以附加零，但 strncpy 也不会附加零，因为您需要一个额外的字节。

于 2012-06-22T18:01:21.693 回答
2

您strncpy(data, pos + 2, (size_t)(pos2-pos));没有\0在字符串末尾添加终止字符。因此，当您稍后尝试打印它时，printf()打印您的整个数据字符串以及紧随其后的内存中的任何内容，直到它达到零 - 这就是您得到的垃圾。您需要在数据末尾显式附加零。也需要它atoi()。

编辑： 您需要为您的 再分配一个字节data，并在那里写一个终止字符。data[len_of_data] = '\0'. 只有在那之后，它才成为一个有效的 C 字符串，并且您可以将它用于atoi()and printf()。

于 2012-06-22T18:06:25.733 回答
1

一个问题：如果没有换行符怎么办？

未定义的行为：

pos = strchr(temp_str, newline);
strcpy(temp_str, pos+1);

的来源和目的地strcpy不得重叠。

于 2012-06-22T17:59:42.413 回答
0

您必须记住在为字符串分配空间时为终止 '\0' 字符添加一个字节。您必须小心使用strncpy，尤其是如果您习惯使用strcpy、strcat或sprintf。这三个函数以 '\0' 结束字符串。strncpy复制您指定的多个字节，并且不假设终止字符串。

您通过确保在复制到的字符缓冲区的末尾放置一个“\0”来承担该责任。这意味着您必须知道副本的起始位置和长度，并在起始位置和长度之和之后放置一个 '\0' 字节。

我选择稍微不同地解决示例问题，但它仍然涉及知道我复制的内容的长度。

在这种情况下，我使用strncpy了前 9 个字符pcszTestStr1并将它们复制到 szTestBuf。然后，我使用 strcpy（以零结尾的字符串）来附加句子的新部分。

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

int n;
int argv_2;

char szTestBuf[100] = {0};
char * pcszTestStr1 = 
"This is a very long, long string to be used in a C example, OK?";

int main(int argc, char *argv[])
{
    int rc = 0;

    printf("The following sentence is too long.\n%s\n", pcszTestStr1);
    strncpy(szTestBuf, pcszTestStr1, 9);
    strcpy(szTestBuf + 9, " much shorter sentence.");
    printf("%s\n", szTestBuf); 

    return rc;
}

这是运行 test.c 的输出gcc -o test test.c。

cnorton@hiawatha:~/scratch$ ./test
The following sentence is too long.
This is a very long, long string to be used in a C example, OK?
This is a much shorter sentence.
cnorton@hiawatha:~/scratch$
于 2012-06-23T16:48:50.260 回答