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

Question

#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 00000000000000000000000000000000000000000000000000000000000000000

423 111111111111111111111111111111111111111111111111111111111111111

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

421 0123456789abcdef0123456789abcde00123456789abcdef0123456789abcde0

0

422 00000000000000000000000000000000000000000000000000000000000000000

0

423 111111111111111111111111111111111111111111111111111111111111111

我不认为它来自字符串的后面，因为每个键都是适当的长度。关于它可能来自什么的任何建议？任何帮助将不胜感激。

Answer 1

函数 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++;
 }

Answer 2

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

但也有一些一般性问题，例如您正在泄漏内存，因为您在循环中调用 malloc 而没有释放（您可以在不使用 malloc 的情况下很好地编写程序）并且您不必要地将整个文件读入内存。