Подскажите правильно ли сделана программа?

#include <iostream> // содержит классы cin, cout;
#include <iomanip> // содержит функцию setw
#include <cmath>
 
using namespace std;  
 
 
int main () 
    {
    setlocale (LC_ALL, "rus"); // это чтобы отображался русский шрифт
    const int max = 500; // максимальное число интераций
    double Xstart,Xend,dX,sum,e,element,P;
    int c;
    bool done ; // признак точности 
    cout <<"Введите Хнач=";
    cin >> Xstart;
    cout << "Введите Хкон=";
    cin>>Xend;
    cout << "Введите Xd=";
    cin>>dX;
    cout << "точность e=";
    cin>>e;
    cout <<"       Таблица значений          "<< endl; 
    cout <<"---------------------------------"<< endl;
    cout <<"  X  | значение функции  |  кол. "<< endl;
    cout <<"---------------------------------"<< endl;
    for (double x=Xstart; x<=Xend; x+=dX) // цикл перебирает все х в интервале Xнач до Xкон c шагом dX
        {
        c = 1; done = true; element = x; sum = element;
        for (int n=0; fabs(element)>e; n++) // цикл вычисления ряда
            {
            element*=(pow(x,(2*n+1)))/(((2*n)-1));   // вычисляем очередной член ряда 
            sum+=element;          // складываем 
            c=c+1;                   // число итераций
            if (n>max)             // число может никогда не превзойти е, следует будет зацикливание, для этого наложим ограничения
                {
                cout << setw (5)<< x << "|" << setw (19)<< "Ряд расходится" << "|" << endl;
                done = false; break; // break служит для предворительного выхода из цикла!
                }
            }
        if (done) // если done = true то выполняется иначе пропускается
            {
            cout << setw (5)<< x << "|" << setw (19)<<  2*sum << "|" << setw (2)<< c << endl; // выводим значения в табличном ввиде
            }
        
        }
     cout <<"---------------------------------"<< endl;
    return 0;
    }

#include <iostream> // содержит классы cin, cout;
#include <iomanip> // содержит функцию setw
#include <cmath>
 
using namespace std; 
 
 
int main () 
{
    setlocale (LC_ALL, "rus"); // это чтобы отображался русский шрифт
    const int max = 500; // максимальное число интераций
    double Xstart, Xend, dX, e;
    int c;
    cout <<"Введите Хнач = ";
    cin >> Xstart;
    cout << "Введите Хкон = ";
    cin>>Xend;
    cout << "Введите dX = ";
    cin>>dX;
    cout << "точность e = ";
    cin>>e;
    cout <<" Таблица значений "<< endl; 
    cout <<"---------------------------------"<< endl;
    cout <<"\t| значение функции | кол. "<< endl;
    cout <<"---------------------------------"<< endl;
    for (double x = Xstart; x <= Xend; x += dX) // цикл перебирает все х в интервале Xнач до Xкон c шагом dX
    {
        c = 1;
        double element = 1 / x, sum = element;
        for (int n = 1; fabs(element) > e && n < max; n++) // цикл вычисления ряда
        {
            element = 1 / pow(x, 2 * n + 1) / (2 * n + 1); // вычисляем очередной член ряда 
            sum += element; // складываем 
            ++c; // число итераций
        }
        sum *= 2;
        if (fabs(element) > e) // если done = true то выполняется иначе пропускается
        {
            cout << "x = " << x << "\t|" << setw (18)<< "Ряд расходится" << "|" << endl;
        }
        else
        {
            cout << "x = " << x << "\t|" << setw (17)<< sum << " |" << setw (2)<< c << endl; // выводим значения в табличном ввиде
        }
        double value = log((x + 1) / (x - 1));
        cout << "значение|" << setw (17) << value << " |" << setw (2)<< c << endl; // выводим значение посчитанное по формуле
        cout << "разница |" << setw (17)<< fabs(value - sum) << " |" << setw (2)<< c << endl; // выводим значение посчитанное по формуле
        cout << "\t|                  |\n";
 
    }
    cout <<"---------------------------------"<< endl;
    return 0;
}