QTableWidget+QTableWidgetItem trouble

**Fastman** · 9th November 2007, 10:41

Originally Posted by jpn

No, that shouldn't leak memory since QTableWidget deletes the old item if you set a new item into cell. But you definitely don't want to re-create all the items every time in setTableData(). Create the items once in constructor or so, and just set the data in setTableData():

Qt Code:

Switch view

QTableWidgetItem* item = ui.tableWidget->item(row, col);
item->setSomething(...);

QTableWidgetItem* item = ui.tableWidget->item(row, col);
item->setSomething(...);

To copy to clipboard, switch view to plain text mode

Hmm... not work

Qt Code:

Switch view

class HSM : public QDialog
{
	Q_OBJECT
...
...
...
private:
	QTableWidgetItem* itemArchive;
	QTableWidgetItem* itemRestore;
	QTableWidgetItem* itemDelete;
...
...
...

class HSM : public QDialog
{
	Q_OBJECT
...
...
...
private:
	QTableWidgetItem* itemArchive;
	QTableWidgetItem* itemRestore;
	QTableWidgetItem* itemDelete;
...
...
...

To copy to clipboard, switch view to plain text mode

Qt Code:

Switch view

HSM::HSM(QWidget *parent, Qt::WFlags flags)
	: QDialog(parent, flags)
{
 
...
...
...
	itemArchive = new QTableWidgetItem(tr("Archive"));
	itemRestore = new QTableWidgetItem(tr("Restore"));
	itemDelete = new QTableWidgetItem(tr("Delete"));
...
...
...

HSM::HSM(QWidget *parent, Qt::WFlags flags)
	: QDialog(parent, flags)
{

...
...
...
	itemArchive = new QTableWidgetItem(tr("Archive"));
	itemRestore = new QTableWidgetItem(tr("Restore"));
	itemDelete = new QTableWidgetItem(tr("Delete"));
...
...
...

To copy to clipboard, switch view to plain text mode

Qt Code:

Switch view

void HSM::setTableData()
{
	sTskInfoGUI sTaskGUI;
	db_work db;
 
	sTaskGUI = db.GetTaskToGUI();
	db.CloseDB();
 
	for (int row = 0 ; row < sTaskGUI.nCmd.size() ;++row)
	{
 
		if (sTaskGUI.nCmd.at(row) == 1)
		{
			itemArchive = ui.tableWidget->item(row,0);
			itemArchive->setIcon(QIcon(QPixmap(":/HSM/Resources/archive.png")));
			itemArchive->setBackground(QColor::fromRgb(200,255,104,100));
			//ui.tableWidget->setItem(row,0,itemArchive);
 
		}
		if (sTaskGUI.nCmd.at(row) == 2)
		{
			itemRestore = ui.tableWidget->item(row,0);
			itemRestore->setIcon(QIcon(QPixmap(":/HSM/Resources/restore.png")));
			itemRestore->setBackground(QColor::fromRgb(115,150,255,100));
			//ui.tableWidget->setItem(row,0,itemRestore);
		}
		if (sTaskGUI.nCmd.at(row) == 3)
		{
			itemDelete = ui.tableWidget->item(row,0);
			itemDelete->setIcon(QIcon(QPixmap(":/HSM/Resources/del.png")));
			itemDelete->setBackground(QColor::fromRgb(255,100,120,100));
			//ui.tableWidget->setItem(row,0,itemDelete);
		}
...
...
...

void HSM::setTableData()
{
	sTskInfoGUI sTaskGUI;
	db_work db;

	sTaskGUI = db.GetTaskToGUI();
	db.CloseDB();

	for (int row = 0 ; row < sTaskGUI.nCmd.size() ;++row)
	{
		
		if (sTaskGUI.nCmd.at(row) == 1)
		{
			itemArchive = ui.tableWidget->item(row,0);
			itemArchive->setIcon(QIcon(QPixmap(":/HSM/Resources/archive.png")));
			itemArchive->setBackground(QColor::fromRgb(200,255,104,100));
			//ui.tableWidget->setItem(row,0,itemArchive);

		}
		if (sTaskGUI.nCmd.at(row) == 2)
		{
			itemRestore = ui.tableWidget->item(row,0);
			itemRestore->setIcon(QIcon(QPixmap(":/HSM/Resources/restore.png")));
			itemRestore->setBackground(QColor::fromRgb(115,150,255,100));
			//ui.tableWidget->setItem(row,0,itemRestore);
		}
		if (sTaskGUI.nCmd.at(row) == 3)
		{
			itemDelete = ui.tableWidget->item(row,0);
			itemDelete->setIcon(QIcon(QPixmap(":/HSM/Resources/del.png")));
			itemDelete->setBackground(QColor::fromRgb(255,100,120,100));
			//ui.tableWidget->setItem(row,0,itemDelete);
		}
...
...
...

To copy to clipboard, switch view to plain text mode

Exception in code :
d:\qt4\src\gui\itemviews\qtablewidget.cpp

Qt Code:

Switch view

inline void QTableWidgetItem::setIcon(const QIcon &aicon)
{ setData(Qt::DecorationRole, aicon); }

inline void QTableWidgetItem::setIcon(const QIcon &aicon)
{ setData(Qt::DecorationRole, aicon); }

To copy to clipboard, switch view to plain text mode

**jpn** · 9th November 2007, 10:49

QTableWidget::item() returns 0 which means there is no item in such cell and so you are dereferencing a null pointer. Did you actually call QTableWidget::setItem() anywhere?

Qt Code:

Switch view

itemArchive = new QTableWidgetItem(tr("Archive"));
itemRestore = new QTableWidgetItem(tr("Restore"));
itemDelete = new QTableWidgetItem(tr("Delete"));
ui.tableWidget->setItem(0,0,itemArchive); // <--
ui.tableWidget->setItem(1,0,itemRestore); // <--
ui.tableWidget->setItem(2,0,itemDelete);  // <--

itemArchive = new QTableWidgetItem(tr("Archive"));
itemRestore = new QTableWidgetItem(tr("Restore"));
itemDelete = new QTableWidgetItem(tr("Delete"));
ui.tableWidget->setItem(0,0,itemArchive); // <--
ui.tableWidget->setItem(1,0,itemRestore); // <--
ui.tableWidget->setItem(2,0,itemDelete);  // <--

To copy to clipboard, switch view to plain text mode

**Fastman** · 9th November 2007, 11:28

Originally Posted by jpn

QTableWidget::item() returns 0 which means there is no item in such cell and so you are dereferencing a null pointer. Did you actually call QTableWidget::setItem() anywhere?

Qt Code:

Switch view

itemArchive = new QTableWidgetItem(tr("Archive"));
itemRestore = new QTableWidgetItem(tr("Restore"));
itemDelete = new QTableWidgetItem(tr("Delete"));
ui.tableWidget->setItem(0,0,itemArchive); // <--
ui.tableWidget->setItem(1,0,itemRestore); // <--
ui.tableWidget->setItem(2,0,itemDelete);  // <--

itemArchive = new QTableWidgetItem(tr("Archive"));
itemRestore = new QTableWidgetItem(tr("Restore"));
itemDelete = new QTableWidgetItem(tr("Delete"));
ui.tableWidget->setItem(0,0,itemArchive); // <--
ui.tableWidget->setItem(1,0,itemRestore); // <--
ui.tableWidget->setItem(2,0,itemDelete);  // <--

To copy to clipboard, switch view to plain text mode

no effect

Exception in code :
d:\qt4\src\gui\itemviews\qtablewidget.cpp

Qt Code:

Switch view

inline void QTableWidgetItem::setIcon(const QIcon &aicon){ setData(Qt::DecorationRole, aicon); }

inline void QTableWidgetItem::setIcon(const QIcon &aicon){ setData(Qt::DecorationRole, aicon); }

To copy to clipboard, switch view to plain text mode

**jpn** · 9th November 2007, 11:48

There are 3 places in your code in which you pick an item at certain cell:

Qt Code:

Switch view

...
itemArchive = ui.tableWidget->item(row,0);
...
itemRestore = ui.tableWidget->item(row,0);
...
itemRestore = ui.tableWidget->item(row,0);
...

...
itemArchive = ui.tableWidget->item(row,0);
...
itemRestore = ui.tableWidget->item(row,0);
...
itemRestore = ui.tableWidget->item(row,0);
...

To copy to clipboard, switch view to plain text mode

On the other hand, you have created an item only into cells (0,0), (1,0) and (2,0). If "row" is anything else than 0-2, you'll get a crash. This is most likely the case of yours.

**Fastman** · 9th November 2007, 13:18

Ok

Thx for you help

**jpn** · 9th November 2007, 17:42

Let me correct one thing. I just re-read my previous message and I want to make sure I don't get misundestood. Of course, calling QTableWidget::item() with any possible parameters doesn't cause a crash:

Qt Code:

Switch view

QTableWidgetItem* item = tableWidget->item(1234, 5678);

QTableWidgetItem* item = tableWidget->item(1234, 5678);

To copy to clipboard, switch view to plain text mode

But the point is that in case no item exists in such cell, QTableWidget::item() returns 0. Then, dereferencing a null pointer is what causes the crash:

Qt Code:

Switch view

item->setSomething();

item->setSomething();

To copy to clipboard, switch view to plain text mode

**Fastman** · 9th November 2007, 18:06

How to make so that last column it was leveled by a right edge of the table ?

Qt Code:

Switch view

void HSM::setTableData()
{
	sTskInfoGUI sTaskGUI;
	db_work db;
 
	sTaskGUI = db.GetTaskToGUI();
	db.CloseDB();...
...
...
		itemRootPatch->setText(sTaskGUI.cRootPath.at(row));
		ui.tableWidget->setItem(row,9,itemRootPatch);
	}
 
	ui.tableWidget->resizeRowsToContents();  
	ui.tableWidget->resizeColumnsToContents(); 
 
}

void HSM::setTableData()
{
	sTskInfoGUI sTaskGUI;
	db_work db;

	sTaskGUI = db.GetTaskToGUI();
	db.CloseDB();...
...
...
		itemRootPatch->setText(sTaskGUI.cRootPath.at(row));
		ui.tableWidget->setItem(row,9,itemRootPatch);
	}

	ui.tableWidget->resizeRowsToContents();  
	ui.tableWidget->resizeColumnsToContents(); 

}

To copy to clipboard, switch view to plain text mode

Here so my form looks.

**jpn** · 9th November 2007, 18:24

You can either use QHeaderView::setResizeMode() and pass QHeaderView::Stretch:

Qt Code:

Switch view

ui.tableWidget->horizontalHeader()->setResizeMode(10, QHeaderView::Stretch);

ui.tableWidget->horizontalHeader()->setResizeMode(10, QHeaderView::Stretch);

To copy to clipboard, switch view to plain text mode

Or you can use a "shortcut" (which QTreeView uses by default):

Qt Code:

Switch view

ui.tableWidget->horizontalHeader()->setStretchLastSection(true);

ui.tableWidget->horizontalHeader()->setStretchLastSection(true);

To copy to clipboard, switch view to plain text mode

**Fastman** · 10th November 2007, 12:19

many Thx !

Thread: QTableWidget+QTableWidgetItem trouble

Thread Tools

Search Thread

Display

Hybrid View

Re: QTableWidget+QTableWidgetItem trouble

Re: QTableWidget+QTableWidgetItem trouble

Re: QTableWidget+QTableWidgetItem trouble

Re: QTableWidget+QTableWidgetItem trouble

Re: QTableWidget+QTableWidgetItem trouble

Re: QTableWidget+QTableWidgetItem trouble

Re: QTableWidget+QTableWidgetItem trouble

Re: QTableWidget+QTableWidgetItem trouble

Re: QTableWidget+QTableWidgetItem trouble

Similar Threads

QComboBox in QTableWidget : display troubles.

QTableWidget won't sort cellwidgets!!!

A QListWidget in a QTableWidget cell ?

QTableWidget (resizing rows, turning off selection, etc.)

Problem inserting in QTableWidget

Bookmarks

Bookmarks

Posting Permissions