факториал в с++

[sp.caster]

Всем привет, задание такое у меня, написать программу считающее факториал какого-то числа n ( 1<=n<=20).
Вроде написал программку и даже работает, но на каком то месте перестает правильно считать, т.е. счет уходит в минус. Может кто поведает юному юзеру в чем проблема?

#include <iostream>
#include <cmath>
using namespace std;

int main()
{
	freopen("test.in", "r", stdin);
	freopen("test.out", "w",stdout);
	int i, n;
	long fact;
    fact=1;
	cin>>n;
	for (i=1; i<=n; i++)
	{ 
		fact = fact*i;
	}
	printf("%d", fact);
	return 0;
}

[RUSt88]

переполнение скорее всего происходит целочисленного типа, используй unsigned long long

[sp.caster]

да вообщем то я тоже так думаю, подправил программу как ты посоветовал, т.е. взял под unsigned long long свой fact, ну и в общем то ничего к сожалению не изменилось

[p51x]

А в принтфе спецификатор поменяли?

[sp.caster]

э-э-э-э, что, что? Немного понятнее пожалуйста можно?=)

[crazy horse]

Мне кажется, что такие вещи как факториал умнее считать рекурсией, цикл тут не нужен. Почти наверняка и задание по этой теме. А по теме - у тебя в n получается неявное преобразование char в int, чтож ты хотел-то. Преобразуй и проверяй всегда принадлежность n к заданному промежутку, чтоб в такие ситуевины не попадать.
Зы. а почему ввод через cin, а вывод не через cout?
Переполнение long на факториале 20? Ну ну...

[p51x]

"%d" - спецификатор формата для принтфа и подобных функций
читате доки для разнообразия что ли

[sp.caster]

спасибо за замечания

[RUSt88]

#include <stdio.h>

#define N 20

unsigned long long F(const unsigned char n) {
  if (n == 0)
    return 1;
  else
    return n * F(n - 1);
}

int main(void) {
  unsigned long long
    fact = 0;
    
  fact = F(N);

  printf("factorial %u: %llu", N, fact);
  //---
  printf("\n\n\tPress any key for exit...");
  flushall();
  getchar();
  return 0;
};