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

Question

sentinelJuly 20, 2018, 1:05 p.m.

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

We recommend hosting TIMEWEB

Stable hosting, on which the social network EVILEG is located. For projects on Django we recommend VDS hosting.

Subscribe to the discussion2

Section subscription252

Do you like it? Share on social networks!

July 20, 2018, 1:31 p.m.
(edited)
The answer was marked as a solution.

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

Comments

Only authorized users can post comments.
Please, Log in or Sign up