Remove selected rows from a QTableView

[niko]

I have a QTableView and a subclassed QStandardItemModel.

I need a Button wich deletes the selected rows. This is my current implementation:

//get selections
QItemSelection selection = tableView->selectionModel()->selection();
 
//find out selected rows
QList<int> removeRows;
foreach(QModelIndex index, selection.indexes()) {
	if(!removeRows.contains(index.row())) {
		removeRows.append(index.row());
	}
}
 
//loop through all selected rows
for(int i=0;i<removeRows.count();++i)
{
	//decrement all rows after the current - as the row-number will change if we remove the current
	for(int j=i;j<removeRows.count();++j) {
		if(removeRows.at(j) > removeRows.at(i)) {
			removeRows[j]--;
		}
	}
	//remove the selected row
	model->removeRows(removeRows.at(i), 1);
}

//get selections
QItemSelection selection = tableView->selectionModel()->selection();

//find out selected rows
QList<int> removeRows;
foreach(QModelIndex index, selection.indexes()) {
	if(!removeRows.contains(index.row())) {
		removeRows.append(index.row());
	}
}

//loop through all selected rows
for(int i=0;i<removeRows.count();++i)
{
	//decrement all rows after the current - as the row-number will change if we remove the current
	for(int j=i;j<removeRows.count();++j) {
		if(removeRows.at(j) > removeRows.at(i)) {
			removeRows[j]--;
		}
	}
	//remove the selected row
	model->removeRows(removeRows.at(i), 1);
}

To copy to clipboard, switch view to plain text mode 

which works - but can't this be easier?

thanks for any suggestions.

niko

[jacek]

QItemSelection selection( tableView->selectionModel()->selection() );
 
QList<int> rows;
foreach( const QModelIndex & index, selection.indexes() ) {
   rows.append( index.row() );
}
 
qSort( rows );
 
int prev = -1;
for( int i = rows.count() - 1; i >= 0; i -= 1 ) {
   int current = rows[i];
   if( current != prev ) {
      model->removeRows( current, 1 );
      prev = current;
   }
}

QItemSelection selection( tableView->selectionModel()->selection() );

QList<int> rows;
foreach( const QModelIndex & index, selection.indexes() ) {
   rows.append( index.row() );
}

qSort( rows );

int prev = -1;
for( int i = rows.count() - 1; i >= 0; i -= 1 ) {
   int current = rows[i];
   if( current != prev ) {
      model->removeRows( current, 1 );
      prev = current;
   }
}

To copy to clipboard, switch view to plain text mode 

[yarovenkoas]

My code im using

QModelIndexList indexes =  ui->tableUserVars->selectionModel()->selectedRows();
int countRow = indexes.count();
 
for( int i = countRow; i > 0; i--)
       modelUserVars->removeRow( indexes.at(i-1).row(), QModelIndex());

QModelIndexList indexes =  ui->tableUserVars->selectionModel()->selectedRows();
int countRow = indexes.count();

for( int i = countRow; i > 0; i--)
       modelUserVars->removeRow( indexes.at(i-1).row(), QModelIndex());

To copy to clipboard, switch view to plain text mode 

And if you are working with a large number of rows ( > 100,000 ) and the selection of lines in order not to sample the rapid removal of this way + if the selection is selectively remove the row from selectionModel

My code im using fast delete rows > 1000000

QModelIndexList indexes =  ui->tableUserVars->selectionModel()->selectedRows();
int countRow = indexes.count();
 
bool flagDif = false;
for( int i = countRow; i > 1; i--)
    if (indexes.at(i-1).row()-1 != indexes.at(i-2).row())
       flagDif = true;
 
if (!flagDif)
   modelUserVars->removeRows(indexes.at(0).row(),countRow,QModelIndex());
else
   for( int i = countRow; i > 0; i--)
       modelUserVars->removeRow( indexes.at(i-1).row(), QModelIndex());

QModelIndexList indexes =  ui->tableUserVars->selectionModel()->selectedRows();
int countRow = indexes.count();

bool flagDif = false;
for( int i = countRow; i > 1; i--)
    if (indexes.at(i-1).row()-1 != indexes.at(i-2).row())
       flagDif = true;

if (!flagDif)
   modelUserVars->removeRows(indexes.at(0).row(),countRow,QModelIndex());
else
   for( int i = countRow; i > 0; i--)
       modelUserVars->removeRow( indexes.at(i-1).row(), QModelIndex());

To copy to clipboard, switch view to plain text mode 

Ps. Sorry bad englsh.

[d_stranz]

And why are you posting this in response to a 10 year old thread???

[yarovenkoas]

maybe someone will come in handy =)