[C++] Źle działający program..

pytanie zadane 17 marca 2018 w C i C++ przez Hiskiel Pasjonat (22,830 p.)

Witam! Piszę sobie od jakiegoś czasu pewien program. W tym momencie natknąłem się na problem. Program nie przechodzi do tego ifa:

else if(Command.substr(0,3).compare("top")==0){
        if(IsTop){
            color(10);
            cout<<"TopMode Enabled!";
            return;
        }else{
            color(12);
            cout<<"TopMode Disabled!";
            return;
        }

        string sub = trim(Command.substr(3, Command.length()-1));
        if(sub.compare("top")){
            HWND hwndMyWnd = FindWindow("ConsoleWindowClass", NULL);
            ::SetWindowPos(hwndMyWnd, HWND_TOPMOST, 0, 0, 0, 0, SWP_DRAWFRAME | SWP_NOMOVE | SWP_NOSIZE | SWP_SHOWWINDOW);
            ::ShowWindow(hwndMyWnd, SW_NORMAL);
            IsTop=true;
            CommandInterpreter("top");
            return;
        }
        else if(sub.compare("untop")){
             HWND hwndMyWnd = FindWindow("ConsoleWindowClass", NULL);
            ::SetWindowPos(hwndMyWnd, HWND_BOTTOM, 0, 0, 0, 0, SWP_DRAWFRAME | SWP_NOMOVE | SWP_NOSIZE | SWP_SHOWWINDOW);
            ::ShowWindow(hwndMyWnd, SW_NORMAL);
            IsTop=false;
            CommandInterpreter("top");
            return;
        }
    }

Nie wchodzi mi do niego wgl. Command wygląda tak: "top". Więc nie kumam o co chodzi. Trim działa poprawnie.

1 odpowiedź

odpowiedź 17 marca 2018 przez adrian17 Ekspert (344,860 p.)
wybrane 17 marca 2018 przez Hiskiel

Najlepsza

if(sub.compare("top")){

Chcesz sprawdzić, czy string się równa "top"? Bo ta linia sprawdza, czy się różni.

Dlaczego używasz `compare()` zamiast zwykłego ==?

if (str == "top") {
}

komentarz 17 marca 2018 przez Hiskiel Pasjonat (22,830 p.)

1. Słyszałem, że powinno się używać compare, a nie operatora porównania.

2. compare zwraca 0 jeśli dwa stringi się zgadzają

komentarz 17 marca 2018 przez adrian17 Ekspert (344,860 p.)

Słyszałem, że powinno się używać compare, a nie operatora porównania.

To jak jeszcze powiesz dlaczego, to chętnie na to odpowiem.

W praktyce, nie widzę powodu by go nie używać do zwykłego porównywania.

compare zwraca 0 jeśli dwa stringi się zgadzają

No właśnie. Dlatego Twój `if(sub.compare("top")){` sprawdza, czy się różnią.

komentarz 17 marca 2018 przez mokrowski Mędrzec (155,460 p.)

... a 0 (zero) jest fałszem.

komentarz 17 marca 2018 przez Hiskiel Pasjonat (22,830 p.)

Wiem. 0 to fałsz, inne niż zero prawda. Ale if sprawdza, czy się zgadza.

if(Command.substr(0,3).compare("top")==0)

Tak poza tym, zmieniłem na == "top" i dalej to samo :/

komentarz 17 marca 2018 przez adrian17 Ekspert (344,860 p.)

Chwila, to który `if` się nie odpala? Ten z 1 czy 13 linii? Jeśli ten z 1 linii (bo go właśnie zmodyfikowałeś), to po co wspomniałeś o `trim`, który jest później?

komentarz 17 marca 2018 przez mokrowski Mędrzec (155,460 p.)

A ten.. sprawdza warunek jakiego oczekujesz?

if(sub.compare("top"))

I następny?

else if(sub.compare("untop"))

komentarz 17 marca 2018 przez Hiskiel Pasjonat (22,830 p.)

Nie wiem, ponieważ nie przechodzi nawet przed tego nadrzędnego if'a

komentarz 17 marca 2018 przez adrian17 Ekspert (344,860 p.)

Nie wiem, ponieważ nie przechodzi nawet przed tego nadrzędnego if'a

To po co pokazałeś cały kod w środku tego ifa i wspomniałeś "Trim działa poprawnie", który nawet nie zostaje odpalony :/

Nie ma sensu zgadywać, zrób `cout` tuż przed tą linią lub najlepiej odpal kod w debuggerze.

komentarz 17 marca 2018 przez mokrowski Mędrzec (155,460 p.)

Wyświetl wynik operacji: Command.substr(0,3).compare("top")==0

na std::clog tak:

#include <iomainp>
#include <iostream>
//....
std::clog << std::boolalpha << (Command.substr(0,3).compare("top")==0) << '\n';

.. i sprawdź czy tam jest wynik true czy false.

komentarz 17 marca 2018 przez Hiskiel Pasjonat (22,830 p.)

A teraz stała się rzecz niesamowita.

 else if(Command.substr(0, 3) == "top"){
        std::clog << std::boolalpha << (Command.substr(0,3).compare("top")==0) << '\n';

(tutaj dalszy kod ifa)

Doszło i pokazało true.. Nie rozumiem już..

komentarz 17 marca 2018 przez Hiskiel Pasjonat (22,830 p.)

Nic innego, żaden cout czy coś. Nic innego nie idzie.

komentarz 17 marca 2018 przez adrian17 Ekspert (344,860 p.)

...

chwila.

        if(IsTop){
            color(10);
            cout<<"TopMode Enabled!";
            return;
        }else{
            color(12);
            cout<<"TopMode Disabled!";
            return;
        }

Zdajesz sobie sprawę, że cały kod pod spodem tego się nigdy nie wykona?

komentarz 17 marca 2018 przez Hiskiel Pasjonat (22,830 p.)

Zdaję. Kiedy jest wywoływane jedynie "top", ma wyświetlić stan. Kiedy top z czymś, ma zrobić x i y. Teraz zauważyłem, że jeśli usunę "return", to wyświetli stan. Potem jest błąd o wyjściu poza zakres, ale to nie ważne.