Как правильно передавать функцию как параметр C++?

Question

[Денис Ковалев]

Есть класс App

В файле App.cpp:

void App::setup() {
  bt_rest.function("startManufacturing", startManufacturing);
}

int App::startManufacturing(String command) {
  ...
  return 1;
}

bt_rest.function принимает (char * function_name, int (*f)(String))

Но компилятор выдает invalid use of non-static member function на bt_rest.function("startManufacturing", startManufacturing);

Уже все поперепробовал...

Comments

[Евгений Шатунов]

Денис Ковалев , откуда взята конструкция bt_rest.function(...)? Из какой это библиотеки?

[Денис Ковалев]

Евгений Шатунов, да Arduino-библиотека aRest, https://github.com/marcoschwartz/aREST/blob/master...

Answer 1

[Roman]

если bt_rest это https://github.com/marcoschwartz/aREST/blob/master...

то там по другому все устроено.

copy is as in original

// Functions array
  uint8_t functions_index;
  int (*functions[NUMBER_FUNCTIONS])(String);
  char * functions_names[NUMBER_FUNCTIONS];

void function(char * function_name, int (*f)(String)){

  functions_names[functions_index] = function_name;
  functions[functions_index] = f;
  functions_index++;
}

Может так

при условии что App будет в единственном экземпляре.

#include<iostream>
#include<string>
/*
* Будем считать, что std::string это String
*/
#define NUMBER_FUNCTIONS 10
struct aRest
{
  uint8_t functions_index;
  int (*functions[NUMBER_FUNCTIONS])(std::string);
  char* functions_names[NUMBER_FUNCTIONS];
  //...
  aRest() : functions_index(0){};
  void function(char * function_name, int (*f)(std::string))
  {
    functions_names[functions_index] = function_name;
    functions[functions_index] = f;
    functions_index++;
  }
};

class App
{
  aRest bt_rest;
public:
  explicit App() : bt_rest(){};
  //...
  int startManufacturing(std::string command);
  void setup();
  void call(std::string s);// Для теста. Там то же по другому...
  //...
};

int App::startManufacturing(std::string command)
{
  std::cout << "startManufacturing\n"
            << "with command " << command
            << " " << this << std::endl;
  return 1;
}

void App::setup()
{
  bt_rest.function("startManufacturing", (int(*)(std::string))&startManufacturing);
  //reinterpret_cast<int(*)(std::string)>(&startManufacturing)
}

void App::call(std::string s)
{
  bt_rest.functions[0](s);
}

int main()
{
  App ap;
  ap.setup();
  ap.call("COMMAND");
}

или форкнуть aRest

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

#define NUMBER_FUNCTIONS 10
template<typename T>
struct aRest
{
  uint8_t functions_index;

  int (T::*functions[NUMBER_FUNCTIONS])(std::string);
  char* functions_names[NUMBER_FUNCTIONS];
  //...
  aRest() : functions_index(0){};

  void function(char * function_name, int(T::*f)(std::string))
  {
    functions_names[functions_index] = function_name;
    functions[functions_index] = f;
    functions_index++;
  }
};

class App
{
  aRest<App> bt_rest;
public:
  App() : bt_rest(){};
  //...
  int startManufacturing(std::string command);
  void setup();
  void call(std::string fn);
  //...
};

int App::startManufacturing(std::string command)
{
  std::cout << "startManufacturing\n"
            << "with command " << command
            << " " << this << " " << std::endl;
  return 1;
}

void App::setup()
{
  bt_rest.function("startManufacturing", &startManufacturing);
}

void App::call(std::string s)
{
  std::mem_fn(bt_rest.functions[0])(this, s);
}

int main()
{
  App ap;

  ap.setup();

  ap.call("COMMAND");
}

Comments

[Mercury13]

Может, и сработает, но связано с особенностями передачи this на AVR.

[Денис Ковалев]

Помогла эта строчка

bt_rest.function("startManufacturing", (int(*)(std::string))&startManufacturing);

Но в итоге выдает ошибку в библиотеке aRest:
In function `void aREST::addToBuffer(bool, bool)':
(.text+0x0): multiple definition of `void aREST::addToBuffer(bool, bool)'

