Std Iterator Problem

Hi All,

I was wondering if anybody can tell me what is wrong with the following
code,

in a .h file I have

std::list<std::vector<Site> > positions ;
std::list<std::vector<Site> >::iterator itSteps ;

then in the .cpp file

void RmfSystem::step() {

if (itSteps == positions.end()) {

itSteps = positions.begin() ;

} else {

itSteps++ ;

}

}

where step gets called by an animate function in another file. The idea
that I have a vector of vector of positions and I want on each call of
step that the iterator points at another set of positions. Then when the
end of the list (well vector) is reached it loops around to the start.

As it stands my program crashes, as itStep == positions.end() never
seems to be found to be true.

Any ideas what I am doing wrong?

Adam

re: Std Iterator Problem

Adam Hartshorne wrote:[color=blue]
> Hi All,
>
> I was wondering if anybody can tell me what is wrong with the following
> code,
>
> in a .h file I have
>
> std::list<std::vector<Site> > positions ;
> std::list<std::vector<Site> >::iterator itSteps ;
>
> then in the .cpp file
>
> void RmfSystem::step() {
>
> if (itSteps == positions.end()) {
>
> itSteps = positions.begin() ;
>
> } else {
>
> itSteps++ ;
>
> }
>
> }
>
> where step gets called by an animate function in another file. The idea
> that I have a vector of vector of positions and I want on each call of
> step that the iterator points at another set of positions. Then when the
> end of the list (well vector) is reached it loops around to the start.
>
> As it stands my program crashes, as itStep == positions.end() never
> seems to be found to be true.
>
> Any ideas what I am doing wrong?
>
> Adam[/color]

Among other things, where is the code that initializes
'itSteps'? Somewhere (after 'positions' is filled with
data) you need to do:

itSteps = positions.begin();

A global iterator can cause unexpected side effects
if multiple functions modify the container that
the iterator references. This is a risky design.

Regards,
Larry

--
Anti-spam address, change each 'X' to '.' to reply directly.

re: Std Iterator Problem

Adam Hartshorne wrote:
[color=blue]
> in a .h file I have
>
> std::list<std::vector<Site> > positions ;
> std::list<std::vector<Site> >::iterator itSteps ;
>
> then in the .cpp file
>
> void RmfSystem::step() {
>
> if (itSteps == positions.end()) {
>
> itSteps = positions.begin() ;
>
> } else {
>
> itSteps++ ;
>
> }
>
> }[/color]

I don't see any errors in the above code per se. However, it's generally
*not* a good idea to define global variables in a header file.

My guess is that the error can be found elsewhere, e.g. calling erase.
It /can/ invalidate your iterator.

--
Peter

re: Std Iterator Problem

Larry I Smith wrote:
[color=blue]
> Adam Hartshorne wrote:
>[color=green]
>>Hi All,
>>
>>I was wondering if anybody can tell me what is wrong with the following
>>code,
>>
>>in a .h file I have
>>
>> std::list<std::vector<Site> > positions ;
>> std::list<std::vector<Site> >::iterator itSteps ;
>>
>>then in the .cpp file
>>
>>void RmfSystem::step() {
>>
>> if (itSteps == positions.end()) {
>>
>> itSteps = positions.begin() ;
>>
>> } else {
>>
>> itSteps++ ;
>>
>> }
>>
>>}
>>
>>where step gets called by an animate function in another file. The idea
>>that I have a vector of vector of positions and I want on each call of
>>step that the iterator points at another set of positions. Then when the
>>end of the list (well vector) is reached it loops around to the start.
>>
>>As it stands my program crashes, as itStep == positions.end() never
>>seems to be found to be true.
>>
>>Any ideas what I am doing wrong?
>>
>>Adam[/color]
>
>
> Among other things, where is the code that initializes
> 'itSteps'? Somewhere (after 'positions' is filled with
> data) you need to do:
>
> itSteps = positions.begin();
>
> A global iterator can cause unexpected side effects
> if multiple functions modify the container that
> the iterator references. This is a risky design.
>
> Regards,
> Larry
>[/color]

Sorry yes I have the following method

void RmfSystem::setUpAnimation() {

itSteps = positions.begin() ;

}

and it works fine, but like I say fails to find match the .end and hence
then loop around.

How should I do this instead?

Adam

re: Std Iterator Problem

