Mam problem - C++

Question

Mój problem polega na tym, że jeśli podam liczby a i b które będą pasować do: if(a%b==0) cout<<b; albo do if(b%a==0) cout<<a; to program się skończy, nie ważne jak dużo bym testów podał. Żeby to zobaczyć: wpiszcie za liczbę testów więcej niż jeden, i w pierwszym teście wpiszcie 11 i 22.

Moj program:

#include <iostream>

 

using namespace std;

int t,a,b,Dzielnik;

int main()

{

cout << "Podaj mi ilosc testow: ";

cin >>  t;

 

for (int k=0; k<t; k++)

{

   cout<<"Podaj pierwsza liczbe: ";

    cin>>a;

    cout<<"Podaj druga liczbe: ";

    cin>>b;

{

        if(a>b)

{if(a%b==0) cout<<b;

Dzielnik = a%b;}

 

 

        if(b>a)

{if(b%a==0) cout<<a;

    Dzielnik = b%a;}

 

cout<<(a*b)/Dzielnik<<endl;

}

}

    return 0;

}

Answer 1

Po pierwsze - zasady. Jeśli dodajesz cokolwiek na stronę i wklejasz kod, użyj czegoś takiego jak "Dodaj lub zaktualizuj bloczek kodu". Ma to na celu ominięcie tego, co zrobiłeś. 

Po drugie - błąd tkwi w zapisie. 
 

{if(a%b==0) cout<<b;

Dzielnik = a%b;}

Jeśli dobrze to rozumiem, to sprawdzasz jakiś warunek, mianowicie czy a%b == 0. A potem... dzielnik = a%b, czyli dzielnik = 0. A potem... dzielisz przez 0. Więcej dodawać komentarza chyba nie trzeba 

Popraw formatowanie kodu, a potem zastanów się nad tym, czy gdzieś nie powinieneś mieć jakichś klamr bądź innych warunków. 

Comments

Zadaniem tego programu jest znalezienie najmniejszej wspólnej wartości. Dzielnik odgrywa tu rolę największego wspólnego dzielnika (nazwałem go dzielnikiem, żeby było dla mnie łatwiej). a warunek if(a%b==0) i if(b%a==0), ponieważ nie działał mi program dla liczb które po dzieleniu przez siebie nie dają reszty. Jak już to poprawiłem, to pojawił mi się problem z działaniem tego programu. Kiedy go włączam wpisuję tam więcej niż jeden test (załóżmy, że 3) i w pierwszym teście wpisując liczby, które pasują do jednego z tych warunków [if(a%b==0) i if(b%a==0)], to nie mogę po tym nic zrobić i nie wiem jak to naprawić.

---

O widzisz, to już są konkretne informacje i część rzeczy nabiera sensu. Tylko czym jest "najmniejsza wspólna wartość" dwóch liczb? Jaka będzie najmniejsza wspólna wartość liczby 2 i 4, a jaka -5 i 10? A może chodzi o znalezienie najmniejszej wspólnej wielokrotności? 

A nie możesz nic zrobić, bo - tak jak już powiedziałem - po spełnieniu warunku a%b == 0 lub b%a == 0 otrzymujesz, że dzielnik = 0. A potem robisz 
 

cout << (a*b)/dzielnik

I jaki wynik dostaniesz, gdy dzielnik = 0? Ano żaden, dzielenie przez 0 nie jest zdefiniowane. W dodatku nie masz rozpatrzonego przypadku, gdy obie liczby są identyczne. Ja tam widzę tylko warunki a > b oraz b > a. 
To, co chcę Ci przekazać, to fakt, że kod jest brzydko napisany, niechlujnie. Masz jakieś zbędne klamry momentami, w dodatku wiesz, co robi poniższy warunek
 

if (a%b == 0) cout << b;

? Otóż jeśli jest spełniony, to wypisuje w konsoli wartość zmiennej b, a następnie do zmiennej Dzielnik przypisuje a%b. Tyle, że jeśli a%b = 0, to Dzielnik = 0. 
Ogółem błędów jest sporo - nie wiadomo, co ma robić program (a przynajmniej ja nie wiem, co to najmniejsza wspólna wartość dwóch liczb, nie uczą tego na matematyce na studiach). Ponadto warunki są mało sensowne. Niby są sprawdzane jakieś warunki, ale ich spełnienie nie robi prawie nic. W dodatku nie masz opisanej sytuacji, gdy dwie liczby są równe. Gdy podasz dwie identyczne liczby, program się wysypie, bo Dzielnik jest niezdefiniowany - nie wiesz, co w nim jest, może jakieś śmieci, może 10, a może 0. 

---

Teraz zauważyłem, że napisałem najmniejsza wspólna wartość a miało być najmniejsza wspólna wielokrotność.

 

---

No widzisz, jest to już zupełnie co innego. Skasuj to wszystko i najpierw napisz algorytm znajdujący największy wspólny dzielnik dwóch liczb - będzie Ci potrzebny w algorytmie. Następnie mając NWD możesz już znaleźć NWW. Wówczas nie trzeba nawet rozróżniać żadnych przypadków (pomijając 0), bo nawet jeśli dwie liczby będą równe, to NWD(a, a) = a i NWW(a, a)  = (a*a)/a = a

Answer 2

Sądząc po nazwie problem leży tu:

Dzielnik = a%b;

Skoro to dzielnik to daj tak

Dzielnik = a/b;

teraz jest okej?

Comments

Nazwałem tą zmienną tak, ponieważ miała ona dawać największy wspólny dzielnik liczb: a i b, więc z tym jest wszystko dobrze.