Потратил на это 2 дня и решил плюнуть на это и просто передавать aREST'у в качестве callback'ов функции из App

[Roman]

Денис Ковалев, это смотрите как у вас заголовочные файлы подключены
multiple definition of это значит множественное определение

[Денис Ковалев]

Roman, Я понимаю. Но оно указывает на функцию в этой библиотеке и на мою фунцию с другим именем, вообще из другого файла.

[Roman]

Денис Ковалев, кусок

мою фунцию с другим именем, вообще из другого файла.

можете показать?

[Денис Ковалев]

Roman,

void buttonStopHeatProcess_PopCallback(void *ptr)
{
  digitalWrite(HEATER_PIN_RELAY, POWER_OFF);
  flagHeating = false;
  pageIngredientes2.show();
}

А это функция из библиотеки:

template <>
void aREST::addToBuffer(bool toAdd, bool quotable) {
  addStringToBuffer(toAdd ? "true" : "false", false);   // Booleans aren't quoted in JSON
}

[Roman]

Денис Ковалев,
хотя это на стадии линковки,
а у вас во всех заголовочных файлах есть #ifndef?

[Денис Ковалев]

Roman, да

Answer 2

[Mercury13]

Не забывайте, что у startManufacturing есть скрытый параметр this.

Четыре варианта.

1. Использовать указатель на метод App:

Rest::function(char * function_name, int (App::* f)(String), App& object)
      // вместо App можно какой-то интерфейс, который App реализует
…
object.*f("string");
...
bt_rest.function("", &App::startManufacturing, *this);

2. Сделать startManufacturing static:

class App {
  static int startManufacturing(String command)
};

3. Сделать обёртку с замыканием:

Rest::function(char * function_name, int (*f)(String, void*), void*);

void doStartManufacturing(String command, void* closure) {
  reinterpret_cast<App*>(closure)->startManufacturing(command);
}
...
bt_rest.function("startManufacturing", doStartManufacturing, this);

4. «Избегай незнакомых женщин и глобальных переменных». Костыль, в общем.

App app;
int doStartManufacturing(String command) { return app.startManufacturing(command); }
...
bt_rest.function("startManufacturing", doStartManufacturing);

Ах да. Вы передаёте String’и по значению. Приспособлены они к такой передаче или всё же лучше по ссылке?

Comments

[Денис Ковалев]

Спасибо за ответ! Пробую первый способ. Ругается на

void function(char * function_name, int (App::* f)(String), App& obj){

error: expected id-expression before '*' token
void function(char * function_name, int (App::* f)(String), App& obj){

expected ')' before '::' token

[Mercury13]

Денис Ковалев,
1. Класс App должен быть хотя бы недоопределён.
2. Я ошибся с вызовом. Вернулся домой и проверил — (obj.*f)(5);

[Mercury13]

#include <iostream>

class App;

void function(char * function_name, int (App::* f)(int), App& obj)
{
    (obj.*f)(5);
}

class App
{
public:
    int start(int x)
    {
        std::cout << x << std::endl;
        return 1;
    }
};

int main()
{
    App app;
    function("111", &App::start, app);
    return 0;
}

[Денис Ковалев]

Mercury13, у меня функция bt_rest.function вызывается внутри метода класса App. Покажите пожалуйста такой пример.

[Mercury13]

#include <iostream>

class App;

void function(const char * function_name, int (App::* f)(int), App& obj)
{
    (obj.*f)(5);
}

class App
{
public:
    int y = 10;

    int start(int x)
    {
        std::cout << x + y << std::endl;
        return 1;
    }

    void exec()
    {
        function("111", &App::start, *this);
    }
};

int main()
{
    App app;
    app.exec();
    return 0;
}

[Денис Ковалев]

Mercury13, хорошо, спасибо, завтра буду пробовать

Answer 3

[GavriKos]

Нельзя просто так взять, и передать нестатический метод. Потому что неясно, у какого экземпляра его вызывать.
Либо делайте статический метод, либо переделывайте bt_rest.function чтобы она еще и экземпляр принимала.