0

Набросал вот такой пример:

#include <iostream>
#include <functional>

// ──────────────────────────────────────────────────────────────────────────────

using CallbackType = std::function<void(void*)>;

// ──────────────────────────────────────────────────────────────────────────────

class Karbofos {
  
    CallbackType *Begin = nullptr;
    CallbackType *End = nullptr;
  
  public:
  
    const std::string Name = "Карбофос";
    
    void SetCallback(CallbackType *B, CallbackType *E) {
      Begin = B;
      End = E;
    }
    
    void Run() {
      if (Begin) (*Begin)(this);
      std::cout << "- Не отдам, слон - мой!\n";
      if (End) (*End)(this);
    }
};

// ──────────────────────────────────────────────────────────────────────────────

void Shef(void* i) {
  std::cout << "- Стой, живодер " << ((Karbofos*)(i))->Name << "!\n";   
}

// ──────────────────────────────────────────────────────────────────────────────

void Kollega(void* i) {
  std::cout << "- " << ((Karbofos*)(i))->Name << ", ты немец и контрабандист!\n";   
}

// ──────────────────────────────────────────────────────────────────────────────

int main() {
  Karbofos Object;  
  CallbackType S = Shef;
  CallbackType K = Kollega;
  Object.SetCallback(&S,&K);
  Object.Run();
  return 0;
}

Вопрос

Можно ли как-то обойтись без временных переменных Chef и Kollega, и сразу нужные ссылки передать в метод SetCallback?

8
  • 1
    Возможно Object.SetCallback(new CallbackType(), new CallbackType());
    – V-Mor
    22 июл 2020 в 3:28
  • Точно! Переработался я)
    – Majestio
    22 июл 2020 в 3:31
  • Знакомо) сейчас вынесу в ответ.
    – V-Mor
    22 июл 2020 в 3:32
  • Осталось как-то возможную утечку памяти подпилить хитрыми указателями.
    – Majestio
    22 июл 2020 в 3:35
  • А зачем вам указатели? 22 июл 2020 в 3:52

2 ответа 2

4

Вам действительно необходимы указатели на std::function? Тоже самое можно сделать и без них:

#include <utility>
#include <iostream>
#include <functional>

//──────────────────────────────────────────────────────────────────────────────

using CallbackType = std::function<void(void*)>;

// ──────────────────────────────────────────────────────────────────────────────

class Karbofos {
  
    CallbackType Begin;
    CallbackType End;
  
  public:
  
    const std::string Name = "Карбофос";

    void SetCallback(CallbackType B, CallbackType E) {
      Begin = std::move(B);
      End = std::move(E);
    }
    
    void Run() {
      if (Begin) Begin(this);
      std::cout << "- Не отдам, слон - мой!\n";
      if (End) End(this);
   }
};

// ──────────────────────────────────────────────────────────────────────────────

void Shef(void* i) {
  std::cout << "- Стой, живодер " << ((Karbofos*)(i))->Name << "!\n";   
}

// ──────────────────────────────────────────────────────────────────────────────
 
void Kollega(void* i) {
  std::cout << "- " << ((Karbofos*)(i))->Name << ", ты немец и контрабандист!\n";   
}

// ──────────────────────────────────────────────────────────────────────────────

int main() {
  Karbofos Object;  
  Object.SetCallback(Shef, Kollega);
  Object.Run();
  return 0;
}
5
  • 1
    Тока хедер <utility> для мува нужен
    – dIm0n
    22 июл 2020 в 5:53
  • Да, так гораздо лучше!
    – Majestio
    22 июл 2020 в 7:11
  • И попутно вопрос, если не использовать std::move, то что?
    – Majestio
    22 июл 2020 в 7:18
  • @Majestio ничего, просто будет копирование, вместо перемещения. 22 июл 2020 в 8:52
  • @dIm0n добавил. 22 июл 2020 в 10:32
1

Т.к. при создании объектов Shef и Kollega используется конструктор по умолчанию, получить аналогичные объекты (разве что с той лишь разницей, что память для них будет выделена в куче, а не в стеке) можно с помощью оператора new и пустых скобок после указания типа. То есть строки

 CallbackType S = Shef;
 CallbackType K = Kollega;
 Object.SetCallback(&S,&K);

Заменятся на лаконичное

Object.SetCallback(new CallbackType(), new CallbackType());

Также обращу внимание на то, что, т.к. память теперь выделена в куче, необходимо будет очистить её соответствующим delete после использования объектов, либо обернуть всё сие в умные указатели, например, так:

#include <iostream>
#include <functional>
#include <memory>

// ──────────────────────────────────────────────────────────────────────────────

using CallbackType = std::function<void(void*)>;

// ──────────────────────────────────────────────────────────────────────────────

class Karbofos {

    // Тут поменял на соотв. типы
    std::shared_ptr<CallbackType> Begin = nullptr;
    std::shared_ptr<CallbackType> End = nullptr;

public:

    const std::string Name = "Карбофос";

    // Поменял типы аргументов
    void SetCallback(std::shared_ptr<CallbackType> B, std::shared_ptr<CallbackType> E) {
        Begin = B;
        End = E;
    }

    void Run() {
        if (Begin) (*Begin)(this);
        std::cout << "- Не отдам, слон - мой!\n";
        if (End) (*End)(this);
    }
};

// ──────────────────────────────────────────────────────────────────────────────

void Shef(void* i) {
    std::cout << "- Стой, живодер " << ((Karbofos*)(i))->Name << "!\n";
}

// ──────────────────────────────────────────────────────────────────────────────

void Kollega(void* i) {
    std::cout << "- " << ((Karbofos*)(i))->Name << ", ты немец и контрабандист!\n";
}

// ──────────────────────────────────────────────────────────────────────────────

int main() 
{
    Karbofos Object;

    // Заменил здесь
    Object.SetCallback(std::shared_ptr<CallbackType>(new CallbackType(Shef)), std::shared_ptr<CallbackType>(new CallbackType(Kollega)));
    Object.Run();
    return 0;
}

P.S. Чтобы привести код к рабочему состоянию, мне потребовалось добавить .c_str() после ((Karbofos*)(i))->Name при выводе в обоих случаях.

9
  • Запили, плс, еще вариант на хитрых указателях - и сразу жму зеленую галочку))
    – Majestio
    22 июл 2020 в 3:38
  • Сейчас попробую. Возможно получится кривенько, но работать должно.
    – V-Mor
    22 июл 2020 в 3:39
  • Увы, пока вылетает с исключением, std::bad_function_call, см.тут - ideone.com/ZY9XDh
    – Majestio
    22 июл 2020 в 3:58
  • Ясн ... new CallbackType() - аргумента нет
    – Majestio
    22 июл 2020 в 4:01
  • Ну в общем - thx!)
    – Majestio
    22 июл 2020 в 4:05

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.