Question

菜鸟刚开始学c++，求大神看看这个代码有什么问题呢？

题目是写个代码求圆周率，
l π的计算公式如下：

l 其中arctan用如下形式的级数计算：

l 直到级数某项绝对值不大于10-15为止；π和x均为double型。
我没有用答案给的方法，数学都忘得差不多了。就想当然地写了下面这样的，求大神指点。

#include "stdafx.h"
#include<iostream>
using namespace std;
double arctan(double x);
double power(double y,int n);
int _tmain(int argc, _TCHAR* argv[])
{
int PI=0;
PI=16*arctan(1/5)-4*arctan(1/239);
cout<<PI<<endl;
return 0;
}
double arctan(double x)
{
double sum=0;
for(int i=0;power(x,2*i+1)/(2*i+1)<=1/power(10,15);i++)
sum+=(power(-1,i))*(power(x,2*i+1)/(2*i+1));
return sum;
}
double power (double y,int n)
{
double val=1.0;
while(n--)
val*=y;
return val;
}

Answer 1

撇开效率和精度不谈，程序本身有三处错

1. int PI=0;

   int应该改成double
   

2. PI=16*arctan(1/5)-4*arctan(1/239);

   1/5和1/239是整数除法，结果都是0，应该用实数除法，比如1.0/5.0

3. for(int i=0;power(x,2*i+1)/(2*i+1)<=1/power(10,15);i++)

   中间的<=改成>=，因为通项足够小是停机条件，而不是循环执行的条件