0

我编写了一个简单的小程序来使用 snprintf 读取具有以下格式的文件，

skip first 15 chars , next 9 chars are sequence number, next 2 char is message and so on.

我对序列号和消息感兴趣，即从字符 16 到 26；

下面是程序。它不会读取每个字段的最后一个字符。它为序列号读取 8 个字符而不是 9 个字符，为消息读取 1 个字节而不是 2 个字节。

#include<stdio.h>

typedef struct
{
  char seqno[9];
  char msg[2];
}Header_T;

int main()
{
  char buf[64]={'\0'};
  FILE *fp;
  int i = 0;
  Header_T hdr1;
  int skipbytes = 15;

  fp=fopen("asdf", "r");
  if (fp == NULL)
  {
    printf("FILE OPEN ERROR\n");
  }
  printf("--sequence--msg--\n");
  while( fgets( buf, sizeof(buf), fp ) != NULL )
  {
    i=skipbytes;
    snprintf(hdr1.seqno, 9, "%s", (buf+i));
    i+=sizeof(hdr1.seqno);
    snprintf(hdr1.msg, 2, "%s", (buf+i));
    i=0;

    printf("--%s--%s--\n", hdr1.seqno, hdr1.msg);
    memset(buf, '\0', 64 );
  }
fclose(fp);
return 0 ;
}

部分文件内容如下 

201301082323458000000001H QB234
201301082323558000000002J QB234
201301082323658000000003N QB234
201301082323758000000004JRQB234
201301082333458000000010JSQB234

所以预期的输出是

--sequence--msg--
--000000001--H --
--000000002--J --
--000000003--N --
--000000004--JR--
--000000010--JS--

但相反，我得到的输出为

--seqno--msgtype--
--00000000--H--
--00000000--J--
--00000000--N--
--00000000--J--
--00000001--J--

谁能解释这种行为以及如何解决它？

我尝试使用for 循环逐个字符赋值而不是snprintf相同的程序，并且程序运行良好；但为此我需要为结构中的字节对齐添加一些填充符。

我也尝试过使用pragma pack()但它也没有区别。

我在 ubuntu 64 位机器上使用 gcc 4.4.3

4

4 回答 4

3

来自cppreference

int snprintf ( char * s, size_t n, const char * format, ... );

n：缓冲区中使用的最大字节数。

生成的字符串的长度最多为 n-1，为额外的终止空字符留出空间。

因此，如果您期望 9 个字符，则应传递n为10，而不是9：

snprintf(hdr1.seqno, 10, "%s", (buf+i));

Header_T需要进行相应的更改：

typedef struct
{
  char seqno[10];
  char msg[3];
}Header_T;
于 2013-08-04T07:00:19.830 回答
1 
#include<stdio.h>

typedef struct {
    char seqno[9+1];//+1 for EOS('\0')
    char msg[2+1];
} Header_T;

int main(void){
    char buf[64]={'\0'};
    FILE *fp;
    Header_T hdr1;

    fp=fopen("asdf", "r");
    if (fp == NULL) {
        printf("FILE OPEN ERROR\n");
        return 1;
    }
    printf("--sequence--msg--\n");
    while( fgets( buf, sizeof(buf), fp ) != NULL ){
        sscanf(buf, "%*15c%9c%2c", hdr1.seqno, hdr1.msg);
        hdr1.seqno[sizeof(hdr1.seqno)-1] = hdr1.msg[sizeof(hdr1.msg)-1] = '\0';
        printf("--%s--%s--\n", hdr1.seqno, hdr1.msg);
        //memset(buf, '\0', 64 );
    }
    fclose(fp);
    return 0 ;
}
于 2013-08-04T08:17:16.850 回答
1

你需要有一个额外的字符来存储尾随'\0'字符，所以你的结构应该看起来像

typedef struct
{ char seqno[10];
  char msg[3];
}Header_T;

编辑：因为这是一个简单的数据传输，为什么不使用memcpy，比如：

memset(hdr1, 0, sizeof(hdr1));
memcpy(hdr1.seqno, &buf[skipbytes], 9);
memcpy(hdr1.msg, &buf[skipbytes + 9], 2);
于 2013-08-04T07:02:29.400 回答
0

这个anwser是在上面两个anwser之后。他们都给出了一些正确的建议。我只是把它们拼接在一起。

首先，使用： 

typedef struct
{ char seqno[10];
  char msg[3];
}Header_T;       //so we have the memory for '\0'

然后，使用：

   while( fgets( buf, sizeof(buf), fp ) != NULL )
   {
        i=skipbytes;
        snprintf(hdr1.seqno, 10, "%s", (buf+i));//here we read 9 number and a '\0'
        i+=sizeof(hdr1.seqno) -1;               //the  '\0' is not part of buffer.
        snprintf(hdr1.msg, 3, "%s", (buf+i));   //also for '\0'
        i=0;

        printf("--%s--%s--\n", hdr1.seqno, hdr1.msg);
        memset(buf, '\0', 64 );
   }
于 2013-08-04T07:31:18.897 回答