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

Question

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

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

Подписка на обсуждение2

Подписка на раздел248

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

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);
    }
}

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

Рекомендуем хостинг TIMEWEB

Комментарии