Acontece que você não inicializou os ponteiros. Tal como o proprio nome indica, eles são ponteiros, logo tem que apontar para algum local na memória que é onde são colocados os valores.
Logo quando faz:
*area = PI * pow(R, 2.0);
Está a dizer, no local de memoria onde o ponteiro area
aponta, coloque este valor. Mas qual local é esse ? area
não foi atribuído sequer, e por isso isto resulta em comportamento indefinido e potencialmente segmentation fault.
O próprio compilador lhe ajuda nesse sentido. Veja o que eu obtenho quando compilo o seu código:
||=== Build: Debug in Test (compiler: GNU GCC Compiler) ===|
... main.c|14|warning: 'area' is used uninitialized in this function [-Wuninitialized]|
... main.c|14|warning: 'volume' is used uninitialized in this function [-Wuninitialized]|
||=== Build finished: 0 error(s), 2 warning(s) (0 minute(s), 0 second(s)) ===|
Como corrige ? Aloca espaço para o ponteiro, que seria o mais normal:
float *area = malloc(sizeof(float));
float *volume = malloc(sizeof(float));
Agora os ponteiros já apontam para um local válido na memória, mesmo que não tenham nenhum valor atribuído por si. Isto já irá fazer as atribuições seguintes funcionarem corretamente. Não se esqueça que deve de incluir <stdlib.h>
para utilizar o malloc
.
Outra alternativa, seria criar variáveis para a area
e volume
como valores e passar os seus endereços à função:
float area, volume;
...
calc_esfera(raio, &area, &volume);
// ^------^-- aqui passa o endereço das variáveis no main
Esta ultima solução até seria preferível pois evita a alocação no heap, que causa mais fragmentação de memória e é mais lenta. Também evita com que tenha de preocupar em liberar a memória com free
quando não precisar mais dela.
scanf("%f", &raio);
após o últimoprintf
, não resolve??raio
pois oscanf
não funciona nesse site