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

Question

sentinelШілде 20, 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);
    }
}

Пікірлер

Тек рұқсаты бар пайдаланушылар ғана пікір қалдыра алады.
Кіріңіз немесе Тіркеліңіз