Russian Qt Forum

Если я храню в QVariant'e объект, то правильно ли он удалится, если я делаю так:

Неправильно, не удалится.

А как делать правильно?

Думаю, что правильнее будет:


А если не хочется нагромождать, то можно так:


Так даже, наверное, будет лучше...

Ну если именно QColor хранить то
QVariant v = QColor(a,r,g,b);
и все, никаких delete, а для извлечения значения -
QColor c = v.value<QColor>();
или qVariantValue.
собственно я только что пересказал один из абзацев документации по QVariant duh.

Rcus, спасибо, но надо не только QColor, а почти всё остальное тоже. Со всеми остальными QVariant-совместимыми классами так же нельзя ведь работать


Это любой с++ объект можно так создавать что ли, вызывая конструктор через Class::Class(...)? Или это не конструктор? (Не знаю некоторых тонкостей с++)

P.S. guajiro, почему отредактировал пост, убрав "delete &v"? Как тогда правильно освобождать память в этом случае?

Нагромождать не хочется, равно как и хранить разные поинтеры в случае

т.к. создание и удаление будет в разных функциях

да нигде там не нужен new и delete. Все тоже самое что и с QColor, только для классов из QtCore есть простые функции для преобразования, типа toString().

Нету new, т.е. память не отбирается у операционной системы, а значит и возвращать ей ничего не нужно (т.е. не нужен и delete). Я сам на такую штуку попадался.

если я правильно помню, такое удаление - undefined behaviour, и в любом случае так делать нельзя в независимости от QVariant или других классов.

Смотрю я на этот код и думаю, а каким компилятором этот код соберается и что делает?

Да, MS VS выдает на этот код ошибку "error C2100: illegal indirection"

Ну да, правильно. Там ведь нет указателя, а значит и разыменовывать нечего.
Вот так компилятор ругаться не будет:

Так проще будет:  ;)

C++ (Qt)
QVariant v = QColor(0,0,0,0);