[C++] problem z implementacją systemu znanego z konsoli (cd..)

Question

Witam! 

Siedzę nad tym jakiś czas i główkuję. Nie potrafię się nagłowić. Dlaczego to nie działa:

   else if(Command.substr(0, 4)=="cd.."){

            string path = CurrentDir;
            if(count(path.begin(), path.end(), '\\')!=1){
                if(path[path.length()-1]=='\\')
                path.erase(path.length()-1, 1);

                cout<<"pre: "<<path<<endl;
                auto beg = path.find_last_of('\\');
                auto end = path.find_last_not_of('\\');
                path.erase(beg, (end-beg)+1);
                cout<<"post: "<<path<<endl;
                CurrentDir = path;
            }
        }

niby post zwraca poprawną ścieżkę (defaultowa jest C:\Windows\System32\, post zwraca C:\Windows) ale za drugim razem wywal błąd out of range :/. Nie rozumiem gdzie leży problem

 

Pozdrawiam, Hiskiel.

Answer 1

Jak wyrzuca std::out_of_range to znaczy, że pierwszy parametr metody std::string::erase jest większy od długości stringa. W kodzie w ogóle nie sprawdzasz wartości zwróconej z metody find_last_of, która może zwrócić std::string::npos. Przekazanie takiej wartości do erase może spowodować wyrzucenie wyjątku.

Answer 2

 path.erase(path.length()-1, 1); 

std::string::pop_back

 

Tak daj:

    auto beg = path.find_last_of('\\');
    if(beg != string::npos) path.erase(beg);
    cout << "post: " << path << endl;