qt - Qt を使用した QTableWidget setItem() セグメンテーション違反

Question

次のコードを使用して、行を QTableWidget に追加しています。

QTableWidgetItem *item = new QTableWidgetItem(fileName);
item->setCheckState(Qt::Checked);

QComboBox *cmb = new QComboBox(this->list);
cmb->addItem("one");
cmb->addItem("two");

this->list->setRowCount(this->list->rowCount()+1);
this->list->setItem(this->list->rowCount()-1,0,item);
this->list->setCellWidget(this->list->rowCount()-1,1,cmb);

テーブルには 2 つの列があります。'item' は 1 番目に配置され、'cmb' は 2 番目に配置されます。このコードを使用すると、テーブルに 1 行を正常に追加できますが、2 行目を追加しようとすると、セグメンテーション エラーが発生します。this->list->setItem() 呼び出しでクラッシュします。

なぜクラッシュするのか考えていますか？

ありがとう！

Answer 1

私のコードのこの問題は、cellChanged SLOT のロジックが原因で発生しました。実際にはまだ作成されていない QTableWidget に setText を設定しようとしていました。

Answer 2

あなたと非常によく似たコードがあり、それもクラッシュしました...

QTableWidgetItem* columnOne = new QTableWidgetItem();
columnOne->setCheckState(Qt::Checked);
QTableWidgetItem* columnTwo = new QTableWidgetItem("Some Text");

int row = tableWidget->rowCount();
tableWidget->insertRow(row);
tableWidget->setItem(row, 0, columnOne);
tableWidget->setItem(row, 1, columnTwo);

ただし、チェック状態が最後に追加されるように setItem 呼び出しの順序を変更すると、機能しました。だから、これはコードが働いている...

QTableWidgetItem* columnOne = new QTableWidgetItem();
columnOne->setCheckState(Qt::Checked);
QTableWidgetItem* columnTwo = new QTableWidgetItem("Some Text");

int row = tableWidget->rowCount();
tableWidget->insertRow(row);
tableWidget->setItem(row, 1, columnTwo);
tableWidget->setItem(row, 0, columnOne);

Answer 3

問題はこの行にあります (いくつかのリストと QTableWidget に同じ名前を付けました: 「リスト」):

this->list->setRowCount(this->list->rowCount()+1)

this->list->rowCount()+1常に 0+1=1 を返すため、テーブルに最初の行を追加できますが、2 番目の行は追加できません。

解決策: QTableWidget に「list」以外の名前を付けます。次に例を示します。

this->myTableWidget->setRowCount(this->list->rowCount()+1);
this->myTableWidget->setItem(this->list->rowCount()-1,0,item);
this->myTableWidget->setCellWidget(this->list->rowCount()-1,1,cmb);

Answer 4

コードがそのまま機能しない理由はわかりませんが、次のようにします。

int N = list->rowCount(); //The problem may lie in multiple rowCount() calls somehow
list->insertRow(N);
list->setItem(N,0,item);
list->setCellWidget(N,1,cmb);

また、「this->」を使用する特定の理由はありますか? 通常、この状況では完全に冗長です。