#include <stdio.h>
int main(void)
{
    int i,n,sum=1;
   
    while(scanf("%d",&n)!=EOF&&n>1&&n<=10)
    {
        for(i=1;i<=n;i++)
            {
                sum=sum*i;
            }
    }
   
    printf("%d\n",sum);
    return 0;
}

错在哪里？按你程序的意思似乎是要不停地求 n 的阶乘，直到输入 EOF 或不在 2-10 之间就停止？
那你得把 前面的 sum=1 移到循环里面，后面的 printf 也移进循环。

当然是哪个循环你应该理解。

为什么要这么编写呢

直接不用scanf函数，一个for循环就搞定了

没搞好呢。
描述
输入一个值n，计算n！

(1<n<=10)



输入每行一个n值
以EOF作为结束。输出输入n!，每行一个。

描述
输入一个值n，计算n！

(1<n<=10)



输入每行一个n值
以EOF作为结束。输出输入n!，每行一个。

是输入一次就计算并输出一次，还是先全部输入后，再全部输出结果？这在逻辑上要作不同的处理，如是前者，你上面的代码改改就行。如是后者，你需要用一个数组来保存输入值或者中间结果。

你顶楼的代码仅仅完成了单次输出。