QTableWidget и QTableWidgetItem, удаляются ли ячейки при QTableWidget::clear(); Опции

[silver47]

Добрый день.

Ячейки создаются таким образом:

table = new QTableWidget();
table->setItem(0, 0, new QTableWidgetItem("Hello World!"));

Собственно вроде как утечка памяти. Память выделена, объект создан, но указатель на него отсутствует. Что будет с этим объектом, если вызвать метод table->clear(); уничтожиться ли? Вроде не должен(родитель не указан, да даже, если и указан был бы, то он живой и здоровый). следовательно перед clear, нужно уничтожить все созданные ячейки вручную? Как это сделать?

Спасибо.

UPD.: ячейки не уничтожаются. Необходимо удалять вручную. Есть идеи как это грамотно реализовать? Не таскать же за собой мешок указателей.

пока думаю так:

QTableWidgetItem *pItem;

// это в цикле в зависимости от размеров таблицы
    pItem = table->item(x, y);
    delete pItem;
//

Сообщение отредактировал silver47 - 15.4.2011, 8:19

[wiz29]

void QTableWidget::setItem ( int row, int column, QTableWidgetItem * item )
Sets the item for the given row and column to item.
The table takes ownership of the item.
Это означает, что таблица полностью отвечает за данный элемент, и, соответственно, уничтожит его в случает очистки таблицы, либо удаления самой таблицы.

QTableWidgetItem *pItem;

// это в цикле в зависимости от размеров таблицы
    pItem = table->item(x, y);
    delete pItem;
//

Такой подход в принципе неверен, существует метод :
QTableWidgetItem * QTableWidget::takeItem ( int row, int column )
Removes the item at row and column from the table without deleting it.

[silver47]

Такой подход в принципе неверен, существует метод :
QTableWidgetItem * QTableWidget::takeItem ( int row, int column )
Removes the item at row and column from the table without deleting it.

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

Это означает, что таблица полностью отвечает за данный элемент, и, соответственно, уничтожит его в случает очистки таблицы, либо удаления самой таблицы.

Если я создам QTableWidgetItem и указатель на него, то после удаления элемента указатель куда будет ссылаться? На тот же адрес? Если да, то прошу прощения запутался. Помню что в сях указатели нужно принудительно занулять после удаления элемента, думал Qt это делает сам.

Сообщение отредактировал silver47 - 15.4.2011, 8:50

[wiz29]

Такой подход в принципе неверен, существует метод :
QTableWidgetItem * QTableWidget::takeItem ( int row, int column )
Removes the item at row and column from the table without deleting it.

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

Нельзя делать

pItem = table->item(x, y);
delete pItem;

нужно

pItem = table->takeItem(x, y);
delete pItem;

По поводу

Это означает, что таблица полностью отвечает за данный элемент, и, соответственно, уничтожит его в случает очистки таблицы, либо удаления самой таблицы.

Если я создам QTableWidgetItem и указатель на него, то после удаления элемента указатель куда будет ссылаться? На тот же адрес? Если да, то прошу прощения запутался. Помню что в сях указатели нужно принудительно занулять после удаления элемента, думал Qt это делает сам.

Qt это не язык программирования, а библиотека, не нужно это путать никогда.... зачем тебе указатель занулять которым ты не пользуешься?...

[abc]

посмотри исходники QTableWidget, там работает модель QTableModel, реализация которой в том же файле
когда делаешь clear(), вызывается QTableModel::clearContents():

for (int i = 0; i < tableItems.count(); ++i) {
        if (tableItems.at(i)) {
            tableItems.at(i)->view = 0;
            delete tableItems.at(i);
            tableItems[i] = 0;
        }
    }
    reset();

и, кроме того, когда делаешь setItem, удаляется старая ячейка (метод QTableModel::setItem):

QTableWidgetItem *oldItem = tableItems.at(i);
    if (item == oldItem)
        return;

    // remove old
    if (oldItem)
        oldItem->view = 0;
    delete tableItems.at(i);

почему ты решил, что не уничтожаются?

[silver47]

зачем тебе указатель занулять которым ты не пользуешься?...

Тот, который ссылается на объект который еще не создан, или уже удален.

[wiz29]

зачем тебе указатель занулять которым ты не пользуешься?...

Тот, который ссылается на объект который еще не создан, или уже удален.

Ты им пользуешься или нет? если не пользуешься, зачем занулять?о_О

[silver47]

почему ты решил, что не уничтожаются?

Потому что запутался. Удалил элемент и проверил указатель, который ссылался на него. Указатель по прежнему ссылается на тот же адрес памяти. Вот меня и переклинило что объект не удален.

Ты им пользуешься или нет? если не пользуешься, зачем занулять?о_О

Нет не пользуюсь. У меня вообще нет указателей на QTableWidgetItem. Сделал один для проверки, происходит ли удаление элемента при очистке таблицы, да и переклинило меня, что после удаления объекта указатель должен быть занулен. Теперь вспомнил что сам это делал всегда.

[wiz29]

ну со всеми бывает, завтра выходной, нужно отдыхать

[Dima1985]

посмотри исходники QTableWidget, там работает модель QTableModel, реализация которой в том же файле
когда делаешь clear(), вызывается QTableModel::clearContents():

for (int i = 0; i < tableItems.count(); ++i) {
        if (tableItems.at(i)) {
            tableItems.at(i)->view = 0;
            delete tableItems.at(i);
            tableItems[i] = 0;
        }
    }
    reset();

и, кроме того, когда делаешь setItem, удаляется старая ячейка (метод QTableModel::setItem):

QTableWidgetItem *oldItem = tableItems.at(i);
    if (item == oldItem)
        return;

    // remove old
    if (oldItem)
        oldItem->view = 0;
    delete tableItems.at(i);

почему ты решил, что не уничтожаются?

у меня не освобождаєтся память после удаления строк

создал приложения в нем есть QTableWidgetItem в конструкторе создаю строк с текстом примерно на 10Мб. приложение хаваєт єту память потом удаляю єти строки а приложения память не освобождаєт!!!