Program obliczający wartość funkcji w postaci rzędu nie zrywa pętli do..while

Question

Mam na zadanie napisać program obliczający wartość funkcji podanej w postaci rzędu (poniżej), dla wartości x z przedziału [0,1] zwiększanych co 0.1, z błędem 0.0001.
Kod nie jest jeszcze ukończony, ale wstępnie, przy próbie obliczenia dla wartości 0.9 jest jakiś błąd. Program uruchamia się prwidłowo, lecz nie zrywa pętli do..while. Powinien zerwać przy k=2 i wyświetlać 2 razy zmienną element, a wyświetla 10 razy.

Proszę o pomoc i z góry dziękuję.

Kod:

#include "stdafx.h"
#include <iostream>
#include <math.h>

using namespace std;
double licznik, mianownik = 1, y = 0;
int znak = 1;
double x = 0.9;

int main()
{
	do
	{
		for (int k = 0; k < 10; k++)
		{
			for (int i = 2 * k + 3; i > 1; i--)
			{
				mianownik = mianownik * i;
			}
			licznik = pow(x, 2 * k + 3);
			licznik = licznik * znak;
			znak = -znak;
		        y = y + (licznik / mianownik);
			mianownik = 1;
			cout << element << endl;

		}
	} while (abs(licznik / mianownik) > 0.0001);

	system("PAUSE");
	return 0;
}

 

Answer 1

Nie chce mi się analizować kodu ale zawsze możesz użyć `cout` i wypisywać wszelkie zmienne aby zaobserwować co się dzieje w programie i czy wszystko przebiega z twoimi założeniami.

---

EDIT: na pierwszy rzut oka masz jeden błąd: wypisujesz zawartość zmiennej `element` ale nie masz jej nigdzie zadeklarowanej. Może chodziło o zawartośc zmiennej `y` ?

Comments

Poprawione. Program uruchamia się, lecz wypisuje: 

a powinien wypisać tylko pierwsze dwie wartości, ponieważ przy trzeciej pętla do..while powinna zwrócić wartość false. Problem w tym, że nie zwraca.

Poprawiony kod:

#include "stdafx.h"
#include <iostream>
#include <math.h>

using namespace std;

double licznik, mianownik = 1, y = 0, element;
int znak = 1;
double x = 0.9;

int main()
{
	do
	{
		for (int k = 0; k < 10; k++)
		{
			for (int i = 2 * k + 3; i > 1; i--)
			{
				mianownik = mianownik * i;
			}
			licznik = pow(x, 2 * k + 3);
			licznik = licznik * znak;
			znak = -znak;
			element = licznik / mianownik;
			y = y + element;
			mianownik = 1;
			cout << element << endl;

		}
	} while (abs(element) > 0.0001);

	system("PAUSE");
	return 0;
}

 

---

Teraz widzę, że oblcizenia są wewnątrz pętli `for` więc `while` zostanie sprawdzone dopiero po zakończeniu tej pętli `for`. Trzeba by zamiast `while` zrobić wewnątrz `if` i użyć `break` do wyskakiwania z pętli przed jej zakońćzeniem.

---

Tak to u mnie wygląda - kompilowane pod Linux więc część rzeczy zakomentowałe, użyłem fabs zamiast abs bo nie chciał tego użyć dla liczb różnych od całkowitych. Ustawiłem rzeczy trochę inaczej dla czytelności kodu.

//#include "stdafx.h"
#include <iostream>
#include <math.h>

using namespace std;

double licznik;
double mianownik;
double element; 

int znak = 1;

double x = 0.9;
double wynik = 0;

int main()
{
  for (int k = 0; k < 10; k++)
  {
    mianownik = 1;

    for (int i = 2 * k + 3; i > 1; i--)
    {
      mianownik = mianownik * i;
    }
    
    licznik = znak * pow(x, 2 * k + 3);
    znak = -znak;
    element = licznik / mianownik;
    wynik = wynik + element;
    
    cout << element << endl;

    if (fabs(element) <= 0.0001)
    {
      break;
    }
  }

//  system("PAUSE");
  return 0;
}