Get index of QValueListIterator

[zlatko]

Hi all!
So i try get index of my QValueListIterator

for (GroupList::const_iterator itGr=gl_GroupList.constBegin(); itGr!=gl_GroupList.constEnd(); ++itGr)
   int  m_nCurrKey = gl_GroupList.findIndex( (*itGr) );

for (GroupList::const_iterator itGr=gl_GroupList.constBegin(); itGr!=gl_GroupList.constEnd(); ++itGr)
   int  m_nCurrKey = gl_GroupList.findIndex( (*itGr) );

To copy to clipboard, switch view to plain text mode 

But i have a compile error from qvaluelist.h

C:\Qt\3.3.5\include\qvaluelist.h(331): error C2678: binary '==' : no operator found which takes a left-hand operand of type 'const SGroup' (or there is no acceptable conversion)

Any suggestions?

[jacek]

Did you implement SGroup::operator==()?

[zlatko]

no...i must do it

[jacek]

no...i must do it

Judging from the error message, yes.

[zlatko]

Understand...So i can't get in other way position(index) of curent iterators item ?

[jacek]

Understand...So i can't get in other way position(index) of curent iterators item ?

This should work:

for( int i = 0, GroupList::const_iterator itGr = gl_GroupList.constBegin(); itGr != gl_GroupList.constEnd(); ++i, ++itGr) {
   int  m_nCurrKey = i; 
   //... 
}

for( int i = 0, GroupList::const_iterator itGr = gl_GroupList.constBegin(); itGr != gl_GroupList.constEnd(); ++i, ++itGr) {
   int  m_nCurrKey = i; 
   //... 
}

To copy to clipboard, switch view to plain text mode 

[zlatko]

No it was my first solution...Actually like this becouse yours cant be compiled

int i;
for( i = 0, GroupList::const_iterator itGr = gl_GroupList.constBegin(); itGr != gl_GroupList.constEnd(); ++i, ++itGr) {
   int  m_nCurrKey = i;

int i;
for( i = 0, GroupList::const_iterator itGr = gl_GroupList.constBegin(); itGr != gl_GroupList.constEnd(); ++i, ++itGr) {
   int  m_nCurrKey = i;

To copy to clipboard, switch view to plain text mode 

But the i think must be more elegant way to solve it...Now i see that its imposible

[jacek]

But the i think must be more elegant way to solve it...Now i see that its imposible

Either you use iterators or access your data by index. Usually there is no need to do both.

You can always do this:

const int size = gl_GroupList.size();
for( int i = 0; i < size; ++i ) {
   int  m_nCurrKey = i; 
   //...
   // gl_GroupList[i] 
   //...
};

const int size = gl_GroupList.size();
for( int i = 0; i < size; ++i ) {
   int  m_nCurrKey = i; 
   //...
   // gl_GroupList[i] 
   //...
};

To copy to clipboard, switch view to plain text mode 

[zlatko]

Yes it can be more flexibility solution
thanks