Adam Hartshorne wrote:
[color=blue]
> Larry I Smith wrote:
>[color=green]
>> Adam Hartshorne wrote:
>>[color=darkred]
>>> Hi All,
>>>
>>> I was wondering if anybody can tell me what is wrong with the following
>>> code,
>>>
>>> in a .h file I have
>>>
>>> std::list<std::vector<Site> > positions ;
>>> std::list<std::vector<Site> >::iterator itSteps ;
>>>
>>> then in the .cpp file
>>>
>>> void RmfSystem::step() {
>>>
>>> if (itSteps == positions.end()) {
>>>
>>> itSteps = positions.begin() ;
>>>
>>> } else {
>>>
>>> itSteps++ ;
>>>
>>> }
>>>
>>> }
>>>
>>> where step gets called by an animate function in another file. The idea
>>> that I have a vector of vector of positions and I want on each call of
>>> step that the iterator points at another set of positions. Then when the
>>> end of the list (well vector) is reached it loops around to the start.
>>>
>>> As it stands my program crashes, as itStep == positions.end() never
>>> seems to be found to be true.
>>>
>>> Any ideas what I am doing wrong?
>>>
>>> Adam[/color]
>>
>>
>>
>> Among other things, where is the code that initializes
>> 'itSteps'? Somewhere (after 'positions' is filled with
>> data) you need to do:
>>
>> itSteps = positions.begin();
>>
>> A global iterator can cause unexpected side effects
>> if multiple functions modify the container that
>> the iterator references. This is a risky design.
>>
>> Regards,
>> Larry
>>[/color]
>
> Sorry yes I have the following method
>
> void RmfSystem::setUpAnimation() {
>
> itSteps = positions.begin() ;
>
> }
>
> and it works fine, but like I say fails to find match the .end and hence
> then loop around.
>
> How should I do this instead?
>
> Adam
>
>[/color]

I think I have worked out the problem, but not sure how to solve it. I
create the iterator at the same time as the list is created. However, I
then go on to fill the list up. I need to really create/renew the
iterator when the list has been finished been filled, but not sure how I
can do that, and still have it globally available.

Adam

re: Std Iterator Problem

"Adam Hartshorne" <oracle3001@yahoo.com> wrote in message
news:d77cu9$hsh$1@wisteria.csv.warwick.ac.uk...[color=blue]
> Adam Hartshorne wrote:
>[color=green]
>> Larry I Smith wrote:
>>[color=darkred]
>>> Adam Hartshorne wrote:
>>>
>>>> Hi All,
>>>>
>>>> I was wondering if anybody can tell me what is wrong with the following
>>>> code,
>>>>
>>>> in a .h file I have
>>>>
>>>> std::list<std::vector<Site> > positions ;
>>>> std::list<std::vector<Site> >::iterator itSteps ;
>>>>
>>>> then in the .cpp file
>>>>
>>>> void RmfSystem::step() {
>>>>
>>>> if (itSteps == positions.end()) {
>>>>
>>>> itSteps = positions.begin() ;
>>>>
>>>> } else {
>>>>
>>>> itSteps++ ;
>>>>
>>>> }
>>>>
>>>> }
>>>>
>>>> where step gets called by an animate function in another file. The idea
>>>> that I have a vector of vector of positions and I want on each call of
>>>> step that the iterator points at another set of positions. Then when
>>>> the
>>>> end of the list (well vector) is reached it loops around to the start.
>>>>
>>>> As it stands my program crashes, as itStep == positions.end() never
>>>> seems to be found to be true.
>>>>
>>>> Any ideas what I am doing wrong?
>>>>
>>>> Adam
>>>
>>>
>>>
>>> Among other things, where is the code that initializes
>>> 'itSteps'? Somewhere (after 'positions' is filled with
>>> data) you need to do:
>>>
>>> itSteps = positions.begin();
>>>
>>> A global iterator can cause unexpected side effects
>>> if multiple functions modify the container that
>>> the iterator references. This is a risky design.
>>>
>>> Regards,
>>> Larry
>>>[/color]
>>
>> Sorry yes I have the following method
>>
>> void RmfSystem::setUpAnimation() {
>>
>> itSteps = positions.begin() ;
>> }
>>
>> and it works fine, but like I say fails to find match the .end and hence
>> then loop around.
>>
>> How should I do this instead?
>>
>> Adam
>>
>>[/color]
>
> I think I have worked out the problem, but not sure how to solve it. I
> create the iterator at the same time as the list is created. However, I
> then go on to fill the list up. I need to really create/renew the iterator
> when the list has been finished been filled, but not sure how I can do
> that, and still have it globally available.
>
> Adam[/color]

