vector<pair..> >::iterator

[Gary Wessle]

Hi
is it ok to do this? if not, then how

for (vector<pair<char,double> >::const_iterator it = v.begin(); it != v.end(); ++it)
switch (*it->first){...

thanks

 

[Victor Bazarov]

is it ok to do this? if not, then how

for (vector<pair<char,double> >::const_iterator it = v.begin(); it
!= v.end(); ++it) switch (*it->first){...

'it->first' is a char. You cannot dereference a char. You have one
too many indirections. Either use

switch (it->first) {

or

switch ((*it).first) {

Which should be the same AFA C++ is concerned.

V

 

[mlimber]

is it ok to do this? if not, then how

for (vector<pair<char,double> >::const_iterator it = v.begin(); it != v.end(); ++it)
switch (*it->first){...

No. Did you even try it? The element "first" is not a pointer, so you
can't dereference it. Drop the star *or* parenthesize (*it) and change
the arrow to a dot, and then it will compile.

Cheers! --M

 

[Gary Wessle]

Hi
is it ok to do this? if not, then how

for (vector<pair<char,double> >::const_iterator it = v.begin(); it != v.end(); ++it)
switch (*it->first){...

never mind, I found it after some trial and error

switch( (*it).first )