Existem vários erros no seu código e, como você não especificou nenhum em específico, vamos passo a passo por cada um.
- Protótipo de funções declarados dentro o corpo da função main
Você está declarando o protótipo das funções lerdados
e printdados
dentro do corpo da função principal (main
).
Apesar alguns compiladores aceitarem esta declaração, o mais indicado é declarar os protótipos em um lugar antes da função main
e não dentro dela.
- Passagem de parâmetros para funções
A declaração void lerdados(aluno dados[qtdalunos])
diz que você vai passar o array por valor, o que vai gerar problemas nas alocações e liberações futuras para este vetor. O correto aqui seria passar o array por referência e adicionar um parâmetro extra informando a quantidade de posições que existem no mesmo. A sintaxe ficaria como no exemplo abaixo:
void lerdados(aluno dados[], int qtde);
void printdados(aluno dados[], int qtde);
- Alocação de memória fora do loop
aluno* alunos = (aluno*)malloc(sizeof(aluno)*100);
alunos->nome = (char*)malloc(50);
alunos->matricula = (char*)malloc(20);
alunos->nascimento = (char*)malloc(10);
alunos->rg = (char*)malloc(20);
lerdados(classe);
Você declara um array de 100 posições, inicializa a primeira posição e chama a função lerdados
que vai tentar, potencialmente, ler n posições, mas passa o vetor por valor e não por referência. Além disto, se você vai ler os dados dentro do loop, cada aluno deveria ser alocado, preferencialmente, por aqui também.
void lerdados(aluno dados[qtdalunos]){
printf("\t\t Nome \\ Matricula \\ Data de Nascimento \\ RG ");
for(i=0; i<qtdalunos; i++){
printf("\nInforme os dados do aluno %d:\n", i+1);
scanf("%s %s %s %s", &dados->nome, &dados->matricula, &dados->nascimento, &dados->rg);
}
- Leitura dos dados
Aqui você tentar ler 4 strings, mas a sintaxe está errada.
scanf("%s %s %s %s", &dados->nome, &dados->matricula, &dados->nascimento, &dados->rg)
Veja, o %s
precisa de um endereço de buffer para colocar o texto lido pelo scanf
. A sintaxe usada (&dados->nome
, por exemplo) vai retornar sempre o endereço do primeiro registro. Você teria que levar em conta o índice do loop. Por exemplo:&dados[i].nome
.
- liberação de memória fora do loop
A mesma coisa das funções anteriores, você está liberando somente 1 item do vetor. O programa termina deixando os outros 99 ponteiros para alunos ainda alocados.
Eu sei que é um bocado de coisa e a linguagem C pode confundir um pouco no início, mas não desanime. Segue uma versão funcional do código com todas as correções sugeridas acima.
Bons estudos.
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <locale.h>
#define QT_ALUNOS 2
#define MAX_ALUNOS 100
typedef struct {
char* nome;
char* matricula;
char* nascimento;
char* rg;
}aluno;
void lerdados(aluno dados[], int qtde);
void printdados(aluno dados[], int qtde);
void liberardados(aluno dados[], int qtde);
int main() {
setlocale(LC_ALL, "pt-BR"); //para funcionar acentos e o ç
aluno* classe = (aluno*) malloc(sizeof(aluno) * MAX_ALUNOS);
lerdados(classe, QT_ALUNOS);
printdados(classe, QT_ALUNOS);
liberardados(classe, QT_ALUNOS);
getchar();
return 0;
}
void alocar_aluno(aluno *a) {
a->nome = (char*)malloc(50);
a->matricula = (char*)malloc(20);
a->nascimento = (char*)malloc(10);
a->rg = (char*)malloc(20);
}
void liberar_aluno(aluno* a) {
free(a->nome);
free(a->matricula);
free(a->nascimento);
free(a->rg);
a->nome = NULL;
a->matricula = NULL;
a->nascimento = NULL;
a->rg = NULL;
}
void lerdados(aluno dados[], int qtde){
int i;
for (i = 0; i < qtde; i++) {
aluno *item = &dados[i];
alocar_aluno( item );
printf("Informe os dados do aluno %i de %i:\n", i+1, qtde);
printf("\tNome : "); scanf("%s", item->nome);
printf("\tMatrícula : "); scanf("%s", item->matricula);
printf("\tDt. Nascimento : "); scanf("%s", item->nascimento);
printf("\tRG : "); scanf("%s", item->rg);
printf("\n");
}
}
void printdados(aluno dados[], int qtde) {
int i;
printf("Listando os %i alunos da classe:\n", qtde);
for (i = 0; i < qtde; i++) {
printf("\t%i\tNome : %s, Matríula: %s, Dt. Nascimento: %s, RG: %s\n ", i,
dados[i].nome,
dados[i].matricula,
dados[i].nascimento,
dados[i].rg);
}
}
void liberardados(aluno dados[], int qtde) {
int i;
for (i = 0; i < qtde; i++) {
liberar_aluno(&dados[i]);
}
}