[] overload

[fox]

Hi
I've two classes

class node {
....
node operator[](int idx);
}

and
class grid {
.....
node operator[](int idx);
}

but when i do :
grid *one=new grid();
cout << grid[1][1]
it uses operator [] for grid class, (which returns a node )
and doesn't use an overloaded one for the returned node value;
Is there a way to force him ?
or do i do sth wrong ?

Cheers

[Jeff Schwab]

fox wrote:

Hi
I've two classes

class node {
...
node operator[](int idx);
}

and
class grid {
....
node operator[](int idx);
}

but when i do :
grid *one=new grid();
cout << grid[1][1]
it uses operator [] for grid class, (which returns a node )
and doesn't use an overloaded one for the returned node value;
Is there a way to force him ?
or do i do sth wrong ?

How do you know it's wrong? Did you overload operator << (
std::ostream&, node const& )? Is indexing a node really supposed to
give another node?

Please post a small program to show the problem. Someone here probably
can show you what has gone wrong, but it is very difficult to tell from
the pseudocode.

-Jeff

[Dan Cernat]

"fox" <ar***@vdi.pl> wrote in message
news:bs**********@nemesis.news.tpi.pl...

Hi
I've two classes

class node {
...
node operator[](int idx);
}

and
class grid {
....
node operator[](int idx);
}

but when i do :
grid *one=new grid();
cout << grid[1][1] ^^^^^^^^^^^^^^^^^^^^^^^^^^ that could not possibly compile
if you meant cout << one[1][1];
then you have some problems here:
1. you allocated only 1 grid element so one[1] is past the array boundaries.
it should have been one[0][1]

2. one is a pointer and when someone does one[n] it actually gets *(one + n)

so what you should have done is:

grid myGrid;
// put here code to fill the collection of nodes
// then do:

cout << grid[1][1];

assuming everything else is working
it uses operator [] for grid class, (which returns a node )
and doesn't use an overloaded one for the returned node value;
Is there a way to force him ?
or do i do sth wrong ?

Cheers

Dan

[Dan Cernat]

err...

"Dan Cernat" <ce****@dan.com> wrote in message
news:vv************@corp.supernews.com...

so what you should have done is:

grid myGrid;
// put here code to fill the collection of nodes
// then do:

cout << grid[1][1]; ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^make it
cout << myGrid[1][1];
Dan

Dan

[Martijn Lievaart]

On Tue, 30 Dec 2003 13:40:34 +0100, fox wrote:

Hi
I've two classes

class node {
...
node operator[](int idx);
}

and
class grid {
....
node operator[](int idx);
}

but when i do :
grid *one=new grid();
cout << grid[1][1]

cout << (*grid)[1][1];

You want to act on the contents of the pointer, not the pointer itself.

HTH,
M4