0

我正在尝试从网络上的教程中学习 C,我想出了这个简单的代码来尝试理解指针和数组的内存分配。

size如果<= 2,则代码可以完美编译和运行,但如果size> 2,则会出现分段错误。有人可以说明如何正确执行此操作吗?

谢谢你。


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


int main () {


int i, size;

printf("Quantos registos pretende inserir? ");
scanf("%d",&size);
getc(stdin);

typedef struct {
    char nome[81];
    int idade;
    char cargo[81];
} Dados;

Dados *data[(size-1)];

data[(size-1)] = (Dados *)malloc(sizeof(Dados));


for(i=0;i<size;i++) {
    printf("\nInsira os dados do funcionário: ");
    printf("\n\n\tNome: ");
    gets(data[i]->nome);
    printf("\n\tIdade: ");
    scanf("%d",&data[i]->idade);
    getc(stdin);
    printf("\n\tCargo: ");
    gets(data[i]->cargo);

    FILE *fdados;
    if(!(fdados = fopen("dados.txt","a+"))) {
         printf("Impossivel aceder ao ficheiro, verfique o erro ocorrido ...");
    }

    fprintf(fdados, "Funcionário %d:",(i+1));
    fprintf(fdados, "\n\n\tNome: %s",data[i]->nome);
    fprintf(fdados, "\n\tIdade: %d",data[i]->idade);
    fprintf(fdados, "\n\tCargo: %s\n\n",data[i]->cargo);

    fclose(fdados);

}

free(data[(size-1)]);

fflush(stdin);
return(0);
}
4

3 回答 3

2

你一次性给自己太多的事情要做。分解问题并测试每个组件:

  1. 解析用户输入:产生一个有效的size_t整数或中止。检查该值既不是零也不是太大,或者中止。

  2. 假设您已经解析了 value nn为结构的副本分配内存:

    Dados * data = malloc(n * sizeof(Dados));
    

    最后,释放内存:

    free(data);
    
  3. 使用带有错误处理的正确解析来填充每个数组成员data[i]

  4. 分别练习文件操作。

于 2013-08-13T21:51:04.120 回答
1

好吧,您有一个结构,并且您似乎想在数组中创建该结构的多个副本。所以你要做的是这个

Dados *data;
data = (Dados *) malloc(sizeof(Dados) * size);

现在您在数据中拥有大小数量的 Dados 结构。

然后最后确保释放你的内存(数据);

于 2013-08-13T22:13:36.863 回答
0

消除 :getc(stdin);

Dados *data[(size-1)];

data[(size-1)] = (Dados *)malloc(sizeof(Dados));// here is wrong . 
                                                  you have data[size-1]
                                             the max index you can use is size-2



for(i=0;i<size;i++) // i < size-1 , since you use data[i] in loop ,

free(data[(size-1)]); // free (data);  just free the pointer you get.
于 2013-08-13T22:07:37.197 回答