Оправданно ли использование static в моем случае?

Question

[Sazoks]

У меня есть некий класс. В этом классе у меня есть функционал обработки команд пользователя. Свой "bash" скажем..
И вот у меня возник вопрос. В классе у меня есть private std::map, map этот состоит из пары .
Так же есть метод CommandListener, который принимает строку от пользователя. Дальше этот метод ищет по ключу совпадение со строкой пользователя и, если нашел, выполняет значение, тобишь метод.
Примерный код:
map

std::map<std::string, void(*)()> Functions = {
		{"SomeF1", SomeF1},
		{"SomeF2", SomeF2},
		{"SomeF3", SomeF3},
		{"SomeF4", SomeF4},
                // ...
	};

Обработчик:

void Client::CommandListener(std::string Command)
{
	std::map<std::string, void(*)()>::iterator it = Functions.find(Command);
	if (it == Functions.end()) std::cout << "This command does not exist\n";
	else it->second();
}

Но! Проблема заключается в том, что map отказывается принимать НЕСТАТИЧЕСКИЕ методы! Т.е. SomeF1 (и остальные) мне пришлось объявить static, и только так заработало.
Я это сделал чисто случайно, и теперь мое решение.. Ну мне оно кажется костыльным =(

Подскажите, как лучше поступить в моем случае?
Спасибо!

Comments

[Roman]

Как вариант

#include<iostream>
#include<functional>
#include<map>
#include<string>

using namespace std;

class Client
{
  map<string, function<void()>> functions;
  void printThis()
  {
    cout << "this: " << this << endl;
  }

public:
  Client()
  {
    functions["printThis"] = [this]{ printThis(); };
  }

  void commandListener(const string& command)
  {
    if(auto it{ functions.find(command) }; it != functions.end())
    {
      it->second();
    }
    else
    {
      cout << "This command does not exist\n"
           << "Add this blank command? (y/n) ";
      if(cin.get() == 'y')
      {
        addCommand(command);
        cout << "Ok!" << endl;
      }
    }
  }

  void addCommand(const string& cmd, function<void()>&& fn = {})
  {
    if(functions.count(cmd) == 0)
    {
      functions[cmd] = fn;
    }
  }
};


int main()
{
  Client cl;
  cl.addCommand("printNumber42", [](){ cout << 42 << endl; });
  cl.addCommand("printTextText", [](){ cout << "Text" << endl; });


  cl.commandListener("printNumber42");
  cl.commandListener("printTextText");
  cl.commandListener("blank");
  cl.commandListener("printThis");

  system("pause");
}

Answer 1

[jcmvbkbc]

Подскажите, как лучше поступить в моем случае?

Узнать чем статические методы класса отличаются от обычных методов.
Узнать, что кроме указателей на функции бывают указатели на функции-члены.
Понять, чем из этого лучше воспользоваться в твоём случае.

Comments

[Sazoks]

Про указатели на методы не знал! Спасибо!
Буду копать.

[Sazoks]

Вот спасибо вам, неужели заработало...
Долго не мог понять, почему не работала запись it->second()
В итоге пришлось делать так (*(this).*(it->second))()
УжОс...

[jcmvbkbc]

(*(this).*(it->second))()

Sazoks, можно записать чуть красивее: (this->*it->second)();