->deleteLater()

[Igors]

"Deleting a QObject while pending events are waiting to be delivered can cause a crash."

Я так понимаю что здесь речь идет о том что нельзя удалять объект если в EventLoop другой нитки есть для него события

Я уверен что будет коре.

"Прошу исполнить" 

Связавшись с deleteLater все время придется думать типа "а что будет в той или иной ситуации?". А для начала мы вообще расписались что это будет работать только в событийной (event-driven) схеме. Потребуется консольная/сетевая версия - и как выцарапывать то deleteLater неясно, а в чужом коде - вообще завал. Впрочем это стандартная проблема любого "deferred delete" и Qt не хуже остальных.

Наверное лучше не спешить лезть головой в петлю, и без крайней необходимости штучки типа deleteLater не использовать

[Авварон]

Наверное лучше не спешить лезть головой в петлю, и без крайней необходимости штучки типа deleteLater не использовать

не согласен

[ieroglif]

простите, конечно, но пока ещё сомневаюсь.
до сих пор не сталкивался с проблемами на deleteLater().

Код

C++ (Qt)
QObject *obj = new QObject(this);
obj->deleteLater();
delete obj;        // новая строка
obj=new QObject(this);

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

[andrew.k]

а что будет если сделать так?

Код

C++ (Qt)
QObject *obj =new QObject();
delete obj;
delete obj;
 

Вот то же самое будет и в том примере.
Как евентлуп получив сообщение, что нужно удалить объект узнает, что он уже был удален?

А нитки там не при чем. Просто при обработке событий может потребоваться обращение к самому объекту, который уже удален.
Просто без множества потоков такую ситуацию надо умудриться получить)

[Igors]

а что будет если сделать так?

Код

C++ (Qt)
QObject *obj =new QObject();
delete obj;
delete obj;
 

Вот то же самое будет и в том примере.

Проверим

Код

C++ (Qt)
#include <QtGui>
 
struct MyObj : public QObject {
	virtual ~MyObj( void )
	{
		qDebug() << "destroy";
	}
};
 
int main(int argc, char **argv)
{
	QApplication app(argc, argv);
	MyObj * obj = new MyObj();
	obj->deleteLater();
	delete obj;
 
	app.exec();
 
 	return 0;
}
 

У меня "destroy" печатается один раз.

Как евентлуп получив сообщение, что нужно удалить объект узнает, что он уже был удален?

Он его просто не получит т.к. деструктор вызовет removePostedEvents (см. исходники)

это бессмысленно.

Просто первые 2 строчки в одном куске кода. 2 последние в другом - и как минимум непросто отследить порядок вызовов

Код

C++ (Qt)
QObject *obj = new QObject(this);
obj->deleteLater();
...
 

Код

C++ (Qt)
delete obj;
obj=new QObject(this);
...
 

[Igors]

не согласен

Ну это Ваше право Годика 2 назад я ухватил прекрасный заказ - 80% у меня уже было сделано. Вот только UI там было на wxWidgets. Пришлось подучиться портированию и всему такому

[Авварон]

не согласен

Ну это Ваше право Годика 2 назад я ухватил прекрасный заказ - 80% у меня уже было сделано. Вот только UI там было на wxWidgets. Пришлось подучиться портированию и всему такому

К чему этот оффтоп?
А насчет делета - попробуйте удалить sender() в обработке QAbstractItemView::doubleClicked(QModelIndex) делетом.

[andrew.k]

ну значит разработчики предусмотрели защиту от дурака. славно.
но это не значит, что так нужно делать.

[Alex Custov]

Проверим

Код

C++ (Qt)
#include <QtGui>
 
struct MyObj : public QObject {
	virtual ~MyObj( void )
	{
		qDebug() << "destroy";
	}
};
 
int main(int argc, char **argv)
{
	QApplication app(argc, argv);
	MyObj * obj = new MyObj();
	obj->deleteLater();
	delete obj;
 
	app.exec();
 
 	return 0;
}
 

У меня "destroy" печатается один раз.

А ты хотел чтобы два раза? Двойное удаление приводит к битью памяти, которое потом вылезти может где угодно.

[Igors]

Двойное удаление приводит к битью памяти, которое потом вылезти может где угодно.

Та неужели? Никогда не слыхал, наверное сильно память бьют (асе электроны разбегаются)