Я хочу найти факториал данного числа с помощью рекурсивной функции в C. Что не так со следующей программой?

Question

#include<stdio.h>

int fact(int i);
void main()
{
    int j;

    j=fact(4);

    printf("%d",j);
}

int fact(int i){
    int x=i;static int tot=1;

    if(x<1){
        tot=x*fact(x-1);
    }

    return tot;
}

Пожалуйста, помогите мне с этим кодом. Что заключено в этом коде?

Answer 1

if(x<1)

Вы уверены, что не имели в виду x > 1?

Кроме того, я бы избавился от static в вашем объявлении tot.Это относится к tot аналогично глобальной переменной.Тебе это не нужно.Поскольку tot всегда присваивается перед прочтением, похоже, что здесь это не вредно, но в целом это выглядит как красный флаг.

Answer 2

У вас нет базового условия в функции факта.

Вам необходимо проверить:

 if(i == 1){
    return 1;
 }else{

   return i * fact(i - 1);

}

Answer 3

Вы не хотите static в вашей декларации tot.

Answer 4

Вы ошиблись в операторе if, это должно быть

if(x > 1) {
   tot=x*fact(x-1);
}

EDIT: Кроме того, tot должно быть нестатичным.