代码评析（系列一）

1.       //阶乘函数
   
2.    public double fact(double x) {
   
3.                   if (x == 0)
   
4.                           return 1;
   
5.                   else
   
6.                           return x * fact(x - 1);
   
7.           }
   

复制代码

以上一段Java代码，大家觉得有啥问题没？

额……感觉小于零、非整数的时候好像会有问题……

回复 2楼daodao的帖子

很犀利么。

G开头的同学，这个好像是你们Team交的Java作品的一个片段。

double类型和零比较用==不合适

这里用递归也不太好，系统堆栈深度有限，很容易就堆栈溢出的。

这个影响严重吗？

回复 4楼Leo的帖子

啊， 是我们组的，上头条了......学习了

回复 1楼Leo的帖子

这是我写的代码，当时感觉能完成解析计算就很开心了。现在回头看看真是有很多考虑不周的地方啊，受教了

回复 6楼Leo的帖子

递归不是问题,因为早溢出了.