Pierwiastek równania kwadratowego daje zły wynik

Question

Program Pierwiastek równania kwadratowego daje zły wynik.
Przykład
Podaje z klawiatury a - 1, b  - 2, c - -3.

Wynik powinien być 2 pierwiastki a wychodzi 1 pierwiastek

Gdzie jest błąd?

#include <iostream>
#include <cmath>


using namespace std;
int main()
{
  int a, b, c, delta;
    cout<<"Podaj a,  b,  c:"<<endl; 
    cin>>a>>b>>c;
 if (a==0)  cout<<"to nie jest  równanie kwadratowe"<<endl; 
    delta = b*b-4*a*c;
 
    if(delta<0)
        cout << "Brak pierwiastkow";
    if(delta==0)
        cout << "1 pierwiastek";
    if(delta>0)
        cout << "2 pierwiastki" ;
        
    return 0;
}

 

Answer 1

cpp.sh/9qety Według Twoich danych, wychodzą dwa pierwiastki. Poza tym, działasz na int'ach, Zmień typy na float/double. 

Comments

Ma wyświetlić się  tylko komunikat że są 2 pierwiastki.

Zmieniłam na double ale jest nadal źle. Nie wiem co jeszcze zmienić. Jestem początkująca.

#include <iostream>
#include <cmath>


using namespace std;
int main()
{
  double a, b, c, delta;
    cout<<"Podaj a,  b,  c:"<<endl; 
    cin>>a>>b>>c;
 if (a==0)  cout<<"to nie jest  równanie kwadratowe"<<endl; 
    delta = b*b-4*a*c;
 
    if(delta<0)
        cout << "Brak pierwiastkow";
    if(delta==0)
        cout << "Jeden pierwiastek";
    if(delta>0)
        cout << "2 pierwiastki" ;
        
    return 0;
}

 

---

Podaj a, b, c:

1 2 -3

2 pierwiastki 

Dostaje coś takiego. Jakiego wyniku oczekujesz?

---

Podaj a, b, c:

1, 2, -3

2 pierwiastki

dostaje brak pierwiastków

---

@tkz,
 Bo ja pisałam z przecinkami. Dzięki

 

---

@tkz, Czy ten program jest dobrze napisany do zadania poniżej do B, BO A opisze?

---

Nie. Nie znalazłaś pierwiastków. Tylko powiedziałaś, że są.

---

To jak to powinien wyglądać poprawny kod?

---

Oblicz pierwiastki jeżeli się da.

---

To jest dobrze?

#include <cmath>


using namespace std;
int main()
{
  double a, b, c, delta;
    cout<<"Podaj a, b, c:"<<endl; 
    cin>>a>>b>>c;
 if (a==0)  cout<<"to nie jest  równanie kwadratowe"<<endl; 
    delta = b*b-4*a*c;
 
    if(delta<0)
        cout << "Brak pierwiastkow";
    if(delta==0)
        cout << "1 pierwiastek"<< "x1= " << -b/2*a;
    if(delta>0)
        cout << "2 pierwiastki"<< "\n";
        cout << "x1= " << (-b-sqrt(delta))/2*a<< "\n";
        cout << "x2= " << (-b+sqrt(delta))/2*a;
        
    return 0;
}

 

---

Niezupełnie. Co z tego, że sprawdzasz, czy równanie jest kwadratowe, skoro i tak liczysz delete. Używaj {} przy if'ach, bo ostatni if jest teoretycznie niepoprawny. Bez znaczenia jaka jest delta i tak liczysz dwa pierwiastki.

---

Czy teraz dobrze?

#include <iostream>
#include <cmath>


using namespace std;
int main()
{
  double a, b, c, delta;
    cout<<"Podaj a, b, c:"<<endl; 
    cin>>a>>b>>c;
{
 if (a==0)  cout<<"to nie jest  równanie kwadratowe"<<endl; 
    delta = b*b-4*a*c;
}
{
    if(delta<0)
        cout << "Brak pierwiastkow";
}
{
    if(delta==0)
        cout << "1 pierwiastek"<< "x1= " << -b/2*a;
}
{
    if(delta>0)
        cout << "2 pierwiastki"<< "\n"<< "x1= " << (-b-sqrt(delta))/2*a<< "\n"<< "x2= " << (-b+sqrt(delta))/2*a;
}        
        
    return 0;
}

 

---

#include <iostream>
#include <cmath>

using namespace std;
int main() {
    double a, b, c, delta;
    cout << "Podaj a, b, c:" << endl;
    cin >> a >> b >> c; 
    if (a == 0){
        cout << "to nie jest  równanie kwadratowe" << endl;  
        return 0; //kończysz działanie programu, bo co masz dalej liczyc?
    }
    delta = b * b - 4 * a * c;
    if (delta < 0){
        cout << "Brak pierwiastkow";
    }
    if (delta == 0){
        cout << "1 pierwiastek" << "\n"
        << "x1= " << -b / 2 * a;
    }
    if (delta > 0){
        cout << "2 pierwiastki" << "\n" 
        << "x1= " << ((-b - sqrt(delta)) / (2 * a)) << "\n" //(2 * a) nawiasy
        << "x2= " << ((-b + sqrt(delta)) / (2 * a));
    }
}

 

---

Dzięki