c - C中文件读取不正确

问题描述

我正在为医院 chanelling 做一个套接字编程。我正在尝试读取这样的文本文件

1   Kavi    card    0   0   0
2   Anparasanesan   gene    0   0   0
3   Thilak  card    0   0   0
4   Akilanesan  immu    0   0   0
5   Aravinthanesan  derm    0   0   0
6   Akalya  derm    0   0   0
7   Vishvapriya derm    0   0   0
8   Kavinga immu    0   0   0
9   Anjalie andr    0   0   0
10  Tom andr    0   0   0

但是当我阅读该文件时，它给我的输出为：

1   Kavi    cardgenecardimmudermdermdermimmuandrandr
2   Anparasanesan   genecardimmudermdermdermimmuandrandr
3   Thilak  cardimmudermdermdermimmuandrandr
4   Akilanesan  immudermdermdermimmuandrandr
5   Aravinthanesan  dermdermdermimmuandrandr
6   Akalya  dermdermimmuandrandr
7   Vishvapriya dermimmuandrandr
8   Kavinga immuandrandr
9   Anjalie andrandr
10  Tom andr

这是我的代码段

char line[MAXCHAR];
        int x = 0;
            while (fgets(line, sizeof(line), fp)){ 
            sscanf(line,"%d\t%s\t%s\t%d\t%d\t%d",&dno,&dname,&dspl,&ti1,&ti2,&ti3);

            id[x]=dno;
            strncpy(name[x], dname, 50);
            strncpy(spl[x], dspl, 4);
            times[x][0]=ti1;
            times[x][1]=ti2;
            times[x][2]=ti3;

            x++;
            }

        int z=0;
        for(z=0;z<10;z++)
        {
            snprintf(line, sizeof(line),"%d\t%s\t%s\n",id[z],name[z],spl[z]);
            n = strlen(line);
            Writen(sockfd,line,n);
        }

标签： cfilefile-handling

让我们来看看其中的一个问题。

邪恶的strncpy

代码使用strncpy幻数4。这并不能确保spl[x]是字符串，因为字符可能缺少最终的null 字符。

        strncpy(spl[x], dspl, 4);  // Avoid code like this

后面的代码尝试用 and 打印一个字符串，并得到“cardgene ...”而不是预期的“card”。当不是字符串时，结果是未定义的行为(UB) - 任何事情都可能发生。"%s"spl[z]spl[z]

        // Alternative: could limit output with
        snprintf(line, sizeof(line),"%.*s\n",(int) (sizeof spl[z]), spl[z]);

怎么修？

不要使用sscanf(line,"%s",&dspl);，因为它没有宽度限制，或者不知道它dspl的大小与line. 我期待

char dspl[4+1];
sscanf(line,"%4s", dspl);

最好确保源字符串和目标数组是足够的，而不是在strncpy()没有测试的情况下使用。

char spl[X_N][4+1];
char dspl[sizeof spl[0]];

// strncpy(spl[x], dspl, 4);
strcpy(spl[x], dspl);

其他修复包括确保sscanf()按预期完成。一种简单的方法用于" %n"记录扫描偏移量，如果它到达那么远，然后寻找额外的垃圾。不必要"\t"的删除。

// Insure  dname is at least 50+1, dspl is 4+1 or as big as the line
char dname[sizeof line];
char dspl[sizeof line];

// sscanf(line,"%d\t%s\t%s\t%d\t%d\t%d",&dno,&dname,&dspl,&ti1,&ti2,&ti3);
int n = 0;
sscanf(line,"%d%50s%4s%d%d%d %n",&dno,dname,dspl,&ti1,&ti2,&ti3, &n);
if (n==0 || line[n]) {
  puts("Bad Input");  // Handle bad input in some fashion
  exit(RETURN_FAILURE);
}

c - C中文件读取不正确

问题描述

解决方案

推荐阅读