Как правильно освободить память в QHash?

[evgenm27]

Добрый день. Не могу разобраться с парой вопросов. Выделяю память под объект Client, потом этот указатель на объект помещаю в QHash с помощью insert.

Код:

QHash<int, Client*> Users;
Client *_client = new Client(var);
Users.insert(num, _client);

И потом когда мне нужно освободить память и удалить объект. Достаточно сделать

Код:

Users.erase(Users.find(num));

или нужно так:

Код:

Users.erase(Users.find(num));
delete _client;

И второй вопрос. В чем все таки разница между методами erase и remove в QHash? Буду благодарен за любые пояснения.

[Авварон]

Да, надо удалять руками, хэш просто хранит указатель на объект, а жизнью объекта должен управлять кто-то другой.

Используйте std::unordered_map и умные указатели (не получится, правда, с QString как ключ, но для int сойдет):

Код:

std::unordered_map<int, std::unique_ptr<Client>> Users;
auto client = std::make_unique<Client>();
Users.insert({num, std::move(client));
Users.erase(Users.find(num));

erase удаляет по итератору (который надо самому найти или как-то вычислить), а remove - по ключу.

[Alex Custov]

>Используйте std::unordered_map

Почему? Я не проверял, но думаю QHash+QSharedPointer подойдёт.

[Авварон]

Почему? Я не проверял, но думаю QHash+QSharedPointer подойдёт.

Подойдёт, но QSharedPointer - это накладные расходы в виде счётчика ссылок, который может быть и не нужен, если owner известен (хэшмапа).

std::unordered_map лучше тем что
1) умеет в movable-only типы (std::unique_ptr, std::thread)
2) умеет в emplace()
3) по нему удобнее итерироваться через range-for
4) для него не надо использовать qAsConst при итерации через range-for

[ViTech]

Тогда ещё немного оффтопа ).

Есть ли резон в Qt программах использовать QSharedPointer, когда есть std::shared_ptr?

[evgenm27]

Спасибо, разобрался!

[Авварон]

Тогда ещё немного оффтопа ).

Есть ли резон в Qt программах использовать QSharedPointer, когда есть std::shared_ptr?

Нету, он строго хуже. Например, он умеет только статическую функцию-deleter, а функтор или лямбду уже нет

[Igors]

- Я люблю булки, плюшки, батоны и кекс! Я люблю хлеб, и торт, и пирожные, и пряники, хоть тульские, хоть медовые, хоть глазурованные. Сушки люблю тоже, и баранки, бублики, пирожки с мясом, повидлом, капустой и с рисом.

Я горячо люблю пельмени, и особенно ватрушки, если они свежие, но черствые тоже ничего. Можно овсяное печенье и ванильные сухари.

А еще я люблю кильки, сайру, судака в маринаде, бычки в томате, частик в собственном соку,икру баклажанную, кабачки ломтиками и жареную картошку.

Вареную колбасу люблю прямо безумно, ..