Функция fact в С++ for DOS 3.1

[Xeon332]

Задание воообщем такое:
Дано натуральное число n. Вычислить 1! + 2! + … + n!.

а код:

#include<stdio.h>
#include<conio.h>
void fact(unsigned char j, long p, unsigned char x)
{
clrscr();
p=1;
for (j=1) to x
 p=p*j;
 fact=p;
}
void main(void)
{
n,i:unsigned char, s:long;
clrscr();
puts("Vvedite chislo");
scanf("%u",&n);
s=0;
for i=1 to n
 s=s+fact(i);
 printf("Factorial=%O",s);
getch();
}

делал всё по образцам.. но прога не компилируется... ошибки непонятны.
(parametr 'x,j,p' is never used; undefined simbol 'n,i,s' )/ Они ж вроде как инициализированы... помогите исправить ошибки...

[Ламер_001]

паскалист?
1) объявление не верное для си
2) передаешь в функцию 1 параметр когда ей необходимо 3.

вот в общем. сравнивай и делай выводы

#include<stdio.h>
#include<conio.h>

long fact(unsigned char j)
{
 long p = 1, i;
 for (i = 1; i <= j; i++)
  p = p*j;
 return p;
};

void main(void)
{
 unsigned char n, i;
 long s;
 clrscr();
 puts("Vvedite chislo:");
 scanf("%u",&n);
 s=0;
 for (i=1; i <= n; i++)
  s = s + fact(i);
 printf("Factorial = %d\n",s);
 getch();
}

[ROD]

Ты передаеш функции

 s=s+fact(i);

А что она от тебя ждет?

void fact(unsigned char j, long p, unsigned char x)

тоесть она получила только переменную i, ожидаемый тип которой unsigned char.

Кроме того, в С++ конструкция

for (j=1) to x
 p=p*j;
 fact=p;
}

будет выполняться вечно. или (скоре всего так и поступит) выдаст ошику по поводу непонятной инструкции "to x" (это не бэйсик)и не правильного написания содержимого скобочек (правильно как у меня в подписе, это бесконечный цикл).
Структура цикла for

for(начальное значение параметра; условие выполнения цикла; изменение параметра)
{инструкции}

Например:
for(i =0; i<10; i++)
{cout<<i<<endl;}

Данный цикл задает переменной i значение 0. Он будет повторяться до тех пор, пока i меньше 10, и после каждого выполнения цикла он будет приплюсовывать к i единицу.

( i++ равносильно i=i+1; аналогично i-- равносильно i=i-1;
а так же
i*=n равносильно i=i*n
i/=n равносильно i=i/n
i+=n равносильно i=i+n
i-=n равносильно i=i-n)

И еще, вроде бы нельзя задавать перменные так:

n,i:unsigned char, s:long;

Это задается так

unsigned char n,i;
long s;

моэжно сразу же задать им значения

unsigned char n,i;
long s=55;

и еще char это символьный масив. т.е. ему надо задать длинну ( n[100]
если тебе надо задать в масив элементы разом, то это делается так:
n[100]={1, 12, 442, 123, ... } если масив числовой
n[100]={"bukafki"} если массив символьный