QTreeWidget Weights/Sorting

[VireX]

I am trying to sort my QTreeWidget, but I was wondering if there was a way to add Weights to the topLevelnodes. I mean I don't want people to sort it by say "name" column, because then it sorts my top level nodes, and the order becomes messed up.

What I want to do is keep top level nodes in a fixed order. But I want their children to be sorted normally by "name" column (which has everything).

Can I do this without subclassing and overriding sort()?
Perhaps install an event filter?
Or Is there some sort of "Weight" property I can add (Didn't see) ?

[wysota]

Reimplement bool operator< ( const QTreeWidgetItem & other ) const for your tree items and use the created subclass instead of QTreeWidgetItem.

[VireX]

Ok I reimplemented that in this way. However, will this sort "names" as well in alphabetical even if I override the original < operator? Or should I code that in too?

#include "XTreeItem.h"
 
bool QTreeWidgetItem::operator< ( const QTreeWidgetItem & other ) const {
  int iScore, iMyScore;
  iScore = 0;
  iMyScore = 0;
  if(other.flags() == 0){
    if(other.text(0) == QString("FIRST"))iScore = 100;
    if(other.text(0) == QString("SECOND"))iScore = 90;
    if(other.text(0) == QString("THIRD"))iScore = 80;
    if(other.text(0) == QString("FOURTH"))iScore = 70;
    if(other.text(0) == QString("FIFTH"))iScore = 60;
    if(other.text(0) == QString("SIXTH"))iScore = 50;
  }
  if(flags() == 0){
    if(text(0) == QString("FIRST"))iMyScore = 100;
    if(text(0) == QString("SECOND"))iMyScore = 90;
    if(text(0) == QString("THIRD"))iMyScore = 80;
    if(text(0) == QString("FOURTH"))iMyScore = 70;
    if(text(0) == QString("FIFTH"))iMyScore = 60;
    if(text(0) == QString("SIXTH"))iMyScore = 50;    
  }
  if(iMyScore < iScore){ return true; } else { return false; }
}
 
#include "XTreeItem.moc"

#include "XTreeItem.h"

bool QTreeWidgetItem::operator< ( const QTreeWidgetItem & other ) const {
  int iScore, iMyScore;
  iScore = 0;
  iMyScore = 0;
  if(other.flags() == 0){
    if(other.text(0) == QString("FIRST"))iScore = 100;
    if(other.text(0) == QString("SECOND"))iScore = 90;
    if(other.text(0) == QString("THIRD"))iScore = 80;
    if(other.text(0) == QString("FOURTH"))iScore = 70;
    if(other.text(0) == QString("FIFTH"))iScore = 60;
    if(other.text(0) == QString("SIXTH"))iScore = 50;
  }
  if(flags() == 0){
    if(text(0) == QString("FIRST"))iMyScore = 100;
    if(text(0) == QString("SECOND"))iMyScore = 90;
    if(text(0) == QString("THIRD"))iMyScore = 80;
    if(text(0) == QString("FOURTH"))iMyScore = 70;
    if(text(0) == QString("FIFTH"))iMyScore = 60;
    if(text(0) == QString("SIXTH"))iMyScore = 50;    
  }
  if(iMyScore < iScore){ return true; } else { return false; }
}

#include "XTreeItem.moc"

To copy to clipboard, switch view to plain text mode 

I know I shoulda used switches and/or functions better (more OO) but still is this basically what I should do to add weights?

[wysota]

What original operator? You have to subclass and reimplement the method in the subclass.

[VireX]

That's what I did, is the code correct?

[wysota]

bool QTreeWidgetItem::operator< ( const QTreeWidgetItem & other ) const

bool QTreeWidgetItem::operator< ( const QTreeWidgetItem & other ) const

To copy to clipboard, switch view to plain text mode 

This doesn't look like a subclass.

[VireX]

Because I forgot to change the name of the class, but yes, assume it is a subclass. You still have not commented on the actual code?

[wysota]

It's hard to understand it... Besides, it doesn't do much sorting... For instance it doesn't seem to compare values of items at all... Only their "flags". Is "flags" the only property of your items?

[VireX]

Well my toplevel nodes have 0 flags, so its only way to detect them, the rest have flags. And I'm assuming Qt will sort the rest by default (in other words my function and original function working together)

[wysota]

Well my toplevel nodes have 0 flags, so its only way to detect them, the rest have flags.

No. Top level nodes don't have parents. That's a way to detect them.

And I'm assuming Qt will sort the rest by default (in other words my function and original function working together)

But you never call the base class implementation so only your method gets called. That's how virtual methods work.

[VireX]

So how do I call it...?

Do I call it like this:
bool XTreeItem:perator< ( const QTreeWidgetItem & other ) const : QTreeWidgetItem:perator< (other) {

[wysota]

Call

QTreeWidgetItem::operator< ( other );

QTreeWidgetItem::operator< ( other );

To copy to clipboard, switch view to plain text mode 

in appropriate place of your reimplementation.

[VireX]

Thought so, thank you, lemme try this stuff.

[VireX]

In file included from XTreeItem.cpp:32:
XTreeItem.moc:36: error: `staticMetaObject' is not a member of `QTreeWidgetItem'

XTreeItem.moc: In member function `virtual void* XTreeItem::qt_metacast(const char*)':
XTreeItem.moc:50: error: `qt_metacast' is not a member of `QTreeWidgetItem'
XTreeItem.moc: In member function `virtual int XTreeItem::qt_metacall(QMetaObject::Call, int, void**)':
XTreeItem.moc:55: error: `qt_metacall' is not a member of `QTreeWidgetItem'

Hmmmmmm very odd errors...

H FILE

#include <QtGui>
 
class XTreeItem : public QTreeWidgetItem {
  Q_OBJECT
  public:
   XTreeItem(QTreeWidget * parent, int type);
   XTreeItem(XTreeItem * parent, int type);
   virtual bool operator< ( const XTreeItem & other ) const;
};

#include <QtGui>

class XTreeItem : public QTreeWidgetItem {
  Q_OBJECT
  public:
   XTreeItem(QTreeWidget * parent, int type);
   XTreeItem(XTreeItem * parent, int type);
   virtual bool operator< ( const XTreeItem & other ) const;
};

To copy to clipboard, switch view to plain text mode 

CPP FILE:

#include "XTreeItem.h"
XTreeItem::XTreeItem(QTreeWidget * parent, int type = Type ){
  QTreeWidgetItem(parent, type);
}
XTreeItem::XTreeItem(XTreeItem * parent, int type = Type ){
  QTreeWidgetItem(parent, type);
}
bool XTreeItem::operator< ( const XTreeItem & other ) const {
  int iScore, iMyScore;
  iScore = 0;
  iMyScore = 0;
  if(other.flags() == 0){
    if(other.text(0) == QString("F"))iScore = 100;
    if(other.text(0) == QString("S"))iScore = 90;
    if(other.text(0) == QString("T"))iScore = 80;
    if(other.text(0) == QString("F_"))iScore = 70;
    if(other.text(0) == QString("F__"))iScore = 60;
    if(other.text(0) == QString("S_"))iScore = 50;
  }
  if(flags() == 0){
    if(text(0) == QString("F"))iMyScore = 100;
    if(text(0) == QString("S"))iMyScore = 90;
    if(text(0) == QString("T"))iMyScore = 80;
    if(text(0) == QString("F_"))iMyScore = 70;
    if(text(0) == QString("F__"))iMyScore = 60;
    if(text(0) == QString("S_"))iMyScore = 50;    
  } 
  QTreeWidgetItem::operator< ( other );
  if(iMyScore < iScore){ return true; } else { return false; }
}
 
#include "XTreeItem.moc"

#include "XTreeItem.h"
XTreeItem::XTreeItem(QTreeWidget * parent, int type = Type ){
  QTreeWidgetItem(parent, type);
}
XTreeItem::XTreeItem(XTreeItem * parent, int type = Type ){
  QTreeWidgetItem(parent, type);
}
bool XTreeItem::operator< ( const XTreeItem & other ) const {
  int iScore, iMyScore;
  iScore = 0;
  iMyScore = 0;
  if(other.flags() == 0){
    if(other.text(0) == QString("F"))iScore = 100;
    if(other.text(0) == QString("S"))iScore = 90;
    if(other.text(0) == QString("T"))iScore = 80;
    if(other.text(0) == QString("F_"))iScore = 70;
    if(other.text(0) == QString("F__"))iScore = 60;
    if(other.text(0) == QString("S_"))iScore = 50;
  }
  if(flags() == 0){
    if(text(0) == QString("F"))iMyScore = 100;
    if(text(0) == QString("S"))iMyScore = 90;
    if(text(0) == QString("T"))iMyScore = 80;
    if(text(0) == QString("F_"))iMyScore = 70;
    if(text(0) == QString("F__"))iMyScore = 60;
    if(text(0) == QString("S_"))iMyScore = 50;    
  } 
  QTreeWidgetItem::operator< ( other );
  if(iMyScore < iScore){ return true; } else { return false; }
}

#include "XTreeItem.moc"

To copy to clipboard, switch view to plain text mode 

[wysota]

1. The item doesn't inherit QObject so it can't contain Q_OBJECT macro
2. This code doesn't make any sense. You call the base class implementation of the operator but you don't do anything with the return value, so what's the point of calling it in the first place? The base implementation will return a boolean value telling you if the current item is smaller than the other in terms understood by the base class.

[VireX]

1. I thought any QWidget was a QObject and therefore I needed Q_OBJECT right? Or do I not even need that?
2. Yes I realized that after I posted heh.
if(iMyScore < iScore){ return true; } else { return QTreeWidgetItem:perator< ( other ); }

[wysota]

1. I thought any QWidget was a QObject and therefore I needed Q_OBJECT right? Or do I not even need that?

QTreeWidgetItem is not a widget. QTreeWidget is.

[VireX]

The name is misleading .

[wysota]

No, it's not. All "items" are not widgets.

[VireX]

This is becoming very very frustrating... And you wonder why I avoid subclassing...

CMain.cpp:322: error: no matching function for call to `XTreeItem::XTreeItem(XTreeItem*&)'
XTreeItem.h:7: note: candidates are: XTreeItem::XTreeItem(XTreeItem*, int)
XTreeItem.h:6: note: XTreeItem::XTreeItem(const XTreeItem&)
XTreeItem.h:5: note: XTreeItem::XTreeItem(QTreeWidget*)

CMain.cpp:329: error: no matching function for call to `XTreeItem::XTreeItem(XTreeItem*&)'
XTreeItem.h:7: note: candidates are: XTreeItem::XTreeItem(XTreeItem*, int)
XTreeItem.h:6: note: XTreeItem::XTreeItem(const XTreeItem&)
XTreeItem.h:5: note: XTreeItem::XTreeItem(QTreeWidget*)
CMain.cpp:340: error: no matching function for call to `XTreeItem::XTreeItem(XTreeItem*&)'
XTreeItem.h:7: note: candidates are: XTreeItem::XTreeItem(XTreeItem*, int)
XTreeItem.h:6: note: XTreeItem::XTreeItem(const XTreeItem&)
XTreeItem.h:5: note: XTreeItem::XTreeItem(QTreeWidget*)

CMain.cpp:351: error: no matching function for call to `XTreeItem::XTreeItem(XTreeItem*&)'
XTreeItem.h:7: note: candidates are: XTreeItem::XTreeItem(XTreeItem*, int)
XTreeItem.h:6: note: XTreeItem::XTreeItem(const XTreeItem&)
XTreeItem.h:5: note: XTreeItem::XTreeItem(QTreeWidget*)
CLobby.cpp:352: error: no matching function for call to `XTreeItem::XTreeItem(XTreeItem*&)'
XTreeItem.h:7: note: candidates are: XTreeItem::XTreeItem(XTreeItem*, int)
XTreeItem.h:6: note: XTreeItem::XTreeItem(const XTreeItem&)
XTreeItem.h:5: note: XTreeItem::XTreeItem(QTreeWidget*)
CMain.cpp:353: error: no matching function for call to `XTreeItem::XTreeItem(XTreeItem*&)'
XTreeItem.h:7: note: candidates are: XTreeItem::XTreeItem(XTreeItem*, int)
XTreeItem.h:6: note: XTreeItem::XTreeItem(const XTreeItem&)
XTreeItem.h:5: note: XTreeItem::XTreeItem(QTreeWidget*)
CMain.cpp:354: error: no matching function for call to `XTreeItem::XTreeItem(XTreeItem*&)'
XTreeItem.h:7: note: candidates are: XTreeItem::XTreeItem(XTreeItem*, int)
XTreeItem.h:6: note: XTreeItem::XTreeItem(const XTreeItem&)
XTreeItem.h:5: note: XTreeItem::XTreeItem(QTreeWidget*)

CMain.cpp:355: error: no matching function for call to `XTreeItem::XTreeItem(XTreeItem*&)'
XTreeItem.h:7: note: candidates are: XTreeItem::XTreeItem(XTreeItem*, int)
XTreeItem.h:6: note: XTreeItem::XTreeItem(const XTreeItem&)
XTreeItem.h:5: note: XTreeItem::XTreeItem(QTreeWidget*)
CMain.cpp:356: error: no matching function for call to `XTreeItem::XTreeItem(XTreeItem*&)'
XTreeItem.h:7: note: candidates are: XTreeItem::XTreeItem(XTreeItem*, int)
XTreeItem.h:6: note: XTreeItem::XTreeItem(const XTreeItem&)
XTreeItem.h:5: note: XTreeItem::XTreeItem(QTreeWidget*)

code for H file:

#include <QtGui>
 
class XTreeItem : public QTreeWidgetItem {
  public:
   XTreeItem(QTreeWidget * parent);
   XTreeItem(const XTreeItem & other);
   XTreeItem(XTreeItem * parent, int type);
   virtual bool operator< ( const XTreeItem & other ) const;
};

#include <QtGui>

class XTreeItem : public QTreeWidgetItem {
  public:
   XTreeItem(QTreeWidget * parent);
   XTreeItem(const XTreeItem & other);
   XTreeItem(XTreeItem * parent, int type);
   virtual bool operator< ( const XTreeItem & other ) const;
};

To copy to clipboard, switch view to plain text mode 

CPPPPPPPPPP

#include "XTreeItem.h"
XTreeItem::XTreeItem(QTreeWidget * parent){
  QTreeWidgetItem(parent, Type);
}
XTreeItem::XTreeItem(XTreeItem * parent, int type = Type){
  QTreeWidgetItem(parent, type);                            
}
XTreeItem::XTreeItem(const XTreeItem & other){
  QTreeWidgetItem(parent);
}
bool XTreeItem::operator< ( const XTreeItem & other ) const {
  int iScore, iMyScore;
  iScore = 0;
  iMyScore = 0;
  if(other.flags() == 0){
    if(other.text(0) == QString("F"))iScore = 100;
    if(other.text(0) == QString("S"))iScore = 90;
  }
  if(flags() == 0){
    if(text(0) == QString("F"))iMyScore = 100;
    if(text(0) == QString("S"))iMyScore = 90;
  } 
  if(iMyScore < iScore){ return true; } else { return QTreeWidgetItem::operator< ( other ); }
}

#include "XTreeItem.h"
XTreeItem::XTreeItem(QTreeWidget * parent){
  QTreeWidgetItem(parent, Type);
}
XTreeItem::XTreeItem(XTreeItem * parent, int type = Type){
  QTreeWidgetItem(parent, type);                            
}
XTreeItem::XTreeItem(const XTreeItem & other){
  QTreeWidgetItem(parent);
}
bool XTreeItem::operator< ( const XTreeItem & other ) const {
  int iScore, iMyScore;
  iScore = 0;
  iMyScore = 0;
  if(other.flags() == 0){
    if(other.text(0) == QString("F"))iScore = 100;
    if(other.text(0) == QString("S"))iScore = 90;
  }
  if(flags() == 0){
    if(text(0) == QString("F"))iMyScore = 100;
    if(text(0) == QString("S"))iMyScore = 90;
  } 
  if(iMyScore < iScore){ return true; } else { return QTreeWidgetItem::operator< ( other ); }
}

To copy to clipboard, switch view to plain text mode 

Please help I give up.... I tried everything, I transmit those headers to QTreeWidgetItem (with same arguments).

All I try to do is:
XTreeItem* t = new XTreeItem(MainList); (MainList is a QTreeWidget*).
That doesn't work so I change the code, and when I do now this doesn't work:
XTreeItem* bee = new XTreeItem(First); (First is a XTreeItem*)