QList . Как узнать, правильно ли удаляются указатели?

[kuzulis]

Доброго времени суток!

как проверить код на утечки памяти?

например я написал код в котором создаются некие объекты, указатели на которые я помещаю в QList

Код:

void Dialog::slNew()
{
    for (int i = 0; i<100000; ++i) {
        TestClass *tc = new TestClass; //создаю мои объекты
        tc->setValue(++cnt); //это просто для проверки объекта потом
        clist.append(tc); //в лист указатели на объекты вношу
    }
}

и очищаю лист как приведено в ассистенте:

Код:

void Dialog::slDel()
{
	while (!clist.isEmpty())
		delete clist.takeFirst();
}

пробовал смотреть в винде через диспетчер задач...
при запуске приложения выделяется к примеру 3 Мб.. жму - создать объекты - становиться к примеру 18 Мб...
жму очистить лист - становится 7Мб ...

т.е получается, что память все больше и больше "кушается" т.е есть утечка?

[lit-uriy]

2 kuzulis, для чистоты проверь в несколько итераций, если возвращается к одному и тому же значению, то скорее всего утечки нет.

[kuzulis]

2 kuzulis, для чистоты проверь в несколько итераций, если возвращается к одному и тому же значению, то скорее всего утечки нет.

не... потребление памяти растет ... я сделал i<1000000 .... и после удаления и последующего создания - память не высвобождается польностью, а накапливается... вот накопил до 60 Мб

но ведь после delete нужно обнулить указатель... а как это в QList можно сделать? ведь после удаления из QList - уже доступ никак к указателю через QList не получить (т.к он уже удален из QList) 

[Winstrol]

но ведь после delete нужно обнулить указатель...

Обнулять указатели после delete не нужно. В профилактических целях даже нельзя.
Что касается утечки, то полный код код, локализованный в одной фунции посмотреть бы.

[Rcus]

Код

C++ (Qt)
void MainWindow::on_pushButton_clicked()
{
    for (int i = 0; i < 1000000; ++i)
        list.append(new QString("Test"));
    while (!list.isEmpty())
        delete list.takeFirst();
}

У меня в несколько модифицированном тесте объем потребляемой памяти стабилизировался после первой итерации, ищите дальше.

[spirit]

а dtor'ы TestClass и всех ваших классов вызываются и отрабатываются верно(поставте бряку или qDebug, что бы убедиться)?
ЗЫ. можно также для удаления юзать qDeleteAll.

[kuzulis]

а dtor'ы TestClass и всех ваших классов вызываются и отрабатываются верно(поставте бряку или qDebug, что бы убедиться)?

если можно, то поподробнее пжлста ! где что ставить? (я ж новичек ! )

вот реализация TestClass к примеру:

*.h

Код:

#ifndef MYCLASS_H
#define MYCLASS_H

class TestClass
{
	public:
		TestClass();
		int getValue();
		void setValue(int value);
	private:
		int testvalue;
};

#endif // MYCLASS_H 

*.cpp

Код:

#include "myclass.h"

TestClass::TestClass()
{
	testvalue = 0;
}

int TestClass::getValue()
{
	return testvalue;
}

void TestClass::setValue(int value)
{
	testvalue = value;
}

[kuzulis]

У меня в несколько модифицированном тесте объем потребляемой памяти стабилизировался после первой итерации, ищите дальше.

у меня если жмакаю "создать 10000 объектов" а потом жмакаю "очистить лист" - то тоже вроде устаканивается (хотя по 1-10 байт набирает потихоньку все-равно)

а вот если жмакаю несколько раз "создать 10000 объектов" - а потом "очистить лист" - то память накапливается!

[spirit]

в класс TestClass добавте деструктор и там уже qDebug.

Код

C++ (Qt)
#include <QDebug>
....
TestClass::~TestClass()
{
     ....
     qDebug() << "TestClass::dtor called";
     ....
}
 

но вроде в коде все нормально.

[kuzulis]

2 spirit

я сделал как посоветовали:
1. в консоли мильон раз (i<1000000) отобразилось сообщение:  TestClass::dtor called
2. пока оно выводилось - память потребляемая увеличивалась + приложение подвисло
3. после того как выводится сообщения перестали - память стабилизировалась + приложение отвисло

[Winstrol]

На вид все вроде правильно.
Но:
-память очень желательно отдавать в обратном порядке тому, в котором она выделялась, если нет веских причин не делать этого. Это снизит фрагментацию кучи.
-выделять стандартным аллокатором кучу маленьких объектов - зло. Надо пользоваться каким либо small object allocator / pool и.т.п.

[kuzulis]

2 Winstrol ,

а где взять :

Надо пользоваться каким либо small object allocator / pool и.т.п.

это спец. библиотека нужна с этими функциями?

[Winstrol]

2 Winstrol ,

а где взять :

Надо пользоваться каким либо small object allocator / pool и.т.п.

это спец. библиотека нужна с этими функциями?

Можно и самому написать - много где описано. Случай-то классический. Можно Boost скачать. Тогда примерно так будет выглядеть

Код

C++ (Qt)
boost::object_pool<TestClass> testClassAlloc;
....
void Dialog::slNew()
{
    for (int i = 0; i<100000; ++i) {
        //TestClass *tc = new TestClass; //создаю мои объекты
        TestClass *tc = testClassAlloc.construct(параметры конструктора);
        tc->setValue(++cnt); //это просто для проверки объекта потом
        clist.append(tc); //в лист указатели на объекты вношу
    }
}
void Dialog::slDel()
{
	while (!clist.isEmpty())
           {
                      //delete clist.takeFirst();
                      TestClass* tc = clist.takeFirst();		
                      testClassAlloc.destroy(tc);
           }
}
 

[kuzulis]

2 Winstrol ,

спасибо, буду пробовать!

[Winstrol]

2 Winstrol ,

спасибо, буду пробовать!

Прошу прощения. Так, как показал пробывать не надо. И скачивать тоже не обязательно.

Код

C++ (Qt)
#include <memory>
std::allocator<TestClass> stdTestClassAlloc;
....
TestClass *tc = stdTestClassAlloc.allocate(1);
new (tc) TestClass(параметры коструктора); //создаю мои объекты
....
TestClass* tc = clist.takeFirst();
tc->~TestClass();
stdTestClassAlloc.deallocate(tc);