Just make sure to set the iterator to begin() *after* filling up the list.
Also, reset it if you remove anything from the list. But why not make those
both members of the RmfSystem class? You can then use member functions of
that class to get or set any Site items using the iterator. (At least move
them out of the header, anyway.)

-Howard

re: Std Iterator Problem

Howard wrote:
[color=blue]
> "Adam Hartshorne" <oracle3001@yahoo.com> wrote in message
> news:d77cu9$hsh$1@wisteria.csv.warwick.ac.uk...
>[color=green]
>>Adam Hartshorne wrote:
>>
>>[color=darkred]
>>>Larry I Smith wrote:
>>>
>>>
>>>>Adam Hartshorne wrote:
>>>>
>>>>
>>>>>Hi All,
>>>>>
>>>>>I was wondering if anybody can tell me what is wrong with the following
>>>>>code,
>>>>>
>>>>>in a .h file I have
>>>>>
>>>>> std::list<std::vector<Site> > positions ;
>>>>> std::list<std::vector<Site> >::iterator itSteps ;
>>>>>
>>>>>then in the .cpp file
>>>>>
>>>>>void RmfSystem::step() {
>>>>>
>>>>> if (itSteps == positions.end()) {
>>>>>
>>>>> itSteps = positions.begin() ;
>>>>>
>>>>> } else {
>>>>>
>>>>> itSteps++ ;
>>>>>
>>>>> }
>>>>>
>>>>>}
>>>>>
>>>>>where step gets called by an animate function in another file. The idea
>>>>>that I have a vector of vector of positions and I want on each call of
>>>>>step that the iterator points at another set of positions. Then when
>>>>>the
>>>>>end of the list (well vector) is reached it loops around to the start.
>>>>>
>>>>>As it stands my program crashes, as itStep == positions.end() never
>>>>>seems to be found to be true.
>>>>>
>>>>>Any ideas what I am doing wrong?
>>>>>
>>>>>Adam
>>>>
>>>>
>>>>
>>>>Among other things, where is the code that initializes
>>>>'itSteps'? Somewhere (after 'positions' is filled with
>>>>data) you need to do:
>>>>
>>>> itSteps = positions.begin();
>>>>
>>>>A global iterator can cause unexpected side effects
>>>>if multiple functions modify the container that
>>>>the iterator references. This is a risky design.
>>>>
>>>>Regards,
>>>>Larry
>>>>
>>>
>>>Sorry yes I have the following method
>>>
>>> void RmfSystem::setUpAnimation() {
>>>
>>> itSteps = positions.begin() ;
>>> }
>>>
>>>and it works fine, but like I say fails to find match the .end and hence
>>>then loop around.
>>>
>>>How should I do this instead?
>>>
>>>Adam
>>>
>>>[/color]
>>
>>I think I have worked out the problem, but not sure how to solve it. I
>>create the iterator at the same time as the list is created. However, I
>>then go on to fill the list up. I need to really create/renew the iterator
>>when the list has been finished been filled, but not sure how I can do
>>that, and still have it globally available.
>>
>>Adam[/color]
>
>
> Just make sure to set the iterator to begin() *after* filling up the list.
> Also, reset it if you remove anything from the list. But why not make those
> both members of the RmfSystem class? You can then use member functions of
> that class to get or set any Site items using the iterator. (At least move
> them out of the header, anyway.)
>
> -Howard
>
>
>
>[/color]

I have just checked, and I do set set the iterator to begin() after
filling up the list.

Also both are memebers of the RmfSystem Class in the header definition.
To achieve what you suggest, I'm not sure how to do that. Also how can
you 'reset' the iterator?

Adam

re: Std Iterator Problem

