Question

此函数将回退文件，创建动态数组（大小），并读入数据，填充_data结构动态数组。注意流这次是通过值传递的。然后该函数返回填充的 struct数组

struct _data
{
    char* name;
    long number;
};

struct _data *load(FILE *stream, int size)
{
    struct _data BlackBox = calloc(size, sizeof(_data));

    char tempName[3];

    stream = fopen("names.txt", "r");
    for (int i=0; i<size; i++)
    {
        fscanf(stream, "%s %ld", tempName, &data.number);
        BlackBox[i].name = calloc(strlen(tempName), sizeof(char));
        strcpy(BlackBox[i].name, tempName);
    }
    fclose(stream);

    return &BlackBox;
}

File Content
ron 7774013
jon 7774014

我是初学者，设计代码有困难。有人可以解释一下。感谢

Answer 1

我认为你有一些gcc的警告可以帮助你。

使用calloc修复内存管理，不要返回堆栈指针

typedef struct _data                                                           
{                                                                              
    char* name;                                                                
    long number;                                                               
} _data;                                                                       

_data *load(FILE *stream, int size)                                            
{                                                                              
    _data *BlackBox = calloc(size, sizeof(_data));                             

    char tempName[3];                                                          

    for (int i=0; i<size; i++)                                                 
    {                                                                          
        fscanf(stream, "%s %ld", tempName, &BlackBox[i].number);               
        BlackBox[i].name = strdup(tempName);                                   
    }                                                                          
    fclose(stream);                                                            

    return BlackBox;                                                           
}                                                                              

int main (void)                                                                
{                                                                              
    FILE *f = fopen("test.data", "r");                                         
    _data *data = load(f, 2);                                                  
    printf("%s %ld\n", data[0].name, data[0].number);                          
    printf("%s %ld\n", data[1].name, data[1].number);                          
    return 0;                                                                  
}

输出

 aurel@vm-pontarlier:~$ ./a.out 
ron 7774013
jon 7774014

考虑改变_data

typedef struct _data{
    char name[256];
    long number;
} _data;

扫描将是：

for (int i=0; i<size; i++)                                                 
{                                                                          
    fscanf(stream, "%s %ld", BlackBox[i].name, &BlackBox[i].number);       
}

Answer 2

您正在制作与之前帖子相同的视频。您也不为name中的_data成员分配内存。至于编译错误：

您使用T或malloc动态分配的calloc类型的数组由句柄控制，指针指向T类型的T*。< / LI>
您定义的结构类型为struct _data，类型包含关键字struct。如果你想要一个单字标识符，你可以使用`typedef'作为Ôrel给你看过的。
您没有名为'dat . The handle to the newly allocated memory is ´BlackBox的变量。

如果您修复了这些错误，您将遇到编译器无法了解的逻辑错误：

您的BlackBox[i].name已通过NULL初始化为calloc，因此未指向有效内存。你不能strcpy任何东西。你可以做的是使用非标准但广泛可用的strdup，它首先根据需要分配内存，然后复制。
您的临时字符串长度为3;它最多可以包含两个字符加上null终止符。您的文件中的名称很短，已被授予，但您的程序必须为任何输入做准备，甚至是非法输入。
FILE *不在您的功能之外使用;它应该是load。
用户必须指定要读取的项目数。这违背了动态分配的目的。用户也不知道读了多少项;根据请求，文件中的项目可能会更少。你的功能设计应该反映出来。（好吧，这不是真的：你初始化内存，这意味着NULL的char指针指示数组的结尾，但是：）
您不测试fscanf的输出，您应该这样做。如果您的文件少于size项，则最后一项将包含垃圾，因为您将垃圾值复制到零初始化内存中。
本身不是错误，但如果您的文件是按行排列的，请先考虑先阅读fgets行，然后使用sscanf进行解析。

下面的示例代码尝试合并这些指南。注意事项：

内存未分配到一个巨大的块中，但随着后续调用realloc，它会根据需要增长。来电realloc(NULL, s)相当于malloc(s)。
函数通过指向整数的指针“填充”元素的数量。
将字符串复制到新分配的内存中。这意味着理论上，字符串只要他们想要就可以。（实际上，最大缓冲区长度为20会减少此类字符串的缩短。）函数duplicate模拟函数strdup。
清理时这些字符串应为free。
load函数采用文件名而不是文件句柄。
文件读取分两个阶段进行：首先读取一行，然后扫描该行。如果格式不是“string without spaces”+“number”，则会写入错误消息。

无论如何，这里是：

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

struct _data
{
    char *name;
    long number;
};

/*
 *      Duplicate a string on the heap (aka strdup)
 */
char *duplicate(const char *str)
{
    char *p = malloc(strlen(str) + 1);

    strcpy(p, str);
    return p;
}

/*
 *      Read a list of names from file fn
 */
struct _data *load(const char *fn, int *psize)
{
    struct _data *data = NULL;
    FILE *stream;
    char line[80];      // buffer for line
    int lnr = 0;        // line number for error message
    int n = 0;          // number of read items

    stream = fopen("names.txt", "r");
    if (stream == NULL) return NULL;    // Can't open file

    while (fgets(line, sizeof(line), stream)) {
        long int number;
        char buf[20];

        lnr++;
        if (sscanf(line, "%19s %ld", buf, &number) == 2) {
            data = realloc(data, (n + 1) * sizeof(*data));
            data[n].number = number;
            data[n].name = duplicate(buf);
            n++;
        } else {
            fprintf(stderr, "[%s, line %d] Illegal format\n", fn, lnr);
        }
    }
    fclose(stream);
    *psize = n;             // Assign number of read items

    return data;
}

/*
 *      Free memory allocated by load
 */
void cleanup(struct _data *data, int n)
{
    while (n--) free(data[n].name);
    free(data);
}

int main()
{
    struct _data *data;
    int i, n;

    data = load("names.txt", &n);
    if (data == NULL) return -1;

    for (i = 0; i < n; i++) {
        printf("%-20s%12ld\n", data[i].name, data[i].number);
    }

    cleanup(data, n);

    return 0;
}

Answer 3

...最后tempName [3]太小了：给定3个字母的输入，它必须至少为4。此外，您忘记为malloc调用中的终止空字符分配空间：

char tempName[4];
...
BlackBox[i].name = malloc(strlen(tempName)+1);

（tempName [3]没有导致错误是因为编译器可能将其四舍五入到偶数个字节 - 但这是典型的初学者错误。）

倒回文件，创建动态结构

3 个答案: