Почему программа работает неверно?

Question

[pwN11]

Суть задачи : Вычислить бесконечную сумму с заданной точностью e(e>0). Считать что требуемая точность достигнута ,если вычислена сумма нескольких первых слагаемых и очередное слагаемое оказалось по модулю меньше чем e -это и все последующие слагаемые можно не учитывать. Изначально I=1 . Вычислить s=-1^(I+1) / i * (i + 1) * (i + 2). Тут представлен мой набросок в консоли,который работает не верно.
1.

double e,  s=0;
            e= double.Parse(Console.ReadLine());
            for (double i = 1, a = 6; i < a; i++) { s = s + Math.Pow(-1,i + 1) / i * (i + 1) * (i + 2); if (s <= e) { a = 1; } else { a = 100; }; Console.Write(i); };
          
            2.
            Console.Write("s=" + s + "e=" + e); Console.Read();

double i=1, e;
            e= double.Parse(Console.ReadLine());
            while (Math.Pow(-1, i + 1) / i * (i + 1) * (i + 2) >= e)
            { Console.Write(i); i++; };
            Console.Read();
            Console.Read();

Answer 1

[Sergey Mozhaykin]

double si = 1.0 / 6; // initial value for i = 1
double s = 0;
int i = 2;
double eps = double.Parse(Console.ReadLine());
while (Math.Abs(si) >= eps)
{
    s += si;
    si = -1 * si * (i - 1) / (i + 2);
    i++;
}
Console.WriteLine(s);
Console.ReadLine();

Answer 2

[Scorpi]

Math.Pow(i + 1, -1) -> Math.Pow(-1, i + 1)

Comments

[pwN11]

Ошибочка вышла,спасибо, но тем не менее она работает не верно.

Answer 3

[Sumor]

Вам больше подходит цикл while, нежели for.
По условиям задачи вам нужно сравнивать только i-ый член, ДО его прибавления к s, а вы сравниваете e с s.
Непонятная игра с параметром a запутывает читателя, а также не даёт сумме вычислиться с точность более 100 слагаемых.

Comments

[pwN11]

Я возможно не совсем понял ваш комментарий ,но по условию сказано что если s меньше e то последующие s можно не учитывать.

[pwN11]

Я не могу понять почему цикл выполняется после первого раза при любых значениях?

[Sumor]

У вас написано:
«и очередное слагаемое оказалось по модулю меньше чем e»
в коде «s = s + Math.Pow(-1, i + 1) / i * (i + 1) * (i + 2)»
в качестве слагаемого именно «Math.Pow(-1, i + 1) / i * (i + 1) * (i + 2)», а не сумма. Плюс, действительно, как заметил carbon88, нужен модуль слагаемого, то есть примерно так (без учёта оптимизации) :
while( Math.Abs(Math.Pow(-1, i + 1) / i * (i + 1) * (i + 2)) < e)
{
…
}

Answer 4

[Алексей Кулаков]

Считать что требуемая точность достигнута ,если вычислена сумма нескольких первых слагаемых и очередное слагаемое оказалось по модулю меньше чем e

Похоже вы забыли про модуль от s когда сравниваете с эпсилон.