Strange Results with Overloaded Operators

[John Harrison]

I'm having problems with the code you can find at http://planetevans.com/c

Particulary with test6(). However even with the earlier tests they work
fine when run, but crash when run in the debugger. I'm also still having
problems with the destructor.

Both your operator[] are wrong. Think carefully about what happens when
arrayLow is not zero.

Should have spotted this in your earlier post, sorry.

john

 

[- Steve -]

I'm having problems with the code you can find at http://planetevans.com/c

Particulary with test6(). However even with the earlier tests they work
fine when run, but crash when run in the debugger. I'm also still having
problems with the destructor.

 

[- Steve -]

Both your operator[] are wrong. Think carefully about what happens when
arrayLow is not zero.

Should have spotted this in your earlier post, sorry.

john

Okay that was a stupid mistake. Changed them to execute return
array[forLocation];

I'm still having problems though. With the destructor enabled it crashes on
test2(). Without it enabled it crashes on test3().

 

[ES Kim]

Both your operator[] are wrong. Think carefully about what happens when
arrayLow is not zero.

Should have spotted this in your earlier post, sorry.

john

Okay that was a stupid mistake. Changed them to execute return
array[forLocation];

I'm still having problems though. With the destructor enabled it crashes on
test2(). Without it enabled it crashes on test3().

Still wrong.
What if a[-3] for IntArray a(-3, 6)?

 

[ES Kim]

Then it returns array[-3] which is a valid variable isn't it?

No. Array index starts from 0. You just changed the interface
using the array. It should be

int& IntArray:perator[] (int forLocation)
{
return array[forLocation - arrayLow];
}

For a[-3], it must return array[0].

 

[- Steve -]

Then it returns array[-3] which is a valid variable isn't it?

Both your operator[] are wrong. Think carefully about what happens when
arrayLow is not zero.

Should have spotted this in your earlier post, sorry.

john

Okay that was a stupid mistake. Changed them to execute return
array[forLocation];

I'm still having problems though. With the destructor enabled it crashes on
test2(). Without it enabled it crashes on test3().

Still wrong.
What if a[-3] for IntArray a(-3, 6)?