2d QList Memory Leak

Question

Я создал этот 2D Qlist. Код «работает», но когда я рассматриваю немного более глубокий синтаксис, мне остается задаться вопросом, приводит ли этот код к утечке памяти.

Было бы здорово, если бы кто-то мог пролить свет на то, приводит ли этот код к утечке памяти или нет. Если это так, ПОЧЕМУ и как исправить?

Мои особые вопросы касаются вопросов, отмеченных как Q1, Q2, Q3, Q4 в комментариях кода.

Также, как я должен уничтожить некоторые строки из lszq2DList и в конечном итоге, как я должен уничтожить lszq2DList.

//The 2d data sructure. Each row will be a QStringList. 
QList<QStringList> lszq2DList; 

//The variable "emptyTempList" is a concern to me. 
//Q1:: Isn't "emptyTempList" just a pointer to some allocated place in memory? 
//If so, I imagine it points to some data structure allocated in memory. 
//Or does this not allocate anything? I have a feeling it does not. 
QStringList emptyTempList; //I use it in my scheme to make a 2D data struct. 

for(int i=0; i<3; i++){ 
    //Q2: Because I am in a loop and reusing the same pointer "emptyTempList", 
    //aren't i pointing to the same thing? 
    //If I think about this, each item in my Qlist (each row) should be 
    //pointing to the same place in memory. To whatever "emptyTempList" 
    //points to.  
    //Of course, this is not the case. What is really going on in line bellow? 
    lszq2DList += emptyTempList; //I'm no really pointing to anything am I? 

    QString szqTempString; //Q3: Gets trashed when out of scope? 
    for(int j=0; j<3; j++){ 
     szqTempString.clear(); 
     szqTempString = "jazz"; 

      //QString value gets copied into datastructure? 
     lszq2DList[i] += szqTempString; 
    } 
} 


QStringList emptyTempList2; 
emptyTempList2 += "blues"; 
emptyTempList2 += "blues"; 
emptyTempList2 += "blues"; 

//I'll add another row. 
lszq2DList += emptyTempList2; 
//Q4: lszq2DList[3] does not point to emptyTempList2, right? 
//Instead it copies all the strings 
//from to emptyTempList2 to lszq2DList[3], right? 

Answer 1

Я предлагаю проверить утечки памяти, используя что-то похожее на valgrind или другое.

Однако, когда память «утечки», это означает, что после использования эта память просто не освободилась, и весь доступ был удален. Это, по сути, означает, что имеется память, выделенная в куче, например, но ничто не имеет к ней доступа, так что это практически бесполезная память. Например, когда вы динамически выделяете память указателю с помощью «new», вы должны «удалить», чтобы освободить память. Приведу пример.

char * ptr = new char; 
ptr = NULL; // Memory leak 
delete ptr; // This is useless since it no longer points to the memory location 

Однако это совершенно действует

char * ptr = new char; 
delete ptr; 
ptr = NULL; 

Причина первая одна утечка, потому что вы не «удалить» память. Поскольку местоположение указателя было изменено на «NULL» до того, как было вызвано удаление, он не знал, чтобы освободить память по любому адресу, связанному с указателем , до, установив его в NULL.

Тем не менее, это не похоже, что ваш код будет иметь какие-либо утечки памяти в нем (ничего не динамически выделяется). Поскольку похоже, что все ваши переменные выделяются в стеке, система управляет этой памятью для вас - как только эти объекты выходят из области видимости (если она находится в main(), а затем, когда программа выходит), они будут выскользнуть из стека система. Это предполагает, однако, что QList не протекает.

EDIT

Помните, что при создании экземпляра класса, память выделяется в стеке (память освобождается сразу после выхода из объема), которые вам не нужно беспокоиться. Динамическое распределение (распределение/освобождение памяти с использованием нового/удаления) сохраняется в куче (постоянная память для наших целей). Поэтому вам не стоит беспокоиться о утечке памяти, которую вы не динамически выделяли.

РЕДАКТИРОВАТЬ 2

В отношении использованием кучи против стека, использование стека каждый раз, когда это возможно. Чем меньше памяти вы должны управлять, тем лучше (и меньше подвержено ошибкам). Что касается второго вопроса, вы не используете указатели. Что вы делаете, это копирование значения из emptyTempList и не повторное использование его памяти (поскольку это не указатель). Вы должны использовать оператор & для извлечения адреса памяти из вашего объекта, так как, опять же, это не указатель.

Надеюсь, это поможет!

С уважением,
Dennis M.

Answer 2

QList, QStringList и QString все "неявное совместное использование", которое в основном означает "копию на записи". Например:

QStringList someFunction() { 
    QStringList list; 
    QString str("Hello World!"); 

    for(int x = 0;x < 100; ++x) { 
     list << str; 
    } 

    QStringList list2 = list; 
    QStringList list3 = list; 
    QStringList list4 = list; 

    return list4; 
} 

QStringList list5 = someFunction(); 

В конце концов, у вас все еще есть строка «Hello World!». в памяти один раз. Вы можете узнать больше об этом Here. Как правило, с классами Qt «объекты, которые наследуют от QObject, переходят в кучу, иначе они идут в стек». Есть исключения, но очень мало и неявно разделяемые классы всегда должны идти в стек.