Сумма бесконечного ряда на C++?

Question

[Name_23]

На картинке формула, для которой нужен код на C++.
Код то сам работает, но я не уверен , правильно ли я прописал формулу:

#include <iostream>
#include <cmath>

using namespace std;

long double Fact2(int n){
if (n < 0)
return 0;
else if (n==1 || n==0)
return 1;
else
return n*Fact2(n-1);
}

 int main()
 {
  int x , k =0;
  double eps, i, sum;

do{
    std::cout << "X=";
    std::cin >> x;
    if (x==0)
    cout << "incorect vaule x" << endl;
}while (x==0);

 do{
    std::cout << "eps=";
    std::cin >> eps;
    if (eps<=0 || eps>=1)
    cout << "incorect vaule eps" << endl;
}while (eps<=0 || eps>=1);

do{
    i = (pow(-1,k) * (pow(x, k+2) / Fact2(k+1) * (4 * k+3)) );
    cout << "k=" << k << "; U=" << i << "; Sum=" << sum << endl;
    sum +=1;
    k++;
}while(abs(i) > eps);

cout << "S=" << sum << endl;
cout << "n=" << k << endl;
return 0;

}

Answer 1

[galaxy]

Как минимум:

double eps, i, sum; -> double eps, i, sum = 0;

i = (pow(-1,k) * (pow(x, k+2) / Fact2(k+1) * (4 * k+3)) );

->

i = (pow(-1,k) * (pow(x, k+2) / Fact2(k+1) / (4 * k+3)) );

sum +=1; -> sum += i;

Comments

[User700]

Также неясно почему не использовать для факториала простой цикл (можно даже тот же, где и идёт вычисление); ну и (-1) в степени k конечно можно найти с помощью переменной, которая становится то +1, то -1 (sgn = -sgn) на каждой итерации.

[Name_23]

galaxy , у тебя Fact2(k+1) / (4 * k+3)) ); поделить вместо умножить. Не понимаю , почему?

[galaxy]

Name_23, про приоритет и ассоциативность операций почитай:

30 / 5 / 6 = 1
30 / 5 * 6 = 36
30 / (5 * 6) = 1