Хеш таблица, групировка значений

[olegla]

так значения у меня и есть числа)

[alexzk]

Вернее так

bool Comp(const std::shared_ptr<Material>& left, const std::shared_ptr<Material>& right)

т.е. избегаем копирования объекта "указатель", хотя тут это не существенно.

[olegla]

Сообщение от alexzk

Вернее так

bool Comp(const std::shared_ptr<Material>& left, const std::shared_ptr<Material>& right)

т.е. избегаем копирования объекта "указатель", хотя тут это не существенно.

а в ифе что изменить?
или ифа уже не нужно , я так понимаю/ тогда как тут правильно в вектор записывать?

gameObjectsByMaterial.push_back(*it);

[alexzk]

std::vector<std::vector<std::shared_ptr<GameObject>>> GroupGameObjectsByMaterial(std::shared_ptr<std::vector<std::shared_ptr<GameObject>>> gameObjects)
{
      std::sort(gameObjects->begin(), gameObjects->end(), Comp); 
       std::vector<std::vector<std::shared_ptr<GameObject>>> gameObjectsByMaterial;
	gameObjectsByMaterial.reserve(gameObjects->size);

        for (const auto &obj : *gameObjects)
        {
              gameObjectsByMaterial.push_back(obj);
        }	
	return gameObjectsByMaterial;
}

[olegla]

Сообщение от alexzk

std::vector<std::vector<std::shared_ptr<GameObject>>> GroupGameObjectsByMaterial(std::shared_ptr<std::vector<std::shared_ptr<GameObject>>> gameObjects)
{
      std::sort(gameObjects->begin(), gameObjects->end(), Comp); 
       std::vector<std::vector<std::shared_ptr<GameObject>>> gameObjectsByMaterial;
	gameObjectsByMaterial.reserve(gameObjects->size);

        for (const auto &obj : *gameObjects)
        {
              gameObjectsByMaterial.push_back(obj);
        }	
	return gameObjectsByMaterial;
}

все равно на этот кусок кода кричит:

gameObjectsByMaterial.push_back(obj);

[alexzk]

а еще вернее - пусть компилер сам копию делают:

std::vector<std::vector<std::shared_ptr<GameObject>>> GroupGameObjectsByMaterial(std::shared_ptr<std::vector<std::shared_ptr<GameObject>>> gameObjects)
{
      std::sort(gameObjects->begin(), gameObjects->end(), Comp); 
       return *gameObjects;
}

т.к. вы работете с указателям, gameObjects уже есть ваш результат. Но вот таким return мы с него еще и копию снимаем.

[alexzk]

можно сделать без указателей, и даже без ссылок:

std::vector<std::vector<std::shared_ptr<GameObject>>> GroupGameObjectsByMaterial(std::vector<std::shared_ptr<GameObject>> gameObjects) //это будет копия оригинала
{
      std::sort(gameObjects.begin(), gameObjects .end(), Comp); 
       return gameObjects; //и возвращаем эту сортированую копию
}

[olegla]

Сообщение от alexzk

а еще вернее - пусть компилер сам копию делают:

std::vector<std::vector<std::shared_ptr<GameObject>>> GroupGameObjectsByMaterial(std::shared_ptr<std::vector<std::shared_ptr<GameObject>>> gameObjects)
{
      std::sort(gameObjects->begin(), gameObjects->end(), Comp); 
       return *gameObjects;
}

т.к. вы работете с указателям, gameObjects уже есть ваш результат. Но вот таким return мы с него еще и копию снимаем.

и всеравно ошибку выдает

[alexzk]

блин, у вас результат вектор векторов...

думаю так нада

std::vector<std::shared_ptr<GameObj ect>>

[alexzk]

std::vector<std::shared_ptr<GameObject>> GroupGameObjectsByMaterial(std::vector<std::shared_ptr<GameObject>> gameObjects) //это будет копия оригинала
{
      std::sort(gameObjects.begin(), gameObjects .end(), Comp); 
       return gameObjects; //и возвращаем эту сортированую копию
}