C++ duże liczby

Question

Witam, zrobiłem sobie to o to zadanko:

https://main2.edu.pl/c/konkurs-wstepu-do-programowania/p/duz/

I w testach na ich stronie odrzuca mi 4 z 96 testów w każdym jest TAK a powino być NIE.

Trochę posiedziałem nad kodem i błędu znaleźć nie mogę.

Wiem , że był już post o to samo zadanie( o tu link do posta https://forum.pasja-informatyki.pl/239464/bardzo-duze-liczby-c), ale nie chodzi mi o samo rozwiązanie tylko raczej o pomoc w szukaniu błędu.

Przy okazji byłbym wdzięczny gdyby ktoś powiedział czy kod jest przejrzysty i czy jest to zrobione w miarę optymalnie. (Dopiero zaczynam).

A tu mój kod:

#include <iostream>

using namespace std;
bool rowny=false, awieksze, prawda=true;
int main()
{
    string a,b,znak;
    cin>>a>>znak>>b;
    int da=a.length(), db=b.length();
    if(da==db)
    {
        for(int i=0; i<da && prawda; i++)
        {
            if((int)a[i]%48>(int)b[i]%48)
            {
                awieksze=true;
                prawda=false;
            }
            else if((int)a[i]%48<(int)b[i]%48)
            {
                awieksze=false;
                prawda=false;
            }
        }
        if(prawda)
            rowny=true;
    }
    else if(da>db)
        awieksze=true;
    else
        awieksze=false;
    if(rowny && (znak=="==" || znak==">=" || znak=="<="))
    {
        cout<<"TAK"<<endl;
    }
    else if(awieksze && (znak==">="|| znak=="!="|| znak==">" ))
    {
        cout<<"TAK"<<endl;
    }
    else if(!awieksze && (znak=="<="|| znak=="!="|| znak=="<" ))
    {
        cout<<"TAK"<<endl;
    }
    else
        cout<<"NIE"<<endl;

}

 

Comments

Zły język bloczka z kodem.

---

Już poprawione, dzięki za uwagę nie zauważyłem tego.

Answer 1

Problem będzie w sytuacji, gdy liczby będą równe, a operatorem będzie != lub <. Wtedy pierwszy if nie będzie spełniony i zacznie sprawdzać pozostałe warunki, aż dojdzie do tego warunku:

else if(!awieksze && (znak=="<="|| znak=="!="|| znak=="<" ))

który będzie spełniony.

Co do uwag, to nazywaj bardziej sensownie zmienne, unikaj zmiennych globalnych i using namespace std. Jak używasz std::string to powinieneś mieć w kodzie #include <string>.

Comments

Zmieniłem na coś takiego:

 else if(!rowny && !awieksze && (znak=="<=" || znak=="!=" || znak=="<" ))

Lecz dalej wyrzuca te same błędy.

I pytanko odnośnie using namespace czemu mam tego unikać?

---

Już działa dodałem tylko !rowny do wczesniejszego else if i jest okej.
Jeśli kogoś interesuje kod to teraz wygląda on tak:
 

#include <iostream>

using namespace std;

int main()
{
    bool rowny=false, awieksze=2, prawda=true;
    string a,b,znak;
    cin>>a>>znak>>b;
    int length_a=a.length(), length_b=b.length();
    if(length_a==length_b)
    {
        for(int i=0; i<length_a && prawda; i++)
        {
            if((int)a[i]%48>(int)b[i]%48)
            {
                awieksze=true;
                prawda=false;
            }
            else if((int)a[i]%48<(int)b[i]%48)
            {
                awieksze=false;
                prawda=false;
            }
        }
        if(prawda)
            rowny=true;
    }
    else if(length_a>length_b)
        awieksze=true;
    else
        awieksze=false;
    if(rowny && (znak=="==" || znak==">=" || znak=="<="))
    {
        cout<<"TAK"<<endl;
    }
    else if(!rowny && awieksze && (znak==">=" || znak=="!=" || znak==">" ))
    {
        cout<<"TAK"<<endl;
    }
    else if(!rowny && !awieksze && (znak=="<=" || znak=="!=" || znak=="<" ))
    {
        cout<<"TAK"<<endl;
    }
    else
        cout<<"NIE"<<endl;

}

 

---

I pytanko odnośnie using namespace czemu mam tego unikać?

https://stackoverflow.com/questions/1452721/why-is-using-namespace-std-considered-bad-practice 

Answer 2

Jedna rada odnośnie przejrzystości, pisz nazwy zmiennych tak, żeby można było stwierdzić do czego służą (z kontekstem)

bo da i db może być niezrozumiałe

Comments

Lepiej pisać długie nazwy niż krótkie i nieczytelne tak?
Czyli zamiast da lepiej dlugosc_a?

---

Tak tylko po angielsku się przyzwyczajaj, czyli np. length_a;

---

@koniak20 Znacznie, moim zdaniem też tak jak kolega niżej pisze: po angielsku. No i nie ograniczałbym się z notatkami// przy większym projekcie naprawdę ci się przydadzą. Kod łatwiej pisać niż czytać...

---

I jak zaczniesz pisać obiektowo to możesz dawać zmienne o krótkich nazwach tylko komentowane i zamieszczone lokalnie (prywatne).

 

---

Zapamiętam ,a odnośnie błędu ktoś z was go może widzi?

---

A w zadaniu może pojawić się liczba z zerem, zerami na początku?

---

I jak zaczniesz pisać obiektowo to możesz dawać zmienne o krótkich nazwach tylko komentowane i zamieszczone lokalnie (prywatne).

Generalnie zasada jest taka, że nazwa powinna być tym dłuższa (analogicznie: krótsza) czym bardziej ogólny (odpowiednio: szczegółowy) jest kontekst (dlatego często iteratory wykorzystywane w pętlach nazywa się it, a zmienne w pętlach for i,j - są to względnie małe zakresy ważności, więc nazwy są krótkie) .