Thread: illegal operand

Does anybody see anything wrong with this code?
I keep getting an "illegal operand" error in my MetroWerks CodeWarrior compiler.

Your help is appreciated.

code


/*wDeck is a two dimensional array with a depth of 13, row and
column are random integers*/

*(wDeck+(row*13+column)) = var;


/code

Does it work with the code reversed. IE var = .......?

How do you mean that it would depend on how I declared wDeck? I'm not sure I understand.

Originally posted by monkey_C
How do you mean that it would depend on how I declared wDeck? I'm not sure I understand.

int wDeck;
int *wDeck;
int **wDeck;
int wDeck[13];
int *wDeck[13];
int wDeck[13*13];
int wDeck[13][13];

some of these make sense for your situation, some dont. but the ones that make sense are used in different ways.

edit: btw these are for a 13x13, not 4x13. oopsie i wasnt thinking when i did it

OK, wDeck was originally declared in main() as

int deck [4] [13] = {0};

and was passed into the function containing my problematic line as

const int wDeck[][13]

but I still don't see how that's affecting my code. I think I've just been staring at the same line for too long.

Originally posted by monkey_C
OK, wDeck was originally declared in main() as

int deck [4] [13] = {0};

and was passed into the function containing my problematic line as

const int wDeck[][13]

but I still don't see how that's affecting my code. I think I've just been staring at the same line for too long.

then access the deck with wDeck[row][column]

and if it's passed to the function as const, why are yuo modifying it?

You can't pass const int wDeck[][13] in C.

Originally posted by monkey_C
I'm sorry, I made two mistakes, first of all I should have mentioned I was specifically trying to avoid array syntax. Secondly, it is not passed into this function as a const, I was looking at a similar line of code in my next function down.

Conclusively, it was declared as:

int wDeck[][13]


again, sorry.

*(*(wdeck + row) + column) is what you need for a multidimentional array.

edit: note however that all this is is a clumsy and difficult to read version of wdeck[row][column]. there's no reason to be using it really.