Não é nenhum absurdo.
Em bibliotecas de função, é muito comum ter rotinas que cuidam de alocação e desalocação de espaço de memória, principalmente para estruturas mais complexas.
O que está errado aqui é o seu entendimento sobre a passagem de ponteiro por argumento de função. Vou explicar.
Na função main()
, você declarou um ponteiro vet
para um inteiro, mas esse ponteiro ainda não foi inicializado:
int *vet, tam, *p;
Já a função alocavet()
espera receber o endereço de um espaço de memória por meio de um ponteiro, também de nome vet
:
void alocavet(int tam, int *vet)
Então, você chama alocavet()
em main()
e passa para o vet
de alocavet()
um endereço inválido, conteúdo que estava lá no espaço de memória da variável não inicializada.
Mas isso não importa muito. Logo na primeira linha de alocavet()
, o endereço inválido recebido por vet
é substituído pelo endereço do novo espaço de memória retornado por malloc()
.
void alocavet(int tam, int *vet){
vet = (int*) malloc (tam * sizeof(int));
}
Quando alocavet()
retorna, o ponteiro vet
em main()
continua apontando para coisa alguma, pois ainda não foi inicializado. E aí, no primeiro loop, você usa esse endereço inválido em scanf
, o que faz o programa quebrar.
O que aconteceu com a alocação feita por alocavet()
?
Ainda está lá, mas perdeu a única referência que tinha. O ponteiro vet
de alocavet()
foi destruído quando a função retornou.
Eu sei. Você esperava que alocavet()
fosse alterar o conteúdo apontado por vet
de main()
.
Um jeito de resolver isso é usando ponteiro de ponteiro:
void alocavet(int tam, int **vet){
*vet = (int*) malloc (tam * sizeof(int));
}
Desta forma, vet
de main()
continuaria sendo declarado do mesmo jeito, como um ponteiro para um inteiro:
int *vet;
E você passaria o endereço deste ponteiro vet
usando o operador &
:
alocavet(tam, &vet);
Isso é suficiente para inicializar o ponteiro em main()
e, além disso, obter o retorno de malloc()
, para checar caso a alocação seja malsucedida, como disseram aqui, e também liberar memória depois de usada.
Veja como ficou:
#include <stdio.h>
#include <stdlib.h>
void alocavet(int tam, int **vet) {
*vet = (int *) malloc ( tam * sizeof(int));
}
void desalocavet(int **vet) {
if (*vet) {
free(*vet);
*vet = 0;
}
}
int main(){
int *vet, tam, *p;
printf("Tamanho do vetor: ");
scanf("%d", &tam);
alocavet(tam, &vet);
if (vet == NULL) abort();
p = vet;
printf("Entre com os elementos do vetor: ");
for (int c=0; c<tam; c++) scanf("%d", &*p++);
p = vet;
printf("Elementos do vetor: ");
for (int c=0; c<tam; c++) printf("%d ", *p++);
desalocavet(&vet);
}