• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Instrukcja warunkowa nie działa c++

VPS Starter Arubacloud
0 głosów
220 wizyt
pytanie zadane 22 lutego w C i C++ przez Doge Gaduła (3,420 p.)

Witam, mam następujący kod:

(dla przyspieszenia powiem, że aktualne != komenda oznacza, ze pobrano znak, (litere lub liczbe))

#include <bits/stdc++.h>

using namespace std;

int main()
{
    fstream wejscie, wyjscie;
    wejscie.open("przyklad.txt");
    wyjscie.open("wyjscie_przyklad.txt");

    string napis;

    string x, aktualne = "komenda";
    while(wejscie>>x)
    {
        cout<<"x: "<<x<<endl;
        if(aktualne != "komenda")
        {
            cout<<"pobralismy znak"<<endl;
            aktualne = "komenda";
            if(x == "DOPISZ")
            {
                cout<<"dopisalismy "<<x<<endl;
                napis += x;
                cout<<"Nowy napis: "<<napis<<endl;
            }
            else if(x == "ZMIEN")
            {
                napis[napis.length() - 1] = x[0];
            }
            else if(x == "USUN")
            {
                napis.pop_back();
            }
            else if(x == "PRZESUN")
            {
                if(x == "Z")
                {
                    x = "A";
                }
                else
                {
                    x[0] += 1;
                }
                napis += x;
            }
            cout<<"po pobraniu"<<endl;
        }
        else
        {
            aktualne = "znak";
            cout<<"pobralismy komende"<<endl;
        }
    }
    cout<<"Napis: "<<napis<<endl;

    cout<<"Dlugosc ciagu: "<<napis.length();

    wejscie.close();
    wyjscie.close();
    return 0;
}

a plik wejscie ("przyklad.txt") wyglada tak:

DOPISZ Z
DOPISZ U
USUN 1
DOPISZ L
DOPISZ A
PRZESUN Z
DOPISZ U
PRZESUN U
ZMIEN M
PRZESUN M
DOPISZ N
USUN 1
DOPISZ T
DOPISZ U
DOPISZ R
DOPISZ H
DOPISZ N
PRZESUN H
DOPISZ V
ZMIEN G

natomiast żadna z instrukcji warunkowych "if" nie działa. O co chodzi? (zadanie maturalne matura 2021 maj, zad. 4.1)

komentarz 22 lutego przez adrian17 Ekspert (346,940 p.)
Musisz dokonkretyzować "nie działa". Co się dzieje, co się nie dzieje? Co się pokazuje w konsoli?
komentarz 22 lutego przez Doge Gaduła (3,420 p.)
Właśnie nic się nie dzieje, nie wiem o co chodzi... instrukcja warunkowa po prostu się nie odpala. Po dodaniu po tych "ifach" "else", odpala się tylko "else"
komentarz 22 lutego przez adrian17 Ekspert (346,940 p.)

Inaczej, czy przynajmniej to

cout<<"x: "<<x<<endl;

przed jakimikolwiek `if`ami, się pokazuje?

Jeśli nie - sprawdziłeś, czy ten plik w ogóle się otworzył?

komentarz 22 lutego przez Doge Gaduła (3,420 p.)
Tak, plik się otworzył i prawidłowo pobiera na zmianę komendę, a później znak.
komentarz 22 lutego przez adrian17 Ekspert (346,940 p.)

OK, czyli nie "nic się nie dzieje", bo coś się dzieje. To jeszcze raz pytam:

Co się dzieje, co się nie dzieje? Co się pokazuje w konsoli?

(w szczególności, możesz po prostu wkleić co się pokazuje i powiedzieć co Ci z tym nie pasuje)

komentarz 22 lutego przez Doge Gaduła (3,420 p.)

W ogóle nie wykonuje się ten fragment kodu:

if(x == "DOPISZ")
            {
                cout<<"dopisalismy "<<x<<endl;
                napis += x;
                cout<<"Nowy napis: "<<napis<<endl;
            }
            else if(x == "ZMIEN")
            {
                napis[napis.length() - 1] = x[0];
            }
            else if(x == "USUN")
            {
                napis.pop_back();
            }
            else if(x == "PRZESUN")
            {
                if(x == "Z")
                {
                    x = "A";
                }
                else
                {
                    x[0] += 1;
                }
                napis += x;
            }