Adam Hartshorne wrote:[color=blue]
> Adam Hartshorne wrote:
>[color=green]
>> Larry I Smith wrote:
>>[color=darkred]
>>> Adam Hartshorne wrote:
>>>
>>>> Hi All,
>>>>
>>>> I was wondering if anybody can tell me what is wrong with the following
>>>> code,
>>>>
>>>> in a .h file I have
>>>>
>>>> std::list<std::vector<Site> > positions ;
>>>> std::list<std::vector<Site> >::iterator itSteps ;
>>>>
>>>> then in the .cpp file
>>>>
>>>> void RmfSystem::step() {
>>>>
>>>> if (itSteps == positions.end()) {
>>>>
>>>> itSteps = positions.begin() ;
>>>>
>>>> } else {
>>>>
>>>> itSteps++ ;
>>>>
>>>> }
>>>>
>>>> }
>>>>
>>>> where step gets called by an animate function in another file. The idea
>>>> that I have a vector of vector of positions and I want on each call of
>>>> step that the iterator points at another set of positions. Then when
>>>> the
>>>> end of the list (well vector) is reached it loops around to the start.
>>>>
>>>> As it stands my program crashes, as itStep == positions.end() never
>>>> seems to be found to be true.
>>>>
>>>> Any ideas what I am doing wrong?
>>>>
>>>> Adam
>>>
>>>
>>>
>>> Among other things, where is the code that initializes
>>> 'itSteps'? Somewhere (after 'positions' is filled with
>>> data) you need to do:
>>>
>>> itSteps = positions.begin();
>>>
>>> A global iterator can cause unexpected side effects
>>> if multiple functions modify the container that
>>> the iterator references. This is a risky design.
>>>
>>> Regards,
>>> Larry
>>>[/color]
>>
>> Sorry yes I have the following method
>>
>> void RmfSystem::setUpAnimation() {
>>
>> itSteps = positions.begin() ;
>> }
>>
>> and it works fine, but like I say fails to find match the .end and
>> hence then loop around.
>>
>> How should I do this instead?
>>
>> Adam
>>
>>[/color]
>
> I think I have worked out the problem, but not sure how to solve it. I
> create the iterator at the same time as the list is created. However, I
> then go on to fill the list up. I need to really create/renew the
> iterator when the list has been finished been filled, but not sure how I
> can do that, and still have it globally available.
>
> Adam[/color]

After the container is filled do:

itSteps = positions.begin();

Perhaps you can just call your RmfSystem::setUpAnimation()
method after the container is filled.

The way you're using these globals is very risky.
It is not normally done, for good reasons. Creating
them in a header file (that may be included by multiple
source files) is a bad idea; you may end up with multiple
copies of them in multiple object files. To ensure that
you get just one copy of each global, try this:

in Site.h:

// state that the global 'positions' and 'itSteps' exist
// somewhere
extern std::list<std::vector<Site> > positions ;
extern std::list<std::vector<Site> >::iterator itSteps ;

in Site.cpp:

// create the global 'positions' and 'itSteps'
std::list<std::vector<Site> > positions ;
std::list<std::vector<Site> >::iterator itSteps ;

Any changes to the container by any function may
invalidate the iterator.

I have no idea what you're trying to do, but perhaps
using a class to encapsulate the container , access to it,
and changes to it, would be a better approach.

Start with a good book on OO Design...

Regards,
Larry

--
Anti-spam address, change each 'X' to '.' to reply directly.

re: Std Iterator Problem

Adam Hartshorne <oracle3001@yahoo.com> wrote in
news:d77cu9$hsh$1@wisteria.csv.warwick.ac.uk:
[color=blue]
> Adam Hartshorne wrote:
>[color=green]
>> Larry I Smith wrote:
>>[color=darkred]
>>> Adam Hartshorne wrote:
>>>
>>>> Hi All,
>>>>
>>>> I was wondering if anybody can tell me what is wrong with the
>>>> following code,
>>>>
>>>> in a .h file I have
>>>>
>>>> std::list<std::vector<Site> > positions ;
>>>> std::list<std::vector<Site> >::iterator itSteps ;
>>>>
>>>> then in the .cpp file
>>>>
>>>> void RmfSystem::step() {
>>>>
>>>> if (itSteps == positions.end()) {
>>>>
>>>> itSteps = positions.begin() ;
>>>>
>>>> } else {
>>>>
>>>> itSteps++ ;
>>>>
>>>> }
>>>>
>>>> }
>>>>
>>>> where step gets called by an animate function in another file. The
>>>> idea that I have a vector of vector of positions and I want on each
>>>> call of step that the iterator points at another set of positions.
>>>> Then when the end of the list (well vector) is reached it loops
>>>> around to the start.
>>>>
>>>> As it stands my program crashes, as itStep == positions.end() never
>>>> seems to be found to be true.
>>>>
>>>> Any ideas what I am doing wrong?
>>>>
>>>> Adam
>>>
>>>
>>>
>>> Among other things, where is the code that initializes
>>> 'itSteps'? Somewhere (after 'positions' is filled with
>>> data) you need to do:
>>>
>>> itSteps = positions.begin();
>>>
>>> A global iterator can cause unexpected side effects
>>> if multiple functions modify the container that
>>> the iterator references. This is a risky design.
>>>
>>> Regards,
>>> Larry
>>>[/color]
>>
>> Sorry yes I have the following method
>>
>> void RmfSystem::setUpAnimation() {
>>
>> itSteps = positions.begin() ;
>>
>> }
>>
>> and it works fine, but like I say fails to find match the .end and
>> hence then loop around.
>>
>> How should I do this instead?
>>
>> Adam
>>
>>[/color]
>
> I think I have worked out the problem, but not sure how to solve it. I
> create the iterator at the same time as the list is created. However,
> I then go on to fill the list up. I need to really create/renew the
> iterator when the list has been finished been filled, but not sure how
> I can do that, and still have it globally available.[/color]

