让我来告诉你们错在哪里

一开始我的代码是这样的：

#include<stdio.h>
int main()
{
 int n,i;
 int s=1,sn=0;
 scanf("%d",&n);
 if(n>0&&n<=20)
 {
 for(i=1;i<=n;i++)
 {
  s=s*i;
  sn=sn+s;
 }
 }
 printf("%d\n",sn);
 return 0;
}

输入5，输出153没问题啊，但是我们忽略了一个问题，那就是当n很大时，s和sn都非常大，已经超出了%d的范围了。

所以s和sn都要定义成long int,输出时也要写成%ld。

正确代码图如下：

#include<stdio.h>
int main()
{
 int n,i;
 long int s=1,sn=0;
 scanf("%d",&n);
 if(n>0&&n<=20)
 {
 for(i=1;i<=n;i++)
 {
  s=s*i;
  sn=sn+s;
 }
 }
 printf("%ld\n",sn);
 return 0;
}