请问这到底是哪里出错了

a=a*i;  是算阶乘吗
s+=a; //s=n+n-1+n-2....
//下面是算i的阶乘，把下边三行放到你的for循环里
for(int j=1;j<i;j++)
  a=a*j;
s+=a;      //阶乘相加  

加个外层循环，累积从1!到n!阶乘的结果。