Instrukcja warunkowa nie działa c++

Question

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)

Comments

Musisz dokonkretyzować "nie działa". Co się dzieje, co się nie dzieje? Co się pokazuje w konsoli?

---

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"

---

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ł?

---

Tak, plik się otworzył i prawidłowo pobiera na zmianę komendę, a później znak.

---

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)

---

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).

---

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

 

---

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"?

---

No tak, nie wiem jak ja tego nie zauważyłem... dawno nie pisałem żadnego kodu. Dziękuję bardzo za pomoc!