Сумма n слагаемых с заданной точностью (C++)

[addibos]

Начал изучать программирование только в этом году, и возник вопрос. Нужно чтобы программа считала значение суммы n слагаемых(вроде так называется, обозначается буквой сигма) (-x)^2/2i! для i от 1 до бесконечности с некоторой точностью E. Программу написал, но у меня создаётся впечатление что она неверно выдаёт результат или я не так понял задание(особенно касается условия, заданного в первом цикле for). Прошу вас проверить и сказать что не так, код привожу ниже

#include <cstdlib>
#include <iostream>
#include <math.h>

using namespace std;

int main(int argc, char *argv[])
{
    float x,y=0,q,i,s,fact=1,n,l=-1;
    cout<<"x=";
    cin>>x;
    cout<<"q=";
    cin>>q;
        for (i=1;s>q;i++);
        {            
            for(n=1;n<=i;n++)
             {
              fact=(i-n)*fact+fact;
             }
        s=((pow ((l*x),(2*i))))/((2*fact));
        y=y+s;
        }                 
    cout<<"y="<<y;
    system("PAUSE");
        return EXIT_SUCCESS;
}

P.S Отладчик жутко тупит, проверять ничего не хочет(стандартный из wxDev C++)

[Sciv]

fact=(i-n)*fact+fact;

Факториал - это произведение чисел, откуда плюс?

И еще - не лучше его (факториал я имею в виду) вынести в отдельную функцию?

int fact(int x)
{...}

[addibos]

Сообщение от Sciv

Факториал - это произведение чисел, откуда плюс?

А тогда факториал единицы считать не будет

Сообщение от Sciv

И еще - не лучше его (факториал я имею в виду) вынести в отдельную функцию?

int fact(int x)
{...}

А что это даст?

[Sciv]

А тогда факториал единицы считать не будет

А это уж как напишете

Не разглядел у Вас fact=1 в объявлении переменных, сорри.

А что это даст?

Уменьшит основной код программы на один цикл (я имею в виду текст, естественно, а не выполнение - при вызове-то функции цикл все-равно выполняться будет)

[addibos]

Ну нам пока что разрешается писать большие коды программ(когда их можно сделать меньше), так что там с решением? Она правильно считает или нет? Я просто запутался с самим условием.

[Sciv]

Вроде бы, да. Хотя вот здесь

for (i=1;s>q;i++); //Кстате, точка с запятой в конце не лишние?

у меня сомнения: обычно сравнение с Е идет по модулю числа:

for (i=1;fabs(s)>q;i++)

[addibos]

Сообщение от Sciv

for (i=1;s>q;i++); //Кстате, точка с запятой в конце не лишние?

Посмотрю, помню экспериментировал с этими точками с запятыми и программа работала каждый раз по разному

у меня сомнения: обычно сравнение с Е идет по модулю числа:

for (i=1;fabs(s)>q;i++)

Нет, у нас в задании не упоминался модуль, спасибо за помощь.

[Sciv]

Сообщение от addibos

Нет, у нас в задании не упоминался модуль, спасибо за помощь.

Это не по заданию, это вроде как математика... Могут и не указать, подразумевая, что вы это уже изучили на матане или численных методах. У нас был один такой "товарисч" в институте, любил выдавать техзадания без формул.
Зато свой плюс в этом однозначно есть - умение искать информацию не в гугле, а в библиотеке

[addibos]

Да это вряд ли, я только поступил в этом году и на матане только пределы изучаем

[vvmcpp]

  for (i=1;s>q;i++);

// если оставить точку с запятой то в теле цикла будет выполняться пустой оператор ; .То есть, как-бы так

 for (i=1;s>q;i++){;}

Затем будет выполняться блок {} - то есть просто блок, который уже к циклу не будет относиться.