Since you're playing with vector<>, you could use an index instead of an
iterator. Just be careful that your vector doesn't get _shorter_ while
you hold an index into the vector. Also watch out for iterator validity
rules. Remember that on insert, all iterators and references into the
vector may be invalidated. So... doing:

std::vector<int> vec;

// assume vec has say 5 items in it already

int index = 2;

int & vecvalue = vec[index];

vec.push_back(6);

vecvalue = 7;


Is "illegal". After the push_back, the reference vecvalue is no longer
valid. However, if you:

std::vector<int> vec;

// assume vec has say 5 items in it already

int index = 2;

int vecvalue = vec[index];

vec.push_back(6);

vec[index] = 7;

Is "correct". After the push_back, we re-index back into the vector, so
it recalculates on the potentially new area of memory that the vector now
occupies...

So, your step() method becomes:

void RmfSystem::step()
{
if (itSteps < positions.size())
{
++itSteps;
}
else
{
itSteps = 0;
}
}

re: Std Iterator Problem

"Adam Hartshorne" <oracle3001@yahoo.com> wrote in message
[color=blue][color=green][color=darkred]
>>>>>>
>>>>>>void RmfSystem::step() {
>>>>>>
>>>>>> if (itSteps == positions.end()) {
>>>>>>
>>>>>> itSteps = positions.begin() ;
>>>>>>
>>>>>> } else {
>>>>>>
>>>>>> itSteps++ ;
>>>>>>
>>>>>> }
>>>>>>
>>>>>>}
>>>>>>[/color][/color][/color]

You haven't shown the context in which this is used, but I think I see the
problem. If I recall correctly, the iterator end() points to one past the
last valid member of the list, *not* to the last member itself. Given that,
the function should be:

++itSteps;
if (itSteps == positions.end())
itSteps = positions.begin();


(I suppose you could pre-increment inside the if statement, but I prefer to
limit actions to one per line, personally.)

-Howard

re: Std Iterator Problem

"Adam Hartshorne" <oracle3001@yahoo.com> skrev i en meddelelse
news:d77b5t$h5j$1@wisteria.csv.warwick.ac.uk...[color=blue]
> Hi All,
>
> I was wondering if anybody can tell me what is wrong with the following
> code,
>
> in a .h file I have
>
> std::list<std::vector<Site> > positions ;
> std::list<std::vector<Site> >::iterator itSteps ;
>
> then in the .cpp file
>
> void RmfSystem::step() {
>
> if (itSteps == positions.end()) {
>
> itSteps = positions.begin() ;
>
> } else {
>
> itSteps++ ;
>
> }[/color]


*** 2
[color=blue]
>
> }
>
> where step gets called by an animate function in another file. The idea
> that I have a vector of vector of positions and I want on each call of
> step that the iterator points at another set of positions. Then when the
> end of the list (well vector) is reached it loops around to the start.
>
> As it stands my program crashes, as itStep == positions.end() never seems
> to be found to be true.
>
> Any ideas what I am doing wrong?
>
> Adam[/color]
Apart from what others say, I must ask you where you "use" your iterator.
You are not allowed to use it in *** 2 indicated above as you might
dereference end().

/Peter

re: Std Iterator Problem

