题
我不能得到下面的代码来工作。
#include <stdio.h>
// I am not sure whethere I should void here or not.
int main() {
// when the first bug is solved, I put here arg[0]. It should be
// similar command line parameter as args[0] in Java.
int a=3;
int b;
b = factorial(a);
// bug seems to be here, since the %1i seems to work only in fprintf
printf("%1i", b);
return 0;
}
int factorial(int x) {
int i;
for(i=1; i<x; i++)
x *= i;
return x;
}
你怎么能得到的代码工作?
解决方案
AInitak给出了正确的答案,但我想补充一点,一个办法可以的找到的代码中的错误是打印出来的阶乘环i
和x
的值。
int factorial(int x) {
int i;
for(i=1; i<x; i++)
{
x *= i;
printf("%d, %d\n", i, x);
}
return x;
}
这给出了输出
1, 3
2, 6
3, 18
4, 72
5, 360
6, 2160
7, 15120
8, 120960
9, 1088640
10, 10886400
11, 119750400
12, 1437004800
13, 1501193216
14, -458131456
-458131456
这使得它更容易看到什么错。循环将不会停止,你指望它为AInitak解释的原因。
其他提示
您正在修改您的循环终止变量(x)的内部循环。目前代码中的几次迭代后吹起来,当 X 溢出一个32位整数的范围内,然后变为负值和非常大的,因此,终止循环。
它应该是:
int factorial(int n) {
int i, x = 1;
for (i = 2; i <= n; ++i) {
x *= i;
}
return x;
}
更重要的是,你应该为变量long
和返回值使用int
代替x
,因为 N!的变得非常大非常快。
这是不好的风格用C定义或声明的函数时,离开了void
。所以把它在
int main(void)
虽然它不关于参数的数目改变任何东西的函数具有(函数具有零个参数而没有任一void
),它将声明功能为一体仅接受零参数,而它不会告诉任何有关数量和类型的参数接受当你忽略void
。但是,两种版本有和没有void
是正确的。
答案那件事了。
#include<stdio.h>
#include<stdlib.h>
int main(int c,char *v[])
{
int x,y;
int *num;
if(c==1)
{
printf("Usage : programName : number");
return 0;
}
num=(int *)malloc(sizeof(int));
*num=atoi(v[1]);
x=1;y=1;
while(x<=*num)
{
y=y*x;
x++;
}
printf("Factorial of %d is %d ",*num,y);
free(num);
return 0;
}
你得到了什么错误讯息?
首先,前 factorial
声明你的函数main
。此外,要注意正确的缩进。你的功能main
的声明是正确的,顺便说一下。
我建议长也使用双或无符号为阶乘运算,以便能够计算阶乘函数的更大的价值。
double fact( double n)
{
if ( n == 1)
return 1;
return n*(fact(n-1));
}
一个更优雅非递归函数。
#include<stdio.h>
long long int fact(long long int);
long long int fact(long long int n){
long long int num = 1;
long long int fi = 0;
for(long long int i=2;i<=n;i++){
for(long long int j=1;j<=i;j++){
fi += num;
}
num = fi;
fi = 0;
}
return num;
}
int main(){
long long int n;
scanf("%lld",&n);
printf("%lld\n",fact(n));
return 0;
}
不隶属于 StackOverflow