sentinel
sentinel20 июля 2018 г. 3: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() - объекты удаляются, падений больше нет. Насколько такой подход неправильный?


Рекомендуем хостинг TIMEWEB
Рекомендуем хостинг TIMEWEB
Стабильный хостинг, на котором располагается социальная сеть EVILEG. Для проектов на Django рекомендуем VDS хостинг.

Вам это нравится? Поделитесь в социальных сетях!

1
Evgenii Legotckoi
  • 20 июля 2018 г. 3:31
  • (ред.)
  • Ответ был помечен как решение.

метод 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);
    }
}




    Комментарии

    Только авторизованные пользователи могут публиковать комментарии.
    Пожалуйста, авторизуйтесь или зарегистрируйтесь
    B

    C++ - Тест 002. Константы

    • Результат:16баллов,
    • Очки рейтинга-10
    B

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

    • Результат:46баллов,
    • Очки рейтинга-6
    FL

    C++ - Тест 006. Перечисления

    • Результат:80баллов,
    • Очки рейтинга4
    Последние комментарии
    k
    kmssr9 февраля 2024 г. 2:43
    Qt Linux - Урок 001. Автозапуск Qt приложения под Linux как сделать автозапуск для флэтпака, который не даёт создавать файлы в ~/.config - вот это вопрос ))
    АК
    Анатолий Кононенко5 февраля 2024 г. 9:50
    Qt WinAPI - Урок 007. Работаем с ICMP Ping в Qt Без строки #include <QRegularExpressionValidator> в заголовочном файле не работает валидатор.
    EVA
    EVA25 декабря 2023 г. 18:30
    Boost - статическая линковка в CMake проекте под Windows Ошибка LNK1104 часто возникает, когда компоновщик не может найти или открыть файл библиотеки. В вашем случае, это файл libboost_locale-vc142-mt-gd-x64-1_74.lib из библиотеки Boost для C+…
    J
    JonnyJo25 декабря 2023 г. 16:38
    Boost - статическая линковка в CMake проекте под Windows Сделал всё по-как у вас, но выдаёт ошибку [build] LINK : fatal error LNK1104: не удается открыть файл "libboost_locale-vc142-mt-gd-x64-1_74.lib" Хоть убей, не могу понять в чём дел…
    G
    Gvozdik19 декабря 2023 г. 5:01
    Qt/C++ - Урок 056. Подключение библиотеки Boost в Qt для компиляторов MinGW и MSVC Для решения твой проблемы добавь в файл .pro строчку "LIBS += -lws2_32" она решит проблему , лично мне помогло.
    Сейчас обсуждают на форуме
    P
    Pisych27 февраля 2023 г. 12:04
    Как получить в массив значения из связанной модели? Спасибо, разобрался:))
    AC
    Alexandru Codreanu19 января 2024 г. 19:57
    QML Обнулить значения SpinBox Доброго времени суток, не могу разобраться с обнулением значение SpinBox находящего в делегате. import QtQuickimport QtQuick.ControlsWindow { width: 640 height: 480 visible: tr…
    BlinCT
    BlinCT27 декабря 2023 г. 16:57
    Растягивать Image на парент по высоте Ну и само собою дял включения scrollbar надо чтобы был Flickable. Так что выходит как то так Flickable{ id: root anchors.fill: parent clip: true property url linkFile p…
    Дмитрий
    Дмитрий10 января 2024 г. 12:18
    Qt Creator загружает всю оперативную память Проблема решена. Удалось разобраться с помощью утилиты strace. Запустил ее: strace ./qtcreator Начал выводиться весь лог работы креатора. В один момент он начал считывать фай…
    Evgenii Legotckoi
    Evgenii Legotckoi12 декабря 2023 г. 14:48
    Побуквенное сравнение двух строк Добрый день. Там случайно не высылается этот сигнал textChanged ещё и при форматировани текста? Если решиать в лоб, то можно просто отключать сигнал/слотовое соединение внутри слота и …

    Следите за нами в социальных сетях