Andre Kostur wrote:
[color=blue]
> Adam Hartshorne <oracle3001@yahoo.com> wrote in
> news:d77cu9$hsh$1@wisteria.csv.warwick.ac.uk:
>
>[color=green]
>>Adam Hartshorne wrote:
>>
>>[color=darkred]
>>>Larry I Smith wrote:
>>>
>>>
>>>>Adam Hartshorne wrote:
>>>>
>>>>
>>>>>Hi All,
>>>>>
>>>>>I was wondering if anybody can tell me what is wrong with the
>>>>>following code,
>>>>>
>>>>>in a .h file I have
>>>>>
>>>>> std::list<std::vector<Site> > positions ;
>>>>> std::list<std::vector<Site> >::iterator itSteps ;
>>>>>
>>>>>then in the .cpp file
>>>>>
>>>>>void RmfSystem::step() {
>>>>>
>>>>> if (itSteps == positions.end()) {
>>>>>
>>>>> itSteps = positions.begin() ;
>>>>>
>>>>> } else {
>>>>>
>>>>> itSteps++ ;
>>>>>
>>>>> }
>>>>>
>>>>>}
>>>>>
>>>>>where step gets called by an animate function in another file. The
>>>>>idea that I have a vector of vector of positions and I want on each
>>>>>call of step that the iterator points at another set of positions.
>>>>>Then when the end of the list (well vector) is reached it loops
>>>>>around to the start.
>>>>>
>>>>>As it stands my program crashes, as itStep == positions.end() never
>>>>>seems to be found to be true.
>>>>>
>>>>>Any ideas what I am doing wrong?
>>>>>
>>>>>Adam
>>>>
>>>>
>>>>
>>>>Among other things, where is the code that initializes
>>>>'itSteps'? Somewhere (after 'positions' is filled with
>>>>data) you need to do:
>>>>
>>>> itSteps = positions.begin();
>>>>
>>>>A global iterator can cause unexpected side effects
>>>>if multiple functions modify the container that
>>>>the iterator references. This is a risky design.
>>>>
>>>>Regards,
>>>>Larry
>>>>
>>>
>>>Sorry yes I have the following method
>>>
>>> void RmfSystem::setUpAnimation() {
>>>
>>> itSteps = positions.begin() ;
>>>
>>>}
>>>
>>>and it works fine, but like I say fails to find match the .end and
>>>hence then loop around.
>>>
>>>How should I do this instead?
>>>
>>>Adam
>>>
>>>[/color]
>>
>>I think I have worked out the problem, but not sure how to solve it. I
>>create the iterator at the same time as the list is created. However,
>>I then go on to fill the list up. I need to really create/renew the
>>iterator when the list has been finished been filled, but not sure how
>>I can do that, and still have it globally available.[/color]
>
>
> Since you're playing with vector<>, you could use an index instead of an
> iterator. Just be careful that your vector doesn't get _shorter_ while
> you hold an index into the vector. Also watch out for iterator validity
> rules. Remember that on insert, all iterators and references into the
> vector may be invalidated. So... doing:
>
> std::vector<int> vec;
>
> // assume vec has say 5 items in it already
>
> int index = 2;
>
> int & vecvalue = vec[index];
>
> vec.push_back(6);
>
> vecvalue = 7;
>
>
> Is "illegal". After the push_back, the reference vecvalue is no longer
> valid. However, if you:
>
> std::vector<int> vec;
>
> // assume vec has say 5 items in it already
>
> int index = 2;
>
> int vecvalue = vec[index];
>
> vec.push_back(6);
>
> vec[index] = 7;
>
> Is "correct". After the push_back, we re-index back into the vector, so
> it recalculates on the potentially new area of memory that the vector now
> occupies...
>
> So, your step() method becomes:
>
> void RmfSystem::step()
> {
> if (itSteps < positions.size())
> {
> ++itSteps;
> }
> else
> {
> itSteps = 0;
> }
> }[/color]

No using a vector, it is a list of vectors. I am iterating through the
list. I have appeared to solve my problem, but replacing

if (itSteps == positions.end()) {

with

if (itSteps == --positions.end()) {

appears to work correctly.

re: Std Iterator Problem

Adam Hartshorne <oracle3001@yahoo.com> wrote in
news:d77gsj$jh9$1@wisteria.csv.warwick.ac.uk:

[color=blue]
> No using a vector, it is a list of vectors. I am iterating through the
> list. I have appeared to solve my problem, but replacing
>
> if (itSteps == positions.end()) {
>
> with
>
> if (itSteps == --positions.end()) {
>
> appears to work correctly.[/color]

Undefined behaviour if positions is empty.

re: Std Iterator Problem

"Adam Hartshorne" <oracle3001@yahoo.com> skrev i en meddelelse
news:d77gsj$jh9$1@wisteria.csv.warwick.ac.uk...[color=blue]
> Andre Kostur wrote:
>[/color]
[snip][color=blue]
>
> if (itSteps == positions.end()) {
>
> with
>
> if (itSteps == --positions.end()) {
>
> appears to work correctly.
>[/color]
As Andre pointed out, this is undefined if the list is empty. It is also an
indication that you are using an invalid iterator as i explained in my other
reply.

/Peter