Błąd w programie, zły wynik

Question

Gdzie jest błąd w tym programie i na czym polega? Otrzymywany wynik nie jest poprawny. 

Treść zadania: Napisz program, który wczytuje dwie liczby a, b i oblicza wartość wyrażenia (a+b)/(a-b). Program powinien sprawdzać poprawność wprowadzonych danych (mianownik != 0)

 

 

Dziękuję za pomoc.

#include <iostream>

using namespace std;

int main()
{
    cout<<"Podaj a i b"<<endl;
    int a, b, wynik;
    cin>>a>>b;
    if ((a-b)==0)
        cout<<"Nie można dzielić przez 0"<<endl;
    else
        wynik=(a+b)/(a-b);
        cout<<"Wynik wynosi: "<<wynik<<endl;

return 0;
}

 

Answer 1

Domyślam się, że wynik nie jest poprawny bo używasz typu int do liczb całkowitych - zmień go na double / float'y.

W dodatku w if'ie / else'ie można pominąć klamry tylko dla jednej linijki - także musisz je dodać przy else'ie

Comments

1. Up typ danych

2. Klamry naljlepiej zawsze, nawet dla jednej linijki:

if () {

} else {

}

Telretycznie dochodzi jednak linijka kodu na calego ifa

Answer 2

Spruboj tak

 

#include <iostream>

using namespace std;

int main()
{
    cout << "Podaj a i b" << endl;
    float a, b , wynik;
    cin >> a >> b;
    if ((a - b) == 0)
        cout << "Nie można dzielić przez 0" << endl;
    else {
        wynik = (a + b) / (a - b);
        cout << "Wynik wynosi: " << wynik << endl;
    }
    return 0;
}

 

Answer 3

Typ int gdzie mamy dzielenie raczej się nie sprawdza, bo często wynik nie jest liczbą całkowitą.

Difference Between float and double

#include <iostream>
#include <iomanip>
 
using namespace std;
 
int main()
{
  float a, b, wynik;

  cout<<"Podaj a i b"<<endl;  
  cin>>a>>b;

  if ((a-b)==0) {
     cout<<"Nie można dzielić przez 0"<<endl;
  }  else {
     wynik=(a+b)/(a-b);
     cout<<"Wynik wynosi: "<<setprecision(2)<<wynik<<endl;
  }
 
  return 0;
}

[ C++ float and double ]