为什么我的代码之间的零打印?

时间:2014-02-06 20:05:47

标签: c io printf fgets

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


#define MAX_KEYS 26
#define MAX_UID 4

struct Key {
    int keynum;
    int id;
    char cryptkeys[65];
};



int main(int argc, char ** argv) {
int MAX_LINE = 69;
struct Key *table = malloc(MAX_UID * sizeof(struct Key));

FILE *fileopen = fopen(filename, "r");
char *a[8];
int size = 0;
char *e;
char *string = NULL;
//reading the file
if(fileopen) {
        while((e =  fgets( line, MAX_LINE, fileopen )) != NULL) {
            printf(e);
            a[size] = strdup(e);
            size++;
        }

}
else {
    printf("File does not exist\n");
    exit(0);
}
for(int i = 0; i < size; i++ ) {
    //printf("%s", a[i]); //no zero showing up

}
//parsing the id and the keys
char id[3];
char key[65];
int ids;
int idnum;
for(int i = 0; i < size-1; i++) {
    struct Key *k = (struct Key*)malloc(sizeof(struct Key));
    string = a[i];
    //set the ID
    for(int ix = 0; ix < 3; ix++) {
        id[ix] = string[ix];
    }
    //printf("%s", id); No zero inbetween ID
    //set the Keys
    for(int j = 4; j < 68; j++) {
        key[j-4] = string[j];
    }
    ids = strtol(string, &id2, 10);
    //printf("Id is %d", ids); //zero shows up here
    //printf("   ");
    k->id = ids;
    strcpy(k->cryptkeys, key);
    k->keynum++;
    table[i] = *k;
    idnum++;
}
for(int i = 0; i < 5; i++) {
    printf("%d", table[i].id);
    printf(" ");
    printf("%s", table[i].cryptkeys);
    printf("\n");

}
return 0;
}
嘿伙计们,我正在尝试在我的结构的指针数组中进行结构操作。我可以很好地添加这些值,但我仍然在我的行之间出现这个0。我的文件只有三行,后面有一个整数和一个字符串。一切似乎都正确解析,但零仍然出现在我的文本中。

如果您想知道

,我的文件会是这样的

421 0123456789abcdef0123456789abcde00123456789abcdef0123456789abcde0

422 0000000000000000000000000000000000000000000000000000000000000000

423 1111111111111111111111111111111111111111111111111111111111111111

当我在阅读表格后尝试打印输出时,它看起来像这样。

421 0123456789abcdef0123456789abcde00123456789abcdef0123456789abcde0

0

422 0000000000000000000000000000000000000000000000000000000000000000

0

423 1111111111111111111111111111111111111111111111111111111111111111

我不认为它来自字符串的后面,因为每个键都是合适的长度。有关它可能来自何方的任何建议?任何帮助将不胜感激。

2 个答案:

答案 0 :(得分:0)

主要问题是您使用strcpy(k->cryptkeys, key);,其中key不是以空值终止的。您只能写入前64个元素,并将最后一个元素保持未初始化状态。

但也存在一些普遍问题,例如:你正在泄漏内存,因为你在循环中调用malloc而没有释放(你可以在没有使用malloc的情况下编写程序就好了)并且你不必将整个文件读入内存。

答案 1 :(得分:0)

函数fgets将读取字符串行中的行,直到读取MAX_LINE字符或它到达新行。 因此,您可以首先分配具有足够内存的行字符串来读取行。在这种情况下,它是MAX_LINE。

char line[MAX_LINE];
char *a[8];
while((e =  fgets( line, MAX_LINE, fileopen )) != NULL) 
{
   printf(line);
   a[size] = strdup(line);
   size++;
 }