Политика конфиденциальностиКонтактыО сайтеОтзывыGitHubDonate
© EVILEG 2015-2018
Рекомендует хостинг
TIMEWEB
20 июля 2018 г. 7:05

Правильное удаление объектов с QGraphicsScene

Есть необходимость периодически удалять объекты со сцены.
Сами объекты хранятся в QMap<QString, DeviceItem *>
Удаление делаю так:
void DeviceGraphView::removeDevices(const QStringList &deviceIds) {
    for (auto deviceId: deviceIds) {
        scene()->removeItem(_deviceItems[deviceId]);
        auto device = _deviceItems.take(deviceId);
if (device) delete device; } }
При этом получаю регулярные SEGFAULT'ы. Сейчас удалил removeItem() - объекты удаляются, падений больше нет. Насколько такой подход неправильный?


Виртуальный хостинг со скидкой 10 процентов
Виртуальный хостинг со скидкой 10 процентов
EVILEG предлагает надёжный хостинг со скидкой 10% на виртуальный хостинг и 5% на VPS
1

метод take удаляет только последний добавленный элемент в QMap, при этом он внутри себя использует метод deleteNode, который при необходимости вызывает деструктор.

При этом qmap может содержать несколько значений с одним ключом, поэтому лучше использовать метод remove(), который не возвращает значения, но при этом удаляет абсолютно все объекты с текущим ключом.  При это в случае необходимости также вызывается деструктор, то есть явный вызов delete не должен потребоваться.
В данной ситуации обязательно нужно удалять объект с графической сцены.
Теоретически исправный код может быть таким
void DeviceGraphView::removeDevices(const QStringList &deviceIds) {
    
    for (const auto& deviceId : deviceIds) 
    {
        for (auto* item : _deviceItems.values(deviceId))
        {
            scene()->removeItem(item);
        }
        _deviceItems.remove(deviceId);
    }
}




0

Комментарии

Только авторизованные пользователи могут публиковать комментарии.
Пожалуйста, авторизуйтесь или зарегистрируйтесь
ПМ
18 марта 2019 г. 15:12
Пётр Махнёв

C++ - Тест 004. Указатели, Массивы и Циклы

  • Результат:50баллов,
  • Очки рейтинга-4
ЯГ
18 марта 2019 г. 7:59
Ян Греку

C++ - Тест 003. Условия и циклы

  • Результат:21баллов,
  • Очки рейтинга-10
МД
15 марта 2019 г. 16:32
Михаил Дементьев

C++ - Тест 001. Первая программа и типы данных

  • Результат:60баллов,
  • Очки рейтинга-1
Последние комментарии
16 марта 2019 г. 13:55
Дмитрий

Спасибо за статью. Давно итересует следующий вопрос: с помощью переменных QMAKE_TARGET_COMPANYQMAKE_TARGET_PRODUCTQMAKE_TARGET_DESCRIPTIONможно задать свойства компилируемой программы, о...
JS
12 марта 2019 г. 10:19
Jean Stefanovich

Большое спасибо за разъяснения!
12 марта 2019 г. 10:04
Евгений Легоцкой

Hello, In fact, this functionality or is not implemented, or is not documented. I'm not sure. But I think, that it should be implemented in Text QML Type. Because of we can write text in...
12 марта 2019 г. 9:51
Евгений Легоцкой

Да вы правы. На самом деле проще через QSqlQueryModel, сколько не пытался использовать эти дженерики типо QSqlTableModel и QSqlRelationalTableModel, то всегда упирался в какие-то их ограничени...
JS
12 марта 2019 г. 9:47
Jean Stefanovich

Единицы измерения лежат там же где и названия продуктов. Просто в таблице ингредиенты нет ещё одного столбца, на который можно было бы установить setRelation. Я в итоге в базе создал ещё один ...
Сейчас обсуждают на форуме
17 марта 2019 г. 22:47
Евгений Легоцкой

Добрый день. Вот, нашлось у меня немного времени. Делается это через шаблон проектирования наблюдатель. GraphKS_mfvSlup.zip
ЧГ
15 марта 2019 г. 21:52
Чарльз Грин

спасибо, попробую, отпишусь
m
15 марта 2019 г. 19:41
mihamuz

Сори догадался)
n
12 марта 2019 г. 16:57
newbie.works.with.QT

Большооооое спасибо!!!!!Не передать как я вам благодарен, спасибо что всегда отзываетесь.Теперь я смогу продолжить работу в QT!!! (пробую писать бота (Я как вы могли догадаться немного не пр...
12 марта 2019 г. 13:20
BlinCT

Ну так если у вас есть готовая программа так ей и воспользуйтесь. Вы же написали ее вот и пользуйтесь.
Присоединяйтесь к нам в социальных сетях

Для зарегистрированных пользователей на сайте присутствует минимальное количество рекламы