A w konsoli nic się nie wyświetla (mam na myśli tę konsolę w codeblocks).

komentarz 22 lutego przez Doge Gaduła (3,420 p.)

A wyjście jest takie:

x: DOPISZ
pobralismy komende
x: Z
pobralismy znak
po pobraniu
x: DOPISZ
pobralismy komende
x: U
pobralismy znak
po pobraniu
x: USUN
pobralismy komende
x: 1
pobralismy znak
po pobraniu
x: DOPISZ
pobralismy komende
x: L
pobralismy znak
po pobraniu
x: DOPISZ
pobralismy komende
x: A
pobralismy znak
po pobraniu
x: PRZESUN
pobralismy komende
x: Z
pobralismy znak
po pobraniu
x: DOPISZ
pobralismy komende
x: U
pobralismy znak
po pobraniu
x: PRZESUN
pobralismy komende
x: U
pobralismy znak
po pobraniu
x: ZMIEN
pobralismy komende
x: M
pobralismy znak
po pobraniu
x: PRZESUN
pobralismy komende
x: M
pobralismy znak
po pobraniu
x: DOPISZ
pobralismy komende
x: N
pobralismy znak
po pobraniu
x: USUN
pobralismy komende
x: 1
pobralismy znak
po pobraniu
x: DOPISZ
pobralismy komende
x: T
pobralismy znak
po pobraniu
x: DOPISZ
pobralismy komende
x: U
pobralismy znak
po pobraniu
x: DOPISZ
pobralismy komende
x: R
pobralismy znak
po pobraniu
x: DOPISZ
pobralismy komende
x: H
pobralismy znak
po pobraniu
x: DOPISZ
pobralismy komende
x: N
pobralismy znak
po pobraniu
x: PRZESUN
pobralismy komende
x: H
pobralismy znak
po pobraniu
x: DOPISZ
pobralismy komende
x: V
pobralismy znak
po pobraniu
x: ZMIEN
pobralismy komende
x: G
pobralismy znak
po pobraniu
Napis:
Dlugosc ciagu: 0

 

komentarz 22 lutego przez adrian17 Ekspert (346,940 p.)

Ok, to na przyszłość nie pisz

A w konsoli nic się nie wyświetla

Bo to ewidentnie nie jest prawdą i myli odpowiadających ;)

Natomiast na oko... to ten kod nie ma prawa działać?

Najpierw wczytujesz "DOPISZ" do `x`, potem wczytujesz "Z" do `x` po czym sprawdzasz czy `x` jest równe "DOPISZ" - no, w ten sposób to nigdy się nie spełni.

Podobnie to nie ma sensu:

            else if(x == "PRZESUN")
            {
                if(x == "Z")

jeśli x jest równe "PRZESUN", to z natury nie będzie równe "Z".

Więc musisz tą logikę przemyśleć. Natomiast, jeśli - nie znam treści zadania - każda linia wejścia zawsze ma komendę i znak, to mógłbyś je oba wczytać na raz w jednej linii, nie kombinując z jakimiś "aktualnymi fazami"?

komentarz 22 lutego przez Doge Gaduła (3,420 p.)
No tak, nie wiem jak ja tego nie zauważyłem... dawno nie pisałem żadnego kodu. Dziękuję bardzo za pomoc!

Zaloguj lub zarejestruj się, aby odpowiedzieć na to pytanie.

Podobne pytania

+1 głos
1 odpowiedź 394 wizyt
pytanie zadane 9 czerwca 2019 w C i C++ przez SCpl Użytkownik (590 p.)
0 głosów
2 odpowiedzi 297 wizyt
pytanie zadane 25 czerwca 2022 w C i C++ przez polandonion Mądrala (7,210 p.)
+1 głos
1 odpowiedź 355 wizyt
pytanie zadane 30 września 2021 w JavaScript przez Doge Gaduła (3,420 p.)

92,782 zapytań

141,711 odpowiedzi

320,588 komentarzy

62,114 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...