SPOJ - równanie liniowe

Question

#include <iostream>
#include <cmath>
#include <iomanip>

using namespace std;
//jezeli chodzio o pierwszy if
//to jezeli pomnozymy przez odwrotnosc wspolczynnik
//przy x to rownanie sie rozwiazuje
float a, b, c;
float d;

int main()
{
    cin >> a >> b >> c;

    if((a!=0)&&(a!=1)&&(b!=c))
    {
        d = pow(a,-1);
       // cout << (c-b)*d << endl;
        cout<<setprecision(4)<<(c-b)*d<<endl;

    }

    else if((a!=0)&&(a!=1)&&(b==c))
    {
       cout << 0 << endl;

    }

    else if((a==0)&&(b!=c))
    {
        cout << "BR" << endl;
    }

    else if ((a==0)&&(b==c))
    {
        cout << "NWR" << endl;
    }

    else if((a==1)&&(b!=c))
    {
        cout << c-b << endl;
    }

    return 0;
}

Dzień dobry wieczór

Od pewnego czasu mierzę się już z tym zadaniem, wydaje mi się, że wszystko jest w porządku i, że rozważyłem wszystkie możliwe przypadki, mimo tego spoj cały czas nie zalicza. Proszę o pomoc lub ewentualny pomysł na inny sposób rozwiązania.

link do zadania: https://pl.spoj.com/problems/JROWLIN/

pozdrawiam

Comments

link do zadania

---

Racja mój błąd, sorka, ale już dodałem

---

Przede wszystkim, to wywal endl na końcu - zobacz, że w example nie masz znaku nowej linii na końcu. Oprócz tego, strasznie przekombinowane są te ify. Jeżeli a==0, to możesz wewnątrz tego ifa stworzyć dwa następne (dalej bym nie schodził bo zbytnie zagnieżdżanie nie jest ok).

Pow(a,-1) też jest przekombinowane, wystarczy podzielić wynik przez a, zamiast tak mnożyć.

---

Dzięki za dotychczasową pomoc udało mi się znacznie uszczuplić kod, ale spoj nadal nie zalicza, czy mógłbyś rzucić jeszcze raz okiem?

#include <iostream>
#include <cmath>
#include <iomanip>

using namespace std;

float a, b, c;

int main()
{
    cin >> a >> b >> c;

    if((a!=0))
    {
        if((a!=1)&&(b!=c)) cout<<setprecision(4)<<(c-b)/a;
        else cout << 0;
    }

    else if(a==0)
    {
        if((b!=c)){cout << "BR";}
        else{cout << "NWR";}
    }

    else cout << c-b;

    return 0;
}

 

---

No dobra, a wpisz sobie input np:

1 1 100000000 (bardzo małe a, małe b i bardzo duże c). Zobaczysz, że nie daje poprawnych wyników.

---

Trafna uwaga! Naprawiłem już to, ale spoj i tak nie zalicza jakieś inne propzycje

#include <iostream>
#include <cmath>
#include <iomanip>

using namespace std;

long double a, b, c;

int main()
{
    cin >> a >> b >> c;

    if((a!=0))
    {
        cout << setprecision(10000000);
        if((a!=1)&&(b!=c)) cout<<setprecision(4)<<(c-b)/a;
        //else if((a==1)&&(b!=c) cout << c-b;
        else if(a==1)
        {
            if(b!=c) cout << c-b;

            else cout << 0;
        }
    }

    else if(a==0)
    {
        if((b!=c)){cout << "BR";}
        else{cout << "NWR";}
    }

    //else cout << c-b;

    return 0;
}

---

Input

W pojedyńczej linii podane są trzy liczby rzeczywiste zaokrąglone do drugiego miejsca po przecinku.

if((a!=0))
{
    std::cout << std::fixed << std::setprecision(2) << (c - b) / a ;
}

---

1 1, 0.3 parametry.

Wpisz w google: "round number to 2 decimal places" albo coś w ten deseń.

---

Już naprawiłem ten błąd z parametrami

 

setprecison(4) musi być, ponieważ jak go zmieniam to na setprecison (2) to zakrągla mi do części całkowitych, tzn. pomija cyfry po przecinki

#include <iostream>
#include <cmath>
#include <iomanip>

using namespace std;

long double a, b, c;

int main()
{
    cin >> a >> b >> c;

    if((a!=0))
    {
        if((a!=1)&&(b!=c)) cout<<setprecision(4)<<(c-b)/a;
        //else if((a==1)&&(b!=c) cout << c-b;
        else if(a==1)
        {
            if(b!=c) cout << c-b;

            else cout << 0;
        }
    }

    else if(a==0)
    {
        if((b!=c)){cout << "BR";}
        else{cout << "NWR";}
    }

    //else cout << c-b;

    return 0;
}

 

 

---

Nie musi być, są inne możliwości do wyświetlania zaokrąglanych liczb do 2 miejsc po przecinku, niż setPrecission. Choćby: https://www.geeksforgeeks.org/rounding-floating-point-number-two-decimal-places-c-c/

---

@cupoforanges nie może być setprecision(4) - poczytaj w dokumentacji jak to działa. Użyj std::fixed i std::setprecision(2) tak jak napisałem wcześniej.

---

Dziękuję wam za pomoc! W końcu zadziałało!

Oto działający kod jeśli jesteście ciekawi:

 

 

#include <iostream>
#include <cmath>
#include <iomanip>

using namespace std;

long double a, b, c;

int main()
{
    cin >> a >> b >> c;

    if((a!=0))
    {
        cout << fixed;
        cout << setprecision(2);
        if((a!=1)&&(b!=c)) cout<<(c-b)/a;
        //else if((a==1)&&(b!=c) cout << c-b;
        else if(a==1)
        {
            if(b!=c) cout << c-b;

            else cout << 0;
        }
    }

    else if(a==0)
    {
        if((b!=c)){cout << "BR";}
        else{cout << "NWR";}
    }

    //else cout << c-b;

    return 0;
}

 

---

A i jeszcze takie pytanie. Mam rozumieć, że setprecision(x) ma być zawsze poprzedzone cout <<fixed? (Mam na myśli przypadek z zaokrąglaniem) I jak jest różnica pomiędzy setprecision(4), fixed setprecision(2), przecież z tego co zaobserwowałem to jest to samo

---

std::fixed i std::setprecision zmienia sposób formatowania na stałe, więc nie musisz go używać za każdym razem. Jeżeli chcesz wrócić do poprzedniego formatowania użyj std::defaultfloat. 

 I jak jest różnica pomiędzy setprecision(4), fixed setprecision(2), przecież z tego co zaobserwowałem to jest to samo

#include <iostream>
#include <iomanip>

int main() {
	float number1 = 1245.1234;
	std::cout << std::setprecision(4) << number1 << "\n";
	std::cout << std::fixed << std::setprecision(2) << number1 << "\n";
}

---

Dzięki za pomoc!