Right way to sort by checkble column in treeview using QSortFilterProxyModel subclass

[umen242]

i have treeview subclass that using QSortFilterProxyModel subclass to sort by column items , when the item is string it is working fine
but when the item is checkble type , and i like to sort by checked items . in the QSortFilterProxyModel it just dosnt work.
here is my code for lessThan methods and where i capture the checkbox state.

@void SettingsDialog::tree_itemChanged(QStandardItem *item)
{

 Qt::CheckState checkState = item->checkState();
 if(checkState == Qt::Checked)
 {

item->setCheckState(Qt::Checked);
item->setData(true,Qt::UserRole);

 }
 else if(checkState == Qt::Unchecked)
 {

item->setCheckState(Qt::Unchecked);
item->setData(false,Qt::UserRole);

 }

}

---

bool SettingsSortFilterProxyModel::lessThan( const QModelIndex &left, const QModelIndex &right ) const
{

int columnLeftIndx = left.column();
int columnRightIndx = right.column();

if(columnLeftIndx == SETTINGS_CHECKBOX_CELL_IDX)
{

bool leftUint = sourceModel()->data( left,Qt::UserRole ).toBool();
bool rightUint = sourceModel()->data( right,Qt::UserRole ).toBool();
return leftUint < rightUint;
}
@

what im doing wrong here ?

[andre]

Is this code complete? It looks like part of the lessThan method is missing?

[umen242]

it just check if the sort coming from the second column and then try to do the sort .
it is complete , as i dont really know what else i need to put

[andre]

The code seems incomplete, because the last brace is not there. So I was wondering if something else might have been dropped.

Perhaps you should be more clear on what kind of result you are expecting. I guess you want your checked items to be sorted above the unchecked ones, but you still want to keep sorting on the text itself as well, right?

Edit:
So, is this what you are after?

Derrek

Zach

Andy

Bernard

Freddy

William

[umen242]

Hi
the last brace just missing when i copied the text , i just want the checked item to be first.
with not text related to the sort .
the check-boxs are in separated item.
Thanks again

[andre]

Try this:

@
bool SettingsSortFilterProxyModel::lessThan( const QModelIndex &left, const QModelIndex &right ) const
{
int columnLeftIndx = left.column();
int columnRightIndx = right.column();

ASSERT(columnLeftIndx == columnRightIndx);

if(columnLeftIndx == SETTINGS_CHECKBOX_CELL_IDX)
{

bool l = sourceModel()->data( left,Qt::UserRole ).toBool();
bool r = sourceModel()->data( right,Qt::UserRole ).toBool();
return (l && !r); //only return true if the left item is checked and the right one is not--
} else {
return QSortFilterProxyModel::qLess(left, right);
}
}
@

[umen242]

Hi thanks for the replay , but it dosn't work . i must doing something wrong .
@bool l = sourceModel()->data( left,Qt::UserRole ).toBool();
bool r = sourceModel()->data( right,Qt::UserRole ).toBool();@
always gives me true. no matter what .
also :
@QSortFilterProxyModel::qLess(left, right);
gives me : error C2039: 'qLess' : is not a member of 'QSortFilterProxyModel'@

[andre]

So why not use the checkStateRole directly then?

Note that the function name should be lessThan(), not qLess()

[umen242]

Finally ! Thanks Andre