O que há de errado em meu código?
int fatorial (int n, int num, int fat)
{
if(num >= 0)
{
fat = 1;
n = num;
while(n > 0)
{
fat *= n; //FATORIAL = FATORIAL*N
n--;
}
}
}
Stack Overflow em Português é um site de perguntas e respostas para programadores profissionais e entusiastas. Leva apenas um minuto para se inscrever.
Inscreva-se para participar desta comunidadeO que há de errado em meu código?
int fatorial (int n, int num, int fat)
{
if(num >= 0)
{
fat = 1;
n = num;
while(n > 0)
{
fat *= n; //FATORIAL = FATORIAL*N
n--;
}
}
}
A função não está retornando nada, seja o valor válido ou não. Usei o -1 para indicar erro.
Além disso, está usando parâmetros completamente desnecessários: fat
pode ser local e n
simplesmente não precisa nem mesmo como variável local.
#include <stdio.h>
int fatorial(int num) {
if (num >= 0) {
int fat = 1;
while (num > 0) {
fat *= num;
num--;
}
return fat;
} else {
return -1;
}
}
int main(void) {
printf("%d\n", fatorial(0));
printf("%d\n", fatorial(1));
printf("%d\n", fatorial(5));
printf("%d\n", fatorial(-5));
}
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.
Por padrão, o C retorna 0 para uma função sem return, por exemplo a função int teste() { }
não tem erro de compilação, porque a linguagem C retorna zero como um return padrão. Então, a função se torna:
int teste() {
return 0; // código adicionado pelo compilador C
}
Na tua função:
int fatorial (int n, int num, int fat)
{
if(num >= 0)
{
fat = 1;
n = num;
while(n > 0)
{
fat *= n; //FATORIAL = FATORIAL*N
n--;
}
}
return 0; //adicionado pelo compilador
}