Russian Qt Forum

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

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

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



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


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

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

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

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

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

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

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();
}

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

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

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

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

*.h

*.cpp

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

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

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

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

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

2 spirit

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

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

2 Winstrol ,

а где взять :
???

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

Можно и самому написать - много где описано. Случай-то классический. Можно 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);
           }
}
 

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);
 

а что значит 1 в вызове allocate(тут) ?

Выделить только 1 объект. Но у меня стоит stlport с оптимизированным std::allocator. Если у Вас будет тормозить, то здесь можно неплохой аллокатор скачать http://ders.stml.net/cpp/xcppcomm/doc/files.html

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

Если есть возможность, проверь на утечки при помощи Valgrind. Глянь вот эту тему (http://www.prog.org.ru/topic_2196_0.html).