Jak napisać własny prosty interpreter?

Question

Witam! Mam problem :/. Otóż potrzebuję napisać interpreter. Chodzi mi o coś takiego, że jeśli prześlę do niego np. !U3"Elo" to ma mi odesłać AdminElo. Program wysypuje się kiedy próbuję dodać więcej niż jeden "modyfikator":

void TAPInterpreter(string Title){

    /*
        Two letters are numbers - text.
        Two letters are !U - Username.
        Two letters are !C - Computer Name.
        Two letters are !P - Current Path.
        Two letters are !D - Current Disk.
        Two letters are !S - Space.
        Two letters are !_ - Pass to a new line.
        Two letters are !! - Exclamation mark.
        Two letters are !\ - Backslash.
        Two letters are !/ - Forward slash.
    */
       int addition;
    for(int i=0;i<Title.length();i++){
        if(i==0)addition = 0;
        else addition +=2;

        string str = Title.substr(0+addition,2+addition);

        if(str.compare("!U")==0){
            cout<<"!U";
        }
        else if(str.compare("!C")==0){
            cout<<"!C";
        }
        else if(str.compare("!P")==0){
            cout<<"!P";
        }
        else if(str.compare("!D")==0){
            cout<<"!D";
        }
        else if(str.compare("!S")==0){
            cout<<"!S";
        }
        else if(str.compare("!_")==0){
            cout<<"!_";
        }
        else if(str.compare("!!")==0){
            cout<<"!!";
        }
        else if(str.compare("!\\")==0){
            cout<<"!\\";
        }
        else if(str.compare("!/")==0){
            cout<<"!/";
        }
        else if(atoi(str.c_str())&&i!=Title.length()){
            int number = atoi(str.c_str());
            cout<<number;
        }
    }
}

 

Niestety nie mogę skorzystać z switch'a.

Comments

Niestety nie mogę skorzystać z switch'a.

Dlaczego?

---

Switch nie chce przyjąć stringa.

---

Stringa nie, ale znak już tak:

size_t i = 0;

while(i < Title.length())
{
	if(Title.at(i) == '!')
	{
		switch(Title.at(++i))
		{
		case 'U': 	std::cout << "!U";	break;
		case 'C': 	std::cout << "!C";	break;
		case 'P': 	std::cout << "!P";	break;
		case 'D': 	std::cout << "!D";	break;
		case 'S': 	std::cout << "!S";	break;
		case '_': 	std::cout << "!_";	break;
		case '!': 	std::cout << "!!";	break;
		case '\\':	std::cout << "!\\";	break;
		case '/': 	std::cout << "!/";	break;
		default: /* błąd?? */
		}

		++i;
	}
	else
	{
		char *str_beg = &Title.at(i);
		char *str_end;
		std::cout << strtol(str_beg , &str_end, 10) << '\n';
		i += str_end - str_beg;
	}
}

 

Answer 1

Metoda substr rzuca wyjątek, jeśli pierwszy argument przekracza długość stringa. I to się dzieje w Twoim programie.

Comments

Ale dlaczego? Jeśli mam np. !C!D to długością jest 4. Za pierwszym razem addition jest równe 0, potem 2. Więc jest 2 i 4. Nic tu nie przekracza.. A w takim razie... Jak mam napisać taki interpreter?

---

Ale pętla wykonuje się tyle razy, ile jest znaków w stringu. Za trzecim razem już masz 6, więc przekracza.

Ps. drugi parametr oznacza ilość znaków, które substr "wyciągnie", nie indeks, na którym skończy.

Answer 2

Na std::string, także można zrobić switch(...) ale to trochę więcej zabawy (constexpr i hash). 

To jest na tyle prosty interpreter, że wystarczy brutalne szukanie sekwencyjne. Prosty szkic:

#include <string>
#include <iostream>
#include <vector>
#include <utility>
#include <functional>

bool startsWith(const std::string& msg, const std::string& pattern) {
    return msg.find(pattern) == 0;
}

std::function<void()> selectFunction(const std::string& pattern) {
    // Komend jest tak mało że brutalne wyszukanie sekwencyjne
    // będzi wystarczająco wydajne.
    static std::vector<std::pair<const char *, std::function<void()>>> commands = {
        { "!U", []{ std::cout << "User\n"; } },
        { "!C", []{ std::cout << "Computer Name.\n"; } },
        { "!P", []{ std::cout << "Current Path.\n"; } },
        { "!D", []{ std::cout << "Current Disk.\n"; } },
        // ...
    };
    auto result = []{ std::cout << "Unknown command.\n"; };
    for(const auto& pr: commands) {
        if(startsWith(pr.first, pattern)) {
            return pr.second;
        }
    }
    return result;
}

int main() {
    std::string command;
    std::cout << "Enter command: ";
    std::cin >> command;
    // TODO: Sprawdzenie czy poprawna etc...
    selectFunction(command)();
}

Jak będziesz chciał mieć bardziej skomplikowane zachowanie (bazujące na komendzie), dodaj do funkcji komendy parametr i tyle :)