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

Question

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.

 

Answer 1

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") {
}

 

Comments

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

---

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

---

...

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?

---

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.

---

Dobra, zorientowałem się o co chodzi. teraz tylko problem z dwiema rzeczami, ale to moja głowa. Dziękuję za pomoc.