1

Есть класс "Camera". В нем есть статический список всех созданных камер:

private:static std::vector<Camera> cameras;

В конструкторе этого класса кладу "себя"(this) в этот список так:

Camera::cameras.push_back(*this);

Хочу удалить "себя" из этого списка в деструкторе. Пытаюсь сделать так:

cameras.erase(std::find(cameras.begin(), cameras.end(), *this));

Выдает ошибку компиляции:

Ошибка  C2678   бинарный "==": не найден оператор, принимающий левый операнд типа "Camera" (или приемлемое преобразование отсутствует)

Что я делаю не правильно? Я думаю нужно переопределить оператор == для сравнения двух камер. Но в этом ли проблема?

10
  • 1
    А вы кладите в вектор указатель, а не объект. std::vector<Camera*> и .push_back(this);
    – gil9red
    26 апр 2016 в 7:00
  • @gil9red, Да так делал до этого но где-то в дебрях stackoverflow вычитал что на производительность это особо не влияет(у меня же не тысячи камер в сцене), и по что по возможности нужно использовать не указатели а просто сами объекты. А вы как храните в основном?
    – cvvvlad
    26 апр 2016 в 7:03
  • 1
    Да по разному, зависит от ситуации. Смотрите, если указатели поместите, вектор сравнит по указателям, если как объекты, тогда создайте оператор ==, чтобы вектор мог сравнить как объекты.
    – gil9red
    26 апр 2016 в 7:07
  • Т.е. дело всё таки в операторе "==". Спасибо! Это и был ответ!!
    – cvvvlad
    26 апр 2016 в 7:10
  • добавляя в вектор объекты по значению, туда заноситься его копия. Вполне возможно, что когда будете удалять, то ничего с вектора не удалиться (все зависит от того, как будет написан оператор ==)
    – KoVadim
    26 апр 2016 в 7:11

2 ответа 2

8

Вот здесь

Camera::cameras.push_back(*this);

вы вносите в вектор копию своего объекта. При удалении вектор пытается сравнивать объекты, а не указатели. Для этого ему нужен оператор сравнивания.

Самое разумное - не копировать свои камеры, копировать адреса. Т.е. использовать

static std::vector<Camera*> cameras;

При этом не будут создаваться лишние сущности, что уже хорошо.

Если вам все-таки необходимо хранить копии, то второй вариант - определить оператор сравнивания, а третий - использовать find_if с соответствующим предикатом (вплоть до лямбды).

Но передача адреса, по-моему, наиболее эффективное решение.

4

Дополню @Harry.

Если не хотите, что бы какой-то зловред положил туда nullptr, можно организовать и вектор "ссылок":

private:static std::vector<std::reference_wrapper<Camera>> cameras;

И ложить туда ссылку на класс:

Camera::cameras.push_back(std::ref(*this));

Ну а поиск нужного элемента можно произвести по его адресу:

cameras.erase(std::find_if(cameras.begin(), cameras.end(), [this](std::reference_wrapper<Camera> wrapper) {
  return &wrapper.get() == this;
}));

Гадость положить туда всё ещё можно, но уже будет сложнее :)

Ваш ответ

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

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