Как передать оператор разыменования в функцию?

Question

[Даниил Демидко]

Есть контейнер с указателями, его нужно преобразовать в контейнер объектов, т. е. просто разыменовать каждый указатель. Сейчас я делаю это с помощью transform и лямбды.

vector<GameObject> ToVector(
    const set<GameObject*, DereferenceCompare<GameObject, greater>> &cont)
{
    vector<GameObject> res(size(cont));
    transform(cbegin(cont), cend(cont), begin(res), [](GameObject *const p) {
        return *p; 
    });
    return res;
}

Есть ли способ сократить код еще больше, может быть какая нибудь стандартная функция есть, которая проводит разыменование указателя?

Comments

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

Даниил Демидко , какая цель у этого преобразования?
Я с полной уверенностью могу сказать что тебе не надо создавать новый вектор игровых объектов из коллекции указателей на другие игровые объекты.

[Даниил Демидко]

Евгений Шатунов, надо, учебное задание такое

[Roman]

Даниил Демидко, не знаю на счет краткости

но оптимизировать чуть чуть можно

UPD:

#include<iostream>
#include<string>
#include<vector>
#include<set>

using namespace std;

class GameObject
{
  string name;

public:
  GameObject(string n) : name{n}
  {
  }
  
  string getName()
  {
    return name;
  }
};

vector<GameObject> ToVector(const set<GameObject*>& cont)
{
  vector<GameObject> res;
  res.reserve(cont.size());
  for(auto& p : cont)
  {
    if(p) res.push_back(*p);
  }
  return res;
}


int main()
{

  set<GameObject*> gset = {
    new GameObject("one"),
    new GameObject("two"),
    new GameObject("three")
  };

 vector<GameObject> test{ToVector(gset)};

  for(auto& v : gset)
  {
    cout << v->getName() << " ";
    delete v;
  }
  cout << '\n';
  for(auto& v : test)
  {
    cout << v.getName() << " ";
  }
  cin.get();
}

[Vitaly]

Roman, И где вы оптимизировали?
1) RVO
2) На одну аллокацию больше. Вектор в куче не надо выделять, потому что 1)
3) Оператор/конструктор перемещения не работает, т.к. нет noexcept, std::vector специально вызывает move_if_noexcept.
4) Опять-таки нет проверки на null при разыменование указателя.

Как итог, имеем полное непонимание C++11 и выше и переусложнённый код. Вариант автора вопроса и быстрее и проще. Ну только никакой проверки на null нет...

[Roman]

Vitaly,
1, 2 вы правы. (только наверно NRVO)

но

быстрее и проще

что здесь происходит vector<GameObject> res(size(cont));?

[Vitaly]

А в 3 и 4 не прав? :)

что здесь происходит vector<GameObject> res(size(cont));?

Создание вектора и заполнение объектами, сконструированными по-умолчанию. В общем и целом я бы предпочёл reserve+back_inserter, что скорее всего будет быстрее. emplace_back бесполезен, т.к. мы объект всё равно конструируем в другом месте, а потом копируем/перемещаем. Но мы ничего не знаем о свойствах объектов, т.ч. в рамках такой неопределённости, я бы предпочёл вариант автора вопроса (разумеется с reserve+back_inserter) вашему.
Ну и сам я бы всё вообще по-другому сделал :)

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

Даниил Демидко , "надо" - это не ответ. :)
Еще раз. Какая цель у этого преобразования?
Для чего тебе из сета указателей получать вектор значений?

[Даниил Демидко]

Евгений Шатунов, так задание и звучит
Это один из методов класса который мне поручено сделать. Он должен возвращать вектор)

Answer 1

[Vitaly]

Короче будет только с библиотекой range (из буста, или range-v3).
К слову, вам надо проверять указатель на null, хотя бы ассертом, если уверены, что там всегда валидные указатели.
Кроме того, вы по сути копируете объекты. Это дорого. Если этого не планировалось, то советую использовать интеллектуальные указатели или reference_wrapper. К тому же, в современном C++ практически нет необходимости использовать сырые указатели.