Program wypisujący trójki pitagorejskie kompletnie nie działa.

Question

Dopiero zaczynam przygodę z C++ i kompletnie nie wiem gdzie popełniłem błąd. Stawiam na pierwszą pętlę bo reszta wydaje się sensowna. Tutaj wklejam kod i liczę, że ktoś mądrzejszy pomoże mi.

#include <iostream>
#include <time.h>
#include <windows.h>
#include <stdlib.h>
#include <math.h>

using namespace std;

int ilosc;
float x=1;
int a=1,b=1,c;


int main()
{
    cout << "Ile chcesz wyswietlic trojek pitagorejskich?: ";
    cin >> ilosc;

    for(int i=1; i<=ilosc; i++)
    {
        if(c==x)
        {
            cout << a << "^2 + " << b << "^2 =" << c << "^2" <<endl;
            a++;
        }

        else
        {
            b++;

            c=pow(a,2)+pow(b,2);
            c=sqrt(c);
            x=sqrt(c);

        }
    }








    return 0;
}

 

Answer 1

W if'ie przy pierwszej iteracji pętli, zmienna c nie jest zainicjalizowana, stąd prawdopodobnie ten błąd ;).

Gdybyś jeszcze opisał jak działa powyższy program, pomogłoby mi to w rozwiązaniu twojego problemu :)

Answer 2

Masz źle ułożony algorytm. Rozumiem, że chcesz otrzymywać dwie liczby:
 - x będące pierwiastkiem a^2 + b^2 
 - c będące liczbą całkowitą pirerwiastka a^2 + b^2 (zaokrąglony x)
a później ze sobą porównać. Taki algorytm ma sens, jednak po pierwsze

c=sqrt(c);
x=sqrt(c);

Zmienna x w tym wypadku stanie się pierwiastkiem c, które już jest pierwiastkiem.

Drugi błąd z pewnością jest w iteracji. Instrukcja if...else... zawsze będzie przechodzić do warunku else, ponieważ nigdy nie znajdzie trójki pitagorasowej w której jednym ze składników (a lub b - w twoim przypadku a) będzie 1. Przemyśl i jeszcze raz rozrysuj sobie na kartce swój algorytm.

Comments

Dzięki za wskazówki, chyba rzeczywiście zacznę od początku. Mam jeszcze jedno pytanie, jeżeli nie spełni się warunek if i wykona else, to czy pierwsza pętla for sama zainicjuje się znowu czy dodać coś co znowu by ją wywoływało.

---

Instrukcje w języku C++ są w domyśle wykonywane sekwencyjnie, czyli jedna po drugiej.

W wypadku twojego oryginalnego kodu będzie to wyglądało tak

for(int i=1; i<=ilosc; i++) {
        if(c==x) {
            cout << a << "^2 + " << b << "^2 =" << c << "^2" <<endl;
            a++;
        }
 
        else {
            b++;
            c=pow(a,2)+pow(b,2);
            c=sqrt(c);
            x=sqrt(c);
        }
}

Zainicjowanie pętli for i pierwsze wykonanie
- sprawdzenie warunku if (nie spełniony)
- wykonanie warunku else
- koniec instrukcji w pętli ==> kolejne wykonanie pętli for

I tak "ilosc" razy